BrickRelic & Voltina
Voltina
Брик, ты когда-нибудь видел код, как будто заброшенные руины? Думаю, нам обоим это не нравится. Давай поговорим о том, как вернуть всё в порядок – будь то строки кода или каменные блоки.
BrickRelic
Я такое повидал немало. Код, как полуразрушенная башня, камень, обточенный ветрами до пыли. Главное – найти основную структуру, прежде чем начнешь разбираться с паутиной, будь то иерархия классов или перемычка. А потом – свежая стяжка: чистые тесты, рефакторинг, документация – чтобы будущие смотрители понимали, что нужно сохранить, а что можно снести. Какой конкретно руины у тебя на уме? Давай разберем её, строка за строкой.
Voltina
Выбери кусочек кода, репозиторий или файл, где комментариев больше, чем смысла. Покажи самые вопиющие примеры, а я укажу, что стоит оставить, а что можно смело удалять. Никакой воды, только чёткий список строк, которые нужно переписать или удалить. Готова приступить?
BrickRelic
Вот примерчик, который сам себя перекомментировал до такой степени, что это просто смех.
Voltina
Точно. Убери все эти пространные пояснения, оставь только самое важное. Получилось вот так:
```java
public Counter() { this.count = 0; }
public void increment() { this.count++; }
public int getCount() { return this.count; }
public static void main(String[] args) {
Counter c = new Counter();
for (int i = 0; i < 5; i++) c.increment();
System.out.println("Final count: " + c.getCount());
}
```
Теперь смысл понятен, код лаконичный, и каждый, кто его прочитает, сразу поймет, что делает каждая часть. Больше не нужно говорить: "Я уже говорил, что эта строка обнуляет count". Скажи, если захочешь, чтобы я придерживался этого стиля и в другом файле.
BrickRelic
Отлично, давай ещё один файл. Я отсеку всё лишнее, оставив только суть.
Voltina
Привет, вот тебе пример "UserService", он довольно многословный. Посмотри и сократи до самого необходимого, ладно?
BrickRelic
Я, конечно, не программист, но, кажется, ты меня куда-то запутала. Что вообще происходит? Поясни, пожалуйста, что за код.