Есть ли более приятный способ сделать это?

Question

Кажется, я часто обращаюсь с такими частными случаями. Должен быть более лаконичный синтаксис или конструкция:

var x = solveForX(); /* some slow calculation here */
if (x < 0)
{
    x = 0;
}

Это эквивалентно, но не выглядит более элегантно:

var x;
x = (x = solveForX()) < 0 ? 0 : x;

Может быть, есть небольшая хитрость?

---

Обновление: Я провел некоторые тесты , чтобы сравнить два моих любимых ответа - тот, который я принял, и Питера Айтая. Оказывается, Питер немного быстрее! Выполнение 1 000 000 итераций каждой (я также запустил версию, которая кеширует Math.max, чтобы увидеть, сколько времени внес вклад в поиск), показывает, что Питер работает меньше, чем половина версии Math.max, даже с кэшированием max. *

Тем не менее, даже самый медленный метод все еще довольно быстр.

Answer 1

Как насчет

var x = Math.max(solveForX(), 0);

Answer 2

Что-то вроде:

x = Math.max(0, solveForX());

Answer 3

(x < 0) && (x = 0);

Редактировать: удален оператор if.Спасибо Андреас.

Это одна строка, и понятно, что она делает (по моему личному мнению) - , если вы знакомы с булевой оценкой короткого замыкания.

Выше используется логическая оценка короткого замыкания .Это может быть очень полезно в определенных ситуациях (особенно арифметика указателей в C ++, но логическое вычисление короткого замыкания также работает в Javascript).

x = 0 вычисляется только, если x < 0.

Вот два примера:

Это оповещение 1:

<script type="text/javascript">
    var x = 1;
    (x < 0) && (x = 0);
    alert(x);
</script>

Это оповещение 0:

<script type="text/javascript">
    var x = -1;
    (x < 0) && (x = 0);
    alert(x);
</script>

Answer 4

Я бы украсил оригинальную функцию solveForX.

function returnNonNegative(fn) {
    function _f() {
        var x = fn();
        if (x < 0) {
            x = 0;
        }
        return x;
    }
    return _f;
}

solveForX = returnNonNegative(solveForX);

В данном конкретном случае использование Math.max кажется вполне приемлемым, но этот шаблон обеспечивает общее решение для этого типа проблем.

Answer 5

Я думаю, что этот способ довольно хорош!

var x = Math.max(solveForX(), 0);

Удачи!

Answer 6

Принятый ответ идеален.Если вы хотите выполнить то же самое без вызова функции, я считаю, что это наиболее кратко:

var x;
(x = solveForX()) > 0 || (x = 0);

---

(В Safari эта реализация на целых 7% быстреечем Math.max(), что, вероятно, не стоит беспокоиться)