[флэйм] if(expr) return 1; else return 0;

[pastor]

по-хорошему надо написать

А где это так сказано? Кто так пишет? Кто так говорит что так нужно?

[Khs]

Лично мне кажется, что тему лучше закрыть, ибо на вкус и цвет товарищей нет. 

не, пусть будет. как `null vs 0`...

ага, и как Qt vs .NET, там аж на 9 страниц развили тему

[ритт]

Лично мне кажется, что тему лучше закрыть, ибо на вкус и цвет товарищей нет. 

не, пусть будет. как `null vs 0`...

ага, и как Qt vs .NET, там аж на 9 страниц развили тему

ну и что? было бы неинтересно, давно тему снесли бы. а так там даже две темы слеплено в кучку

зы. кстати, на вкус и цвет товарищи есть

[Khs]

зы. кстати, на вкус и цвет товарищи есть

Ну это как сказать, ведь никем не доказано что два разных человека одинаково чувствуют и видят одно и то же А следовательно и не факт, что им нравится одно и то же

Хы..теперь в этом топике тоже несколько рассуждений

[pastor]

xintrea, так, ради интереса, загляни в исходники в Qt и напиши письмо в Qt Software, мол "ай-я-яй, вы тут все каноны стилистики и С++ ломаете, куда же этогодится так писать":

Код

C++ (Qt)
bool QWidget::autoFillBackground() const
{
    return d->extra && d->extra->autoFillBackground;
}

Код

C++ (Qt)
QInputContext *QWidget::inputContext()
{
    if (!testAttribute(Qt::WA_InputMethodEnabled))
        return 0;
 
    return d->inputContext();
}

И тернарные операции используете:

Код

C++ (Qt)
extra->explicitMinSize = (minw ? Qt::Horizontal : 0) | (minh ? Qt::Vertical : 0);

Код

C++ (Qt)
Qt::LayoutDirection QWidget::layoutDirection() const
{
    return testAttribute(Qt::WA_RightToLeft) ? Qt::RightToLeft : Qt::LeftToRight;
}

А этот код вобще кошмар:

Код

C++ (Qt)
QCursor QWidget::cursor() const
{
    if (testAttribute(Qt::WA_SetCursor))
        return (d->extra && d->extra->curs)
            ? *d->extra->curs
            : QCursor(Qt::ArrowCursor);
    if (isWindow() || !parentWidget())
        return QCursor(Qt::ArrowCursor);
    return parentWidget()->cursor();
}

ужос!

[xintrea]

по-хорошему надо написать

А где это так сказано? Кто так пишет? Кто так говорит что так нужно?

В книжках это сказано. К сожалению, в электронном виде не могу найти, а то бы ссылки дал. Называются "Практика программирования" товарищей Кернигана и Пайка, и "Совершенный код" товарища Маркконнелла. Кстати там не только то что тут обсуждаем пишут. Там заодно и советы по стилю форматирования есть, причем не на пустом месте, или "потому что так красиво", или "потому что тут так принято", а на нескольких страницах с диаграммами структуры кода. Я тут эту тему вообще боюсь поднимать, ибо чую что получица холивар шояипу.

[xintrea]

xintrea, так, ради интереса, загляни в исходники в Qt и напиши письмо в Qt Software, мол "ай-я-яй, вы тут все каноны стилистики и С++ ломаете, куда же этогодится так писать"

ужос!

Ужос налицо.

Вы же не хотите сказать что "коль даже в Qt пишут уродливый код, так и я тоже буду всё уродовать"? Если видишь откровенное уродство, чего на него равняться?

[ритт]

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

[Rcus]

Не помню в CC высказываний о вреде тернарного оператора или наставлений о единственно верном стиле программирования. Помню только что в начале главы о форматировании кода есть знак предупреждения о священных войнах Макконнелл больше говорит о снижении сложности через снижение вложенности

[kuzulis]

2 xintrea,
+ пицот тыщ ! 

остальные - нечаво выпендривацца!

[pastor]

Ужос налицо.

Вопрос снят...

[pastor]

2 xintrea,
+ пицот тыщ ! 

остальные - нечаво выпендривацца!

Молодца! Аргументируй давай!

[kuzulis]

Аргументирую:
исхожу из сугубо эстетических взглядов. Зачем писать нечто подобное:

Код:

QCursor QWidget::cursor() const
{
    if (testAttribute(Qt::WA_SetCursor))
        return (d->extra && d->extra->curs)
            ? *d->extra->curs
            : QCursor(Qt::ArrowCursor);
    if (isWindow() || !parentWidget())
        return QCursor(Qt::ArrowCursor);
    return parentWidget()->cursor();
}

т.к в таком случае еще более нагружается мозг и без того нагруженный! зачем усложнять и без того сложную жизнь?!

и в конечном счете ведь впринципе эта запись (ну и аналогичные ей) не дают никаких плюсов при работе приложения (ИМХО)

нужно писать код - как стих в литературе! чтобы было легко его читать! без всяких там подвыподвертов!

[BRE]

нужно писать код - как стих в литературе! чтобы было легко его читать! без всяких там подвыподвертов!

Перепиши пожалуйста эту функция так, как написал бы ее сам.

[kuzulis]

Я не умею! Я только учусь!

Ага, ща все брошу и буду фигней заниматься (с)