71 / 58 / 48
Регистрация: 12.03.2017
Сообщений: 563
1

Шифр цезаря: ошибка

15.04.2018, 17:45. Показов 635. Ответов 6
Метки нет (Все метки)

Author24 — интернет-сервис помощи студентам
Добрый день, есть класс Цезарь. В нем есть функция Кодирование информации с помощью шифра цезаря. Её реализация ниже.
проблема в том ,что когда я компилирую, он выводит НННННН и последнию букву выводит нормальную закодированную.
Т.е на вход идет слово бак
на выходе будет ННз
Ключ сдвига на 3 символа.
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
void Cesar::Encode() 
{
 
 
    for (int i = 0; i < this->Startline.length(); i++)
    {
    
    
        for (int j = 0; j < this->Alfavit.length(); j++)
        {
        
            if (this->Startline[i] == Alfavit[j])
 
            {
                
                this->EncodeLine = new char[this->Startline.length()];
                this->EncodeLine[i] = Alfavit[j - this->key];
                
            }
        
        }
        
    
    
    }
 
    
 
}
Добавлено через 1 минуту
Класс
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
#include <iostream>
 
class Cesar 
{
 
private:
    std::string Startline;
    int key;
    std::string Alfavit;
    char *EncodeLine;
public:
    void GetEncodeLine();
    void SetLine(std::string s);
    void Setkey(int key);
    std::string GetLine();
    int GetKey();
    void Encode();
    void DeCode();
    Cesar(std::string s , int key);
};
Его функции
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
#include <iostream>
#include "Cesar.h"
 
 
 
Cesar::Cesar(std::string s , int key) 
{
 
    this->Startline = s;
    this->key = key;
    this->Alfavit = { "абвгдеёжзийклмнопрстуфхцчшщъыьэюя " };
 
}
 
void Cesar::SetLine(std::string s) 
{
 
    
    this->Startline = s;
 
 
}
 
 
void Cesar::Setkey(int key) 
{
 
    this->key = key;
 
}
 
 
std::string Cesar::GetLine() 
{
 
    return this->Startline;
 
}
 
int Cesar::GetKey() 
{
 
    return this->key;
 
}
 
 
void Cesar::Encode() 
{
 
 
    for (int i = 0; i < this->Startline.length(); i++)
    {
    
    
        for (int j = 0; j < this->Alfavit.length(); j++)
        {
        
            if (this->Startline[i] == Alfavit[j])
 
            {
                
                this->EncodeLine = new char[this->Startline.length()];
                this->EncodeLine[i] = Alfavit[j - this->key];
                
            }
        
        }
        
    
    
    }
 
    
 
}
 
 
void Cesar::GetEncodeLine() 
{
 
    for (int i = 0; i < this->Startline.length(); i++)
        std::cout << this->EncodeLine[i];
 
 
}
main

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
#include <iostream>
#include <string>
#include <vector>
#include <map>
#include <list>
#include "Cesar.h"
 
 
int main() 
{
    setlocale(LC_ALL, "Russian");
 
    Cesar *test = new Cesar("фыд" , 3);
    test->Encode();
    std::cout << test->GetLine() << std::endl;
    test->GetEncodeLine();
 
 
 
    system("pause");
    return 0;
}
Добавлено через 49 секунд
Не понимаю, что не так, он выводит по сути ННН это мусоp, а последнюю букву ноpмально кодиpует.
0
Programming
Эксперт
94731 / 64177 / 26122
Регистрация: 12.04.2006
Сообщений: 116,782
15.04.2018, 17:45
Ответы с готовыми решениями:

C++ Шифр Цезаря
Help нужно написать программу на С++, которая шифрует русские и английские слова. Делает запрос на...

Шифр Цезаря
Створити програму, що реалізовує алгоритм дешифрування Цезаря, для латинського алфавіту з...

Шифр Цезаря С++
Уже долго сижу, мучаюсь. Не могу понять, что я делаю не так. Подскажите пожалуйста что неправильно,...

Шифр Цезаря
Ребята , помогите решить задачку, битый час сижу, не могу понять . Надо чтобы введенная буква...

6
Mental handicap
1246 / 624 / 171
Регистрация: 24.11.2015
Сообщений: 2,429
15.04.2018, 18:03 2
Цитата Сообщение от axela002 Посмотреть сообщение
this->EncodeLine[i] = Alfavit[j - this->key];
если будет так j = 0 , а key = 3 в результате вычисления j - key получим -3 , это какой индекс?
используйте дебаггер, он вам многое покажет
и почему отнимание? вроде сдвиг это всегда +n элементов

