将布尔值分配给局部变量并重用它还是使用if重复比较更好？

Question

想象一下我有这个代码（Razor 语法）：

<script type="text/javascript">
@{
    var i = 0;
    foreach (var notify in @Model)
    {
        if (i > 1) // <------ First comparison 
        {
            <text>setTimeout(function() {</text>
        }

        <text>
        // JavaScript
        </text>

        if (i > 1) // <------  Second same comparison
        {
            <text>}, 1000 * @i);</text>
        }

        i++;
    }
}
</script>

请注意，int i 被比较了两次。将第一次比较的结果与局部变量进行比较然后检查它的值还是进行第二次简单比较会更好吗？像这样：

<script type="text/javascript">
@{
    var i = 0;
    foreach (var notify in @Model)
    {
        bool higherThanOne = i > 1;
        if (higherThanOne)
        {
            <text>setTimeout(function() {</text>
        }

        <text>
        // JavaScript
        </text>

        if (higherThanOne)
        {
            <text>}, 1000 * @i);</text>
        }

        i++;
    }
}
</script>

我多次遇到类似的情况，但我不确定哪个更好。由于不友好的 Razor + JavaScript 连接语法，我想避免将代码分配给变量。

Answer 1

在这种情况下，是的，记录一下这个神奇的条件意味着什么：

bool wrapInTimeout = i > 1;

不要称它为higherThanOne。那将是一个无用的名字。

根据变量所代表的内容命名您的变量，而不是根据它们的计算方式。

Answer 2

在您的情况下比较非常简单，所以我看不出使用临时变量有什么好处，但是，如果您确实有更多相同的比较，那么您可能希望使用该变量，因为它简化了代码结构和可读性。

我还注意到，如果每次都调用同一个 javascript 函数，您实际上可以将这两个条件合二为一：

<script type="text/javascript">
@{
    var i = 0;
    foreach (var notify in @Model)
    {
        if (i > 1) // <------ First comparison 
        {
            <text>setTimeout(myFunction, 1000 * @i);</text>
        }

        i++;
    }

    <text>function myFunction(){ //your code here, if the same for all models }</text>
}
</script>