Как исправить данную ошибку?

Question

[Влад]

Есть массив с русским ключом:


пытаюсь найти его данным способом:

el.status = ("Статус") in el.changelog ? el.changelog["Статус"].after : 'не назначен';

В консоли выдает ошибку:

Comments

[Дмитрий]

Он же пишет английским по белому что не может искать черную кошку в undefined, потому что ее там нет и быть не может. Пользуйтесь отладчиком и проверяйте, что у вас содержится в el

[SagePtr]

el чему равна?

[Влад]

SagePtr,

const initialState = [];
const changed = ' изменён статус запроса на';
var log = [];
var comments = [];

export function queryLog(state = initialState, action): object {
    if (action.type === constants.LOAD_QUERY_LOG_STOP) {
        log = action.queryLog;
    }
    if (action.type === constants.LOAD_COMMENTS_STOP) {
        comments = action.comments;
    }

    let summ = log.concat(comments).map(el => {
        el.date = el.dt_changelog * 1000 || el.datetime * 1000;
        el.type = el.dt_changelog ? 'changelog' : 'comment';
        el.text = el.dt_changelog ? changed : el.text;
        el.author = el.dt_changelog ? el.usr.source : el.author;
        el.status = ("Статус") in el.changelog ? el.changelog["Статус"].after : 'не назначен';
        return el;
    });

    console.log('!!!!!!!!!!!!!!!!!!!!!');
    console.log(comments);
    console.log(log);
    console.log(summ);

    summ.sort(function(a, b) {
        if (a.date > b.date) {
            return 1;
        }
        if (a.date < b.date) {
            return -1;
        }
        return 0;
    });
    return summ;
}

[Дмитрий]

Влад, Он спросил вас чему равно значение el - посмотрите в отладчике перед ошибкой...

[Влад]

Дмитрий,

[Роман]

Влад, el у вас массив. Тоесть чтобы обратится к самому первому элементу вам надо везде заменить el.changelog на el[0].changelog, но в этом случае ваш код обработает только первый элемент.
чтобы обработать все элементы, вам надо сделать примерно так:

el.forEach(item=>{
   item.status = "Статус" in item.changelog ? item.changelog["Статус"].after : "не назначен";
});

[Роман]

Влад, вернее даже так:

// ...
    let summ = log.concat(comments).map(items => {
      items.forEach(el=>{
        el.date = el.dt_changelog * 1000 || el.datetime * 1000;
        el.type = el.dt_changelog ? 'changelog' : 'comment';
        el.text = el.dt_changelog ? changed : el.text;
        el.author = el.dt_changelog ? el.usr.source : el.author;
        el.status = ("Статус") in el.changelog ? el.changelog["Статус"].after : 'не назначен';
      });
      return items;
    });
// ...

[Дмитрий]

Влад, В браузере в девтулз есть вкладка watch, там можно ввести любую переменную и посмотреть ее значение предварительно брейкпойнтом остановив программу в нужном месте.

Answer 1

[Роман]

Ошибка говорит о том, что el.changelog не определен

Покажите что выводит в консоль команда console.log(el);

Comments

[Влад]

[Роман]

Влад, el у вас массив. Тоесть чтобы обратится к самому первому элементу вам надо везде заменить el.changelog на el[0].changelog, но в этом случае ваш код обработает только первый элемент.
чтобы обработать все элементы, вам надо сделать примерно так:

// ...
    let summ = log.concat(comments).map(items => {
      items.forEach(el=>{
        el.date = el.dt_changelog * 1000 || el.datetime * 1000;
        el.type = el.dt_changelog ? 'changelog' : 'comment';
        el.text = el.dt_changelog ? changed : el.text;
        el.author = el.dt_changelog ? el.usr.source : el.author;
        el.status = ("Статус") in el.changelog ? el.changelog["Статус"].after : 'не назначен';
      });
      return items;
    });
// ...

[Влад]

Роман, ага, понял.
Но в консоле теперь другая ошибка

