Skip to main content
adicionou 379 caracteres ao conteúdo
Fonte Link
luiscrjr
  • 1,5mil
  • 8
  • 17

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.

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.

Para evitar isto, ao invés de verificar se há conexão, verifique se a url está enviando os parâmetros para salvar. Sugiro substituir seu if($link) { por algo assim:

if ( $idSeg && $seg && $idTerca && $tercaa && $idQuarta && $quartaa &&
     $idQuinta && $quintaa && $idSexta && $sextaa && $idSabado && $sabadoo ) {

Isso implicaria em só atualizar o banco e redirecionar se todos estes parâmetros estiverem presentes. Quando você redirecionar, os parâmetros não irão e, consequentemente, não cairão neste if.

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 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 colocar acima do seu if($link) { 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). Uma 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.

Fonte Link
luiscrjr
  • 1,5mil
  • 8
  • 17

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.

Para evitar isto, ao invés de verificar se há conexão, verifique se a url está enviando os parâmetros para salvar. Sugiro substituir seu if($link) { por algo assim:

if ( $idSeg && $seg && $idTerca && $tercaa && $idQuarta && $quartaa &&
     $idQuinta && $quintaa && $idSexta && $sextaa && $idSabado && $sabadoo ) {

Isso implicaria em só atualizar o banco e redirecionar se todos estes parâmetros estiverem presentes. Quando você redirecionar, os parâmetros não irão e, consequentemente, não cairão neste if.