Улучшать код основного PHP [в резерве]

Хорошие это фрагмент CRUD, в моем приложении у меня есть следующий код, который функционирует хорошо, он приносит необходимые данные. Вопрос, если он, если я могу улучшать или осуществлять другой способ звонить в консультацию SQL. Я хочу довести до сведения, что у меня нет ничто из опыта в PHP и - я первый приложение этого типа. Спасибо.

<!DOCTYPE html>
<html lang="es">

<?php include('includes/connection.php') ?>
<?php include('includes/extras.php') ?>

<body>
<main>
<?php include('includes/header.php') ?>
<section class="container mt-5">
<form action="#" method="POST" class="form-group">
<div class="card">
<div class="card-header bg-info text-white text-center">
<div class="lead">Busqueda entre fechas</div>
</div>
<div class="card-body">
<input name="fechadesde" type="date" class="form-control mt-2" required>
<input name="fechahasta" type="date" class="form-control mt-2" required>
<input name="buscar" type="submit" class="form-control btn btn-success mt-2" value="Buscar">
<?php
if (isset($_POST['buscar'])) {
$fechadesde = $_POST['fechadesde'];
$fechahasta = $_POST['fechahasta'];
$sql = "SELECT registros.Apellido, registros.Nombre, liquidaciones.Importe, liquidaciones.Observaciones,
liquidaciones.Fecha
FROM registros
LEFT JOIN liquidaciones ON (registros.id = liquidaciones.idSocio)                            
WHERE liquidaciones.Fecha BETWEEN '{$fechadesde}' AND ' {$fechahasta}'";
$result = mysqli_query($conn, $sql);
if ($result) {
?>
<table class="table table-striped table-active mt-5 table-responsive">
<thead class="thead-dark">
<tr>
<th class="text-center" scope="col">Apellido</th>
<th class="text-center" scope="col">Nombre</th>
<th class="text-center" scope="col">Importe</th>
<th class="text-center" scope="col">Fecha</th>
<th class="text-center" scope="col">Observaciones</th>
</tr>
</thead>
<tbody>
<?php while ($row = $result->fetch_assoc()) { ?>
<tr>
<td class="text-center align-middle"><?php echo $row["Apellido"] ?></td>
<td class="text-center align-middle"><?php echo $row["Nombre"] ?></td>
<td class="text-center align-middle"><?php echo $row["Importe"] ?></td>
<td class="text-center align-middle"><?php echo $row["Fecha"] ?></td>
<td class="text-center align-middle"><?php echo $row["Observaciones"] ?></td>
</tr>
</tbody>
<?php }; ?>
</table>
</div>
<div class="card-footer text-success text-right">
<?php
$sql = "SELECT SUM(liquidaciones.Importe) AS Suma
FROM liquidaciones
WHERE liquidaciones.Fecha BETWEEN '{$fechadesde}' AND ' {$fechahasta}'";
$result = mysqli_query($conn, $sql);
while ($row = $result->fetch_assoc()) {
?> <h5>Suma: <?php echo $row['Suma'] ?> </h5>
<?php }; ?>
</div>
<?php
} else {
?>
<script type="text/javascript">
var msg = alertify.error('');
msg.delay(2).setContent('No se encontraron datos');
</script>
<?php
}
mysqli_close($conn);
};
?>
</div>
</div>
</form>
</section>
</main>
</body>

</html>

Хорошие, это фрагмент CRUD, в моем приложении у меня есть следующий код, который функционирует хорошо, он приносит необходимые данные. Вопрос, если он, если я могу улучшать или осуществлять другой способ звонить в консультацию SQL. Я хочу довести до сведения, что у меня нет ничто из опыта в PHP и - я первый приложение этого типа. Спасибо.

0
задан 21.10.2019, 07:52
1 ответ

Первое, что ты должен делать, состоит в том, чтобы быть немного лучше организовано. Я имею в виду помещать include все вместе. Это оказывается очень дешевым, includes помещают один здесь и другой там.

Другая сшей, что у тебя есть, что aprencer - это, чтобы не включать код PHP внутри формуляра. Это не обозначает элегантность в способе кодировать твое приложение. Но это что-то, что ты должен изучать со временем, так как ты новый.

Был бы хорошим таким же образом как у тебя был файл aparete, чтобы присоединяться в базу данных и потом ты используешь include.

Из равного способа ты можешь использовать это:

$query = “SELECT firstname, lastname, employee_id
FROM employees”;
$results = mysql_query($query);
while(list($firstname, $lastname, $employee_id) = 
mysql_fetch_row($results)) {
// ...
}

ты догадаешься, что ты можешь использовать for loop, это другие случаи. Метод List () будет давать тебе большую гибкость, потому что у тебя могут быть переменные в местах, которые ты хочешь.

-2
ответ дан 24.10.2019, 23:35