时间戳

首页 > 解决方案 > 返回不工作和过于复杂的代码

javascript - 返回不工作和过于复杂的代码

问题描述

作为更大功能的一部分，我试图简单地运行检查以查看数组中的值是否“增加” - 例如：

var a = [1,2,3,4,5] // is increasing
var a = [1,4,6,7,36] // is increasing
var a = [1,6,3,6,5] // not increasing

如果前一个值 a[previous] 小于 a[next]，它会增加。出于某种原因，问题是当它不增加时它不会返回 -1。而且，当我学习 JavaScript 时，我的代码似乎比它应该的要复杂得多。

我的问题：
1. 为什么当 a 不能增加时它不返回 -1？
2. 为什么这么简单的测试我的代码看起来那么复杂？我错过了什么吗？（也就是说，如果您认为它过于复杂，我的代码通常就是这样）

编写这样一个函数的更好方法是什么？我应该将“测试是否已经增加”部分放在单独的函数中吗？如果有人也可以提供关于编写更好、更简单、可读、更干燥的代码的提示，那将不胜感激:)

var a = [1,2,3,4,5]
var canIncrease = 0; // boolean

// test if already increased
for(i=0;i<a.length;i++) {
	if((a[i] < a[i+1] && i !== a.length-1)||(a[i] > a[i-1] && i==a.length-1)) {
		console.log('index ' + i + ' cannot increase');
	} else {
		console.log('index ' + i + ' can increase');
		canIncrease = 1;
	}
}

if (!canIncrease) {
	console.log('array a cannot increase');
	return -1;
} else {
	console.log('would continue');
	// continue with main function...
}

标签： javascript

解决方案

every()如果元素是最后一个元素或小于下一个元素，则可以使用并返回 true。

function func(arr) {
  return arr.every((o, i, a) => (i + 1) === a.length ||  o < a[i + 1] );
}

console.log(func([1, 2, 3, 4, 5]));
console.log(func([1, 4, 6, 7, 36]));
console.log(func([1, 6, 3, 6, 5]));

文档：每个（）

推荐阅读