Javascript: Использование пользовательского ввода для поиска массива массивов

Question

Возможно, я неправильно структурировал этот код (впервые работаю с Javascript), но я хочу взять пользовательский ввод и выполнить поиск массива массивов, чтобы проверить, существует ли массив с таким именем существует.

Сначала я попытался использовать функцию eval(), которую мне сказали не очень хорошо, но она работает, когда есть совпадение, если не существует совпадения, которое не существует.

Основная предпосылка программы - это массив, содержащий местоположения, которые впоследствии являются массивами, содержащими продукты питания. Пользователь вводит имя пищевого продукта и там, где он хочет его поместить, и он создает объект питания, вставляет этот объект в нужное место в массивах и также вводит элемент в список, отображаемый в html.

Вот весь код до сих пор:

var fridge = []; 
 
var freezer = []; 
 
var pantry = []; 
 

 
var locations = [fridge, freezer, pantry]; 
 

 

 
function Food(name, location, locationName){ 
 
    this.name = name; 
 
    this.location = location; 
 
    this.locationName = locationName; 
 
    this.displayName = function(){ 
 
     alert(this.name); 
 
    }; 
 
}; 
 
function createFood(){ 
 
    var name = prompt("Enter the items name:"); 
 
    var locationName = prompt("Enter a location to store it:") 
 
    var location = eval(locationName); 
 

 
    while(locations.indexOf(location) == -1){ 
 
     alert("This is not an actual location."); 
 
     locationName = prompt("Please enter another location:"); 
 
     location = eval(locationName); 
 
    }; 
 

 
    var x = new Food(name, location, locationName) 
 

 
    function insertFood(Food){ 
 
     var a = locations.indexOf(Food.location); 
 
     locations[a].push(Food); 
 

 
     var list = document.getElementById(Food.locationName + "_items"); 
 
     var li = document.createElement("li"); 
 
     li.innerHTML = Food.name; 
 
     list.insertBefore(li, list.lastChild); 
 
    }; 
 

 
    insertFood(x); 
 
};

Пожалуйста, дайте мне знать, если это структурировано неправильно причина это была моя идея структурирования на первый взгляд.

Спасибо!

Answer 1

Как было отмечено выше, было бы лучше, чтобы сделать locations объект, так что вы можете иметь ключ (строка), указывающий на массив с таким же именем.

var fridge = []; 
    var freezer = []; 
    var pantry = []; 

    var locations = { 
     "fridge":fridge, 
     "freezer":freezer, 
     "pantry":pantry 
    }; 

Преимущество этого в том, что вам не нужно иметь locationName, поскольку он никогда не вступает в игру. Все, что вам нужно, это проверить, имеет ли объект locations свойство с тем же именем, что и пользовательский ввод, используя встроенную функцию hasOwnProperty. Что-то вроде:

if(!locations.hasOwnProperty(userInputLocation)) 
     alert("That is not a designated location"); 

Тогда ваш конструктор еды также становится проще, и нужно только имя и местоположение свойства:

function Food(name, location){ 
     this.name = name; 
     this.location = location; 
    } 

Вы также можете затем вызвать любое конкретное место непосредственно по его имени (если вы (если вы объявите массивы внутри объекта, как в коде SGD), locations[location], где location - это просто строка, содержащая либо «морозильник», либо «кладовая» или «кладовая», холодильник". Вы также можете вызвать массив через locations[someFood.location].

Во всяком случае я не так много для подсказок и предупреждений (не говоря уже о Eval), поэтому я создал jsBin используя поля ввода, вы можете проверить его здесь: http://jsbin.com/pamoquwuhu/1/edit

отредактирован добавить: Если цель заключается в том, что вы позже захотите найти пищу по его названию во всех местах, в которых она сохранена, было бы лучше добавить/нажать foodObj.name вместо целого foodObj в locations[location]. Затем вы можете использовать собственный цикл for(property in object) в объекте местоположений, чтобы увидеть, где можно хранить всю данную пищу, и вставлять ее в массив results. Так что ваша findFood функции может содержать следующее (в предположении food является входным пользователем строки названия продуктов питания для поиска:

var results = []; 

    for(var loc in locations){ // loops through the property names in `locations` 
     if(locations[loc].indexOf(food)>=0) 
      results.push(loc); 
    } 

    if(!results.length){ 
     alert(food+' was not found'); 
    else 
     alert(food+' was found in '+results.toString()); 

Предполагая ту же пищу, может храниться в нескольких местах, и что вы хотите, чтобы найти данную пищу по его названию, ваше имуществовашего пищевого объекта станет менее важным (или, возможно, бесполезным).

Answer 2

Вы используете функцию как функцию/конструктор и как имя параметра (это не должно быть проблемой, но может вызвать проблемы позже), и вы никогда не звоните insertFood. Также места должны скорее быть объектом, чем массивом, чтобы вы могли обращаться к вспомогательным массивам, как в своем коде.

var locations = {fridge : [], freezer:[], pantry:[]]; 
 

 

 
function Food(name, locationName){ 
 
    this.name = name; 
 
    this.locationName = locationName; 
 
    this.displayName = function(){ 
 
     alert(this.name); 
 
    }; 
 
}; 
 
function createFood(){ 
 
    var name = prompt("Enter the items name:"); 
 
    var locationName = prompt("Enter a location to store it:") 
 

 
    while(locationName in locations){ 
 
     alert("This is not an actual location."); 
 
     locationName = prompt("Please enter another location:"); 
 
    }; 
 

 
    var x = new Food(name, locationName) 
 

 
    function insertFood(food){ 
 
     locations[food.locationName].push(food); 
 

 
     var list = document.getElementById(food.locationName + "_items"); 
 
     var li = document.createElement("li"); 
 
     li.innerHTML = food.name; 
 
     list.insertBefore(li, list.lastChild); 
 
    }; 
 
    // call the method now? 
 
    insertFood(x); 
 
}; 
 
createFood();