skrócony zapis instrukcji If - wykonywanie fałszu

Stworzyłem funkcję, która reaguje na naciśnięcie klawisza Enter:

function stopEnter(e)
{
    // 1 wersja
    if (e.keyCode == 13)
    {
        e.preventDefault();
    }

    // 2 wersja
    (e.keyCode == 13) ? e.preventDefault() : false;
}

Czy drugi sposób (skrócony zapis) jest dopuszczalny? Czy mogę "wykonać fałsz"?

Możesz, ale czy warto?
można różne skróty robić np.:
e.keyCode == 13 && e.preventDefault()
(ja robię czasem coś w tym stylu foo.someMethod && foo.someMethod() jak chcę wywołać metodę obiektu tylko wtedy, gdy istnieje).
tylko pytanie o czytelność. Ify dobrze komunikują intencję, patrzysz na kod i rozumiesz, co ma robić, a np. tutaj:

(e.keyCode == 13) ? e.preventDefault() : false;

to po co to false? Nawet jeśli będzie działać, to może budzić wątpliwości "po co"?

poza tym e.keyCode jest już odradzane: https://developer.mozilla.org/en-US/docs/Web/API/KeyboardEvent/keyCode

// 2 wersja
    (e.keyCode == 13) ? e.preventDefault() : false;

Ta wersja zapewnia pałowanie na code review :P

Ta instrukcja jest przeznaczona do tego żeby zwrócić wartość zależną od warunku logicznego. Czyli żeby zamiast:

let x;
if (blah) { x = 10; } else { x = 11; }

Móc napisać:

let x = (blah) ? 10 : 11

Nie należy jej używać w innych przypadkach.

I to też tylko wtedy gdy obie gałęzie są bardzo proste np. stała lub wywołanie funkcji. Przy bardziej skomplikowanej logice lepszy if lub osobna metoda.

Czyli za takie coś nie będzie pałowania :)

let x = (blah) ? mojaFunkcja(parametrA) : mojaFunkcja(parametrB)

Odnośnie

(e.keyCode == 13) ? e.preventDefault() : false;

Widziałem że community Webpacka używało czegoś takiego:

e.keyCode != 13 || e.preventDefault();

lub odwrotnie z &&
Do pomijania drugiej części ternary.

Można czytelniej :)

document.addEventListener('keydown', stopEnter);

function stopEnter(e)
{   
    if (e.code == 'Enter') 
...