operator - ¿Cómo escribo una compleja condición multilínea en Ruby?

ruby selenium if else

Question

¿Cómo escribo esta declaración de condición compleja multi-línea en Ruby?

 if ( (aa != nil && self.prop1 == aa.decrypt)
|| (bb != nil && self.prop2 == bb.decrypt)
) && (self.id.nil? || self.id != id)
return true
end

Estoy obteniendo un Syntax error; unexpected tOROP Syntax error; unexpected tOROP .

En Java, podría escribir

if (
( (aa != null && aa.prop1.equals(aa.decrypt()))
|| (bb != null && bb.prop2.equals(bb.decrypt()))
)
&& (this.id != id)
) {
return true;
}

Answer 1

Debe escapar del espacio en blanco con la barra diagonal inversa, es feo pero lo necesita si desea dividir las condiciones en varias líneas. O eso o deja el operador booleano en la línea anterior. Entonces cualquiera de estos funcionará:

if ( (aa != nil && self.prop1 == aa.decrypt)/
|| (bb != nil && self.prop2 == bb.decrypt)/
) && (self.id.nil? || self.id != id)
return true
end

o:

if ( (aa != nil && self.prop1 == aa.decrypt) ||
(bb != nil && self.prop2 == bb.decrypt)) &&
(self.id.nil? || self.id != id)
return true
end

Personalmente, por lo general, decido poner todas o cada una de las condiciones en un método que documenta por sí mismo lo que se está decidiendo:

def everythings_cool?
( (aa != nil && self.prop1 == aa.decrypt) ||
(bb != nil && self.prop2 == bb.decrypt)) &&
(self.id.nil? || self.id != id)
end

entonces:

if everythings_cool?
# do stuff

Answer 2

La respuesta corta es que el operador debe estar al final de la línea para decirle a Ruby que continúe leyendo la siguiente línea como parte de la declaración, así que esto funcionaría:

if ( (aa != nil && self.prop1 == aa.decrypt) ||
(bb != nil && self.prop2 == bb.decrypt) ) &&
(self.id.nil? || self.id != id)
return true
end

Dicho esto, probablemente puedas reducir la lógica lanzando excepciones basadas en valores de entrada y eliminando algunas comprobaciones redundantes (estoy haciendo algunos saltos aquí sobre cómo se verán tus variables, pero entiendes la idea).

raise ''aa must support decrypt'' unless aa.respond_to? :decrypt
raise ''bb must support decrypt'' unless bb.respond_to? :decrypt
if prop1 == aa.decrypt || prop2 == bb.decrypt
if self.id != id
return true
end
end