代码之家 › 专栏 › 技术社区 › espresso_coffee

有没有更好的方法来减少多个if语句?

ternary-operator if-statement javascript

espresso_coffee · 技术社区 · 7 年前

我有一个函数可以验证几个不同的条件。以下是我的函数示例:

function checkData() {
  var errorMsg = "",
    fld1 = 0, //Number(document.getElementById('fld1').value),
    fld2 = 5, //Number(document.getElementById('fld2').value),
    fld3 = 1, //Number(document.getElementById('fld3').value),
    fld4 = 0; //Number(document.getElementById('fld4').value);

  if (!fld1) {
    errorMsg += "Error 1\n\n";
  }

  if (fld1 === fld4) {
    errorMsg += "Error 2\n\n";
  }

  if (fld2 > fld4) {
    errorMsg += "Error 3\n\n";
  }

  if (fld3 > 3) {
    errorMsg += "Error 4\n\n";
  }

  if (errorMsg !== "") {
    var check = confirm(errorMsg + "\n Do you want to submit the form?");

    if (check) {
      return true;
    } else {
      return false;
    }
  }

  return true;
}

<button onclick="checkData();">Click Here</button>

在上面的示例中,为了测试目的,我硬编码了一些值。但是,我想知道是否可以重构这段代码,并找到更好的方法来获得相同的结果?三元运算符是否更适合?或者有其他方法让它发挥作用?谢谢您。

3 回复 | 直到 7 年前

MarcoS 7 年前

在这个用例中,我认为“多个IFS”解决方案是非常清楚的,所以它是要使用的解决方案。

如果你想优化一点,我只能建议

        if(check){
            return true;
        }else{
            return false;
        }

成为

return !!check;

(这两个感叹号只是将任何对象强制转换为布尔值:-)

Alen.Toma 7 年前

整体 check 变量是无意义的。所以 return confirm 你只需要

function checkData() {
	var errorMsg = "",
		fld1 = 0, //Number(document.getElementById('fld1').value),
		fld2 = 5,//Number(document.getElementById('fld2').value),
		fld3 = 1,//Number(document.getElementById('fld3').value),
		fld4 = 0;//Number(document.getElementById('fld4').value);

	if(!fld1){
		errorMsg += "Error 1\n\n";
	}

	if(fld1 === fld4){
		errorMsg += "Error 2\n\n";
	}

	if(fld2 > fld4){
		errorMsg += "Error 3\n\n";
	}

	if(fld3 > 3){
		errorMsg += "Error 4\n\n";
	}

    return errorMsg !== "" ? confirm(errorMsg + "\n Do you want to submit the form?") : true

}

<button onclick="checkData();">Click Here</button>

Adam 7 年前

可以使用三元运算符重构if语句。但很可能它会使您的代码更难阅读。你可以替换

if(check){
    return true;
}else{
    return false;
}

用公正 return check; 因为这是一个布尔语句。

此外,就可读性而言,最好给字段变量贴上更有意义的标签,因为知道 fld2 应该总是大于 fld4 从名字上看并不明显。

如果您不关心突出显示特定的错误代码,那么您当然可以将一些检查合并在一起,然后返回 false 如果没有指定的错误代码,但我怀疑您希望保留该功能。

推荐文章

DIES · 多动作语句的公共if或三元运算符?

8 年前

Leo Messi · 三元内部的空合并运算符

8 年前

Haikel · 是否可以将这两行写为三元运算符?

8 年前

puregeek · 在C#中,三元运算符给出错误:只能将赋值、调用、递增、递减和新对象表达式用作语句

8 年前

Srinivasan · 在C中使用多个三元运算符调用时,不会区分bool和enum类型重载的函数++

8 年前

JustADude · setter与Classic If中的三元运算

8 年前

elight · 为什么两个单独计算的javascript字符串表达式不能正确地一起计算?[副本]

8 年前

wells · Ruby each Iterator-根据if条件替换要迭代的哈希?

8 年前

Franck Boudraa · Ruby/Rails三元和范围

8 年前

Aldos · PHP null coalesce+三元运算符奇怪的行为

8 年前