Советы по ускорению работы кода + оценка самого кода

SlavaSokolov · Регистрация: 14.04.2016

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

Вчера вечером сел написать 3 консольные программки для работы с шаблонами размножения текста:
1.Выборка групп синонимов(создание словаря)
2.Корректировка словаря(поиск и удаления эквивалетных строк, а также все символы в нижн.регистр).
3.Объединение нескольких словарей.

Первую смысла выкладывать нет, так как всё очень просто.

В общем получился такой корректор(программа 2).

"Корректор"

Кликните здесь для просмотра всего текста

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
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
//Корректор
#include <iostream>
#include <Windows.h>
#include <clocale>
#include <string>
#include <fstream>
#include <vector>
 
std::string corrector(std::string);
std::string no_copy(std::string);
 
void fill_vector(std::vector<std::string> *, std::string *);
void correct_vector(std::vector<std::string> *);
 
void str_to_vector(std::vector<std::string> *, std::string);
 
bool str_equals(std::string, std::string);
 
//Функция для проверки результ. раб. с вектором
void inf_of_vector(std::vector<std::string> *, int);
 
int main()
{
    setlocale(LC_CTYPE, "RUS");
 
    SetConsoleCP(1251);
    SetConsoleOutputCP(1251);
 
    std::string file_name;
 
    std::cout << "Enter file name: ";
    std::cin >> file_name;
 
    while (!std::ifstream(file_name).is_open())
    {
        std::cout << "File not found! Enter file name: ";
        std::cin >> file_name;
    }
 
    std::ifstream input_file(file_name);
 
    std::string line;
    std::string end_text;
 
    while (getline(input_file, line))
    {
        line = corrector(line);
 
        if (!line.empty())
            end_text = end_text + line + '\n';
    }
 
    input_file.close();
 
    end_text = no_copy(end_text);
 
    std::ofstream output_file(file_name);
    output_file << end_text;
    output_file.close();
 
    std::cin.ignore();
    std::cout << "Click Enter for exit";
    std::cin.get();
}
 
 
std::string corrector(std::string line)
{
    bool flag = false;
 
    for (int i = 0; i < (int)(line.size()); i++)
    {
        line[i] = (char)(tolower(line[i]));
 
        if (line[i] == '|')
            flag = true;
    }
 
    if (!flag)
        line.clear();
 
    return line;
}
 
//Очистка текста от эквивалетных строк
std::string no_copy(std::string text)
{
    std::vector<std::string> *vector = new std::vector<std::string>;
 
    fill_vector(vector, &text);
    inf_of_vector(vector, 0);
 
    correct_vector(vector);
    inf_of_vector(vector, 1);
 
    text.clear();
 
    for (int i = 0; i < (int)(vector->size()); i++)
    {
        if (!(*vector)[i].empty())
            text = text + (*vector)[i] + '\n';
    }
 
    delete vector;
 
    return text;
}
 
 
void fill_vector(std::vector<std::string> *vector, std::string *text)
{
    std::string line;
 
    for (int i = 0; i < (int)(text->size()); i++)
    {
        if ((*text)[i] == '\n')
        {
            vector->push_back(line);
            line.clear();
        }
        else
        {
            line += (*text)[i];
        }
    }
}
 
 
void inf_of_vector(std::vector<std::string> *vector, int par)
{
    for (int i = 0; i < (int)(vector->size()); i++)
    {
        std::cout << (*vector)[i] << std::endl;
        if (i % 25 == 0 && par != 0)
        {
            std::cin.get();
        }
    }
 
    std::cout << "Vector length = " << vector->size() << std::endl;
    std::cin.get();
}
 
 
//Очистка вектора от эквивалетных строк
void correct_vector(std::vector<std::string> *vector)
{
 
    for (int x = 0; x < (int)(vector->size()); x++)
    {
        if (x == (int)(vector->size()))
            break;
 
        for (int i = x + 1; i < (int)(vector->size()); i++)
        {
            if (!(*vector)[x].empty() && !(*vector)[i].empty()) {
 
                if (str_equals((*vector)[x], (*vector)[i]))
                {
                    if ((int)(*vector)[i].size() < (int)(*vector)[x].size())
                        (*vector)[i].clear();
                    else
                        (*vector)[x].clear();
                }
            }
 
        }
    }
}
 
 
bool str_equals(std::string line_1, std::string line_2)
{
    std::cout << "Line1: " << line_1 << std::endl;
    std::cout << "Line2: " << line_2 << std::endl;
 
    bool equal = false;
 
    if (line_1 == line_2)
        return true;
 
    std::vector<std::string> *vector_1 = new std::vector<std::string>;
    std::vector<std::string> *vector_2 = new std::vector<std::string>;
 
    str_to_vector(vector_1, line_1);
    str_to_vector(vector_2, line_2);
 
    int count;
    int c = 0;
 
    if ((int)(vector_1->size()) > (int)(vector_2->size()))
        count = (int)(vector_2->size());
    else
        count = (int)(vector_1->size());
 
    for (int x = 0; x < (int)(vector_1->size()); x++)
    {
        for (int i = 0; i < (int)(vector_2->size()); i++)
        {
            if ((*vector_1)[x] == (*vector_2)[i])
                c++;
        }
    }
 
    if (c == count)
        equal = true;
 
 
    delete vector_1;
    delete vector_2;
 
    return equal;
}
 
 
void str_to_vector(std::vector<std::string> *vector, std::string line)
{
    std::string word;
 
    for (int i = 0; i < (int)(line.size()); i++)
    {
        if (line[i] != '|')
            word += line[i];
        else
        {
            vector->push_back(word);
            word.clear();
        }
    }
}

