Старая шутка

[ViTech]

При чём тут "до" или "после"? Код надо писать так, чтобы другие его нормально читали, а не шарады разгадывали. Если это единственное место в коде, то можно комментарием сопроводить. Если несколько раз встречается, то в функцию выносить. Есть математические функции стандартной библиотеки. Одна isnormal чего стоит. Если их недостаточно, то добавлять свои. Например, isNegativeOrNan, для условия из первого сообщения. В ней, в целях оптимизации, можно хоть !(w >= 0) писать, хоть на ассемблере, на читаемость основного кода это уже влиять не будет.

[gil9red]

При чём тут "до" или "после"? Код надо писать так, чтобы другие его нормально читали, а не шарады разгадывали. Если это единственное место в коде, то можно комментарием сопроводить. Если несколько раз встречается, то в функцию выносить. Есть математические функции стандартной библиотеки. Одна isnormal чего стоит. Если их недостаточно, то добавлять свои. Например, isNegativeOrNan, для условия из первого сообщения. В ней, в целях оптимизации, можно хоть !(w >= 0) писать, хоть на ассемблере, на читаемость основного кода это уже влиять не будет.

Плюсую

[Пантер]

При чём тут "до" или "после"? Код надо писать так, чтобы другие его нормально читали, а не шарады разгадывали. Если это единственное место в коде, то можно комментарием сопроводить. Если несколько раз встречается, то в функцию выносить. Есть математические функции стандартной библиотеки. Одна isnormal чего стоит. Если их недостаточно, то добавлять свои. Например, isNegativeOrNan, для условия из первого сообщения. В ней, в целях оптимизации, можно хоть !(w >= 0) писать, хоть на ассемблере, на читаемость основного кода это уже влиять не будет.

ППКС

[Igors]

Код надо писать так, чтобы другие его нормально читали, а не шарады разгадывали. Если это единственное место в коде, то можно комментарием сопроводить. Если несколько раз встречается, то в функцию выносить. Есть математические функции стандартной библиотеки. Одна isnormal чего стоит. Если их недостаточно, то добавлять свои. Например, isNegativeOrNan, для условия из первого сообщения. В ней, в целях оптимизации, можно хоть !(w >= 0) писать, хоть на ассемблере, на читаемость основного кода это уже влиять не будет.

Ну и к чему эта тирада для начинающих? Разве кто-то другого мнения или призывал так писать? Совсем наоборот

Хотя я не одобряю такой стиль, ни к чему этот вынос мозга, правильно так

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

Такая же ситуевина и здесь

Код

C++ (Qt)
float a;
...
float b = sqrt(a);
...
if (a < 0) b= 0;

Ну ведь знал же он что sqrt может и не понадобится, но зачем-то поперся его сразу вычислять. Неужели опять лох писал? 

[ViTech]

Ну ведь знал же он что sqrt может и не понадобится, но зачем-то поперся его сразу вычислять. Неужели опять лох писал? 

- Меня будете называть Евгений... Александр Саныч! Ясно? Я спрашиваю, ясно?
- Ясно.
- Сесть. Как стемнеет, кассу будем брать.
- И он пойдет?
- Все!
- Так он же на этом скачке расколется, редиска, при первом же шухере!

- А ну-ка, поди сюда, Федя, сядь. Возьми бумагу, пиши: редиска - поставь тире - нехороший человек. Раскалываться - это значит предавать, сознаваться. Мягкий знак поставь. Шухер - опасность, скачок - тире - ограбление. Записал?
- Да.
- А теперь, Федя, скажи Васе все, что ты ему сказал раньше, на нормальном, на гражданском языке.
- Этот нехороший человек предаст нас при первой опасности.

У меня такая вот ассоциация возникает .

Вы лучше опишите контекст, в котором такой код писался. Назначение этой программы. Как давно это было, под какое железо/компилятор. Может условия какие специфичные были. Чтобы можно было понять и посочувствовать этому гражданину .

