Код шифрования Цезаря не работает должным образом C

Question

Я работаю над проблемой шифрования Цезаря для проблемы, но я столкнулся с незначительной проблемой. Всякий раз, когда значение шифрования получает больше ascii из 'z', я хочу, чтобы он возвращался к 'a', но я не могу понять, как это сделать. Вот код:

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

int main(int argc, string argv[]) 
{ 
    if(argc !=2 && !isdigit(argv[1])) 
    { 
     return 1; 
    } 

    //convert input to int and get the string 
    int k = atoi(argv[1]); 
    k=k%26; 
    //printf("%d" ,k); 
    //get he text 

    char *s; 
    s=GetString(); 
    int i, n=strlen(s); 

    //checking each character 
    for(i=0;i<n;i++) 
    { 
     if(s[i]==' ') 
     { 
      s[i]=' '; 
     } 
     else 
     { 
      s[i]=s[i]+k; 
     } 
     printf("%c" ,s[i]); 
    } 

} 

Это довольно основной код. Любая помощь будет оценена.

P.S. вот пример с ключом 4.

input- Vinay Dawani 
output- Zmre} He{erm 

Answer 1

После добавления k к письму, проверьте, если она выше, чем z. Если это так, просто вычтите количество букв в алфавите, чтобы обернуть его.

s[i] = s[i]+k; 
if (s[i] > 'z') { 
    s[i] = s[i] - 26; 
} 

Чтобы иметь дело с верхним и нижним регистром, вам нужны отдельные тесты в основном цикле.

for(i=0;i<n;i++) 
{ 
    if(islower(s[i])) 
    { 
     s[i] += k; 
     if (s[i] > 'z') { 
      s[i] -= 26; 
     } 
    } 
    else if (isupper(s[i])) 
    { 
     s[i] += k; 
     if (s[i] > 'Z') { 
      s[i] -= 26; 
     } 
    } 
    printf("%c" ,s[i]); 
} 

Обратите внимание, что я удалил тест для if (s[i] == ' '). Это не обязательно - все, что не письмо, просто осталось в покое.

Answer 2

Предполагая, что введенный код не является EBCDIC (который имеет неалфавитные символы между буквенными символами), то буквы латинского алфавита (без акцента a-z) кодируются как непрерывные кодовые точки во всех общих наборах кодов. Вращение алфавита просто, когда вы знаете, как это сделать.

Фундаментально, конвертировать каждую букву в 0..25 смещении из письма a или A (в зависимости от случая), добавьте ключ кодирования, взять результат по модулю 26, и добавить начальную букву (a или A) назад. Обратите внимание, что арифметика имеет место как int, но результат присваивается обратно (возможно, подписанному) char.

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

int main(int argc, string argv[]) 
{ 
    if (argc != 2 || !isdigit((unsigned char)argv[1][0])) 
    { 
     fprintf(stderr, "Usage: %s shift\n", argv[0]); 
     return 1; 
    } 

    // convert input to int and get the string 
    int k = atoi(argv[1]) % 26; 
    if (k < 0) 
     k += 26; 

    char *s = GetString(); 
    int n = strlen(s); 

    printf("Original: [%s]\n", s); 

    // encoding each character 
    for (int i = 0; i < n; i++) 
    { 
     if (isupper((unsigned char)s[i])) 
     { 
      s[i] = 'A' + (s[i] - 'A' + k) % 26; 
     } 
     else if (islower((unsigned char)s[i])) 
     { 
      s[i] = 'a' + (s[i] - 'a' + k) % 26; 
     } 
    } 

    printf("Encrypted: [%s]\n", s); 
    return 0; 
} 

Пример запуска:

$ ./caesar13 3 
Caesar's cipher is hardly secure against the lazy dog jumping over the quick brown fox, is it? 
Original: [Caesar's cipher is hardly secure against the lazy dog jumping over the quick brown fox, is it?] 
Encrypted: [Fdhvdu'v flskhu lv kdugob vhfxuh djdlqvw wkh odcb grj mxpslqj ryhu wkh txlfn eurzq ira, lv lw?] 
$ ./caesar50 23 
Fdhvdu'v flskhu lv kdugob vhfxuh djdlqvw wkh odcb grj mxpslqj ryhu wkh txlfn eurzq ira, lv lw? 
Original: [Fdhvdu'v flskhu lv kdugob vhfxuh djdlqvw wkh odcb grj mxpslqj ryhu wkh txlfn eurzq ira, lv lw?] 
Encrypted: [Caesar's cipher is hardly secure against the lazy dog jumping over the quick brown fox, is it?] 
$ 

CS50 library легко доступны в Интернете.