В основу алгоритма объединения словарей запихал прошлый код с некоторыми изменениями. Там нет новых функций, так что закинул только main.

Кликните здесь для просмотра всего текста

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
//Объединение словарей
#include <iostream>
#include <Windows.h>
#include <clocale>
#include <string>
#include <fstream>
#include <vector>
 
std::string corrector(std::string);
std::string no_copy(std::string);
 
void fill_vector(std::vector<std::string> *, std::string *);
void correct_vector(std::vector<std::string> *);
 
void str_to_vector(std::vector<std::string> *, std::string);
 
bool str_equals(std::string, std::string);
 
//Функция для проверки результ. раб. с вектором
void inf_of_vector(std::vector<std::string> *, int);
 
int main()
{
    setlocale(LC_CTYPE, "RUS");
 
    SetConsoleCP(1251);
    SetConsoleOutputCP(1251);
 
    std::string file_name_1;
    std::string file_name_2;
 
    std::cout << "Enter file name: ";
    std::cin >> file_name_1;
 
    while (!std::ifstream(file_name_1).is_open())
    {
 
        std::cout << "File not found! Enter file name: ";
        std::cin >> file_name_1;
    }
 
    std::cout << "Enter file name: ";
    std::cin >> file_name_2;
 
    while (!std::ifstream(file_name_2).is_open())
    {
        std::cout << "File not found! Enter file name: ";
        std::cin >> file_name_2;
    }
 
    std::ifstream input_file_1(file_name_1);
    std::ifstream input_file_2(file_name_2);
 
    std::string line;
    std::string end_text;
 
    while (getline(input_file_1, line))
    {
        end_text = end_text + line + '\n';
    }
 
    while (getline(input_file_2, line))
    {
        end_text = end_text + line + '\n';
    }
 
 
    input_file_1.close();
    input_file_2.close();
 
    end_text = no_copy(end_text);
 
    std::ofstream output_file("union.txt");
    output_file << end_text;
    output_file.close();
 
    std::cin.ignore();
    std::cout << "Click Enter for exit";
    std::cin.get();
}

1.Если кто осилил мой говнокод и понял что там, то можно ли как-то ускорить работу корректора(МБ сменить алгоритм)? Сам заметил потенциальное увеличение скорости только в 160-163 стрках. Элементы вектора можно удалить и его длина будет сокращаться? Я правильно понимаю?

2.Если у кого есть что сказать о низком качестве кода, то говорите).

3.+Возникла проблема с пониманием написанного, когда объём уходит за 1000 строк. Пишу небольшой проектик там уже более 1000 строк кода и понимаю, что я не могу понять на следующий день как и что в нём работает(Хотя оно работает и правильно). Это нормально? Тут только опыт поможет?

@_stanislav · 29.10.2017, 12:33

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

что я не могу понять на следующий день как и что в нём работает

