Как правильно организовать поиск самой короткой строки в массиве?

Question

[Vlad_beg]

Нужно найти самую длинную и самую короткую строку в массиве, но данный код находит первое короткое слово а дальше не проверяет. Подскажите, что делаю не так?

var input = document.getElementsByClassName("input")[0].value.split(" ");
 	 var longest = 0, shortest = input[0], longWord, shortWord;
   var output = document.getElementsByClassName("output")[0];
 	
   for(var i = 0; i < input.length; i++) {

   		if (input[i].length < longest ) {
   				longest = input[i].length;
   				longWord = input[i]; 
   		} else if(input[i].length < shortest.length) {
   			shortest = input[i].length;
   			shortWord = input[i]; 
   		} 
   }
  output.innerHTML += "Longest Word is: \"" + longWord + "\".Length is: " + longest + "<br/>";
  output.innerHTML += "Shortest Word is: \"" + shortWord + "\".Length is: " + shortest + "<br/>";
  
  }

Answer 1

[Алексей Уколов]

При инициализации должно быть так:
shortest = input[0].length
Хотя, я бы вообще так сделал для наглядности:
shortest = Infinity

if (input[i].length < longest )
Неправильный знак, должно быть ">".

else if
Лучше просто отдельный if сделать, на случай, когда в исходном массиве одно слово - оно и самое длинное и самое короткое одновременно.

Ещё, я бы не хранил длину самого короткого и самого длинного слов, только их самих - длину легко получить через word.length, а код упрощается. Даже если здесь и будет какой-то штраф по производительности, им абсолютно точно можно пренебречь.

Comments

[Vlad_beg]

Большое спасибо, а не могли бы пояснить, почему в данном случаи не правильно использовать длинну для "shortest"
else if(input[i].length < shortest.length)
а именно при инициализации переменной "shortest"

[Алексей Уколов]

Во-первых, таким образом вы избавляетесь от ситуации, когда в переменной хранится то строка, то число. А во-вторых, если в shortest уже хранится число, а не строка, вызов shortest.length вернёт undefined.

[Vlad_beg]

Благодарю за объяснение