Блог

 this.breakintoletters=()=>
        this.lengthi!==0?(this.title2=this.title,this.title2.split(),this.title2.
            map((x)=>this.arol.push(new letter(x))))
                        :!!false 
  

Итак, в основном это код. Предполагается, что он разбивает строку на буквы, а затем помещает соответствующие объекты в массив..
Выполняется проверка длины строки, если не 0, возвращает ошибку, в которой находится функция MAP …) TypeError сообщает мне, что это не функция. Редактор не показывает ошибки. Был бы признателен за помощь

Я предлагаю использовать другой подход, проверяя this.lengthi заранее и возвращая либо false , либо позже отображенную длину переданных значений.

 this.breakintoletters = () => {
    if (!this.lengthi) return false;
    this.title2 = this.title;
    // this.title2.split(); the result is not taken, not even splitted
    return this.title2.map(x => this.arol.push(new letter(x)));
};  

Вы не присваиваете разделенное значение обратно this.title2 и не используете map для this.split2 , которое является строкой

 this.breakintoletters=()=>
        this.lengthi  ? (this.title2=this.title,
                         this.title2=this.title2.split(), 
                         this.title2.map((x)=>this.arol.push(new letter(x))))
                      :false
  

ИМО, вы должны попытаться сделать свой код понятным только до такой степени, чтобы он оставался доступным для чтения, вы можете просто сделать это следующим образом

 this.breakintoletters = ( ) => {
     if(this.lengthi === 0 ) return false;
     this.title2=this.title;
     this.title2=this.title2.split(); 
     return this.title2.map((x) => this.arol.push(new letter(x)))) 
 }
  

(this.title2=this.title, this.title2.split(), this.title2. map((x)=>this.arol.push(новая буква(x))))

this.arol Это имя массива?

Попробуйте реструктурировать его, чтобы быть:

(this.title.split().map((x) => this.arol.push(новая буква(x)))

Такие методы, как split() join() map () и т.д., Могут быть объединены в цепочку.

Я бы переосмыслил использование функции map здесь, хотя и троичный, который другие комментаторы рассмотрели выше. Я имею в виду, что технически это работает, но если цель состоит в том, чтобы выполнить итерацию по строке, чтобы получить определенные значения, было бы лучше использовать цикл for. Функции отображения больше, когда вы хотите выполнить итерацию, чтобы применить один и тот же указанный метод к каждому отдельному символу

Кроме того, это всего лишь форматирование, но оно значительно облегчает чтение и понимание вашего кода, когда у вас есть некоторые пробелы между переменными и операторами, и вы выбираете имена переменных, которые имеют смысл для того, что вы делаете (this.bookLength , this.reverseAr), или, по крайней мере, используя общий this.array или this.arr. Это облегчает задавать подобные вопросы, потому что вы получите меньше уточняющих вопросов об опечатках, а также, если вы когда-либо планируете работать над большей базой кода, это важно писать чистый код, понятный тем, кто вас не знает