var a = 1;
function myFunction() {
++a;
return true;
}
// Alert pops up.
if (myFunction() && a === 2) {
alert("Hello, world!");
}
// Alert does not pop up.
if (a === 3 && myFunction()) {
alert("Hello, universe!");
}
https://jsfiddle.net/3oda22e4/6/
myFunction会增加一个变量并返回某些东西。如果我在包含它递增的变量的if语句中使用这样的函数，则条件将取决于顺序。
这样做是好还是坏的实践，为什么？
条件的顺序是否有变化，取决于您是否更改了条件中使用的变量。您用作示例的两个if语句是不同的，无论您是否使用myFunction()，它们都将是不同的。它们等价于：
if (myFunction()) {
if (a === 2) {
alert("Hello, world!")
}
}
// Alert does not pop up.
if (a === 3) {
if (myFunction()) {
alert("Hello, universe!")
}
}
我认为你代码中的不良实践不在于你在条件语句中更改条件操作数的值，而在于你的应用程序状态被暴露和操纵在一个函数内部，而该函数甚至不接受此状态变化变量作为参数。我们通常尝试将函数与其范围外的代码隔离，并使用它们的返回值来影响代码的其余部分。全局变量90%的时间都是一个坏主意，随着代码库的逐渐增大，它们往往会产生难以追踪、调试和解决的问题。
这是不好的编程习惯，原因如下：
与良好构建的代码相比，该代码可读性较差。如果第三方稍后检查该代码，则这一点非常重要。
如果稍后更改myfunction，则代码流程将变得完全不可预测，并可能需要进行重大文档更新。
小而简单的更改可能会对代码的执行产生极大的影响。
看起来很业余。
MyFunction 违反了一项原则，称为“Tell, Don't Ask”（说出来，不要问）。
MyFunction 改变了某些东西的状态，因此它成为了一个命令。如果 MyFunction 成功或无法增加 a，则不应返回 true 或 false。它被赋予了一项任务，必须尝试成功，如果发现该任务目前不可能完成，则应抛出异常。
在 if 语句的谓词中，MyFunction 被用作查询。
一般而言，查询不应呈现副作用（即不更改可以观察到的事物）。好的查询就像计算一样，对于相同的输入，它应该产生相同的输出（有时被描述为“幂等性”）。
还要知道这些都是指导方针，可以帮助您和其他人推理代码。会引起混乱的代码终究会出错。
有一些好的模式，比如 Trier-Doer 模式，可以像您的代码示例那样使用，但每个阅读它的人都必须通过名称和结构理解发生了什么。
put the function call before the if:
function tryChangeA() {
a++;
return true;
}
var ok = tryChangeA();
if (ok && a == 2) … // alternatively: if (a == 2 && ok)
make the mutation explicit inside the if:
function testSomething(val) {
return true;
}
if (testSomething(++a) && a == 2) …
put the logic inside the called function:
function changeAndTest() {
a++;
return a == 2;
}
if (changeAndTest()) …
changeAndTest不再是一个纯谓词，但有些模式（特别是循环）需要在条件中执行类似于regex.match(string)或i--的操作，而我并没有看到使用临时变量的更简洁的变体相比之下有多大的优势。 - Bergivar a = 1;
function myFunction() {
++a; // 1
return true;
}
if (myFunction() && a === 2) { // 2, 3, 4
alert("Hello, world!")
}
if (a === 3 && myFunction()) { // 2, 3, 4
alert("Hello, universe!")
}
在不同的作用域中改变变量的值。这可能会有问题，但通常是有问题的。
在if语句条件中调用函数。
本身并不会引起问题，但并不是很清晰。更好的做法是将该函数的结果赋值给一个变量，可能使用一个描述性的名称。这将帮助读代码的人理解您想要在if语句中检查什么。顺便说一下，该函数总是返回true。
使用了一些魔术数字。 想象一下其他人阅读该代码，而它是大型代码库的一部分。这些数字代表什么意思？更好的解决方案是用良好命名的常量替换它们。
如果您想支持更多消息，则需要添加更多条件。 更好的方法是使其可配置。
我会将代码重写如下：
const ALERT_CONDITIONS = { // 4
WORLD_MENACE: 2,
UNIVERSE_MENACE: 3,
};
const alertsList = [
{
message: 'Hello world',
condition: ALERT_CONDITIONS.WORLD_MENACE,
},
{
message: 'Hello universe',
condition: ALERT_CONDITIONS.UNIVERSE_MENACE,
},
];
class AlertManager {
constructor(config, defaultMessage) {
this.counter = 0; // 1
this.config = config; // 2
this.defaultMessage = defaultMessage;
}
incrementCounter() {
this.counter++;
}
showAlert() {
this.incrementCounter();
let customMessageBroadcasted = false;
this.config.forEach(entry => { //2
if (entry.condition === this.counter) {
console.log(entry.message);
customMessageBroadcasted = true; // 3
}
});
if (!customMessageBroadcasted) {
console.log(this.defaultMessage)
}
}
}
const alertManager = new AlertManager(alertsList, 'Nothing to alert');
alertManager.showAlert();
alertManager.showAlert();
alertManager.showAlert();
alertManager.showAlert();
使用具有精确功能的类，该类使用自己的内部状态，而不是一堆依赖于任何可能位于任何位置的变量的函数。是否使用类取决于您的选择，也可以以不同的方式完成。
使用配置文件。这意味着如果您想要添加更多消息，则无需修改代码。例如，假设配置来自数据库。
正如您可能注意到的那样，这会改变函数外部作用域中的变量，但在这种情况下不会造成任何问题。
使用具有清晰名称的常量。（好吧，它可能更好，但请容忍我给出的示例）。
const hasFuel = driveCar(1000);
if (!hasFuel) {
refill();
} - Giacomo Cosimato一个改变事物的函数。这个世界要变成什么样子？这个函数必须改变事物并在每次调用时返回不同的值。
考虑一副扑克牌的dealCard函数。它发牌1-52。每次调用它时，它应该返回一个不同的值。
function dealCard() {
++a;
return cards(a);
}
/* 我们假设数组cards已经洗牌 */
/* 为了简洁起见，我们假设这副牌是无限的，并且不会在52张牌后重新开始 */
a“属于”dealCards，并且每个人都同意对该方法的调用更改a，那么就没有问题。Ruby数组shuffle只会给你一个全新的已经被洗牌过的数组，所以甚至不会影响你的东西。 - Greg