1

Добро дамам и господам, i have массив

var results = {
'a': '1',
'b': '2'};

и мне нужно пробежаться по массиву так чтобы если у хоть одного ключа значение == 1 то кое-что сделать в jquery, также если хоть у одного ключа значение == 2 сделать кое-что в jquery = так на все цифирки от 1 до 5

То что у меня получилось:

for (var key in results) {
    if (results[key] == '1') {
        $('.avatar > img').attr('src', '/images/avatars/chief_1.svg') 
        break
    }
    if (results[key] == '2') {
        $('.avatar > img').attr('src', '/images/avatars/chief_2.svg') 
        break
    }
    if (results[key] == '3') {
        $('.avatar > img').attr('src', '/images/avatars/chief_3.svg') 
        break
    }
    if (results[key] == '4') {
        $('.avatar > img').attr('src', '/images/avatars/chief_4.svg') 
        break
    }
    if (results[key] == '5') {
        $('.avatar > img').attr('src', '/images/avatars/chief_5.svg') 
        break
    }
}

При break цикл заканчивает при первом же итерации, а

при continue изменение картинки подсчитывается только при второй итерации

1
  • используйте конструкцию switch.
    – Alex
    2 июл 2017 в 18:33

1 ответ 1

1

Можно создать булевый массив, ключи которого — числа от 1 до 5, а значения — true или false, в зависимости от того, встречался ли уже такой ключ.

let results = {
  'a': '1',
  'b': '2'
};

let isActionPerfomed = [];
for (let i = 1; i <= 5; ++i) {
  isActionPerfomed[i] = false;
}

for (let key in results) {
  let value = results[key];
  for (let i = 1; i <= 5; ++i) {
    if (value == i && !isActionPerfomed[i]) {
      console.log(`выполняем действие ${i}`);
      $('.avatar > img').attr('src', `/images/avatars/chief_${i}.svg`);
      isActionPerfomed[i] = true;
    }
  }
}
<script src="https://cdnjs.cloudflare.com/ajax/libs/jquery/3.2.1/jquery.min.js"></script>

11
  • Спасибо за выделенное время, но тут все равно меняется картинка при последней итерации выполняем действие 4 выполняем действие 2 тут должно было поменяться картика на 2, а меняется на 4
    – Welis
    2 июл 2017 в 18:41
  • Звучит очень странно. А чему равно $('.avatar > img').attr('src') после цикла?
    – diralik
    2 июл 2017 в 18:47
  • самая минимальная - chief_1.svg
    – Welis
    2 июл 2017 в 18:48
  • Должно менятся картинка на самую минимальную значение ключей в объекте)
    – Welis
    2 июл 2017 в 18:50
  • Вам всегда нужно чтобы осталась самая минимальная? Может тогда просто взять минимум из значений вашего объекта (minValue = Math.min(...Object.values(results))), а потом один раз выполнить $('.avatar > img').attr('src', "/images/avatars/chief_${minValue}.svg");?
    – diralik
    2 июл 2017 в 18:51

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.