Почему возникает TypeError?

Question

[atsin4469]

Задание
Реализуйте функцию convertText(), которая принимает на вход строку и, если первая буква не заглавная, возвращает перевернутый вариант исходной строки. Если первая буква заглавная, то строка возвращается без изменений.
reverse() импортируется

Решение

const convertText = (text) => (text[0].toUpperCase() === text[0] || text === '' ? text : reverse(text));

Ошибка

TypeError: Cannot read property 'toUpperCase' of undefined

Непонятно, почему "undefined".

Comments

[Lynn «Кофеман»]

Например вам передали пустую строку.

Переставьте условия местами

[Tim]

Lynn «Кофеман», пустая строка не вызовет такую ошибку. Там передают не строку и отсутсвует метод toUpperCase

[Lynn «Кофеман»]

Tim, у пустой строки text[0] будет undefined и будет ровно эта ошибка

[Tim]

Lynn «Кофеман», если у пустой строки брать нулевой индекс, то там уже будет не строка

[Lynn «Кофеман»]

Tim, правильно. Там будет undefined. И именно в этом и есть ошибка автора

Answer 1

[Rsa97]

console.log(""[0])
// undefined

Comments

[atsin4469]

Если вместо text[0].toUpperCase()
написатьtext.toUpperCase()[0]
то код сработает.

Так можно делать? Это нормальная практика?

[Rsa97]

atsin4469,

console.log("".toUpperCase()[0])
// undefined

В результате получаете сравнение undefined === undefined
Технически работать будет, но я бы так делать не стал и добавил бы отдельную проверку на пустую строку.

Answer 2

[Developer]

Чему text равен?
Полагаю, что undefined. Потому и ошибка

Answer 3

[Kirill Mager]

В Вашем случае когда в массиве значение под индексом 0 равно undefined функция вылетает с ошибкой.
Для того, чтобы избежать такого поведения необходимо добавить проверку на undefined

В вашей функции это будет выглядеть следующим образом:

const convertText2 = (text) => {

  if (text[0] === undefined) {
    return 'Undefined value!';
  }

   return text[0].toUpperCase() === text[0] || text === '' ? text : reverse(text)
};

по хорошему перед тем как писать логику функции Вам необходимо проверить что text это массив (text.isArray()), раз Вы обращаетесь к его первому элементу.
+ раз уж это массив, а Вы дальше будете писать логику по прохождению каждого элемента массива, то перед этой логикой для каждого элемента проверку на undefined и null необходимо добавить