на следующий день не нормально, через хотя бы пол года нормально. если не понятно раздели код на функции или на классы.

SlavaSokolov · 29.10.2017, 13:17 **[ТС]**

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

на следующий день не нормально, через хотя бы пол года нормально. если не понятно раздели код на функции или на классы.

Немного преувеличил. Как работает то помню, но что и где находится вот тут сложно ориентироваться. Там 20 объектов и я в них просто запутываюсь из-за идиотских идентификаторов.

@_stanislav · 29.10.2017, 14:45

SlavaSokolov, если путаешься значит нужна декомпозиция, это точно.

Добавлено через 43 секунды
разделить на файлы, классы, на библиотеки наконец.

Croessmah · 29.10.2017, 14:49

C++
1
2
    std::vector<std::string> *vector_1 = new std::vector<std::string>;
    std::vector<std::string> *vector_2 = new std::vector<std::string>;

И зачем?

SlavaSokolov · 29.10.2017, 14:57 **[ТС]**

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

И зачем?

Не знаю)... Ещё будут какие-нибудь замечания?

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

разделить на файлы, классы, на библиотеки наконец.

Сложно с слабым английским придумывать нормальные названия функций, что б потом понимать зачем они)

Croessmah · 29.10.2017, 15:03

SlavaSokolov, убрать указатели, переделать на код с ссылками.

SlavaSokolov · 29.10.2017, 15:10 **[ТС]**

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

убрать указатели, переделать на код с ссылками

Что изменится? Легче читать будет без операций разыменования?

@_stanislav · 29.10.2017, 15:24

SlavaSokolov, как сказал один умный дядька:
Для функций допускающих в качестве своего аргумента "отсутствующий объект" (представленный значением nullptr), следует использовать указатели (и не забывать о проверке на nullptr).
В противном случае следует использовать ссылку.
Проще код меньше ошибок

Добавлено через 4 минуты
SlavaSokolov, может Корректор в класс инкапсулировать?

SlavaSokolov · 29.10.2017, 15:38 **[ТС]**

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

может Корректор в класс инкапсулировать?

Рано, исходники то я оставлю. Конечный "проект" в QT всё равно переделывать с нуля. Я там такой дичи натворил, что с огромным трудом понимаю что где и как. Блин, там работает всё как задумано, но мне очень трудно двигать дальше.

Да и что объединение, что корректор очень медленно работают. Файл на 710 строк "корректируется" около 6 - 10 минут.

@_stanislav · 29.10.2017, 15:43

SlavaSokolov, в логику работы чужого кода никто с радостью вникать не будет, тем более большого, тем более кудрявого. темы на оценку большого кода часто без ответа оставляют.

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

Не по теме:

SlavaSokolov, тем более ты спасибо не тыкаешь:jokingly:

SlavaSokolov · 29.10.2017, 15:46 **[ТС]**

Не по теме:

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

SlavaSokolov, тем более ты спасибо не тыкаешь

Ещё б знать, как это сделать...

@_stanislav · 29.10.2017, 15:49

Не по теме:

SlavaSokolov, не знаешь как сказать спасибо, а нарушение уже схватил, :D

SlavaSokolov · 29.10.2017, 16:56 **[ТС]**

Не по теме:

_stanislav, я б тебе поведал одну трушную историю об одной очень неудачной шутке в адрес администрации...

Добавлено через 1 час 4 минуты
Во славу Croessmah и ссылок!

Кликните здесь для просмотра всего текста

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
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
//Корректор
#include <iostream>
#include <Windows.h>
#include <clocale>
#include <string>
#include <fstream>
#include <vector>
 
 
std::string corrector(std::string &);
std::string no_copy(std::string &);
 
void fill_vector(std::vector<std::string> &, std::string &);
void correct_vector(std::vector<std::string> &);
 
void str_to_vector(std::vector<std::string> &, std::string);
 
bool str_equals(std::string, std::string);
 
//Функция для проверки результ. раб. с вектором
void inf_of_vector(std::vector<std::string> &, int);
 
