Advertisement
Guest User

Untitled

a guest
Jul 25th, 2017
91
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
text 2.91 KB | None | 0 0
  1. Плюсы:
  2. * кеширование и переиспользование игровых объектов
  3.  
  4. * LINQ
  5. ** Более высокоуровневая библиотека работы с массивами использование в неперфоманс критичных местах приветствуется.
  6.  
  7. * нет неуместного в игровых проектах использования языка (sealed/readonly/прочее)
  8. ** Что прочее? Не у нас в проекте не гделибо в игровой индустрии еще никто эту хрень не испольует. Она антипродуктивна.
  9.  
  10. * жесточайшая библиотека работы с массивами (из Extensions)
  11. ** это несколько суперпростых и коротких функций делающих код читаемей и короче
  12.  
  13. * смерть астероидов анимирована, хотя скрыта от глаза
  14. ** У меня все анимировано нормально
  15.  
  16. Минусы:
  17. * использование FixedPoint там, где в этом нет необходимости (например, в измерении временных интервалов)
  18. ** В задании сказано использовать FixedPoint но не сказано зачем, вероятность что человек догадается до того где это хочет применять автор задания минимальна, сказно для что для ВСЕХ расчетов
  19.  
  20. * использование чисел с плавающей запятой в логических вычислениях
  21. ** Нашел в одном месте, там из-за того что эта библиотека для FixedPoint оч. слабая и там нету Atan2, он вычислил на float, но так опять же требование считать на FixedPoint совершенно в вакуме и необосновано, то это можно считать валидным приемом.
  22.  
  23. * использование дженериков там, где в этом нет необходимости (например, в CollisionProcessor.RemoveCollider)
  24. ** Да, небольшая описка, или забыл зарефакторить, или спать хотел, в целом никак не влияющая на производительность или еще что
  25.  
  26. * после смерти НЛО, анимация смерти не встаёт на паузу
  27. ** Да, баг, слегка уменьшает перформанс.
  28.  
  29.  
  30. ** В паре мест реально был немного косячный код в плане логики и структуры, однако внимание на это обращено не было.
Advertisement
Add Comment
Please, Sign In to add comment
Advertisement