shouldUseConditionalExpression para una condición múltiple con un return default

[fdodino]

En este código

class Parcial {
	const property cantidadPreguntas
}

class Alumno {
	const notas = []
	var tipoAlumno = "E"
	
	method rendir(nota) {
		notas.add(nota)
	}

  method estudiaPara(parcial) {
		if (tipoAlumno == "E") {
			// Estudioso
			return true
		}
		if (tipoAlumno == "V") {
			// Vago
			return notas.last() < 6
		}
		if (tipoAlumno == "H") {
			// Hijo del rigor
			return parcial.cantidadPreguntas() > 5
		}
		return false
	}
}

está saltando en el último if esta validación: Estás usando incorrectamente el if. Devolvé simplemente la expresión booleana.(shouldUseConditionalExpression). Raro.

[fdodino]

El warning se va si en la última instrucción tiramos error:

[PalumboN]

Y cuál sería el problema? Dónde se marca el error? O que en este caso querés aceptar esta estructura de ifs que devuelven booleanos?

El comportamiento que describís parece el correcto.

[fdodino]

Fijate que acá::

method estudiaPara(parcial) {
		if (tipoAlumno == "E") {
			// Estudioso
			return true
		}
		if (tipoAlumno == "V") {
			// Vago
			return notas.last() < 6
		}
              // acá me lo marca al if
		if (tipoAlumno == "H") {
			// Hijo del rigor
			return parcial.cantidadPreguntas() > 5
		}
              // acá me lo marca al if
		return false
	}

me está diciendo que estoy usando mal el if, que debería usar la expresión booleana pero yo estoy usando una expresión booleana en cada caso, no me convence que tenga que cambiarlo a

  method estudiaPara(parcial) {
		if (tipoAlumno == "E") {
			// Estudioso
			return true
		}
		if (tipoAlumno == "V") {
			// Vago
			return notas.last() < 6
		}
		return tipoAlumno != "H" || parcial.cantidadPreguntas() > 5
	}

(ojo, yo lo hago para programar en Wollok pero no para enseñar el strategy).

[PalumboN]

Ahh ok, sí es algo particular de este patrón de ifs.

Habría que definir una heurística de cuándo devolver booleanos usando if está bien y cuándo no... podría ser que si hay más de un if no tire esta validación 🤔