search

DolgushevEvgeny / change_substring

0 stars 0 forks source link

Замечания по программе Replace #1

Open alexey-malov opened 9 years ago

alexey-malov commented 9 years ago

Разбить main на подфункции с понятными именами и назначением

Не соблюдаются СоглашенияОИменованииФункций аТакжеПеременных

Что за переменная tmp зачем она?

Что за javascr_out

alexey-malov commented 9 years ago 
void Change_strings(FILE* input, FILE* output, string string_for_change, string string_to_change)

строки должны передаваться по константной ссылке: если они не изменяются функцией

alexey-malov commented 9 years ago 
    char line_for_subsidiary_implementation[500];
    fgets(line_for_subsidiary_implementation, 500, input);

А если в файле будет строка длиннее 500 символов?

alexey-malov commented 9 years ago

Что такое line_for_subsidiary_implementation (Строка для вспомогательного выполнения)?

alexey-malov commented 9 years ago

Если у нас в файле будет большая строка (10 миллионов символов a) И надо будет заменить a на aa У Вас квадратичная сложность на одних только сдвигах.

alexey-malov commented 9 years ago

Программа не работает: в файле 12312312345 не заменяет подстроку 1231234 на abcabcde

воспроизвести проблему в тестах исправить

alexey-malov commented 9 years ago

Для тестирования нормальных ситуаций используйте данные, приближенные к реальности.

k=0,5 за исполнение, за срок: 0,9