- 5,018
- 47
- 783
hohserg написал(а):Больше стремись повторно использовать код вместо копипасты.Maxik001 написал(а):Но мне интересно послушать, что бы ты сделал "лучше"
Прости, не понял ?
hohserg написал(а):Больше стремись повторно использовать код вместо копипасты.Maxik001 написал(а):Но мне интересно послушать, что бы ты сделал "лучше"
У меня горит, когда люди не придерживаются какого-нибудь кодстайла. Желания читать поплывший код нет. Не стоит пробел там, где он должен быть - сразу видно, писал аутист (не в обиду, но это так). Табуляция вместо пробелов - аналогично. Про переносы вообще молчу.Maxik001 написал(а):Каждый по своему понимает значение "ужасный код". Ты слишком обожествляешь себя.
Кодстайл.Maxik001 написал(а):Но мне интересно послушать, что бы ты сделал "лучше"
Названия методов... setRegistr... Установить регистр. Что? Логичнее назвать registerItem / registerRenderer.
Не ко мне претензии, я по учебнику делал гайд.Если бы там было написано "вот методы, но их необязательно писать, если вы не собираетесь их использовать". Я бы их не стал брать.Методы preInit, init, postInit - не обязательные.
С уверенностью могу сказать это.Maxik001 написал(а):Ты же не можешь сказать, что я делаю совсем игнорируя любой кодестайл...
Во-первых, ты его не форматируешь.Maxik001 написал(а):Но по форматирует каждый так, как ему удобнее читать.
Dahaka написал(а):С уверенностью могу сказать это.Maxik001 написал(а):Ты же не можешь сказать, что я делаю совсем игнорируя любой кодестайл...
Во-первых, ты его не форматируешь.Maxik001 написал(а):Но по форматирует каждый так, как ему удобнее читать.
Во-вторых, если для себя, то пофиг. Если выкладываешь в опен сурс или делаешь видео урок по этому коду, изволь сделать нормально. А если бы ты работал в команде, то с таким отношением сразу пошел бы в одно место.
В-третьих, в чем проблема сделать нормально? В любой IDE есть авто форматтер. Не способен ручками, она сделает все за тебя. Самому же приятнее будет смотреть на красиво отформатированный код.
Код нужно стараться использовать повторно. Будет легче изменять функционал в будущем.Maxik001 написал(а):Прости, не понял ?
В первом видео не критично.Dahaka написал(а):кодстайла
hohserg написал(а):ИМХО, лучше бы ты записывал уроки по коду из основного учебника
Да у тебя даже игра не запустится с этим кодом. Как ты думаешь, как инициализируется поле proxy? Само по себе?Maxik001 написал(а):Блин, я знаю что такое аннотация) Я знаю что ее надо ставить над методом или классом. К какому методу мне установить эту SidedProxy? (кстати она была прямо над @EventHandler, я хз почему она съехала)
Тут важно делать либо так, либо так. А не абы как.Maxik001 написал(а):Я не думаю что это плохо читается
Dahaka написал(а):А подчеркнул я там из-за того, что нет пробела.
Thunder написал(а):!world.isRemote это проверка что действие происходит на серверной стороне, а ты всегда говоришь что это проверка что мир клиентский. На будущее)