boas práticas de programação em php
Gostaria de saber boas práticas de programação e melhorias que eu poderia fazer nessa página php, mesmo vocês não sabendo sobre o que se trata, isso nao importa muito, pois só quero saber se existe como melhorar meus updates, pois como vocês podem ver tem horas que executo 2 updates ao mesmo tempo.
<?php
include_once("../../conexao/conexao.php");
include_once("../../visualizarFilas/includes/filaSOL.php");
function gravarHistorico($usuario, $data, $horario, $status, $id_ordem, $id_solicitacao, $conn)
{
$status = '7';
$query_insert = "insert into tbl_Historico
(analista_responsavel, data_modificacao, hora_modificacao, fk_status, fk_ordem_judicial, fk_id_solicitacao)
values
('$usuario', '$data', '$horario', '$status', '$id_ordem', '$id_solicitacao')";
mysql_query($query_insert, $conn);
$linhas_afetadas = mysql_affected_rows();
if($linhas_afetadas > 0)
{
$mensagem = "Histórico salvo com sucesso.";
/*
echo "<script>alert('$mensagem')</script>";
*/
}
}
function verificaStatus($id_ordem, $id_solicitacao, $usuario, $conn)
{
$status = '7';
$query_consulta = "select count(fk_status) as total,
(SELECT quant_sol from tbl_ordem_judicial where id_ordem='$id_ordem') as totalSol
from tbl_Identificacao_Usu
where fk_ordem_judicial = '$id_ordem' and
fk_status = '$status'";
$result = mysql_query($query_consulta, $conn);
$total = mysql_result($result, 0, "total");
$totalSol = mysql_result($result, 0, "totalSol");
if($total == $totalSol)
{
$query_update = "update tbl_ordem_judicial
set fk_status = '5'
where id_ordem = '$id_ordem'";
mysql_query($query_update, $conn);
$linhas_afetadas = mysql_affected_rows();
if($linhas_afetadas > 0)
{
$mensagem = "Ordem Judicial número: $id_ordem foi concluída!";
echo "<script>alert('$mensagem')</script>";
gravarHistorico($usuario, date('Y-m-d'), date('H:i:s'), '7', $id_ordem, $id_solicitacao, $conn);
/*
echo "<table width=500 border=1 cellpadding=3 cellspacing=0 bordercolor=f2f2f2>";
echo "<tr>";
echo "<td>Solicitação Finalizada</td>";
echo "</tr>";
echo "</table>";
*/
echo "<script>location.href='../../visualizarFilas/filaSolicitacoes.php'</script>";
}
}
else
{
gravarHistorico($usuario, date('Y-m-d'), date('H:i:s'), $status, $id_ordem, $id_solicitacao, $conn);
/*
echo "<table width=500 border=1 cellpadding=3 cellspacing=0 bordercolor=f2f2f2>";
echo "<tr>";
echo "<td class=titulo>Solicitação Finalizada</td>";
echo "</tr>";
echo "</table>";
*/
echo "<script>location.href='../../visualizarFilas/filaSolicitacoes.php'</script>";
}
}
function finalizarSolicitacao($id_ordem, $id_solicitacao, $usuario, $conn)
{
//verificando a cor que a solicitação está sendo finalizada
$cor = verificaSinal($id_ordem);
if($cor == "<img src='../imagens/sinalVerde.gif'>")
{
$cor = '1';
}
else if($cor == "<img src='../imagens/sinalAmarelo.gif'>")
{
$cor = '2';
}
else if($cor == "<img src='../imagens/sinalVermelho.gif'>")
{
$cor = '3';
}
//fim
//status 'Finalizada'
$status = '7';
$query_update = "update tbl_Identificacao_Usu
set fk_status = '$status', fk_sinal = '$cor'
where id_solicitacao = '$id_solicitacao'";
$query_update2 = "update tbl_TOJ_Identificacao
set dataTratamentoFinal = now()
where fk_id_solicitacao = '$id_solicitacao'";
mysql_query($query_update, $conn);
mysql_query($query_update2, $conn);
$linhas_afetadas = mysql_affected_rows();
if($linhas_afetadas > 0)
{
echo "<script>alert('A Solicitação: $id_solicitacao foi finalizada com sucesso!')</script>";
verificaStatus($id_ordem, $id_solicitacao, $usuario, $conn);
}
else
{
echo "error na ordem judicial<br>";
}
}
function verificarSolicitacao($id_ordem, $id_solicitacao, $usuario, $conn)
{
$query = "select analista_responsavel
from tbl_TOJ_Identificacao
where fk_id_solicitacao = '$id_solicitacao'";
$result = mysql_query($query, $conn);
$linhas = mysql_num_rows($result);
$analista = mysql_result($result, 0, "analista_responsavel");
if(!empty($analista))
{
finalizarSolicitacao($id_ordem, $id_solicitacao, $usuario, $conn);
}
else
{
echo "<script>alert('Esta solicitação não pode ser finalizada pois ainda não foi tratada!')</script>";
echo "<script>history.back();</script>";
}
}
$id_ordem = $_GET["id_ordem"];
$id_solicitacao = $_GET["id_solicitacao"];
$usuario = $login_acesso;
verificarSolicitacao($id_ordem, $id_solicitacao, $usuario, $conn);
?>
<html>
<head>
<link rel="stylesheet" type="text/css" href="../css/estilo.css">
</head>
<body>
</body>
</html>Discussão (2)
Carregando comentários...