Почему выводится больше картинок?

Question

[Genri_Rus]

Вообщем мне нужно вывести с помощью FileReader не больше 5 картинок
Проблема в том, что у меня выводится сначала 5 картинок и появляется Alert, но потом если выбрать еще картинки, то они тоже появляются в виде превью:

spoiler

Но если кликнуть по картинке, то в консоли в массиве всегда 5 картинок:


Вот пример

Answer 1

[Дмитрий Гололобов]

if (files.length > 5) {
    alert('Добавлено больше 5 картинок');
    return
  }

Возврат сделайте.

Comments

[Genri_Rus]

Теперь картинки не добавляются: https://codepen.io/Genri_Rus/pen/GRoWgpp

[Дмитрий Гололобов]

Genri_Rus, это уже в чем-то другом проблема.
У меня тоже ваш пример через раз работал.
Но по условию, которое я указал выше, функция просто перестает работать.

[Genri_Rus]

Дмитрий Гололобов, я для этого и задал вопрос, потому как вариант с return более очевиден

[Genri_Rus]

Дмитрий Гололобов, после return никакой код ниже не запускается, т.к. картинки в массив выше условия уже запушились

[Дмитрий Гололобов]

Genri_Rus,

if (files.length >= 5) {
    alert('Добавлено больше 5 картинок');
    return
  }
  files.push.apply(files, file);

[Genri_Rus]

Дмитрий Гололобов, спасибо большое

Я перенес проверку в самый верх

[Genri_Rus]

Дмитрий Гололобов, есть проблема, если выбрать 3 картинки, а потом добавить еще больше картинок то условие не срабатывает

Т.е. я выбираю 3 картинки, потом больше 3 и картинки все равно добавляются

[Дмитрий Гололобов]

Genri_Rus, если вы хотите, чтобы скрипт контролировал количество выбираемых файлов, то нужно доп условие.

$("#selector").on("change", function() {
    if ($("#selector")[0].files.length > 2) {
        alert("You can select only 2 images");
    } else {
       //
    }
});

[Genri_Rus]

Дмитрий Гололобов, спасибо, правильно: https://codepen.io/Genri_Rus/pen/GRoWgpp ?

Просто почему-то не работает