Problema logico if

**programmerscommunity** · 21-03-2014, 18.37.38

Salve a tutti;
ho un problema nell'esecuzione di un if: questa pagian riceve da un form vari dati tra cui il numero di linea ($num) con cui verificare la disponibilità di un prodotto. Il problema è che sebbene questo prodotto sia disponbile ($numoff = 5) l'if lo considera come nullo e da il messaggio d'errore preimpostato. Qui il codice interessato:

Codice PHP:


<?php

    session_start();

    require_once('mysqlconn.php');

    mysql_select_db($database, $offerte);

    $bSuccesso = true;

    $num = $_GET['num'];

    $nome = $_POST['nome'];

    $cognome = $_POST['cognome'];

    $mail = $_POST['email'];

    $nascita = $_POST['nascita'];

    $provincia = $_POST['provincia'];

    $indirizzo = $_indirizzo['indirizzo'];

    $control = mysql_query("SELECT * FROM offerte WHERE num = '".$num."' ") or die(mysql_error());

    echo "SELECT * FROM offerte WHERE num = '".$num."' ";

    $info = mysql_fetch_array($control);

    $numoff = $resrow['numoff'];

    echo $numoff;

    if ($numoff == 0) {

        echo "Non è possibile completare l'operazione: l'offerta richiesta è esaurita";

        } else {.......

**miki92** · 21-03-2014, 19.01.06

Il problema penso sia questo:

Codice PHP:


    $info = mysql_fetch_array($control); 
    $numoff = $resrow['numoff'];

Cambialo in:

Codice PHP:


    $info = mysql_fetch_array($control); 
    $numoff = $info['numoff'];

**programmerscommunity** · 21-03-2014, 19.14.48

Era proprio quello il problema, ora però mi da:
Fatal error: Cannot redeclare rpass() (previously declared in /membri/programmerscommunity/confbuy.php:26) in /membri/programmerscommunity/confbuy.php on line 26

**programmerscommunity** · 21-03-2014, 19.15.39

Qui il codice completo:

Codice PHP:


<?php

    session_start();

    require_once('mysqlconn.php');

    mysql_select_db($database, $offerte);

    $bSuccesso = true;

    $num = $_GET['num'];

    $nome = $_POST['nome'];

    $cognome = $_POST['cognome'];

    $mail = $_POST['email'];

    $nascita = $_POST['nascita'];

    $provincia = $_POST['provincia'];

    $indirizzo = $_indirizzo['indirizzo'];

    $control = mysql_query("SELECT * FROM offerte WHERE num = '".$num."' ") or die(mysql_error());

    $info = mysql_fetch_array($control);

    $numoff = $info['numoff'];

    if ($numoff == 0) {

        echo "Non è possibile completare l'operazione: l'offerta richiesta è esaurita";

        } else {

    $numoff = $numoff - 1;

    $countdown = mysql_query("UPDATE offerte SET numoff = '".$numoff."' WHERE num = '".$num."' ") or die(mysql_error());

    //INIZIO GENERAZIONE CODICE 

    do {

    mysql_select_db($database, $acquisti);

    function rPass($length = 6) {



      // Questo array contiene i caratteri con i quali

      // saranno create le password

      $lettere = explode(" ",

      "A B C D E F G H I J K L M N O P Q R S T U V W X Y Z "

      ."a b c d e f g h i j k l m n o p q r s t u v w x y z "

      ."0 1 2 3 4 5 6 7 8 9");



      // questo ? il clico tramite si crea la password

      for($i=0;$i<$length;$i++) {

      srand((double)microtime()*8625342);  

        $foo = rand(0, 61);

        $pass = $pass.$lettere[$foo];

      }

    $control = mysql_query("SELECT * FROM acquisti WHERE codeoff = '".$pass."'") or die (mysql_error());

    if (mysql_num_rows($control) != 0) {

        $esist = 0;

    } else { $esist = 1; }

      return $pass;

      return $esist;

    

    }

    

    } while ( $esist == 0);

    

    //FINE GENERAZIONE CODICE

    mysql_select_db($database, $acquisti);

    $buy = mysql_query("INSERT INTO acquisti (num, nome, cognome, email, nascita, provincia, indirizzo, codeoff) values ('".$num."', '".$nome."', '".$cognome."', '".$email."', '".$nascita."', '".$provincia."', '".$indirizzo."', '".$pass."')") or die (mysql_error());

    if ($bSuccesso)

        {

            $intestazioni  = "MIME-Version: 1.0\r\n";

            $intestazioni .= "Content-type: text/html; charset=iso-8859-1\r\n";

            $messaggio = "

            <html><head><title>Codice acquisto</title></head>

            <body>

            <h1>Grazie per aver effettuato l'acquisto</h1><br><br>

            Stampa questa mail e consegnala presso $azienda; il pagamento dovrà essere effettuato in loco secondo le modalità stabilite dall'azienda erogatrice dell'offerta.<br><br>

            CODICE OFFERTA: $pass

            </body></html>";

            mail ($mail, "Codice acquisto", $messaggio, $intestazioni);

            echo "Acquisto avvenuto con successo; una mail con il codice per attivare l'offerta è stata inviata alla tua casella di posta elettronica.";

            header('Location: index.php');

            mysql_select_db($databse, $offerte);

            $countdown = mysql_query("UPDATE offerte SET numoff = (numoff - 1) WHERE num = '".$num."' ") or die(mysql_error());

            

        }

    else

        {

            echo '<p align="center">Spiacenti, &egrave; avvenuto un errore! Indietro per riprovare.</p>';

        }

    }

?>

**miki92** · 21-03-2014, 19.20.33

Cambia questa parte:

Codice PHP:


    function rPass($length = 6) { 

      // Questo array contiene i caratteri con i quali 
      // saranno create le password 
      $lettere = explode(" ", 
      "A B C D E F G H I J K L M N O P Q R S T U V W X Y Z " 
      ."a b c d e f g h i j k l m n o p q r s t u v w x y z " 
      ."0 1 2 3 4 5 6 7 8 9"); 

      // questo ? il clico tramite si crea la password 
      for($i=0;$i<$length;$i++) { 
      srand((double)microtime()*8625342);   
        $foo = rand(0, 61); 
        $pass = $pass.$lettere[$foo]; 
      } 
    $control = mysql_query("SELECT * FROM acquisti WHERE codeoff = '".$pass."'") or die (mysql_error()); 
    if (mysql_num_rows($control) != 0) { 
        $esist = 0; 
    } else { $esist = 1; } 
      return $pass; 
      return $esist; 
     
    }

Con:

Codice PHP:


    /* function rPass($length = 6) { 

      // Questo array contiene i caratteri con i quali 
      // saranno create le password 
      $lettere = explode(" ", 
      "A B C D E F G H I J K L M N O P Q R S T U V W X Y Z " 
      ."a b c d e f g h i j k l m n o p q r s t u v w x y z " 
      ."0 1 2 3 4 5 6 7 8 9"); 

      // questo ? il clico tramite si crea la password 
      for($i=0;$i<$length;$i++) { 
      srand((double)microtime()*8625342);   
        $foo = rand(0, 61); 
        $pass = $pass.$lettere[$foo]; 
      } 
    $control = mysql_query("SELECT * FROM acquisti WHERE codeoff = '".$pass."'") or die (mysql_error()); 
    if (mysql_num_rows($control) != 0) { 
        $esist = 0; 
    } else { $esist = 1; } 
      return $pass; 
      return $esist; 
     
    } */

Oppure con:

Codice PHP:


    if(!function_exists("rPass")){ function rPass($length = 6) { 

      // Questo array contiene i caratteri con i quali 
      // saranno create le password 
      $lettere = explode(" ", 
      "A B C D E F G H I J K L M N O P Q R S T U V W X Y Z " 
      ."a b c d e f g h i j k l m n o p q r s t u v w x y z " 
      ."0 1 2 3 4 5 6 7 8 9"); 

      // questo ? il clico tramite si crea la password 
      for($i=0;$i<$length;$i++) { 
      srand((double)microtime()*8625342);   
        $foo = rand(0, 61); 
        $pass = $pass.$lettere[$foo]; 
      } 
    $control = mysql_query("SELECT * FROM acquisti WHERE codeoff = '".$pass."'") or die (mysql_error()); 
    if (mysql_num_rows($control) != 0) { 
        $esist = 0; 
    } else { $esist = 1; } 
      return $pass; 
      return $esist; 
     
    }}

**programmerscommunity** · 21-03-2014, 19.32.19

Fatal error: Maximum execution time of 30 seconds exceeded in /membri/programmerscommunity/confbuy.php on line 24

**programmerscommunity** · 22-03-2014, 21.07.40

Cosa devo quindi fare visto che mi si presenta questo errore?

**alemoppo** · 22-03-2014, 22.04.47

Che senso ha il do...while?
Comunque puoi riscrivere il codice per intero che stai usando?

Ciao!

**programmerscommunity** · 22-03-2014, 22.38.15

Codice PHP:


<?php

    session_start();

    require_once('mysqlconn.php');

    mysql_select_db($database, $offerte);

    $bSuccesso = true;

    $num = $_GET['num'];

    $nome = $_POST['nome'];

    $cognome = $_POST['cognome'];

    $mail = $_POST['email'];

    $nascita = $_POST['nascita'];

    $provincia = $_POST['provincia'];

    $indirizzo = $_indirizzo['indirizzo'];

    $control = mysql_query("SELECT * FROM offerte WHERE num = '".$num."' ") or die(mysql_error());

    $info = mysql_fetch_array($control);

    $numoff = $info['numoff'];

    if ($numoff == 0) {

        echo "Non è possibile completare l'operazione: l'offerta richiesta è esaurita";

        } else {

    $numoff = $numoff - 1;

    $countdown = mysql_query("UPDATE offerte SET numoff = '".$numoff."' WHERE num = '".$num."' ") or die(mysql_error());

    //INIZIO GENERAZIONE CODICE 

    do {

    mysql_select_db($database, $acquisti);

      if(!function_exists("rPass")){ function rPass($length = 6) { 



      // Questo array contiene i caratteri con i quali 

      // saranno create le password 

      $lettere = explode(" ", 

      "A B C D E F G H I J K L M N O P Q R S T U V W X Y Z " 

      ."a b c d e f g h i j k l m n o p q r s t u v w x y z " 

      ."0 1 2 3 4 5 6 7 8 9"); 



      // questo ? il clico tramite si crea la password 

      for($i=0;$i<$length;$i++) { 

      srand((double)microtime()*8625342);   

        $foo = rand(0, 61); 

        $pass = $pass.$lettere[$foo]; 

      } 

    $control = mysql_query("SELECT * FROM acquisti WHERE codeoff = '".$pass."'") or die (mysql_error()); 

    if (mysql_num_rows($control) != 0) { 

        $esist = 0; 

    } else { $esist = 1; } 

      return $pass; 

      return $esist; 

     

    }}  

    

    } while ( $esist == 0);

    

    //FINE GENERAZIONE CODICE

    mysql_select_db($database, $acquisti);

    $buy = mysql_query("INSERT INTO acquisti (num, nome, cognome, email, nascita, provincia, indirizzo, codeoff) values ('".$num."', '".$nome."', '".$cognome."', '".$email."', '".$nascita."', '".$provincia."', '".$indirizzo."', '".$pass."')") or die (mysql_error());

    if ($bSuccesso)

        {

            $intestazioni  = "MIME-Version: 1.0\r\n";

            $intestazioni .= "Content-type: text/html; charset=iso-8859-1\r\n";

            $messaggio = "

            <html><head><title>Codice acquisto</title></head>

            <body>

            <h1>Grazie per aver effettuato l'acquisto</h1><br><br>

            Stampa questa mail e consegnala presso $azienda; il pagamento dovrà essere effettuato in loco secondo le modalità stabilite dall'azienda erogatrice dell'offerta.<br><br>

            CODICE OFFERTA: $pass

            </body></html>";

            mail ($mail, "Codice acquisto", $messaggio, $intestazioni);

            echo "Acquisto avvenuto con successo; una mail con il codice per attivare l'offerta è stata inviata alla tua casella di posta elettronica.";

            header('Location: index.php');

            mysql_select_db($databse, $offerte);

            $countdown = mysql_query("UPDATE offerte SET numoff = (numoff - 1) WHERE num = '".$num."' ") or die(mysql_error());

            

        }

    else

        {

            echo '<p align="center">Spiacenti, &egrave; avvenuto un errore! Indietro per riprovare.</p>';

        }

    }

?>

**programmerscommunity** · 22-03-2014, 22.39.46

Il do...while serve per far ripetere la generazione del codice fino a che questo non è diverso da uno già presente nel db

**alemoppo** · 23-03-2014, 01.44.47

Ok. Ho riletto meglio il sorgente.

Non capisco perché dichiari la funzione "rPass()", e perché hai due return. Non puoi fare semplicemente così?:

Codice PHP:


    //INIZIO GENERAZIONE CODICE 

    $length = 6;

    // Questo array contiene i caratteri con i quali  
    // saranno create le password  
    $lettere = explode(" ",  
    "A B C D E F G H I J K L M N O P Q R S T U V W X Y Z "  
    ."a b c d e f g h i j k l m n o p q r s t u v w x y z "  
    ."0 1 2 3 4 5 6 7 8 9");  
    do { 
            // questo ? il clico tramite si crea la password  
            for($i=0;$i<$length;$i++) {  
                srand((double)microtime()*8625342);    
                $foo = rand(0, 61);  
                $pass = $pass.$lettere[$foo];  
            }  
            $control = mysql_query("SELECT * FROM acquisti WHERE codeoff = '".$pass."'") or die (mysql_error());  
            if (mysql_num_rows($control) != 0) {  
                $esist = 0;  
            } else { $esist = 1; }  
    } while ( $esist == 0); 
     
    //FINE GENERAZIONE CODICE

Inoltre non capisco perché selezioni più volte il database: il database è unico e le tabelle le selezioni dalla query.

Se ancora ti va in loop, probabilmente c'é qualche problema nella gestione del database.

p.s: occhio alle sql injection.

Ciao!

**programmerscommunity** · 23-03-2014, 20.26.21

Grazie. Funziona tutto.

Discussione: Problema logico if

LinkBack

Strumenti discussione

Display

Problema logico if

Regole di scrittura