1

Tenho um loop que deveria inserir um senha aleatória para cada registro, e quando terminar o loop (ter criado a senha diferente para cada registro), listar os resultados visualizando a senha.

 <?php
ob_start();
session_start();

//conexão com banco
include ("conn.php");

//definição de variáveis
$tabela = "chatoperator";
$campos = "operatorid, vclogin, dtmlastvisited, istatus, vcpassword"; 
$quant = 10; //número de ações que será realizada de cada vez 
$sec = 10; //tempo entre o envio de um pacote e outro (em segundos) 
$ok = 0;
$inicio = 0;
$fim = $inicio + $quant; 
$acentos = "<meta http-equiv='Content-Type' content='text/html; charset=utf-8' />";

//verifica se a senha confere com a da sessão
echo $acentos;

$sql = "select $campos from $tabela where istatus = 0 limit $inicio,$fim" or die (mysql_error($sql)); 
$query = mysql_query($sql,$conexao); 
$registros = mysql_num_rows($query); 

if($registros==0){
    mysql_query("update $tabela set istatus = 0" or die (mysql_error($conexao))); 
        echo("<font face=’tahoma’>todas as senhas foram alteradas!</font>"); 
    $ok = 1;
}else{
    while($result = mysql_fetch_array($query)) {
        $operatorid = $result[0]; //operatorid
        $vclogin = $result[1]; //vclogin
        $dtmlastvisited = $result[2]; //dtmlastvisited
        $istatus = $result[3]; //istatus
        $vcpassword = $result[4]; //vcpassword

        //gera senha aleatória
        function geraSenha(){
        //caracteres que serão usados na senha randomica
            $chars = 'abcdxyswzABCDZYWSZ0123456789';
            //ve o tamnha maximo que a senha pode ter
            $max = strlen($chars) - 1;
        //declara $senha
            $senha = null;

        //loop que gerará a senha de 8 caracteres
            for($i=0;$i < 8; $i++){
            $senha .= $chars{mt_rand(0,$max)};
    }
    return $senha;          
}
$senha_randomica   =  geraSenha();
$senha = md5($senha_randomica);

mysql_query("update $tabela set istatus = 1 AND vcpassword = '$senha_randomica' where    operatorid = $operatorid") or die (mysql_error($conexao));
    echo("
<table width='100%' border='1'>
    <tr>
        <th>Identificador do Usuário</th>
        <th>Login do Usuário</th>
        <th>Data do Último Acesso</th>
        <th>Status do Acesso</th>
        <th>Senha do Usuário</th>
   </tr>
   <tr>
        <td> $operatorid </td>
        <td> $vclogin </td>
        <td> $dtmlastvisited </td>
        <td> $istatus </td>
        <td> $senha_randomica </td>
  </tr>
 </table>");
      }
   }
    mysql_free_result($query); 
    mysql_close($conexao);

    if(!$ok){
        echo("<meta http-equiv=\"refresh\" content=\"" . $sec . "\">"); 
}
?>
3
  • Qual é sua dificuldade?
    – Maniero
    2/04/2015 às 12:22
  • O script está gerando a MESMA senha para todos os registros, a idéia é que fosse gerada UMA senha diferente para cada registro. Obrigado pelo retorno @bigown 2/04/2015 às 12:28
  • Sem indentação fica difícil entender o que está acontecendo. Se voc~e editar a pergunta com melhor formatação do código, quem sabe até você mesmo encontre algum erro.
    – Maniero
    2/04/2015 às 12:55

1 Resposta 1

4

O problema parece ser apenas que a função geraSenha está dentro do laço. Coloque ela antes de todo este código.

function geraSenha() {
    //caracteres que serão usados na senha randomica
    $chars = 'abcdxyswzABCDZYWSZ0123456789';
    //ve o tamnha maximo que a senha pode ter
    $max = strlen($chars) - 1;
    //declara $senha
    $senha = null;
    
    //loop que gerará a senha de 8 caracteres
    for ($i=0;$i < 8; $i++) {
        $senha .= $chars{mt_rand(0,$max)};
    }
    return $senha;          
}
$i = 0;
while ($i < 10) {
    echo geraSenha() . "\n";
    $i++;
}

Veja funcionando no ideone. E no repl.it. Também coloquei no GitHub para referência futura.

A pergunta fala claramente em problema para criar a senha e de fato sem a alteração acima eu testei e só gera uma. Agora nos comentários o problema parece ser outro. Há um erro também na gravação dos dados. Precisaria ser assim (se eu entendi a intenção).

mysql_query("update $tabela set istatus = 1, vcpassword = '$senha_randomica' where    operatorid = $operatorid") or die (mysql_error($conexao));

Então ficarias assim (não tem como eu testar um código que não segue o MVCE.

<?php
function geraSenha(){
    $chars = 'abcdxyswzABCDZYWSZ0123456789';
    $max = strlen($chars) - 1;
    $senha = "";
    for($i=0;$i < 8; $i++){
        $senha .= $chars{mt_rand(0,$max)};
    }
    return $senha;          
}

ob_start();
session_start();

//conexão com banco
include ("conn.php");

//definição de variáveis
$tabela = "chatoperator";
$campos = "operatorid, vclogin, dtmlastvisited, istatus, vcpassword"; 
$quant = 10; //número de ações que será realizada de cada vez 
$sec = 10; //tempo entre o envio de um pacote e outro (em segundos) 
$ok = 0;
$inicio = 0;
$fim = $inicio + $quant; 
$acentos = "<meta http-equiv='Content-Type' content='text/html; charset=utf-8' />";

//verifica se a senha confere com a da sessão
echo $acentos;

$sql = "select $campos from $tabela where istatus = 0 limit $inicio,$fim" or die (mysql_error($sql)); 
$query = mysql_query($sql,$conexao); 
$registros = mysql_num_rows($query); 

if($registros==0){
    mysql_query("update $tabela set istatus = 0" or die (mysql_error($conexao))); 
        echo("<font face=’tahoma’>todas as senhas foram alteradas!</font>"); 
    $ok = 1;
}else{
    while($result = mysql_fetch_array($query)) {
        $operatorid = $result[0]; //operatorid
        $vclogin = $result[1]; //vclogin
        $dtmlastvisited = $result[2]; //dtmlastvisited
        $istatus = $result[3]; //istatus
        $vcpassword = $result[4]; //vcpassword

        $senha_randomica   =  geraSenha();
        $senha = md5($senha_randomica);

        mysql_query("update $tabela set istatus = 1, vcpassword = '$senha_randomica' where    operatorid = $operatorid") or die (mysql_error($conexao));
        echo("
<table width='100%' border='1'>
    <tr>
        <th>Identificador do Usuário</th>
        <th>Login do Usuário</th>
        <th>Data do Último Acesso</th>
        <th>Status do Acesso</th>
        <th>Senha do Usuário</th>
    </tr>
    <tr>
        <td> $operatorid </td>
        <td> $vclogin </td>
        <td> $dtmlastvisited </td>
        <td> $istatus </td>
        <td> $senha_randomica </td>
    </tr>
</table>"
        );
    }
}
mysql_free_result($query); 
mysql_close($conexao);

if(!$ok){
    echo("<meta http-equiv=\"refresh\" content=\"" . $sec . "\">"); 
}
?>
13
  • obrigado pelo retorno, porém tua solução gera 10 senhas (ou quantas eu parametrizar) mas ela não gera uma para cada registro dentro do loop, pode ajudar quanto a isto? Obrigado 2/04/2015 às 13:29
  • 1
    A minha solução gera quantas você quiser. basta tirar a função geraSenha de dentro do laço. Eu não tenho como mostrar seu código funcionando por isto eu adaptei ele para mostrar que funciona. Eu vi que você não consegue indentar o código corretamente, então provavelmente se testou a minha solução com seu código deve ter mudado parte do código errado. Sem você entender claramente onde começa e termina uma coisa, fica difícil saber quais partes do código estão dando problema e quais precisam ser mexidas. A função deve ser movida para o início do código, mais nada.
    – Maniero
    2/04/2015 às 13:34
  • Bom, indentar o código corretamente é só uma questão de visualização, isso não reflete o saber ou não de algo, mas tudo bem, não alterei nada do seu código, aliás é o mesmo que eu já havia testado anteriormente, porém, mesmo fora do laço/loop a senha é gerada, mas o problema é inserir uma em cada registro, e não a mesma em todos os registros. Se precisar de mais alguma informação para entender a MINHA dúvida para poder me ajudar, só informar, obrigado @bigown 2/04/2015 às 13:46
  • 2
    @Eduardo, se a solução apresentada não resolveu seu caso, acho melhor você detalhar melhor a sua dificuldade, pois testando com um código simplificado aqui as senhas não repetem. E eu espero que você não entenda como crítica, mas eu demorei mais tempo para conseguir ler seu código do que para testar a solução do bigown. Até entendo que é seu direito usar um código todo bagunçado, mas já que você quer ajuda, podia indentar direito o código para facilitar pra quem está tendo boa vontade de ajudar. Inclusive, quanto mais organizado o programador, mais chance de achar ou nem sequer cometer erros.
    – Largato
    2/04/2015 às 13:56
  • 1
    @EduardoCorreia eu editei para colocar a parte do problema que você não havia citado antes em nenhum local.
    – Maniero
    2/04/2015 às 15:39

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 .