22.02.2014 / 00:26 | |
DominaN Пользователь Сейчас: Offline
Имя: Кирилл Откуда: Смоленск Регистрация: 28.09.2011
| вот мой отборный говплохой код трехлетней давности. Too fat
Прикрепленные файлы: AsteroiX_src.zip (12.07 кб.) Скачано 29 раз |
22.02.2014 / 00:44 | |
aNNiMON Супервизор Сейчас: Offline
Имя: Витёк Регистрация: 11.01.2010
| Наконец-то нашел исходник PakCreator'a, которому почти 5 лет. Индусский код, уродские названия переменных (pnpn, a, sof, NOF), дублирование кода, неинформативные комментарии if (!fc.exists()) {
//а нима пака;
}
Стыд да и только) __________________
let live
Прикрепленные файлы: Pak_Creator_src.zip (40.65 кб.) Скачано 33 раза |
22.02.2014 / 00:49 | |
aNNiMON Супервизор Сейчас: Offline
Имя: Витёк Регистрация: 11.01.2010
| А вот пример не только хорошего кода, но и хорошего проектирования. Игра от Malcolm'a.
__________________
let live
Прикрепленные файлы: HuntTheWumpus.zip (61.32 кб.) Скачано 42 раза |
22.02.2014 / 07:47 | |
RGT Пользователь Сейчас: Offline
Имя: Snork Откуда: Yerevan Регистрация: 03.02.2010
| (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 Пользователь Сейчас: Offline
Имя: %name% Регистрация: 14.03.2010
| Как улучшить условие? private void verifyAndOccupate(int x, int y, Unit unit) {
if (x < 0 || x > wTiles || y < 0 || y > hTiles) {
// over map
} else {
Unit free = getUnit(x, y);
if (free != null && free.isFree()) {
free.setGamer(unit.getGamer());
}
}
}
|
22.02.2014 / 09:42 | |
samodelkin Пользователь Сейчас: Offline
Имя: Сергей Откуда: Липецк Регистрация: 05.11.2013
| Naik, смотря что делает каждая строка.
Изменено samodelkin (22.02 / 09:42) (всего 1 раз) |
22.02.2014 / 09:47 | |
Naik Пользователь Сейчас: Offline
Имя: %name% Регистрация: 14.03.2010
| samodelkin, выполняется в JVM, что она еще может делать? |
22.02.2014 / 09:47 | |
aNNiMON Супервизор Сейчас: Offline
Имя: Витёк Регистрация: 11.01.2010
| samodelkin, а разве не видно? Тут даже комментарии не нужны, потому что код сам себя описывает (название этому самодокументирующийся код).
__________________
let live |
22.02.2014 / 09:49 | |
samodelkin Пользователь Сейчас: Offline
Имя: Сергей Откуда: Липецк Регистрация: 05.11.2013
| aNNiMON, ясно. Вроде бы итак красивый код, даже и делать не надо вродебы ничего.
Изменено samodelkin (22.02 / 09:49) (всего 1 раз) |
22.02.2014 / 09:52 | |
aNNiMON Супервизор Сейчас: Offline
Имя: Витёк Регистрация: 11.01.2010
| v2 private void verifyAndOccupate(int x, int y, Unit unit) {
Unit free = getUnit(x, y);
if (free != null && free.isFree()) {
free.setGamer(unit.getGamer());
}
}
private Unit getUnit(int x, int y) {
if (x < 0 || x > wTiles || y < 0 || y > hTiles) return null;
//..
__________________
let live Изменено aNNiMON (22.02 / 10:08) (всего 1 раз) |