Вопрос по стилю написания кода.

[red5]

Здравствуйте. Я начинающий програмист и меня интересует как грамотнее будет написать такой участок кода:

Код:

bool Dock5::isFrameDeclareVariable()
{
    if(f1->frameStyle() == QFrame::NoFrame){
	return false;
    }
    else {
	return true;
    }
}

Может быть, стоит написать всё в одну строку?

Код:

bool Dock5::isFrameDeclareVariable()
{
    return ( f1->frameStyle() != QFrame::NoFrame )
}

Есть уважаемые люди, которые говорят, что первый вариант "некрасивый". А в книге "Совершенный код" С.Макконнелл пишет что нужно писать код, который прежде всего понятен людям. Мне например, первый вариант гораздо понятнее.
Понимаю, что это дело вкуса, вот и интересует мнение опытных специалистов.

[kuzulis]

Код:

    ...
    return (f1->frameStyle() == QFrame::NoFrame) ? false:true;
    ...

или

Код:

bool Dock5::isFrameDeclareVariable()
{
    return ( f1->frameStyle() != QFrame::NoFrame )
}

[mell]

я, например, вижу красоту кода объектно-ориентированных языков (тем более программируя на qt) именно в его понятности. Вообще, если правильно использовать ООП, комментарии к коду можно даже не писать, достаточно правильно выбирать названия методам. Если программа пишется в команде, или будет выкладываться с исходным кодом, или программа большая и в ней нужно будет потом разбираться, естественно предпочтителен "понятный" вариант.
а вот красивый вариант, по моему мнению, предпочтителен в процедурных языках в тех случаях, когда пишется "серьезный" код, например решение сложных алгоритмов, когда не просто кодишь потоком gui или там sql, а решаешь "красивую задачу красивым стилем"
Видимо тут отличие, как между стихами и прозой....

Кстати вот еще "понятный" вариант:

Код

C++ (Qt)
bool Dock5::isFrameDeclareVariable()
{
    bool isFrameDeclare = (f1->frameStyle() != QFrame::NoFrame)
    return isFrameDeclare;
}
 

[Rcus]

Во, нашел. Помню что уже по этому поводу флейм http://www.prog.org.ru/topic_8578_0.html

[spirit]

вот статейка про стиль троллей.

[red5]

Спасибо.
Вот пример из стиля троллей:

Код:

Use curly braces when the body of a conditional statement contains more than one line,
and also if a single line statement is somewhat complex.
// Wrong
if (address.isEmpty()) {  
    return false;  
}  

for (int i = 0; i < 10; ++i) {  
    qDebug("%i", i);  
}  

// Correct  
if (address.isEmpty())  
    return false;  

for (int i = 0; i < 10; ++i)  
    qDebug("%i", i);  

Но ниже у них несколько исключений из этого правила :-) И эти исключения больше путают, чем помогают.
А вот в книге "Совершенный код" Макконнелл пишет, что всегда следует применять {}, даже когда всего одна строка. Это позволяет избежать ощибок-опечаток при дальнейшем добавлении второй строки. Конечно, это всё дело вкуса и споры вокруг этой темы говорят, что нет единственно правильного решения. Всем спасибо.

[Rcus]

Не вижу противоречия. Все "исключения" подчиняются правилу

General exception

    * Feel free to break a rule if it makes your code look bad.