Почему мое решение задачи не верно?

Question

[Ily4farmer]

Задача такая:
Напишите функцию areArraysSame, которая принимает два массива с числами в качестве аргументов. Она должна возвращать true, если эти массивы одинаковые (то есть содержат одни и те же числа в одном и том же порядке), или false, если массивы различаются. Убедитесь, что ваша функция работает правильно, запустив такой код:

areArraysSame([1, 2, 3], [4, 5, 6]);
false
areArraysSame([1, 2, 3], [1, 2, 3]);
true
areArraysSame([1, 2, 3], [1, 2, 3, 4]);
false

Решение задачи такое:

var areArraysSame = function (argument1, argument2) {
			for (var i = 0; i < argument1.length; i++) {
				if (argument1[i] !== argument2[i] || argument1.length !== argument2.length) {
				return false;
			} 
			}
			return true;
		}

Я же написал так:

var areArraysSame = function (argument1, argument2) {
			for (var i = 0; i < argument1.length; i++) {
				if (argument1[i] !== argument2[i] || argument1.length !== argument2.length) {
				return false;
			} else {
				return true;
			}
			}
		}

Пожалуйста объясните мне, почему мое решение неверно?

Answer 1

[Александр]

На сравнении первого элемента она выдаст ответ не проверяя следующие элементы. В первом варианте выдаст true пройдя все элементы массива.

Comments

[Ily4farmer]

а почему так? он же по идее должен еще и по остальным элементам пройтись

[0ffff0]

Ily4farmer, else не используется, когда в условии есть или. Тупой компилятор не понимает, что вы от него хотите, потому что вы его совсем запутали. Поэтому. Если хотите через else, то надо не или, а и

// если хотим через конструкцию if else 
(
if (ушки большие и серенький и большой и длинный нос)
 { назвать слоником }//только при совпадении всех примет сходство истинно
else { не называть слоником }//сходство ложно при несовпадении всех примет 
)
// конструкция выше имеет недостаток, при находжении //несоответствия продолжает проверку и возвращает результат //только после вычисления истинности всего выражения условия

*************************************************************************
//как избавиться от ненужных вычислений
(
if (ушки маленькие или не серенький или небольшой или недлинный нос)
 { не назвать не слоником }
  назвать слоником
) 
//программа построенная таким образом ищет "неслоника" поэтому //при первом же несовпадении сообщит нам об этом.
// но если всунуть else, который для такого случая не предусмотрен, то  корректной работы ждать не придётся, вычислив первое условие,
интерпретатор пойдет по ветке else, игнорируя все, что там дальше

Answer 2

[Павел Корнилов]

https://codepen.io/KorniloFF/pen/jdWXWj?editors=1011

function areArraysSame (argument1, argument2) {
  if(argument1.length !== argument2.length) return false;
    for (var i = 0; i < argument1.length; i++) {
    if (argument1[i] === argument2[i]) {
        continue;
      } else {
        return false;
      }
    }
  return true;
 };

// test
console.log(areArraysSame([1,2,3], [1,2,3]));
console.log(areArraysSame([1,2,3], [1,2,'3']));
console.log(areArraysSame([1,2,3], [1,2,3, 4]));

Answer 3

[mastevoy]

/*areArraysSame([1, 2, 3], [4, 5, 6]); false
areArraysSame([1, 2, 3], [1, 2, 3]); true
areArraysSame([1, 2, 3], [1, 2, 3, 4]); false
*/
function areArraysSame (m1,m2)
{
for(var i = 0; i < m1.length; i++)
{
if(m1[i] !== m2[i] || m1.length !== m2.length )
{
console.log (false);
}
else
{
console.log (true);
}
}

}

у меня все получилось. Просто return не использовал.