Много ошибок и неточностей:
1) onClick не требует указания протокола "javascript
:". Его указывать следует только в подстановке вместо url, например в аттрибуте href элемента <a>.
2) JavaScript выполняется последовательно только один раз, поэтому вам, в данном случае, нужно создавать функции, которые потом будут вызываться по определённым событиям.
3) Для того, чтобы получить целое число следует пользоваться генератором случайных чисел, а целочисленное деление, которое вы использовали, в данном случае работает неверно: вы могли бы легко получить числа вне диапазона 0-3. Лучше использовать подобный способ получения случайного числа:
parseInt(Math.random()*4)
Math.random() возвращает случайное число от 0 до 1, умножение на 4 увеличивает диапазон на отрезок от 0 до 3. А parseInt приводит это число к целому виду. Конечно связи с введенным текстом не будет никакой, но зато работает
