- 1
- 2
nPosition = !bInvert ? data->pos_back
: data->pos_front;
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+150
nPosition = !bInvert ? data->pos_back
: data->pos_front;
Lure Of Chaos 12.04.2011 16:46 # 0
- сверху или снизу?
- что, вот так сразу?
Govnocoder#0xFF 12.04.2011 17:19 # 0
Lure Of Chaos 12.04.2011 17:22 # 0
хватить уже ныть )
Govnocoder#0xFF 12.04.2011 17:27 # +2
Сидит программист в кафе, прогу пишет. Тут к нему подваливает тян:
- Если хочешь со мной познакомиться, то меня зовут Маша.
- А если я не хочу с тобой знакомиться, то как тебя зовут?
hidespb 12.04.2011 21:46 # +3
-да!
-и чем же он занят?
-сбережением энергии!
absolut 12.04.2011 22:38 # 0
iddqd 13.04.2011 09:18 # +6
absolut 14.04.2011 14:05 # 0
TheCalligrapher 14.04.2011 21:51 # +2
В данном случае можно предположить, что 'bInvert == 0' является штатной и естественной ситуацией, почему автор кода и предпринял усилия к тому, чтобы первой в '?:' была именно ветка, предназначенная для обработки ситуации 'bInvert == 0'.
Aleskey 15.04.2011 09:11 # 0
//id 2 - двигаемся назад
//id 1 - двигаемся вперед
BYTE bInvert = (data->Flags & 0x0010) == 0x0010;
int nPosition = 0;
if (id == 2) {
nPosition = !bInvert ? data->pos_back : data->pos_front;
moveto(nPosition);
} else if (id == 1) {
nPosition = !bInvert ? data->pos_front : data->pos_back;
moveto(nPosition);
}
Aleskey 15.04.2011 09:20 # +2
препод: функция меняет знак?
студент: да!
препод: (удивленно) Каааак?
студент: Но не очень!!!
gegMOPO4 18.04.2011 18:50 # 0
absolut 15.04.2011 09:12 # 0
TheCalligrapher 15.04.2011 10:07 # −1
Settings settings = { 0 };
и ожидать, что в результате вот таким вот компактным образом я безо всякой головной боли получу "штатный" набор установок, работающий в 9 случаях из 10.
А уж если мне надо соорудить что-то необычное, то я тогда уже выборочно подкручу то, что мне надо подкрутить, типа
settings.bInvert = 1;
При таком подходе "логичнее" выглядит именно 'bInvert', а не 'bDirect'. Но это, конечно, дело вкуса...
SmackMyBitchUp 23.04.2011 21:05 # 0
guest 24.04.2011 16:31 # −1
guest 23.04.2011 21:45 # 0
Делай так:
Settings settings = {};
а лучше так:
Settings settings;