[Racheengel]

Неужели опять лох писал?

Ну не знает человек языка толком, почему сразу лох то. Мож студент...

float b = a < 0 ? 0 : sqrt(a);

[Igors]

Вы лучше опишите контекст, в котором такой код писался. Назначение этой программы. Как давно это было, под какое железо/компилятор. Может условия какие специфичные были. Чтобы можно было понять и посочувствовать этому гражданину .

Компилятор icc.

Ну не знает человек языка толком, почему сразу лох то. Мож студент...

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

Да, собсно я и сам здесь не знаю ответа. Есть кое-какие соображения, но "не факт"

[ViTech]

Вижу что писал человек очень даже грамотный, похоже в сочувствии совсем не нуждается.

Про сочувствие я в том плане, что приходилось писать в каких-нибудь специфичных условиях, под какой-нибудь определённый компилятор с каким-нибудь особенным поведением. Равно как и вам теперь, похоже, можно посочувствовать в том, что приходится разгребать этот код. Хотя даже в этом случае не вижу причин, по которым "человек очень даже грамотный" не мог бы выражаться "на нормальном, на гражданском языке" .

Конкретно про sqrt: кто его знает, на какую реализацию и поведение он рассчитывал. sqrt может и исключение выкинуть, в коде есть что-нибудь про исключения? Это риторический вопрос . Да и закладываться на какое-то частное поведение в особом случае - не лучшая идея.

[Igors]

Про сочувствие я в том плане, что приходилось писать в каких-нибудь специфичных условиях, под какой-нибудь определённый компилятор с каким-нибудь особенным поведением. Равно как и вам теперь, похоже, можно посочувствовать в том, что приходится разгребать этот код. Хотя даже в этом случае не вижу причин, по которым "человек очень даже грамотный" не мог бы выражаться "на нормальном, на гражданском языке" .

Ну зачем такой минор? Почему бы и не посмотреть как другие пишут, это интересно, а при случае и что-то перенять у них? (разумеется не все подряд).

Хотя и не знаю "правельный ответ", но код выше (с немедленным вычислением sqrt) становится совершенно естественным "гражданским языком" если.... сказать или хотите подумать?   

[ViTech]

Хотя и не знаю "правельный ответ", но код выше (с немедленным вычислением sqrt) становится совершенно естественным "гражданским языком" если.... сказать или хотите подумать?   

Давайте уже, срывайте покровы .

[Racheengel]

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

[Old]

Давайте уже, срывайте покровы .

Очередная попытка выдать говнокод за сакральные знания.
Подождем....

[Igors]

Давайте уже, срывайте покровы .

Он явно рассчитывает на то что sqrt будет вычисляться асинхронно

Код

C++ (Qt)
float b = sqrt(a);  // сразу запускает вычисление
// если тут какой-то код, 
// то он будет выполняться параллельно

Тогда все сразу становится очень натуральным. Ну это всего лишь "я так думаю" 

неважно, какая за этим стояла идея (хотя скорее всего, никакой).

Ну а вдруг стояла?

[ViTech]

Он явно рассчитывает на то что sqrt будет вычисляться асинхронно

Вот это поворот сюжета . А куда, и главное когда, возвращается результат вычисления sqrt при таком "асинхронном" вызове? И какие предпосылки в коде говорят о том, что что-то на этом участке может выполняться параллельно?

[Igors]

Вот это поворот сюжета . А куда, и главное когда, возвращается результат вычисления sqrt при таком "асинхронном" вызове?

Когда вычисленное "а" потребуется. Это небезопасно, напр если sqrt вернет nan, то будет выставлен флаг FPU. И если параллельный код вздумает его читать (ну а вдруг?), то... Упоминание об этом мелькает в доке icc, но ничего о том каким же образом организуется параллельность (может и мифическое )

И какие предпосылки в коде говорят о том, что что-то на этом участке может выполняться параллельно?

Тогда "все сходится", иначе и не написать