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

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

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