Помогите найти баг в коде (язык С, шифр Цезаря)

[Daniela Daniela]

Добрый день!
Получила задание написать программу для шифровки текста (шифр Цезаря).
С этим вроде бы проблем не возникло, но при запуске программы к буквам нижнего регистра почему-то добавляется еще какой-то символ.

Код

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

int main (int argc, string argv[])
{
    if (argc !=2)
    {
        printf("Usage: ./caesar key\n");
        return 1;
    }

    int key = atoi(argv[1]);
    string s = get_string();

    for (int i=0, n=strlen(s); i<n ; i++ )
    {
        if (isalpha (s[i]) && islower (s[i]))
        {
            printf("%c", tolower(((s[i]-97+key)%26)+97));
        }
        if (isalpha (s[i]) && isprint(s[i]))
        {
            printf("%c", (((s[i]-65+key)%26)+65));
        }
        if (ispunct (s[i]) || isspace (s[i]))
        {
            printf ("%c", s[i]);
        }
    }
    printf("\n");
}

после запуска с ключем 0 (для проверки) выдает:
~/workspace/pset2/ $ ./caesar 0
ABC abc
ABC aGbHcI

Откуда взялись дополнительные символы??? Где я ошиблась?

П.С.
Не по теме: что значит сообщение
Ошибка в программном коде: 'ascii' codec can't encode characters in position 109-116: ordinal not in range(128)

[Black Fregat]

Одни if и ни одного else. Для маленьких букв выполняются оба куска - и для больших, и для маленьких

[Daniela Daniela]

Спасибо за ответ!
else нет, т.к. если условие не выполняется, то цикл завершается и программа переходит к следующему (во всяком случае, так объясняют на лекции)

Разделила проверку условия: if (isalpha (s[i]) ) и потом внутри две проверки отдельно if (islower (s[i])) и if (isupper(s[i]))
ООО чудо! Заработало!

Но теперь при проверке выдает сообщение (проверяется не человеком, а автоматически компом):
Результат теста - неверно.
Ошибки:
encrypts "a" as "b" using 1 as key
expected prompt for input, found none
encrypts "barfoo" as "yxocll" using 23 as key
expected prompt for input, found none
encrypts "BARFOO" as "EDUIRR" using 3 as key
expected prompt for input, found none
encrypts "BaRFoo" as "FeVJss" using 4 as key
expected prompt for input, found none
encrypts "barfoo" as "onesbb" using 65 as key
expected prompt for input, found none

Что значит "ожидаемое приглашение для ввода, не найдено ни одного"?
Я в отчаянии...

[p51x]

Только то, что написано. Очевидно тестирующая система ждет какой-то строки - приглашения для ввода, типа >. Но не находит и тст проваливается.

Кстати, кто вам сказал, что у мейна в параметрах стринг?

[Daniela Daniela]

Может я неправильно поняла задание?
Цитата: "ваша цель состоит в том, чтобы написать в caesar.c программу, которая шифрует сообщение с помощью шифра Цезаря. Программа должна принимать один аргумент командной строки: неотрицательное целое число. Если вашу программу запускают без каких-либо данных командной строки, или более чем одним аргументом, должно появляться сообщение об ошибке и возвращаться значение 1 (что в данном случае означает ошибку) как указано ниже:
return 1;
В противном случае, программа должна перейти к запросу текста, который нужно зашифровать; символы, которые не являются буквами алфавита, будут выведены без изменений. После вывода зашифрованного текста программа должна завершиться, main должен вернуть 0 следующим образом:
return 0. "

Так как мы автоматизируем для вас некоторые тесты, программа должна вести себя как показано ниже. Считаем, что подчеркнутый текст это то, что набрал пользователь.
~/Dropbox/pset2: ./caesar 13

Be sure to drink your Ovaltine!

Or fher gb qevax lbhe Binygvar!

у меня все именно так и выводится.

По поводу стринг - так задано изначально ("так как эта программа должна стартовать с того, что в командной строке она должна принять значение k, в main надо прописать:

int main (int argc, string argv [])")

[Black Fregat]

Действительно, непонятно, какой промпт ожидается.
Судя по примеру, может быть, перевод строки?

[Daniela Daniela]

Сообщение от Black Fregat

Судя по примеру, может быть, перевод строки?

Что значит "перевод строки"? это \n ? так вроде бы ж есть... или еще где-то нужно?

Извините за такие вопросы, просто тяжело быть "чайником" , а хочется понять в чем ошибка

И, если не сложно, объясните что может значить сообщение (не из этого задания):
"Ошибка в программном коде: 'ascii' codec can't encode characters in position 109-116: ordinal not in range(128)"

[Black Fregat]

Что именно не так с Вашим заданием - непонятно. Если не у кого проконсультироваться, можно толко попытаться подобрать. Попробуйте таки вывести хоть какой-то текст перед вводом строки. Возможны варианты: просто перевод строки, какой-то текст с переводом строки (например, как предлагали, ">"), текст без перевода строки.

Сообщение от Daniela Daniela

"Ошибка в программном коде: 'ascii' codec can't encode characters in position 109-116: ordinal not in range(128)"

Эта ошибка означает, что где-то в позициях 109-116 ожидаются символы чисто ascii, а на самом деле стоят символы с кодом больше 127, скорее всего, русские буквы

[Daniela Daniela]

Я его победила! Спасибо Вам за помощь.
Подбирала пол дня. В итоге тест прошел, когда я вывела текст перед вводом пользователя и вывела текст перед ответом!
Вот так получилось:
~/workspace/pset2/ $ ./caesar 0
plaintext: abc ABC!
ciphertext: abc ABC!
~/workspace/pset2/ $ ./caesar 13
plaintext: abc ABC!
ciphertext: nop NOP!

П.С. по поводу ascii: это мне нужно создать новую тему , чтоб выяснить ошибку ? (суть задачи из этой же темы: запросить ФИО пользователя и вывести инициалы большими буквами)

[Black Fregat]

Сообщение от Daniela Daniela

это мне нужно создать новую тему , чтоб выяснить ошибку ?

Не знаю..

Сообщение от Daniela Daniela

запросить ФИО пользователя и вывести инициалы большими буквами

Напишите подробнее, где именно ошибка.
Может, комментарии русскими буквами? Иногда и на это бывает ошибка