Em que pese não ser a melhor prática, não há problema em redirecionar para a mesma página, desde que você saiba o que está fazendo e consiga escrever sua lógica de uma forma que nunca caia em um redirecionamento infinito (o navegador na verdade cancela o loop quando verifica que há muitos redirecionamentos para a mesma página, lançando algo como ERR_TOO_MANY_REDIRECTS
). Assim, não chega a travar a navegação.
No seu problema específico, você está apenas verificando se o a variável $link
(que pelo contexto, depreende-se que é uma conexão aberta com o mysql
) não é null
, ou seja: sempre (se tudo estiver certo).
Para evitar isto, ao invés de verificar se há conexão, verifique se a url está enviando os parâmetros para salvar. Sugiro substituircolocar acima do seu if($link) {
por algo assim:
if ( $idSeg && $seg && $idTerca && $tercaa && $idQuarta && $quartaa &&
$idQuinta && $quintaa && $idSexta && $sextaa && $idSabado && $sabadoo ) {
if($link) {
/* funções de update */
header("Location: AdmAgenda.php");
} else{
die("Erro: " .mysqli_error($link));
}
}
Isso implicaria em só atualizar o banco e redirecionar se todos estes parâmetros estiverem presentes (não sei se esta é a lógica da sua aplicação, mas é apenas para fins de explicação, você adapta). QuandoUma abordagem mais clean é verificar um a um. Feito isto, quando você redirecionar, os parâmetros não irão ser enviados e, consequentemente, não cairão neste if
.