Вниз  Совершенный код
- 22.02.2014 / 00:26
DominaN
  Пользователь

DominaN 
Сейчас: Offline
вот мой отборный говплохой код трехлетней давности. Too fat :-D


Прикрепленные файлы:
AsteroiX_src.zip (12.07 кб.) Скачано 29 раз
- 22.02.2014 / 00:44
aNNiMON
  Супервизор

aNNiMON 
Сейчас: Offline
Наконец-то нашел исходник PakCreator'a, которому почти 5 лет.
Индусский код, уродские названия переменных (pnpn, a, sof, NOF), дублирование кода, неинформативные комментарии
  1. if (!fc.exists()) {
  2.     //а нима пака;
  3. }
Стыд да и только)
__________________
 let live


Прикрепленные файлы:
Pak_Creator_src.zip (40.65 кб.) Скачано 33 раза
- 22.02.2014 / 00:49
aNNiMON
  Супервизор

aNNiMON 
Сейчас: Offline
А вот пример не только хорошего кода, но и хорошего проектирования. Игра от Malcolm'a.
__________________
 let live


Прикрепленные файлы:
HuntTheWumpus.zip (61.32 кб.) Скачано 42 раза
- 22.02.2014 / 07:47
RGT
  Пользователь

RGT 
Сейчас: Offline
  1. (SETF str '(Плевал я на ваше форматирование)) ((LAMBDA (x) (LIST (CAR x) (CAR (CDR x)) (CAR (CDR (CDR x))) (CAR (CDR (CDR (CDR (CDR x))))))) str)

__________________
 don't tread on me

Изменено RGT (22.02 / 07:47) (всего 2 раза)
- 22.02.2014 / 08:28
Naik
  Пользователь

Naik 
Сейчас: Offline
Как улучшить условие? :gg:
  1. private void verifyAndOccupate(int x, int y, Unit unit) {
  2.     if (x < 0 || x > wTiles || y < 0 || y > hTiles) {
  3.         // over map
  4.     } else {
  5.         Unit free = getUnit(x, y);
  6.         if (free != null && free.isFree()) {
  7.             free.setGamer(unit.getGamer());
  8.         }
  9.     }
  10. }

- 22.02.2014 / 09:42
samodelkin
  Пользователь

samodelkin 
Сейчас: Offline
Naik, смотря что делает каждая строка.

Изменено samodelkin (22.02 / 09:42) (всего 1 раз)
- 22.02.2014 / 09:47
Naik
  Пользователь

Naik 
Сейчас: Offline
samodelkin, выполняется в JVM, что она еще может делать? :gg:
- 22.02.2014 / 09:47
aNNiMON
  Супервизор

aNNiMON 
Сейчас: Offline
samodelkin, а разве не видно? Тут даже комментарии не нужны, потому что код сам себя описывает (название этому самодокументирующийся код).
__________________
 let live
- 22.02.2014 / 09:49
samodelkin
  Пользователь

samodelkin 
Сейчас: Offline
aNNiMON, ясно. Вроде бы итак красивый код, даже и делать не надо вродебы ничего.

Изменено samodelkin (22.02 / 09:49) (всего 1 раз)
- 22.02.2014 / 09:52
aNNiMON
  Супервизор

aNNiMON 
Сейчас: Offline
v2
  1. private void verifyAndOccupate(int x, int y, Unit unit) {
  2.     Unit free = getUnit(x, y);
  3.     if (free != null && free.isFree()) {
  4.         free.setGamer(unit.getGamer());
  5.     }
  6. }
  7.  
  8. private Unit getUnit(int x, int y) {
  9.     if (x < 0 || x > wTiles || y < 0 || y > hTiles) return null;
  10.     //..

__________________
 let live

Изменено aNNiMON (22.02 / 10:08) (всего 1 раз)
Наверх  Всего сообщений: 233
Фильтровать сообщения
Поиск по теме
Файлы топика (7)