Стоит задача реализовать функцию, через которую можно управлять телефонной книгой.
Для управления телефонной книгой нужно реализовать три команды:
ADD — добавляет контакт
SHOW —возвращает содержимое телефонной книг
REMOVE_PHONE — удаляет номер
Команда ADD Добавляет контакт в телефонную книгу со списком телефонов. Телефоны перечисляются через запятую. Если такой контакт существует, то команда пополняет список телефонов контакта Пример команды ADD: ADD Ivan 555-10-01,555-10-03.
Команда REMOVE_PHONE Удаляет телефон из телефонной книги. Если телефон успешно удалён, то функция должна вернуть true. Если такого телефона в телефонной книге не существует, то возвращается false.
Команда SHOW Возвращает массив контактов с их телефонами. Массив содержит строчки вида: "Имя: Телефон1, Телефон2". Массив должен быть отсортирован по имени контакта. Телефоны идут в порядке добавления их в телефонную книгу. Контакт с пустым списком телефонов не должен возвращаться.
Я очень слаб в js,поэтому написал (говно)код,который проходит 3 теста из 13(они закрытые). Если выполнять код на локальной машине,то все соответствует образцу. Пожалуйста,подскажите, где программа ведет себя или может вести себя неправильно,уже все перерыл. Буду очень благодарен за любую помощь. Спасибо. P.s - написал комментарии скорее для себя, что бы было понимание происходящего. Если они тупые и очевидные,прошу прощения.
var phoneBook = {}; // используем структуру объекта,где свойство - имя, значение - массив номеров,принадлежащих этому имени.
/**
* @param {String} command
* @returns {*} - результат зависит от команды
*/
module.exports = function (command) { //для того,что бы вызывать из другого файла
commands = command.split(' ');
indef = commands[0]; //первый элемент в строке - команда
if(indef == 'ADD'){
name = commands[1]; //2 - имя
numbers = commands[2].split(','); //3 - строка с номерами через запятую,которую приводим к массиву
if(!phoneBook.hasOwnProperty(name)){ //если такого имени нету в объекте
phoneBook[name] = numbers; //добавляем его и соотносим с массивом номеров
}
else { //если имя есть
for(i = 0; i < numbers.length; i++){
phoneBook[name].push(numbers[i]); //добавляем в уже существующий массив номера
}
}
}
else if(indef == 'REMOVE_PHONE') {
numb_remove = commands[1];
for(name in phoneBook){ //перебираем имена
if(phoneBook[name].includes(numb_remove)){ //перебираем номера и если есть
x = phoneBook[name].indexOf(numb_remove); //получаем его индекс
phoneBook[name].splice(x,1);//удаляем из объекта
return true;
}
}
return false;
}
else if(indef = 'SHOW'){
var names = Object.keys(phoneBook).sort(); //получаем список имен и сортируем его
var book = ['\n']; //создаем массив строк, который будет выводится на экран
for (i = 0; i < names.length; i++){
book.push(names[i] + ': ' + phoneBook[names[i]].join(',') + '\n'); //задание структуры name1: phone1,phone2
// name2: phone1,phone2
}
if(book.length > 0){
return book;
}
}
};
else if (indef = 'SHOW') {
— присваивание вместо сравнения. Исправьте и посмотрите, сколько тестов будет пройдено.if(book.length > 0)
всегда верно).const
илиlet
. Сейчас вы создаёте кучу глобальные переменных. Вроде в данном коде это не проблема, если я ничего не пропустил, но это рано или поздно может иметь плохие и трудно уловимые последствия.