int main()
{
    setlocale(LC_CTYPE, "RUS");
 
    SetConsoleCP(1251);
    SetConsoleOutputCP(1251);
 
    std::string file_name;
 
    std::cout << "Enter file name: ";
    std::cin >> file_name;
 
    while (!std::ifstream(file_name).is_open())
    {
        std::cout << "File not found! Enter file name: ";
        std::cin >> file_name;
    }
 
    std::ifstream input_file(file_name);
 
    std::string line;
    std::string end_text;
 
    while (getline(input_file, line))
    {
        line = corrector(line);
 
        if (!line.empty())
            end_text = end_text + line + '\n';
    }
 
    input_file.close();
 
    end_text = no_copy(end_text);
 
    std::ofstream output_file(file_name);
    output_file << end_text;
    output_file.close();
 
    std::cin.ignore();
    std::cout << "Click Enter for exit";
    std::cin.get();
}
 
 
std::string corrector(std::string &line)
{
    bool flag = false;
 
    for (int i = 0; i < (int)(line.size()); i++)
    {
        line[i] = (char)(tolower(line[i]));
 
        if (line[i] == '|')
            flag = true;
    }
 
    if (!flag)
        line.clear();
 
    return line;
}
 
//Очистка текста от эквивалетных строк
std::string no_copy(std::string &text)
{
    std::vector<std::string> vector;
 
    fill_vector(vector, text);
 
    correct_vector(vector);
 
    text.clear();
 
    for (int i = 0; i < (int)(vector.size()); i++)
    {
        if (!vector[i].empty())
            text = text + vector[i] + '\n';
    }
 
    return text;
}
 
 
void fill_vector(std::vector<std::string> &vector, std::string &text)
{
    std::string line;
 
    for (int i = 0; i < (int)(text.size()); i++)
    {
        if (text[i] == '\n')
        {
            vector.push_back(line);
            line.clear();
        }
        else
        {
            line += text[i];
        }
    }
}
 
 
void inf_of_vector(std::vector<std::string> &vector, int par)
{
    for (int i = 0; i < (int)(vector.size()); i++)
    {
        std::cout << vector[i] << std::endl;
        if (i % 25 == 0 && par != 0)
        {
            std::cin.get();
        }
    }
 
    std::cout << "Vector length = " << vector.size() << std::endl;
    std::cin.get();
}
 
 
//Очистка вектора от эквивалетных строк
void correct_vector(std::vector<std::string> &vector)
{
 
    for (int x = 0; x < (int)(vector.size()); x++)
    {
        if (x == (int)(vector.size()))
            break;
 
        for (int i = x + 1; i < (int)(vector.size()); i++)
        {
 
            if (str_equals(vector[x], vector[i]))
            {
                if ((int)(vector[i].size()) < (int)(vector[x].size()))
                {
                    std::vector<std::string>::iterator it;
                    it = vector.begin() + i;
                    vector.erase(it);
                }
                else
                {
                    std::vector<std::string>::iterator it;
                    it = vector.begin() + x;
                    vector.erase(it);
                }
            }
 
        }
    }
}
 
 
bool str_equals(std::string line_1, std::string line_2)
{
    //std::cout << "Line1: " << line_1 << std::endl;
    //std::cout << "Line2: " << line_2 << std::endl;
 
    bool equal = false;
 
    if (line_1 == line_2)
        return true;
 
    std::vector<std::string> vector_1;
    std::vector<std::string> vector_2;
 
    str_to_vector(vector_1, line_1);
    str_to_vector(vector_2, line_2);
 
    int count;
    int c = 0;
 
    if ((int)(vector_1.size()) > (int)(vector_2.size()))
        count = (int)(vector_2.size());
    else
        count = (int)(vector_1.size());
 
    for (int x = 0; x < (int)(vector_1.size()); x++)
    {
        for (int i = 0; i < (int)(vector_2.size()); i++)
        {
            if (vector_1[x] == vector_2[i])
                c++;
        }
    }
 
    if (c == count)
        equal = true;
 
    return equal;
}
 
 
void str_to_vector(std::vector<std::string> &vector, std::string line)
{
    std::string word;
 
    for (int i = 0; i < (int)(line.size()); i++)
    {
        if (line[i] != '|')
            word += line[i];
        else
        {
            vector.push_back(word);
            word.clear();
        }
    }
}

