Siempre va ser mas sencillo leer un if sin anidar y dentro de lo posible es buena idea evitarlo por claridad, pero eso no garantiza que el código sea legible. Para mi legible es que lo pueda leer con facilidad otro programador así que una forma de validar eso, es que le preguntes a alguien mas dentro de tu equipo si entiende lo que hiciste. Por ejemplo para mi algo mas legible seria que tuvieras algo en el controller como esto:
Código PHP:
Ver originalif (!is_request('post')) {
return view('registrarse');
}
if (!is_valid($_POST)) {
return json_response('Datos no validos', 400);
}
return json_response($data, 200);
Por la sintaxis debes de estar usando CI, pero debe de ser posible que puedas agregar tus propios helpers y clases que hagan mas sencillo a otro programador la lectura de tu código. Un buen síntoma de que algo anda mal es que tengas controladores con métodos muy grandes eso indica que esos métodos están realizando mas cosas y es momento de refactorizarlos para que metas esas operaciones en otras clases o helpers.
Te recomiendo que investigues sobre los siguientes temas:
- tell don´t ask
- SOLID
- GRASP
- Refactoring
- Testing