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

@axela002 · Регистрация: 12.03.2017

Студворк — интернет-сервис помощи студентам

Добрый день, есть класс Цезарь. В нем есть функция Кодирование информации с помощью шифра цезаря. Её реализация ниже.
проблема в том ,что когда я компилирую, он выводит НННННН и последнию букву выводит нормальную закодированную.
Т.е на вход идет слово бак
на выходе будет ННз
Ключ сдвига на 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ует.

@Azazel-San · 15.04.2018, 18:03

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

this->EncodeLine[i] = Alfavit[j - this->key];

если будет так j = 0 , а key = 3 в результате вычисления j - key получим -3 , это какой индекс?
используйте дебаггер, он вам многое покажет
и почему отнимание? вроде сдвиг это всегда +n элементов

Добавлено через 3 минуты

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

Т.е на вход идет слово бак
на выходе будет ННз
Ключ сдвига на 3 символа.

буква "б" в вашем массиве Alfavit под индексом 1
буква "а" в вашем массиве Alfavit под индексом 0
соответственно дважды вы доступаетесь в чужую память

@axela002 · 15.04.2018, 18:05 **[ТС]**

Сообщение от 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еализации.

@Azazel-San · 15.04.2018, 18:10

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

Шифp цезаpя пpедполагает всегда сдвиг влево.

Хозяин-барин

Добавлено через 1 минуту

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

this->EncodeLine = new char[this->Startline.length()];

C++
1
new char[this->Startline.length() + 1]

@axela002 · 15.04.2018, 18:19 **[ТС]**

Сообщение от 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аво + ключ.

@Azazel-San · 15.04.2018, 18:57

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

Почитайте о шифpе цезаpя пpежде чем отвечать на мой вопpос.

Как раз наоборот, вероятно вам стоит почитать..
https://ru.wikipedia.org/wiki/... 1%80%D1%8F

@axela002 · 15.04.2018, 19:13 **[ТС]**

Сообщение от Azazel-San

Как раз наоборот, вероятно вам стоит почитать..

Cтpанно на лекциях мы смещали влево.
В общем почитал на дpугих сайтах, и также на вики, суть куда смещается не имеет значения либо лево либо пpаво.
Тогда без обид.

Новые блоги и статьи Все статьи Все блоги /
делаю науч статью по влиянию грибов на сукцессию anaschu 13.03.2026 прикрепляю статью	SDL3 для Desktop (MinGW): Создаём пустое окно с нуля для 2D-графики на SDL3, Си и C++ 8Observer8 10.03.2026 Содержание блога Финальные проекты на Си и на C++: hello-sdl3-c. zip hello-sdl3-cpp. zip Результат:	Установка CMake и MinGW 13.1 для сборки С и C++ приложений из консоли и из Qt Creator в EXE 8Observer8 10.03.2026 Содержание блога MinGW - это коллекция инструментов для сборки приложений в EXE. CMake - это система сборки приложений. Здесь описаны базовые шаги для старта программирования с помощью CMake и. . .	Как дизайн сайта влияет на конверсию: 7 решений, которые реально повышают заявки Neotwalker 08.03.2026 Многие до сих пор воспринимают дизайн сайта как “красивую оболочку”. На практике всё иначе: дизайн напрямую влияет на то, оставит человек заявку или уйдёт через несколько секунд. Даже если у вас. . .
Модульная разработка через nuget packages DevAlt 07.03.2026 Сложившийся в . Net-среде способ разработки чаще всего предполагает монорепозиторий в котором находятся все исходники. При создании нового решения, мы просто добавляем нужные проекты и имеем. . .	Модульный подход на примере F# DevAlt 06.03.2026 В блоге дяди Боба наткнулся на такое определение: В этой книге («Подход, основанный на вариантах использования») Ивар утверждает, что архитектура программного обеспечения — это структуры,. . .	Управление камерой с помощью скрипта OrbitControls.js на Three.js: Вращение, зум и панорамирование 8Observer8 05.03.2026 Содержание блога Финальная демка в браузере работает на Desktop и мобильных браузерах. Итоговый код: orbit-controls-threejs-js. zip. Сканируйте QR-код на мобильном. Вращайте камеру одним пальцем,. . .	SDL3 для Web (WebAssembly): Синхронизация спрайтов SDL3 и тел Box2D 8Observer8 04.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-sync-physics-sprites-sdl3-c. zip На первой гифке отладочные линии отключены, а на второй включены:. . .