Hola paisa:
Te voy a dar pistas sobre uno de los errores... aunque debrían de habértelo enseñado mejor
if(valor<0 || >3)
Está línea significa coloquialmente
Si valor es menor que 0 o mayor que 3...
... pero al hacer una evaluación "OR" (||)... se chequea el valor booleano de la izquierda y lo mismo de la derecha...
Ahora desglosamos lo que está dentro del paréntesis y evaluamos...
valor < 0: si la variable valor existe se puede evaluar y dará un resultado lógico cierto/falso... true/false
luego evaluamos la parte de la derecha y nos encontramos:
>3:
Ahora como ejercicio adicional (los malditos deberes)
fijate en el resultado de poner alert(valor < 0) y alert(>3)
Espero haberte ayudado, pero coincido con KarlanKas (
)en que tienes que intentar resolverlo tú mismo...
Saludos
