Código prolijo
Aquí tenemos un código con un problema de legibilidad. Y esto hace difícil entender las estructuras que corresponde cada bloque de código. Además, tenemos el problema de cómo posicionar las llaves.
<?php
if(isset($_POST['txtFechainicial']) && $_POST['txtFechainicial']!="")
{
$where[] = "fecha_recibo > ".$_POST['txtFechainicial']."";
}
if(isset($_POST['txtFechafinal']) && $_POST['txtFechafinal']!="")
{
$where[] = "fecha_recibo < ".$_POST['txtFechafinal']."";
}
if(isset($_POST['txtTrabajador']) && $_POST['txtTrabajador']!="")
{
$where[] = "trab_recibo = ".$_POST['txtTrabajador']."";
}
if(isset($_POST['txtSerie']) && $_POST['txtSerie']!="")
{
$where[] = "serie1 = ".$_POST['txtSerie']."";
}
La solución a este problema es agregar tabulaciones. La convención sobre las llaves que la llave de apertura debería estar al lado del paréntesis que cierra la condición de if
<?php
if (isset($_POST['txtFechainicial']) && $_POST['txtFechainicial'] != "") {
$where[] = "fecha_recibo > " . $_POST['txtFechainicial'] . "";
}
if (isset($_POST['txtFechafinal']) && $_POST['txtFechafinal'] != "") {
$where[] = "fecha_recibo < " . $_POST['txtFechafinal'] . "";
}
if (isset($_POST['txtTrabajador']) && $_POST['txtTrabajador'] != "") {
$where[] = "trab_recibo = " . $_POST['txtTrabajador'] . "";
}
if (isset($_POST['txtSerie']) && $_POST['txtSerie'] != "") {
$where[] = "serie1 = " . $_POST['txtSerie'] . "";
}
✨ Recuerda: El código fuente se escribe para humanos.
Otro problema que podemos encontrarnos es el siguiente, aquí hay una falta de criterio en común, algunas de las líneas utilizan llaves y otras no. Esto significa que tenemos una Falta de estándar de codificación.
<?php
if (isset($_GET['p']))
$p = $_GET['p'];
else
$p = 1;
$t = '';
if (isset($_GET['t']))
$t = $_GET['t'];
else if(isset($_GET['codigo']))
$t = $_GET['codigo'];
else if(isset($_POST['TxtBuscar']))
$t = $_POST['TxtBuscar'];
if (!empty($_POST['TxtBuscar'])) {
$cadena = $valor[0];
}
Este el problema resuelto, en nuestro caso decidimos que lo mejor es utilizar las llaves.
<?php
if (isset($_GET['p'])) {
$p = $_GET['p'];
} else {
$p = 1;
}
$t = '';
if (isset($_GET['t'])) {
$t = $_GET['t'];
} else if(isset($_GET['codigo'])) {
$t = $_GET['codigo'];
} else if(isset($_POST['TxtBuscar'])) {
$t = $_POST['TxtBuscar'];
}
if (!empty($_POST['TxtBuscar'])) {
$cadena = $valor[0];
}
Estándar de codificación
Es una serie de reglas que nos dice como escribir nuestro código. Esto nos ayuda generar un código claro y consistente. También, nos evita que pierdas tiempo en decisiones triviales.
Cómo mejorar la legibilidad de nuestro código
- Define un estándar
- Respétalo
- Apóyate en algún linter (es una herramienta)