Оцените решенную задачу

[Vitaliy_m]

Оцените решенную задачу...Она простая, так что не судите строго, только учусь... Дайте дельные советы: как можно упросить код? где сделал ошибки? Спасибо.


/*
* Программа должна запрашивать несколько слов, выдавая на экран их характеристику.
* Например, «животное, муж. рода, ед. числа., имен. падеж)».
* После ввода всех слов, на экран выдается стихотворение, в котором часть слов заменены введенными с клавиатуры словами.
*/

var name = prompt("Укажите имя существа!");
name = name.toLowerCase();
var pol = prompt("Какой у него род?");
pol = pol.toLowerCase();
var stringLine = "У меня живёт то ли спит то ли я на достоинство стал и в обморок ";
var mass = stringLine.split(' ');


function addName(){
if (name != false){
massName = name.split();}
else throw alert("Вы не ввели имя существа!") ;
if (mass != undefined && mass instanceof Array)
{ return mass.splice(3,0,massName);}
else throw alert("Не создан массив!");}

function addPol(){
if (pol && typeof(pol) == "string"){

if (pol == "мужской"){
mass.splice(9,0,"здох");
mass.splice(13,0,"ему");
mass.splice(15,0,"он");
mass.splice(20,0,"упал");
}
else if (pol == "женский"){
mass.splice(9,0,"здохла");
mass.splice(13,0,"ей");
mass.splice(15,0,"она");
mass.splice(20,0,"упала");
}
else throw alert("Вы не правельно ввели род!"); }
else {throw alert("Вы не ввели Род!");}
return mass; }
addName();
addPol();
mass.splice(4,0,",");
mass.splice(11,0,",");
mass.splice(17,0,"-");
mass.splice(27,0,".");
alert(mass.join(' '));

[Deff]

Вы не правельно ввели род
Вы не правильно ввели род

Собственно вот эта конструкция занудна
mass.splice(20,0,"упал");
наверно нужно создать три массива(а не один), разбив их по тестовым проверкам,
Тогда можно искать текущее слово по индексу в массиве, не задавая жестко позиции проверяемых элементов массива
Удаление элемента массива по его названию

[Vitaliy_m]

Вы не правельно ввели род
Вы не правильно ввели род

Собственно вот эта конструкция занудна
mass.splice(20,0,"упал");
наверно нужно создать три массива(а не один), разбив их по тестовым проверкам,
Тогда можно искать текущее слово по индексу в массиве, не задавая жестко позиции проверяемых элементов массива
Удаление элемента массива по его названию

А тогда конструкцию - "mass.splice(20,0,"упал"); " , как можно обойти?

[Bandicoot]

mass.splice(9,0,"здох");

mass.splice(9,0,"здохла");

Я один не понимаю - как, не зная русского языка, можно писать код?))

[Vitaliy_m]

Я один не понимаю, как, не зная русского языка, можно писать код?))

Я не хотел парить себе голову стихотворением, решил написать такую фразу.. Суть то не в ней а в задаче=)

[Deff]

1.Плохо понятен алгоритм
2. Нет проверочного массива с тестовыми словами
"здox"
"eмy"
"oн"
"yпaл"
и спец символами , . - ?
3.Откуда беруться эти цифры ?
mass.splice(11,0,",");

[Vitaliy_m]

mass.splice(11,0,","); цифры это позиции слов куда они будут внедренны в массив.

[Vitaliy_m]

По логике того как я писал скрипт, я делал всё таким образом -
1) создал массив с стихотворением
2) проверка введённых данных
3) внедрение слов в массив

[Deff]

mass.splice(11,0,","); цифры это позиции слов куда они будут внедренны в массив.

Там есть mass.splice(20,0,"упал"); - в текущем массиве элементов всего 16, последний -пустой

[Vitaliy_m]

Да но по мере добавление элементов их будет 20, так как слово "упал" последнее то есть четвёртое..Сейчас сделаю с массивами пример скину код..он более оригинальней чем этот...