Я писал код на шифре Цезаря (кодирую буквы, сдвигая их на введенное число). Он работал правильно, пока я не ввел условие, гарантирующее, что введенный ключ является цифрой и каким-то образом нарушает работу кода.

Я не понимаю, как это условие соотносится с кодом потом и что не так.

Не мог бы кто-нибудь помочь?

#include <stdio.h>
#include <cs50.h>
#include <stdlib.h>
#include <string.h>
#include <ctype.h>

int main(int argc, string argv[])

{
    if (argc != 2)
    {
        printf("Usage: ./caesar key\n");
        return 1;
    }

    for (int j = 0; j < strlen(argv[1]); j++)
    {
        if (!(isdigit(argv[1][j])))
        {
            printf("Usage: ./caesar key\n");
            return 1;
        }

    }


    int k = atoi(argv[1]);
    if (argc == 2 && k > 0)

    {
        string s = get_string("plaintext: ");

        int m = k % 26;

        //introducing an array type char of string length size
        char cypher[strlen(s)];


        for (int i = 0; i < strlen(s);  i++)
        {
            //checking if it's a letter, if yes then next if
            if (isalpha(s[i]))
            {
                //checking if it's uppercase
                if (isupper(s[i]))
                {
                    cypher[i] = (s[i] + m);
                    //make a round if it's beyond Z
                    if (cypher[i] > 'Z')
                    {
                        cypher[i] = cypher[i] - 26;
                    }

                }
                // checking if it's lowercase
                else if (islower(s[i]))
                {
                    //if it falls beyond ascii table
                    if ((s[i] + m) > 126)
                    {
                        cypher[i] = (s[i] - 26 + m );
                    }
                    else
                    {
                        cypher[i] = (s[i] + m);
                    }
                    //make a round if it's beyond z
                    if (cypher[i] > 'z')
                    {
                        cypher[i] = cypher[i] - 26;
                    }
                }

            }
            // if it's not a letter - keep the initial character, write it in the cypher
            else
            {
                cypher[i] = s[i];
            }
        }

        printf("ciphertext: %s\n", cypher);
        return 0;

    }



}
0
Sabie 26 Ноя 2020 в 03:16

1 ответ

Лучший ответ

Strlen в C не учитывает нулевой символ. Длина массива должна быть [strlen (s) +1], и должен быть добавлен символ \ 0

0
Sabie 26 Ноя 2020 в 01:18