Замечания по программе RemoveExtraSpaces - Githubissues

MaxMartST / OOP

Курс по объектно ориентированному программированию

1 stars 0 forks source link

Замечания по программе RemoveExtraSpaces #6

Open alexey-malov opened 4 years ago

alexey-malov commented 4 years ago

https://github.com/MaxMartST/OOP/blob/1cba2b3478b5948d1a508870308e285f80cae503/Lw%202/string_remove_extra_spaces/RemoveExtraSpaces.cpp#L29-L45

[x] При ошибке чтения из потока, этот цикл зависнет
[x] Вывод программы неудобен - ввод новой строки начинается в конце выведенной:

alexey-malov commented 4 years ago

https://github.com/MaxMartST/OOP/blob/1cba2b3478b5948d1a508870308e285f80cae503/Lw%202/string_remove_extra_spaces/RemoveExtraSpaces.cpp#L12-L14

[x] Этот код будет работать некорректно на 64 битных платформах

alexey-malov commented 4 years ago

https://github.com/MaxMartST/OOP/blob/1cba2b3478b5948d1a508870308e285f80cae503/Lw%202/string_remove_extra_spaces/RemoveExtraSpaces.cpp#L21-L25

[x] Найдите способ избежать удаления символа в начале строки (ведь при этом придётся сдвинуть все последующие символы)

alexey-malov commented 4 years ago

https://github.com/MaxMartST/OOP/blob/1cba2b3478b5948d1a508870308e285f80cae503/Lw%202/string_remove_extra_spaces/RemoveExtraSpaces.cpp#L10-L12

[x] Здесь происходит двойное копирование символов строки. Первый раз - когда мы создаём копию. Второй раз - когда выполняем unique_copy

alexey-malov commented 4 years ago

https://github.com/MaxMartST/OOP/blob/1cba2b3478b5948d1a508870308e285f80cae503/Lw%202/string_remove_extra_spaces/RemoveExtraSpaces.cpp#L14-L25

[x] Переменная length не хранит длину
[x] После удаления символа в конце строки, она может стать пустой (если до этого содержала 1 символ). Здесь вы без каких либо проверок обращаетесь к символу в 0 позиции

alexey-malov commented 4 years ago

https://github.com/MaxMartST/OOP/blob/1cba2b3478b5948d1a508870308e285f80cae503/Lw%202/tests_for_the_string/unit_tests.cpp#L12

[x] Для объявления пустой строки достаточно объявить переменную типа string. Тогда сработает конструктор по умолчанию, а не инициализирующий конструктор

alexey-malov commented 4 years ago

https://github.com/MaxMartST/OOP/blob/1cba2b3478b5948d1a508870308e285f80cae503/Lw%202/tests_for_the_string/unit_tests.cpp#L16-L20

[x] Неудачное имя str

alexey-malov commented 4 years ago

https://github.com/MaxMartST/OOP/blob/1cba2b3478b5948d1a508870308e285f80cae503/Lw%202/tests_for_the_string/unit_tests.cpp#L22-L39

[x] Эти тесты относятся к другой функции. Надо вынести их в отдельную секцию
[x] Имена тестов непонятные. Что они проверяют?