Last active
December 18, 2015 07:49
-
-
Save xdegtyarev/5749297 to your computer and use it in GitHub Desktop.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
1) private не надо ставить, существует дефолтный модификатор видимости) | |
2) использу google translate если не уверен как по-английски что-то пишется. или спроси меня. и попробуй http://lingualeo.ru/ | |
3) все типы умеют инициализироваться по умолчанию, не нужно писать Object o = null; Только если ты явно хочешь обнулить значение; | |
4) если ты не добавляешь какого либо функционала в get'еры/set'еры не испульзуй проперти, оставь поле, и если будет необходимо изменить поведение ебанешь проперти | |
5) любая конструкция if/else даже если состоит из 1 оператора должна быть написана со скобками if(){}else{} никаких if();else; аналогично для for/foreach | |
6) правильно название подбирай: UseAbility возвращает bool что вприницпе хуево и непонятно переименуй во что-то что из названия подсказывает что вернет bool например TryUseAbility() | |
7) (delegate(AbilityBase item) { return item.myType == abilityType; }); в контексте Comparer'oв вместо явных делегатов используй лямбды и анонимные функции читать легче | |
8) толково растаскивай код, зачем Actor'у который по сути является инструментом для просчета передвижения чувачка знать про то, есть ли у него абилки, может ли он их использовать и так далее; | |
9) ObstacleType selected = copy[Random.Range(0, 100000) % copy.Count]; неведомая хуйня зачем?! | |
10) пиздато что ты научился кэшировать трансформы но это надо использовать если ты к трансформу часто обращаешься, например в Update | |
11) атрибуты обычно на строку выше от цели задаются + [hideInInspector],тем более [serializefield] если нет нужны в сериализации пиcать не стоит | |
12) в коде ты к объектам в 3ем лице обращаешься; типо у васи есть абилки: Vasya.HasAbilities() а не Vasya.IHaveAbilities(); |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment