А можете поревьювить?

Question

[yabivipil]

Решил написать свой велосипед (клиент) для телеграма. Там реализован еще не весь функционал, но хочется какой-то конструктивной критики. Можете помочь с этим?

ссылка на репу

Comments

[Стас Заварухин]

Ну в идеале бы видеть php7.2, так как сейчас 7.3 довольно геморно ставить, а так, если не считать функционал, то в принципе неплохо

[BoShurik]

Зачем разделение на билдеры и дто? По идеи их можно было бы объединить в один класс. С точки зрения академичности может и правильно, но вот пользоваться и поддерживать будет не очень удобно.

[sizzeR]

а собственно зачем?)

[yabivipil]

BoShurik, проблема в том, что у апи есть обязательные и необязательные параметры + дто-экшн, по моему мнению, должен быть иммутабельным, поэтому появились билдеры, которые решают помогают достичь этого

Answer 1

[grinat]

Даже код смотреть неохота, коммиты вида refactoring part 2 уже говорят обо всем. Нафига думать, зачем описывать имзенения, если можно просто делать: refactoring part 2, refactoring part 3, code part 1, code part 2, my mom love pizza