Написание избыточных программных операторов потока для ясности или соображений оптимизации?

Question

Я помечен как C, хотя это, безусловно, относится ко многим языкам. Причина этого - часть вопроса, связанного с оптимизацией, которая зависит от компилятора.

Иногда мы сталкиваемся с ситуациями, как это в программах:

if(bob == 42) 
{ 
    /* ... */ 
    return; 
} 
else 
{ 
    /* ... */  
} 

else блока здесь не является строго необходимым, как вы, вероятно, можете видеть. То же самое происходит и с другими структурами управления потоком программ; некоторые «обычные» конструкции становятся излишними из-за особых условий. Возникает вопрос: есть ли причина писать эти избыточные блоки кода? Ясность? Может ли это помочь компилятору с оптимизацией, если ситуация была достаточно сложной?

Answer 1

Я имею в виду то, что вы написали как «самодокументирующий код». Используя блок else, вы упрощаете людям видеть, что если bob не равен 42, загорится 2-й блок кода.

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

Самодокументирующий код означает, что объяснение происходящего можно увидеть в именах функций или путем чтения фактического кода, что уменьшает необходимость в комментариях кода.

Что касается запуска кода, выполните некоторые тесты с некоторым временным кодом, чтобы узнать, что работает лучше.

Answer 2

Для себя, я часто пишу

if(bob == 42) 
{ 
    /* ... */ 
    return; 
} 
/* else */ 

/* ... */ 

хотя я мог бы быть то здесь отсутствует.

РЕДАКТИРОВАТЬ: Я также только что понял вопрос о сфере применения здесь. По крайней мере, на C++ (не в старом C, не уверенном в C99?) Блок else имел бы свою «собственную» разную область, о которой стоит обратить внимание.

Answer 3

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

/* Is bob 42 ? */ 
if(bob == 42) 
{ 
    /* Bob is 42, return from function */ 

    return; 
} 
else 
{ 
    /* Bob is not 42, do nothing */ 
} 

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

Answer 4

Я предпочел бы только одна линия (если это равносильно тому, что вы имели в виду)

if (bob == 42) return; 

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

Повторяя одно и то же сообщение с небольшими изменениями в комментариях, читатель читает код для глупого.

Если бы это было не то, что вы имели в виду с кодом snipset, если есть на самом деле что-то делать в обоих случаях, я бы предпочел

if (bob == 42) { 
    /* do something if bob is the chosen one */ 
} else { 
    /* for all others apply the standard treatment */ 
} 
return; 

Это не скрывает ваш оператор возврата глубоко внутри блока ,

Таким образом, если вы чувствуете, что вам нужно будет прокомментировать управление потоком, а не семантику вашего кода, вы делаете что-то неправильно.