Почему fputc сохраняет кракозябры вместо символов?

Question

Имеется следующий код на си (шифрование по шифру Вижнера):

#include "stdafx.h"
#include <iostream>

#define _CRT_SECURE_NO_WARNINGS
#pragma warning(disable: 4996)

int main()
{
    char secret_key[255];
    int c, s, i = 0;    
    int secret_key_len;
    int select_menu = 0;

    FILE * normal_text = fopen("normal.txt", "r");
    FILE * crypt_text = fopen("crypt.txt", "w");
    FILE * decrypt_text = fopen("decrypt.txt", "w");

    setlocale(LC_ALL, "Russian");

    printf("Введите ключ: ");
    scanf("%s", secret_key);

    printf("Зашифровать normal.txt (0), или расшифровать crypt.txt (1)");
    scanf_s("%d", &select_menu);

    secret_key_len = strlen(secret_key);

    if (select_menu)
    {
        if (crypt_text == NULL)
        {
            printf("Файл с зашифрованным текстом пустой.");
        }

        while ((c = fgetc(crypt_text)) != EOF)
        {
            s = secret_key[i++ % secret_key_len];
            fputc( (c - s + 26) % 26, decrypt_text);
        }
    }
    else
    {
        if (normal_text == NULL)
        {
            printf("Нормальный текст отсутствует");
        }

        while ((c = fgetc(normal_text)) != EOF)
        {
            s = secret_key[i++ % secret_key_len];
            fputc( (c + s) % 26, crypt_text);
        }
    }


    fclose(normal_text);
    fclose(crypt_text);
    fclose(decrypt_text);

    system("pause");
    return 0;
}

Почему fputc сохраняет кракозябры вместо символов? Понять не могу.

Answer 1

С функцией fputc, разумеется, всё в порядке. Проблема в кодах символов.

Вы почему-то предполагаете, что коды символов a..z есть числа 0..25. А это не так. Код символа 'a' — 97, например.

Если ваш текст на входе — строчные латинские буквы, вместо

fputc( (c - s + 26) % 26, decrypt_text);

вам нужно вот что:

int letternum = c - 'a';
int encodednum = (letternum - s + 26) % 26;
char encodedletter = 'a' + encodednum;
fputc(encodedletter, decrypt_text);

Ну или просто

fputc('a' + (c - 'a' - s + 26) % 26, decrypt_text);

Аналогичная проблема у вас с расшифровкой, надеюсь, тут вы разберётесь сами по аналогии.

Подумайте, как вы хотите кодировать текст, в котором есть пробелы, заглавные буквы, цифры, знаки препинания.

---

Дополнение: если вы хотите работать со всеми символами, возможно, имеет смысл работать не по модулю 26, а по модулю 256:

fputc(c - s, decrypt_text);

и соответственно

fputc(c + s, crypt_text);

При этом байты ключа должны быть не в промежутке 0..25, а 0..255. Это автоматически решает проблему с пробелами, заглавными буквами, цифрами, знаками препинания и т. п.