zorro76
@zorro76

Как этот код можно оптимизировать (сократить)?

Есть функция (которая валидирует много чего) - меня ж интересует два куска практически идентичного кода, который проходиться for по группе с checkbox и выводит сообщения с перечнем ошибок для каждой группы отдельное

function populateErrors() {
        var reasons = [];
        var dragdropChoices = $('.drag-and-drop .choices');
        for (var c = 0; c < dragdropChoices.length; c++)
           if ($(dragdropChoices[c]).find('input:checkbox:checked').length == 0) {
               reasons.push('select at least one correct answer for each <b>drag & drop</b> question');
           }

        var multipleChoices = $('.multiple-choice .choices');
        for (var c = 0; c < multipleChoices.length; c++)
           if ($(multipleChoices[c]).find('input:checkbox:checked').length == 0) {
               reasons.push('select at least one correct answer for each <b>multiple choice</b> question');
           }

        return _.uniq(reasons);
    }

разметка HTML/Ruby
<div class="activity-content">
  <div class="activity-content-item drag-and-drop" id="drag-editor71815805" data-id="46">
    <p>Enter answer options:</p>
    <div class="choices">

  <div class="choice">
    <div class="checkbox-switch-wrapper">
      <div class="checkbox-switch "><span></span></div>
      <input type="checkbox" value="1" name="activity[item_attributes][questions_attributes][][content][choices][][correct]">
    </div>&nbsp;
  </div>


  <div class="choice">
    <div class="checkbox-switch-wrapper">
      <div class="checkbox-switch "><span></span></div>
      <input type="checkbox" value="1" name="activity[item_attributes][questions_attributes][][content][choices][][correct]">
    </div>&nbsp;
  </div>


  <div class="choice">
    <div class="checkbox-switch-wrapper">
      <div class="checkbox-switch "><span></span></div>
      <input type="checkbox" value="1" name="activity[item_attributes][questions_attributes][][content][choices][][correct]">
    </div>&nbsp;
  </div>
</div>
</div>

  <div class="activity-content-item multiple-choice" id="choice-editor6345333" data-id="47">
    <p>Enter answer options:</p>
    <div class="choices">

  <div class="choice">
    <div class="checkbox-switch-wrapper">
      <div class="checkbox-switch "><span></span></div>
      <input type="checkbox" value="1" name="activity[item_attributes][questions_attributes][][content][choices][][correct]">
    </div>&nbsp;
  </div>


  <div class="choice">
    <div class="checkbox-switch-wrapper">
      <div class="checkbox-switch "><span></span></div>
      <input type="checkbox" value="1" name="activity[item_attributes][questions_attributes][][content][choices][][correct]">
    </div>&nbsp;
  </div>


  <div class="choice">
    <div class="checkbox-switch-wrapper">
      <div class="checkbox-switch "><span></span></div>
      <input type="checkbox" value="1" name="activity[item_attributes][questions_attributes][][content][choices][][correct]">
    </div>&nbsp;
  </div>
</div>
</div>
</div>


как сократить этот повторяющийся код?
  • Вопрос задан
  • 54 просмотра
Решения вопроса 1
@vabka
Токсичный
Не вижу, что тут радикально плохое.
JS можно сократить:
const dragndropReasons = $('.drag-and-drop .choices')
    .filter(c => c.find('input:checkbox:checked').length == 0)
    .map(() => 'select at least one correct answer for each <b>drag & drop</b> question');
const multipleReasons = $('.multiple-choice .choices')
    .filter(c => c.find('input:checkbox:checked').length == 0)
    .map(() => 'select at least one correct answer for each <b>multiple choice</b> question');
return _.uniq(dragndropReasons.concat(multipleReasons))
Ответ написан
Пригласить эксперта
Ответы на вопрос 1
Simkav
@Simkav
На вход передаете свои массивы по одному
function foo(array) {
  for (let i = 0; i < array.length; i++) {
    if ($(array[c]).find('input:checkbox:checked').length === 0) {
      reasons.push(
        'select at least one correct answer for each <b>multiple choice</b> question'
      );
    }
  }
}
Ответ написан
Ваш ответ на вопрос

Войдите, чтобы написать ответ

Войти через центр авторизации
Похожие вопросы
Artezio Нижний Новгород
от 130 000 до 180 000 ₽
Artezio Москва
от 160 000 до 220 000 ₽
Artezio Минск
от 2 800 до 3 300 $
04 мар. 2021, в 16:10
10000 руб./за проект
04 мар. 2021, в 15:49
500 руб./в час
04 мар. 2021, в 15:42
8000 руб./за проект