search

VIZ-Blockchain / viz-cpp-node

VIZ C++ node
Other
14 stars 5 forks source link

Remove unneeded hardfork switches from the code #19

Closed bitphage closed 5 years ago

bitphage commented 6 years ago

Because we're launching a new chain, we don't need old steemit legacy code. So we can make a code cleanup and remove old code to improve readability.

antonblockchain commented 6 years ago

Читка старого кода. Это ошибка новичка. 1) Теряется обратная совместимость со стимом. 2) Добавляется неоттетированный код. 3) Теряется время разработчика.

Голос тоже занимался чисткой кода. В результате код плохо совместим со стимом.

On1x commented 6 years ago

@AntonBliznets имеется ввиду развилка хардфорков. Нет никакого смысла держать файл полный ифов, которые НИКОГДА не сработают.

antonblockchain commented 6 years ago

Я понимаю что имеется в виду. Это для совместимости. Терять совместимость со стимит дорого. Пример голос: рефакторинг кода сделал импорт из стимита дорогим. Как надо было - делать потомки классов в отдельных (новых файлах) и переопределять поведение в потомках. Таким образом весь ВАШ код будет в отлеьных классах и отдельных файлах. Обновления со стимом будут надежными и быстрыми. Ваш код локализован в новых объектах. То что вы изменили видно и полностью описано.

Совместимость ценнее красоты кода. IDE маркирует все #IFDEF .... как комментарии.

On1x commented 5 years ago

Done in 2018.