То что вы пытаетесь написать:
if (question === 1||3)
Значит: "условие ИЛИ какое-то число, эквивалентно true". То есть 3 - это эквивалент true. Ноль будет эквивалентом false. Посмотрите, чему будут эквиваленту отрицательные числа, мне лень смотреть документацию или пробовать консоль.
В вашем случае, решение будет выглядеть так:
if (question === 1|| question === 3)
Если охота использовать перечисление в надежде сократить условие, то это может выглядеть так:
if ([1,3].indexOf(question) > -1)