Почему появляется ошибка reading 'toUpperCase'?

Question

[Максим Головлёв]

Подскажите, пожалуйста, что делаю не так.

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

import reverse from './reverse';

const convertText = (string) => {

  if (string[0] === '') {
    return ''
  }

  if (string[0] === string[0].toUpperCase()) {
    return string
  }
  else if (string[0] !== string[0].toUpperCase()) {
    return reverse(string)
  }
    else {
      return ''
    }
};

export default convertText;

Comments

[Rsa97]

Для какой входной строки выходит ошибка?
И ещё, в строке '123' первая буква заглавная или нет?

Answer 1

[AgentSmith72]

Раз попробуешь тернарные операторы и уже не сможешь от них отказаться)

const convertText = (string) => {
    return !string ? '' : (string[0] === string[0].toUpperCase() ? string : string.split('').reverse().join(''));
};

Comments

[Rsa97]

Вложенные тернарные операторы - моветон.

Answer 2

[Rsa97]

const isLetter = (c) => {
  if (c === '_' || c === '$') {
    return false;
  }
  try {
    eval(`let ${c};`);
    return true;
  } catch (e) {
    return false;
  }
}

const isUpperCase = (c) => c === c.toUpperCase();

const convertText = (string) => {
  for (let i = 0; i < string.length; i += 1) {
    if (isLetter(string[i])) {
      return isUpperCase(string[i]) ? string : reverse(string);
    }
  }
  return reverse(string);
}

С современными регулярками вспомогательные функции меняются на

const isLetter = (c) => /\p{L}/u.test(c);

const isUpperCase = (c) => /\p{Lu}/u.test(c);

Comments

[VoidVolker]

try {
    eval(`let ${c};`);
    return true;
}

Да вы извращенец, батенька!

[Rsa97]

VoidVolker, Ну, это вполне работающий способ определить, что символ - это буква. Алфавитов развелось столько, что перечислять вручную не получится.

[VoidVolker]

Оп, сюрприз от стандарта JS:

let ᾩ = "something";
let 〱〱〱〱 = "less than? wtf";

isLetter('ᾩ')
true
isLetter('〱〱〱〱')
true

[Rsa97]

VoidVolker,
ᾩ - Греческая заглавная буква омега с дасия и просгеграммены
〱- Вертикальная кана знак повтора
С точки зрения Unicode и то и другое - буквы (категория L)
В некоторые языки уже завезли регулярку по классам кодовых точек Unicode, в JS пока нет.
https://regex101.com/r/X3e7BH/1

[Rsa97]

VoidVolker, На самом деле у меня хуже с проверкой isUpperCase.
Во многих алфавитах нет понятия uppercase/lowercase, зато есть unicase (буквы всегда в одном регистре) или titlecase (диграфы или триграфы, используемые в начале слова, например Dz, Dž, ᾩ). По хорошему, для правильного распознавания нужна, опять же, современная регулярка \p{Lu}, но в JS её не завезли.

[SagePtr]

Rsa97, за все браузеры не ручаюсь, но в Хром, Firefox, а также в Nodejs - завезли.

[Rsa97]

SagePtr, Действительно. Видимо я где-то не так проверял, скорее всего не указал флаг u. Тогда всё упрощается

const isLetter = (c) => /\p{L}/u.test(c);

const isUpperCase = (c) => /\p{Lu}/u.test(c);

[VoidVolker]

О как, ну тогда, если рассматривать проблему именно с точки зрения "одно решение для всех алфавитов" - то да, вот такие костыльные извращения через эвал вероятно имеет смысл использовать. Я данную проблему рассматриваю с точки зрения "в каком языке/алфавите надо" и использую решение именно в рамках конкретного языка. За регулярку SagePtr спасибо! А вот за код в картинке - нет )