betslus1 / bestbot

конкурс ТИ
Apache License 2.0
21 stars 5 forks source link

Фидбек по конкурсной работе от проверяющих #1

Open b1ng0o opened 2 years ago

b1ng0o commented 2 years ago

1) Чистый JS -- очень смелое решение для робота, который может потерять деньги :) 2) Наличие секретного метода die внутри logger'а смущает. 3) хранение истории в БД -- большой плюс 4) магические константы 5) отсутствие тестов

betslus1 commented 2 years ago

1) Чистый JS -- очень смелое решение для робота, который может потерять деньги :)

2) Наличие секретного метода die внутри logger'а смущает.

3) хранение истории в БД -- большой плюс

4) магические константы

5) отсутствие тестов

1) А чем чистый JS так плох в контексте фондового рынка? Отсутствием строгой типизации? Потеряет или нет робот деньги зависит от алгоритма ) У меня тысячи форков вроде норм работает на чистом JS и весьма быстро)

2) согласен, все придумывалось на лету, не во всем оптимальная структура интерфейсов. 3) согласен 4) согласен, обычно глобалс не использую, тут решил попробовать и пожалел об этом 5) согласен, как то исторически с ними не сложилось ) все время надо написать много кода, и не остаётся времени на тесты)

b1ng0o commented 2 years ago

Отдельно отметили большим плюсом: классный Readme наличие различных интерфейсов для взаимодействие Ну и конечно же игру )