task 187

Этот сайт лучше всего просматривать в современном браузере с включённым JavaScript.

task 187

Maxim

Добрый день, подскажите в чем ошибка.

export const isIdentical = (obj1, obj2) => {
 const item = Object.values(obj1);
 const item2 = Object.values(obj2);
 let counter = 0;
 for (let i = 0; i < item.length; i++){
  for ( let j = 0; j < item2.length;j++){
    if( item[i] === item2[j]) {
    counter++;
  }
 }}
 return counter === item.length && counter === item2.length?true:false;
};

Проверки которые я делаю сам проходят корректно.

salton

У вас вас проверка зависит от последовательностей элементов в массиве. На самом деле элемент1 в первом объекте может соответствовать элементу 2 во втором объекте. Элемент 2 в первом объекте элементу1 во втором объекте. итд. Вы объекты превратили в массив и тем самым пытаетесь в первую очередь сравнить порядок элементов. В то время как в объектах возможно проверить наличие элемента не используя обход всего объекта.

Maxim

salton Мне казалось что я использую супер неэффективное решение через 2 цикла, но оно для меня самое надежное и простое, в моей голове механика работы моего когда такова -
1) я каждый элемент [i] сравниваю со всеми элементами [j].
2) при положительном результате сравнения, я плюсую counter.
3)если counter равен длинне массива то все элементы идентичные.
Я делав тесты и объекты менял местами и добавлял кучу своих, и менял те что есть. Проверки проходят корректно.

Я сейчас еще пойду покопаюсь в теории, но мне казалось такой вариант написания как раз таки дает ту самую независимость от последовательности элементов.

Coderslang_Master

Maxim ход мыслей очень логичный. Единственный нюанс в том, что порядок полей в объектах не имеет значения, поэтому ты не можешь сравнивать их используя индекс.

Эти два объекта равны, но твой код вернет false.

const obj1 = { b: 'f', a: 12, c: true },
const obj2 = { c: true, b: 'f', a: 12 },

Maxim

Coderslang_Master Я прошу прощение, я просто для себя хочу разобраться , где моя ошибка)
Не сочтите за хамство)
Ваш пример:

Результат:
Меняем одно поле в объекте:

Результат:

Я через conlose.log(item[1],item1[2]) проверил все элементы, и обращение к ним проходит верное, мне выводит ровно тот элемент к индексу которого я обращаюсь.

Coderslang_Master

Maxim я немного неправильно прочитал твой код и добавил неточный пример.

Твой код не проходит проверки потому что ты игнорируешь имена ключей объекта.

Проверь на таких объектах:

const cat = {
  name: 'Tom',
  age: 'Tom',
  isHappy: 'Tom',
}

const dog = {
  name: 'Tom',
  age: '2',
  isHappy: true,
}

Maxim

Coderslang_Master
Большое Спасибо!)