1

Bom dia, estou aprendendo PHP e criei um sistema de login. Gostaria de ajuda com o código para alterar o status do usuário logado.

Eu tenho uma lista com usuários que estão online e, ao rodar esse script, queria alterar o status do usuário que já tiver passado mais de 50 minutos com status online, só que ele está alterando o status de todos os usuários.

Esse é meu código:

$pdo = new PDO("mysql: host=localhost; dbname=pdo","root","");

$view = $pdo->prepare("SELECT agora,lastaccesso FROM users");
$view->execute();

foreach ($view as $mostra):
    $lastaccesso = $mostra['lastaccesso']; 
    $agora = $mostra['agora']; 
    $horaNova = strtotime("$lastaccesso + 5 minutes");
    $horaNovaFormatada = date("H:i", $horaNova);
    if ($horaNovaFormatada  < $agora):
        $altera = $pdo->prepare("UPDATE users SET online='" . $sim_nao . "'");
        $altera->execute();
        if($altera):
            echo "<script>window.location = 'online.php'</script>";
        else:
            echo "erro";
        endif;
endforeach;
6
  • voce está setando essa variavel $sim_nao em algum lugar? 12/09/2018 às 12:35
  • Aparentemente você não está fechando o primeiro if 12/09/2018 às 12:38
  • eu havia esquecido da variavel e do if , ja coloquei e o resultado é o mesmo . ele altera status de todos os usuarios , mas eu so quero que seja aterado dos usuarios que estao a mais de 1 hora com status online 12/09/2018 às 12:47
  • tentei esse codigo mas esta me retornando erro . 12/09/2018 às 12:48
  • $con=mysqli_connect("localhost","root","","pdo"); if (mysqli_connect_errno()) { echo "Failed to connect to MySQL: " . mysqli_connect_error(); } $result = mysqli_query($con,"SELECT id_user, TIMEDIFF(lastacesso,agora) AS diferenca FROM users"); $row = mysqli_fetch_array($result); $sim_nao = ($row["diferenca"] > 10) ? '0' : '1'; mysqli_query($con,"UPDATE users SET online='".$sim_nao."' where diferenca >= -00:25:18 "); mysqli_free_result($result); mysqli_close($con); ?> 12/09/2018 às 12:48

1 Resposta 1

0

Veja que você está dando um update sem especificar um id, gerando assim uma alteração em todos os usuários.

Outra coisa, se você está fazendo uma preparação da query, por que não usar um placeholder na mesma?

No se if, Tente algo como:

if ($horaNovaFormatada  < $agora):
    $altera = $pdo->prepare("UPDATE users SET online= ? WHERE id = ?");
    $altera->bindValue(array($sim_nao, $iddouser));
    $altera->execute();
    if($altera):
        echo "<script>window.location = 'online.php'</script>";
    else:
        echo "erro";
    endif;

Troque a variável $iddouser pelo id do usuário.

Espero que que esteja gravando o id do usuário logado em algum local.

2
  • entendi , fiz com id e deu certo , mas queria poder varrer banco e verifica todos os usuarios quando tento fazer assim , todos os status mudam de uma vez só . 12/09/2018 às 14:01
  • Usando o id do usuário logado,a alteração irá se aplicar a todos os usuários do sistema. Como você está identificando os usuários logados do seu sistema? Está gravando o id em algum lugar?
    – Jorge.M
    12/09/2018 às 14:10

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .