Как можно сократить этот код?

Question

[FuGou]

Я новичок в js, как можно сократить этот код и есть ли тут какие-либо ошибки которые можно исправить или как-то правильнее написать код?

var type_input = document.getElementById("type_input").value;
var type_name = ['фильм', 'сериал', 'мультсериал', 'мультфильм', 'аниме'];
var count_type = 0; // счетчик ошибок
var type = 0; // значение для базы данных
            for (var i = 0; i < type_name.length; i++){  
                if(type_input != type_name[i]){
                    count_type++;
                    if (count_type > 4){
                        alert('Добавьте или исправьте тип');
                        break;                           
                    }  
                }
                if(type_input == type_name[i]){
                    type = count_type + 1;
                }
            }

Comments

[Михаил Р.]

Вы нейро-сетью ошиблись

[FuGou]

Михаил Р., ?

Answer 1

[Сергей delphinpro]

const type_input = document.getElementById("type_input").value.toLowerCase();
const type_names = ['фильм', 'сериал', 'мультсериал', 'мультфильм', 'аниме'];
const type_index = type_names.indexOf(type_input);
let type = 0;

if (type_index === -1) {
    alert('Добавьте или исправьте тип');
} else {
    type = type_index + 1; // значение для базы данных (если нужно 1-based)
}

Comments

[FuGou]

Спасибо огромное

[FuGou]

А можете еще объяснить как это работает(что означает -1, indexOf и тд?)

[Сергей delphinpro]

FuGou, это узнаётся из документации https://developer.mozilla.org/ru/docs/Web/JavaScri...

[FuGou]

Сергей delphinpro, Спасибо я все понял

Answer 2

[Rsa97]

const type_input = document.getElementById("type_input").value;
const type_name = ['фильм', 'сериал', 'мультсериал', 'мультфильм', 'аниме'];
const type_idx = type_name.indexOf(type_input);
let type = 0;
if (type_idx === -1) {
    alert('Добавьте или исправьте тип');
} else {
    type = type_idx + 1;
}

А вообще, такое делается с помощью select, тогда невозможно ввести неправильное значение и весь этот код просто не нужен.
И в целом, не используйте var, она размещает переменную в глобальной области видимости. Вместо этого используйте const или let, в зависимости от дальнейшей иммутабельности переменной.
Используйте строгое сравнение. Нестрогое означает, что вы не уверены в типах данных.

Comments

[FuGou]

А можете написать как делается с помощью select'а?

[FuGou]

хорошо, спасибо

[Rsa97]

FuGou, Вместо input в HTML-коде делаете select с нужными опциями.

[FuGou]

Rsa97, спасибо

[FuGou]

Rsa97, а select это же меню с выбором? А мне нужно чтобы я вводил тип фильма, а не выбирал и select мне попросту не подойдет

[Rsa97]

FuGou, А смысл вводить, если вы потом всё равно фильтруете и не пропускаете то, чего нет в массиве.

[FuGou]

Rsa97, просто мне нужно чтобы в базу данных отправлялись цифрами, а не строками типы

[Rsa97]

FuGou, Так и сделайте у select'а в options'ах значениями нужные числа.

[FuGou]

вот я хочу чтобы при вводе в инпут выдавалась строка, а в базу данных нужна именно цифра

[Rsa97]

FuGou,

[FuGou]

Rsa97, а ну если так, хорошо спасибо, так и сделаю, так действительно проще

[Михаил Ливач]

FuGou, я Вас сейчас толкну на путь зла: есть такая библиотека Choices.js

[FuGou]

Михаил Ливач, опять что-то новое изучать…. Мне бы с этим разобраться