Должен ли я избавиться от операторов продолжения

Question

Я читал in this Q&A, что, как правило, следует избегать утверждений в циклах continue. Правило стоит придерживаться кода ниже? Если да, то каков был бы лучший способ реорганизовать его, чтобы избавиться от них?

for (property in formInput) { 
     if (!formInput.hasOwnProperty(property) || property === "Id") { 
      continue; 
     } 
     if (property.slice(-3) === "_Id") { 
      setMagicSuggestFromFormInput(property); 
      continue; 
     } 
     if (property.slice(-3) === "_bl" && formInput[property] === true) { 
      $("#" + property).prop("checked", true); 
      continue; 
     } 
     $("#" + property).val(formInput[property]); 
    } 

Edit: Если вы думаете, что цикл должен быть переработан, к тому же с указанием о том, как это можно сделать, не могли бы вы сказать мне, почему вы считаете предложил рефакторинга лучший выбор дизайна?

Answer 1

Ваш код:

for (property in formInput) { 
    if (!formInput.hasOwnProperty(property) || property === "Id") { 
     continue; 
    } 
    if (property.slice(-3) === "_Id") { 
     setMagicSuggestFromFormInput(property); 
     continue; 
    } 
    if (property.slice(-3) === "_bl" && formInput[property] === true) { 
     $("#" + property).prop("checked", true); 
     continue; 
    } 
    $("#" + property).val(formInput[property]); 
} 

Является эквивалентом:

for (property in formInput) { 
    if (property.slice(-3) === "_Id") { 
     setMagicSuggestFromFormInput(property); 
    } else if (property.slice(-3) === "_bl" && formInput[property] === true) { 
     $("#" + property).prop("checked", true); 
    } else if (formInput.hasOwnProperty(property) && property !== "Id") { 
     $("#" + property).val(formInput[property]); 
    } 
} 

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

Answer 2

Почему бы вам не попробовать «еще если»?

Ввод

$("#" + property).val(formInput[property]); 

внутри еще заявление гарантирует, что он не будет работать, когда один из другого, если заявления выполняются. Вы должны сделать остальные утверждения if else if.

Answer 3

Когда вы используете break/continue в цикле или блоке в начале, они действуют как предварительное условие, которое является хорошим дизайном и помогает понять блок легко. Если вы используете continue или break в середине блока с некоторым кодом, он действует как скрытая ловушка и трудно понять, что плохо.

Вы можете проверить эту ссылку для более подробного обсуждения вокруг перерыва/продолжить https://softwareengineering.stackexchange.com/questions/58237/are-break-and-continue-bad-programming-practices

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

for (property in formInput) { 
    if (formInput.hasOwnProperty(property) && property !== "Id") { 
    var propVal = property.slice(-3); 
    if (propVal === "_Id") { 
     setMagicSuggestFromFormInput(property); 
    } else if (propVal === "_bl" && formInput[property]) { 
     $("#" + property).prop("checked", true); 
    } else { 
     $("#" + property).val(formInput[property]); 
    } 
    } 
} 

Answer 4

Если не хотите использовать продолжать заявления, то вы можете идти по пути вложенного кода:

for (property in formInput) { 
    if (formInput.hasOwnProperty(property) && property !== "Id") { 
     if (property.slice(-3) === "_Id") { 
      setMagicSuggestFromFormInput(property); 
     } 
     else if (property.slice(-3) === "_bl" && formInput[property] === true) { 
      $("#" + property).prop("checked", true); 
     } 
     else{ 
      $("#" + property).val(formInput[property]); 
     } 
    } 
} 

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

Answer 5

Вы можете вложить условные проверки и инвертировать условия.

for (property in formInput) { 
    if (formInput.hasOwnProperty(property) && !property === "Id") 
     if (property.slice(-3) === "_Id") { 
      setMagicSuggestFromFormInput(property); 
     else if (property.slice(-3) === "_bl" && formInput[property] === true) 
      $("#" + property).prop("checked", true); 
     else 
      $("#" + property).val(formInput[property]); 
    } 
}