Ви не увійшли. Будь ласка, увійдіть або зареєструйтесь.
Ласкаво просимо вас на україномовний форум з програмування, веб-дизайну, SEO та всього пов'язаного з інтернетом та комп'ютерами.
Будемо вдячні, якщо ви поділитись посиланням на Replace.org.ua на інших ресурсах.
Для того щоб створювати теми та надсилати повідомлення вам потрібно Зареєструватись.
Український форум програмістів → C++ → Подвоєння першої літери при виводі рядка
Сторінки 1
Для відправлення відповіді ви повинні увійти або зареєструватися
Не розумію чому опісля заміни літер у всьому рядку при виводі подвоюється перша літера.
А я так сподівався, що є:).
Ще там помилка при невірному вводі цілого числа k я можу повторно ввести вірне, але функція при цьому не спрацьовує.
Ось, тільки я дужки квадратні біля "і" прибрав, бо писало, що це не дозволено:
#include <stdio.h>
#include <string.h>
#include <ctype.h>
#include <stdlib.h>
char caesar_crypt(char *input, int key);
int main(int argc, char **argv)
{
int k = atoi(argv[1]);
char to_crypt[100];
while(k < 1 || k > 25)
{
if (k < 1 || k > 25)
printf("Wrong k, try again: ");
scanf("%d", &k);
}
fgets(to_crypt, 100, stdin);
char crypted = caesar_crypt(to_crypt, k);
puts(&crypted);
return 0;
}
char caesar_crypt(char *input, int key)
{
for(int i = 0, n = strlen(input); i < n; i++)
{
if (isalpha(inputi) && (int) inputi <= 90 && (int) inputi >= 65 && ((int) inputi + key <= 90))
{
inputi = (int)inputi + key;
}
else if (isalpha(input[i) && (int) inputi <= 90 && (int) inputi >= 65 && ((int) inputi + key > 90))
{
inputi = 65 + (((int)inputi + key) - 91);
}
else if (isalpha(inputi) && (int) inputi >= 97 && (int) inputi <= 122 && ((int) inputi + key <= 122))
{
inputi = (int)inputi + key;
}
else if (isalpha(inputi) && (int) inputi >= 97 && (int) inputi <= 122 && ((int) inputi + key > 122))
{
inputi = 97 + (((int)inputi + key) - 123);
}
}
return *input;
}
0. Не забувайте тег code
1. atoi при неправильному параметрі дає UB: http://www.cplusplus.com/reference/cstdlib/atoi/
Тобто зовсім не гарантується, що якщо рядок 10 не спрацював, то потім спрацює рядок 13 - у k буде сміття.
2. Якщо рядок 13 спрацював, то спрацює і рядок 15.
3. Якщо те, що ввели в scanf, буде неправильним, то буде встановлено ferror на stdin, і подальше читання без скидання цього прапорця буде неможливим. Значення k при цьому, знову ж таки, невизначене. http://www.cplusplus.com/reference/cstd … /?kw=scanf
Викликайте
clearerr(stdin);а ще краще - читайте стрічку і обробляйте всередині програми, а не покладайтеся на стандартні функції, вони в C незручні, як на цей час.
4. За розміщення рядків 16 і 17 на одному рівні вбив би.
З одною помилкою розібрався, але подвоєння першої літери при виводі обробленого рядка все ще зберігається.
А ви в курсі, що код, який ви надали, не компілюється?
А ви про прибрані квадратні дужки не забули? Ось його нова версія:
#include <stdio.h>
#include <string.h>
#include <ctype.h>
#include <stdlib.h>
char caesar_crypt(char *input, int key);
int main(int argc, char **argv)
{
int k = atoi(argv[1]);
char to_crypt[100];
if ((argc > 2) || (argc == 1) || (k < 0))
{
printf("Error.\n");
return 1;
}
else
{
k = k % 26;
fgets(to_crypt, 100, stdin);
char crypted = caesar_crypt(to_crypt, k);
puts(&crypted);
return 0;
}
}
char caesar_crypt(char *input, int key)
{
for(int i = 0, n = strlen(input); i < n; i++)
{
if (isalpha(input[i]) && (int)input[i] <= 90 && (int) input[i] >= 65 && ((int)input[i] + key <= 90))
{
input[i] = (int)input[i] + key;
}
else if (isalpha(input[i]) && (int)input[i] <= 90 && (int) input[i] >= 65 && ((int)input[i] + key > 90))
{
input[i] = 65 + (((int)input[i] + key) - 91);
}
else if (isalpha(input[i]) && (int)input[i] >= 97 && (int) input[i] <= 122 && ((int)input[i] + key <= 122))
{
input[i] = (int)input[i] + key;
}
else if (isalpha(input[i]) && (int)input[i] >= 97 && (int) input[i] <= 122 && ((int)input[i] + key > 122))
{
input[i] = 97 + (((int)input[i] + key) - 123);
}
}
return *input;
}Нічого собі - "не забув"! Це ви їх по-людськи не виклали.
Приберіть всі "магічні константи".
І я щось не дуже розумію, що ви робите в рядку 23 - передаєте в puts адресу символа, щоб він вивів що завгодно, починаючи з цього символа і невідомо де це закінчується? От у вас казна-що і виводиться.
у 23-му рядку я виводжу на екран результат роботи функції "caesar_crypt", яку я зберіг до змінної "crypted".
Там вводиться стрічка для кодування шифром цезаря і виводиться відповідно теж стрічка.
Де саме "там"?
Все зрозумів. Не треба було тягнути вивід у main, а просто вивести відразу у функції і return 0.
Тоді все добре виводиться без подвоєнь. Ну і ще деякі правки зробив у коді.
Ви б краще реагували б на зауваження та відповідали на питання, тоді б і самі щось зрозуміли.
Сторінки 1
Для відправлення відповіді ви повинні увійти або зареєструватися