[Роман]

Влад, исправил, скопируйте еще раз

[Влад]

Роман, вот такая штука теперь

[Роман]

Влад, покажите что выведет команда
console.log(typeof items)

вставить сразу после
let summ = log.concat(comments).map(items => {

[Влад]

Роман,

[Роман]

Влад, а теперь покажите что выведет команда
console.log(items)

вставить сразу после
let summ = log.concat(comments).map(items => {

[Влад]

Роман,

[Роман]

Влад, вообще ничего теперь не понятно))))

то у вас в el был массив


то теперь в нем объект

[Влад]

Роман, делал всё по вашей инструкции

[Роман]

Влад, давайте теперь так сделаем:

export function queryLog(state = initialState, action): object {
    if (action.type === constants.LOAD_QUERY_LOG_STOP) {
        log = action.queryLog;
    }
    if (action.type === constants.LOAD_COMMENTS_STOP) {
        comments = action.comments;
    }
    const list = log.concat(comments);
    console.log("list:", list); // ----------- 1 ----------------

    let summ = list.map(el => {
        console.log("el:", el); // ----------- 2 ----------------

        el.date = el.dt_changelog * 1000 || el.datetime * 1000;
        el.type = el.dt_changelog ? 'changelog' : 'comment';
        el.text = el.dt_changelog ? changed : el.text;
        el.author = el.dt_changelog ? el.usr.source : el.author;
        el.status = ("Статус") in el.changelog ? el.changelog["Статус"].after : 'не назначен';
        return el;
    });

    console.log('!!!!!!!!!!!!!!!!!!!!!');
    console.log(comments);
    console.log(log);
    console.log(summ);

    summ.sort(function(a, b) {
        if (a.date > b.date) {
            return 1;
        }
        if (a.date < b.date) {
            return -1;
        }
        return 0;
    });
    return summ;
}

и покажите что дают эти два console.log-а которые я вставил в этот код

[Влад]

Роман,

[Роман]

Отлично, теперь скажите откуда вы взяли эту картинку (ту что в первом вашем комментарии к ответу) ????
Как только мы это поймем, ответ будет готов)))

другими словами почему тогда у вас
console.log("el:", el); выдавал массив а сейчас выдает объект.

[Влад]

Роман, не все данные прилетели


вот, что сейчас выдает консоль

[Роман]

Влад, и как я понимаю теперь ошибок не дает и все работает?

[Роман]

Влад, хотя нет, судя по тому что вывело в 6-м элементе массива list, то на этом элементе выкинет ошибку, так как в нем нет свойства changelog

[Влад]

Роман,

[Роман]

Влад, вам надо понять, почему для некоторых авторов changelog есть а для некоторых его нет. Понять должно ли так быть. Если должно, то перед тем как обращаться к el.changelog в строке

el.status = ("Статус") in el.changelog ? el.changelog["Статус"].after : 'не назначен';

нужно проверять его на существование примерно так:

el.status = el.changelog ? ("Статус") in el.changelog ? el.changelog["Статус"].after : 'не назначен' : 'нет лога изменений';

[Влад]

Роман, Вооооооо, сразу все заработало!!
Огромное спасибо вам.

[Роман]

Влад, тут вам пишет:

TypeError: Cannot use `in` operator to search for `Статус` for undefined

что означает:

вы не можете использовать оператор in для поиска "Статус" если переменная в которой производится поиск НЕОПРЕДЕЛЕНА

тоесть (смотрим на этот кусочек ("Статус") in el.changelog)
el.changelog не определена!!!!!

таким образом перед обращением к el.changelog надо проверить, существует ли он:

el.status = el.changelog ? ("Статус") in el.changelog ? el.changelog["Статус"].after : 'не назначен' : 'нет лога изменений';

[Влад]

Роман,
Восхитительно. Спаасибо еще раз!
Я только начинаю вливаться в разработку. Так что, не судите строго )

[Роман]

Влад, да не за что. Обращайтесь))))