Skip to content

Instantly share code, notes, and snippets.

@xdegtyarev
Last active December 18, 2015 07:49
Show Gist options
  • Star 0 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
  • Save xdegtyarev/5749297 to your computer and use it in GitHub Desktop.
Save xdegtyarev/5749297 to your computer and use it in GitHub Desktop.
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