Моё внимание в интервью с разработчиками OpenBSD привлёк «клоп», который является, цитирую: …

[poige]

… «хорошим примером, почему оператор ?: следует использовать с осторожностью, по кр. мере в замысловатых выражениях с макросами». Код этот из FreeBSD. Что же, больше нападок, чем на «?:», было, наверное, только на «goto». :-) Я лично считаю, что проблема тут совсем не в «?:», а в style(9), и это действительно проблема. Посмотрите на оригинал (с «

Comments

[dnovikoff]

вообще, выравниваение любых скобочных конструкций есть хороший тон =)) я бы сделал немного в другой стилистике и с меньшим количеством пробелов, но суть от этого не меняется. да и всё надо использовать в меру.

> я бы сделал немного в другой стилистике

[poige]

Запиши? - любопытно посмотреть.

Re: > я бы сделал немного в другой стилистике

[dnovikoff]

для начала, я бы сделал это в виде inline-функции =) и эту тучу скобочек заменил бы if'ами:

int
sblock(sb, wf)
{
if (sb->sb_flags & SB_LOCK) {
if (wf != M_WAITOK)
return EWOULDBLOCK;

return sb_lock(sb);
}

sb->sb_flags |= SB_LOCK;

return 0;
}

что-нибудь такое

> для начала, я бы сделал это в виде inline-функции =)

[poige]

Хм, я не про это. Вот есть макрос - как ты его записал бы, чтобы он стал читабельней? В данном случае, то есть, меня стиль оформления текста кода интересует.

[some41]

не шарите.

#define sblock(sb, wf) (((sb)->sb_flags & SB_LOCK) == 0 ? (sb)->sb_flags |= SB_LOCK, 0 : \
(wf) == M_WAITOK ? sb_lock(sb) \
: EWOULDBLOCK)

> не шарите.

[poige]

Я же не про то, как оптимальнее это сделать говорю, а как читабельнее.

Re: > не шарите.

[some41]

а при чем тут оптимальность? код один и тот же. только оформление

> код один и тот же. только оформление

[poige]

Ну не совсем уж и тот же. А оформление - «табличное», какое ты и продемонстрировал, вовсе не так уж хорошо - оно завязано на подгонку (зачастую пробелами), и занимает больше места в ширину. Наглядность тоже ниже.

[gadm]

Человек, который использует в C'шном коде «?:» - педераст. Dixi.

[fenikso]

хе-хе, что, и даже для такого:

bool b=...;
int i = b?5:7;

?

Я же сказал в самом начале: ...

[poige]

«…
Что же, больше нападок, чем на «?:», было, наверное, только на «goto». :-)
…»

[fenikso]

Да я видел, видел. Мне просто стало интересно довести до крайне простого примера :) Хотя конечно можно писать и
int i=5;
if (...) i+=2;

:)) тоже две строчки

[ex_sighup150]

Не знаю. Мне нравится style(9)-conformant. Хотя, конечно, сам style(9) небезугрёбищен.