Я написал небольшую программу для шифра Ceasar на C ++.
Я думаю, что у меня все хорошо с логикой. Тем не менее, добавление ASCII идет не так по какой-то странной причине, которую я не могу понять. Вот мой код:
Что-то не так в этой строке:
«s [i] = (abs (s [i])) + k;»
Я даю вклад в следующем:
2
ху
87
Вывод должен быть:
Г.Х.
Я получаю вывод как: ред.
#include <cmath>
#include <cstdio>
#include <vector>
#include <iostream>
#include <algorithm>
using namespace std;
int main(){
int n;
cin >> n;
string s;
cin >> s;
int k;
cin >> k;
k = k % 26;
for(int i = 0; i<s.size(); i++){
if((abs(s[i]) >=65 && abs(s[i]) <=90) || (abs(s[i]) >= 97 && abs(s[i]) <= 122)){
cout << "k is: "<<k << endl; //k should be 9
//for x abs(s[i]) should be 120
cout << "Absolute value is: "<<abs(s[i]) <<endl;
s[i] = (abs(s[i])) + k; // thish is not 129.. i am getting 127
cout << "After adding K: "<<abs(s[i]) << endl;
if((abs(s[i]) > 90) && (abs(s[i]) < 97))
s[i] = abs(s[i]) - 26;
if(abs(s[i]) > 122){
s[i] = abs(s[i]) - 26;
}
}
}
for(int i =0 ; i< s.size(); i++)
cout<<s[i];
return 0;
}
Любая помощь будет оценена. Благодарю.
эта строка:
s[i] = (abs(s[i])) + k;
вычисляет значение больше 127. Использование строк char
типы внутри, так что вы кладете значение вне диапазона s[i]
: определена реализация: по модулю, усечение …
Вычитая 26
потом не помогает: данные уже уничтожены.
Исправлено: полностью работать с целым числом и возвращать обратно s[i]
в конце:
for(int i = 0; i<s.size(); i++){
int v = s[i];
if((abs(v) >=65 && abs(v) <=90) || (abs(v) >= 97 && abs(v) <= 122)){
cout << "k is: "<<k << endl; //k should be 9
//for x abs(v) should be 120
cout << "Absolute value is: "<<abs(v) <<endl;
v = (abs(v)) + k; // integer won't overflow
cout << "After adding K: "<<abs(v) << endl;
if((abs(v) > 90) && (abs(v) < 97))
s[i] = abs(v) - 26;
if(abs(v) > 122){
s[i] = abs(v) - 26;
}
}
}
Я проверил это, и я получаю gh
все в порядке (и это быстрее, потому что меньше доступа к массиву s[i]
)
Других решений пока нет …