169 похожих чатов

Подскажите пожалуйста, какой "хороший тон" в ситуации, когда : есть класс с

разными методами (я пишу автотесты)
один из них мне не нравится или допустим я не понимаю как он работает или почему он работал когда-то
я не хочу от него избавляться, потому что думаю что возможно он мне будет полезен (неважно по каким причинам).

я пишу рядом метод с таким же названием добавляя какой-то символ или v2 , но понимаю что это не очень хорошо.

а как хорошо? или хорошо было бы просто нормально переписать?

4 ответов

11 просмотров

Может быть, использовать гит?

i think it's not okay- Автор вопроса
Rakhmonov
Может быть, использовать гит?

типа... видна история изменений? хороший вариант (наверное), но сейчас у меня в гите беспорядок.

если никто кроме тебя его использовать не будет и этот метод нигде не используется, то думаю норм закомментировать.

Правильно - написать тесты на текущую логику. Удалить код, который не нравится, написать код который нравится, проверить, что тесты зеленые. Но если у вас проблемы с гитом, то в общем-то всё это не важно.

Похожие вопросы

Обсуждают сегодня

30500 за редактор? )
Владимир
47
а через ESC-код ?
Alexey Kulakov
29
Чёт не понял, я ж правильной функцией воспользовался чтобы вывести отладочную информацию? но что-то она не ловится
notme
18
Добрый день! Скажите пожалуйста, а какие программы вы бы рекомендовали написать для того, чтобы научиться управлять памятью? Можно написать динамический массив, можно связный ...
Филипп
7
У меня есть функция где происходит это: write_bit(buffer, 1); write_bit(buffer, 0); write_bit(buffer, 1); write_bit(buffer, 1); write_bit(buffer, 1); w...
~
14
Недавно Google Project Zero нашёл багу в SQLite с помощью LLM, о чём достаточно было шумно в определённых интернетах, которые сопровождались рассказами, что скоро всех "ибешни...
Alex Sherbakov
5
Ребят в СИ можно реализовать ООП?
Николай
33
Как передать управляющий символ в открытую через CreateProcess консоль? Собсна, есть процедура: procedure TRedirectThread.WriteData(Data: OEMString); var Written: Cardinal;...
Serjone
6
https://github.com/erlang/otp/blob/OTP-27.1/lib/kernel/src/logger_h_common.erl#L174 https://github.com/erlang/otp/blob/OTP-27.1/lib/kernel/src/logger_olp.erl#L76 15 лет назад...
Maksim Lapshin
20
Всем привет! Имеется функция: function IsValidChar(ch: UTF8Char): Boolean; var i: Integer; ValidChars: AnsiString; begin ValidChars := 'abcdefghijklmnopqrstuvwxyzABCDE...
Евгений
44
Карта сайта