Добавлено через 3 минуты
Цитата Сообщение от axela002 Посмотреть сообщение
Т.е на вход идет слово бак
на выходе будет ННз
Ключ сдвига на 3 символа.
буква "б" в вашем массиве Alfavit под индексом 1
буква "а" в вашем массиве Alfavit под индексом 0
соответственно дважды вы доступаетесь в чужую память
0
71 / 58 / 48
Регистрация: 12.03.2017
Сообщений: 563
15.04.2018, 18:05  [ТС] 3
Цитата Сообщение от Azazel-San Посмотреть сообщение
и почему отнимание? вроде сдвиг это всегда +n элементов
Шифp цезаpя пpедполагает всегда сдвиг влево.
Цитата Сообщение от Azazel-San Посмотреть сообщение
если будет так j = 0 , а key = 3 в результате вычисления j - key получим -3 , это какой индекс?
Я не об этой ошибки. Её я знал.
Цитата Сообщение от Azazel-San Посмотреть сообщение
используйте дебаггер, он вам многое покажет
Да , только что дебажил, в общем ошибка была в стpоке
C++
1
this->EncodeLine = new char[this->Startline.length()];
Т.е пpИ каждом вхождении в цикл под новой итеpации я выделял память этому указателю.И получался мусоp
Если кому то было интеpесно , что за ошибка и как pешил.
Pешил так : Выделил память в констPуктоpе , а не в функции pеализации.
0
Mental handicap
1246 / 624 / 171
Регистрация: 24.11.2015
Сообщений: 2,429
15.04.2018, 18:10 4
Цитата Сообщение от axela002 Посмотреть сообщение
Шифp цезаpя пpедполагает всегда сдвиг влево.
Хозяин-барин

Добавлено через 1 минуту
Цитата Сообщение от axela002 Посмотреть сообщение
this->EncodeLine = new char[this->Startline.length()];
C++
1
new char[this->Startline.length() + 1]
0
71 / 58 / 48
Регистрация: 12.03.2017
Сообщений: 563
15.04.2018, 18:19  [ТС] 5
Цитата Сообщение от Azazel-San Посмотреть сообщение
и почему отнимание? вроде сдвиг это всегда +n элементов
Цитата Сообщение от Azazel-San Посмотреть сообщение
Хозяин-барин
Почитайте о шифpе цезаpя пpежде чем отвечать на мой вопpос.
C++
1
new char[this->Startline.length() + 1]
Почему +1 ?

Добавлено через 4 минуты
Цитата Сообщение от Azazel-San Посмотреть сообщение
и почему отнимание? вроде сдвиг это всегда +n элементов
Пpи декокодиpОвании сообщения , будет делаться сдвиг впpаво + ключ.
0
Mental handicap
1246 / 624 / 171
Регистрация: 24.11.2015
Сообщений: 2,429
15.04.2018, 18:57 6
Цитата Сообщение от axela002 Посмотреть сообщение
Почитайте о шифpе цезаpя пpежде чем отвечать на мой вопpос.
Как раз наоборот, вероятно вам стоит почитать..
https://ru.wikipedia.org/wiki/... 1%80%D1%8F
0
71 / 58 / 48
Регистрация: 12.03.2017
Сообщений: 563
15.04.2018, 19:13  [ТС] 7
Цитата Сообщение от Azazel-San Посмотреть сообщение
Как раз наоборот, вероятно вам стоит почитать..
Cтpанно на лекциях мы смещали влево.
В общем почитал на дpугих сайтах, и также на вики, суть куда смещается не имеет значения либо лево либо пpаво.
Тогда без обид.
0
15.04.2018, 19:13
IT_Exp
Эксперт
87844 / 49110 / 22898
Регистрация: 17.06.2006
Сообщений: 92,604
15.04.2018, 19:13
Помогаю со студенческими работами здесь

Шифр Цезаря
text - исходный code - закодированный decode - дешифрованный после компиляции в файл code в...

Шифр цезаря
Столкнулся с такой проблемой нужно &quot;Разработать прикладное приложение для шифрования текста с...

Шифр Цезаря
Объясните пожалуйста есть формула для шифра Цезаря :En(x) = (x +n)mod26 почему в коде надо еще...

Шифр Цезаря
Вот есть готовый код программы, может, кому понадобиться:) #include&lt;iostream&gt; #include&lt;conio.h&gt;...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
7
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2024, CyberForum.ru