Проблема со скорость сортировки решена))). Какой-то идиот думал, что вывод в консоль не замедляет работу. Убрал из функции сравнения вывод строк и сортировка выполняется за 9 секунд вместо 239.

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Обработчик клика мыши в браузере ПК и касания экрана в браузере на мобильном устройстве 8Observer8 02.02.2026 Содержание блога Для начала пошагово создадим рабочий пример для подготовки к экспериментам в браузере ПК и в браузере мобильного устройства. Потом напишем обработчик клика мыши и обработчик. . .	Философия технологии iceja 01.02.2026 На мой взгляд у человека в технических проектах остается роль генерального директора. Все остальное нейронки делают уже лучше человека. Они не могут нести предпринимательские риски, не могут. . .	SDL3 для Web (WebAssembly): Вывод текста со шрифтом TTF с помощью SDL3_ttf 8Observer8 01.02.2026 Содержание блога В этой пошаговой инструкции создадим с нуля веб-приложение, которое выводит текст в окне браузера. Запустим на Android на локальном сервере. Загрузим Release на бесплатный. . .	SDL3 для Web (WebAssembly): Сборка C/C++ проекта из консоли 8Observer8 30.01.2026 Содержание блога Если вы откроете примеры для начинающих на официальном репозитории SDL3 в папке: examples, то вы увидите, что все примеры используют следующие четыре обязательные функции, а. . .
SDL3 для Web (WebAssembly): Установка Emscripten SDK (emsdk) и CMake для сборки C и C++ приложений в Wasm 8Observer8 30.01.2026 Содержание блога Для того чтобы скачать Emscripten SDK (emsdk) необходимо сначало скачать и уставить Git: Install for Windows. Следуйте стандартной процедуре установки Git через установщик. . . .	SDL3 для Android: Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 29.01.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами. Версия v3 была полностью переписана на Си, в. . .	Инструменты COM: Сохранение данный из VARIANT в файл и загрузка из файла в VARIANT bedvit 28.01.2026 Сохранение базовых типов COM и массивов (одномерных или двухмерных) любой вложенности (деревья) в файл, с возможностью выбора алгоритмов сжатия и шифрования. Часть библиотеки BedvitCOM Использованы. . .	SDL3 для Android: Загрузка PNG с альфа-каналом с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 28.01.2026 Содержание блога SDL3 имеет собственные средства для загрузки и отображения PNG-файлов с альфа-каналом и базовой работы с ними. В этой инструкции используется функция SDL_LoadPNG(), которая. . .

@_stanislav "C with Classes" 2022 / 1404 / 523 Регистрация: 16.08.2014 Сообщений: 5,885 Записей в блоге: 1
	29.10.2017, 14:45
	SlavaSokolov, если путаешься значит нужна декомпозиция, это точно. Добавлено через 43 секунды разделить на файлы, классы, на библиотеки наконец. 0

Croessmah Неэпический 18146 / 10730 / 2066 Регистрация: 27.09.2012 Сообщений: 27,029 Записей в блоге: 1
	29.10.2017, 15:03
	Сообщение было отмечено SlavaSokolov как решение Решение SlavaSokolov, убрать указатели, переделать на код с ссылками. 0

@_stanislav "C with Classes" 2022 / 1404 / 523 Регистрация: 16.08.2014 Сообщений: 5,885 Записей в блоге: 1
	29.10.2017, 15:24
	SlavaSokolov, как сказал один умный дядька: Для функций допускающих в качестве своего аргумента "отсутствующий объект" (представленный значением nullptr), следует использовать указатели (и не забывать о проверке на nullptr). В противном случае следует использовать ссылку. Проще код меньше ошибок Добавлено через 4 минуты SlavaSokolov, может Корректор в класс инкапсулировать? 0

@_stanislav "C with Classes" 2022 / 1404 / 523 Регистрация: 16.08.2014 Сообщений: 5,885 Записей в блоге: 1
	29.10.2017, 15:43
	SlavaSokolov, в логику работы чужого кода никто с радостью вникать не будет, тем более большого, тем более кудрявого. темы на оценку большого кода часто без ответа оставляют. Добавлено через 1 минуту Не по теме: SlavaSokolov, тем более ты спасибо не тыкаешь:jokingly: 0

@_stanislav
	29.10.2017, 15:49
	Не по теме: SlavaSokolov, не знаешь как сказать спасибо, а нарушение уже схватил, :D 0

Советы по ускорению работы кода + оценка самого кода

Решение