problem z IF i linkiem Opcje

[invx]

mam taki kod

[PHP] pobierz, plaintext 
<?php
 
if ( isset($_GET[m]) )
    {
         if( empty($_POST[login]) )
        {
            $login=1;
        }
        if( empty($_POST[pass]) )
        {
            $pass=1;
        }
        if( empty($_POST[pass2]) )
        {
            $pass2=1;
        }
        if( empty($_POST[pytanie]) )
        {
            $pytanie=1;
        }
        if( empty($_POST[odpowiedz]) )
        {
            $odpowiedz=1;
        }
        if( empty($_POST[kolor]) || $_POST[kolor] == wybierz )
        {
            $kolor=1;
        }
    }
if ( $login !== 1 && $pass !== 1 && $pass2 !== 1 && $pytanie !== 1 && odpowiedz !== 1 && kolor !==1 ) 
    {
        $action=&#092;"loguj.php?mode=add_user_save\";           //link1
    }
    else
    {
        $action=&#092;"loguj.php?mode=add_user&m=spr\";        //link2
    }
 
 
    echo&#092;"
    <center>
    <p align='center' class='title'>REJESTRACJA</p>
    <form method='POST' action='$action'>
    <table>
    &#092;";
#********************************************
#Sprawdzanie loginu
#********************************************
    if ( isset($_GET[m]) )
    {
        if ( $login == 1 )
        {
            $clas=&#092;"formularz_error\";
        }
        else
        {
            $clas=&#092;"formularz\";
        }
    }
    else
    {
        $clas=&#092;"formularz\";
    }
 
    echo&#092;"
    <tr class=$clas>
    <td  align='right'><b>Login:</b>
    </td>
    <td class='formularz'><input type='text' value='$_POST[login]' name='login' size='20'><br>
    </td>
    </tr>
    &#092;";
 
// reszta pul do sprawdzania
 
?>
[PHP] pobierz, plaintext 

chodzi o to, ze jak by nie bylo zwsze jest spelniany pierwszy warunek, i zawsze $action, przyjmuje wartosc loguj.php?mode=add_user_save,a chce zeby dopier jesli wszystkie pola nie sa puste to przeszlo do pliku z mode=add_user_aave

Ten post edytował invx 19.06.2004, 03:59:13

[Kinool]

$pass !== 1 co to za twor?? jesli to ma byc negacja to tak: $pass!=1

[Ludvik]

$a !== $b - Nie identyczny - TRUE jeśli $a nie jest równy $b, lub nie są tego samego typu. (tylko w php 4) 

Więc to chyba nie jest błąd.

W tym if-ie pierwszym masz w dwóch ostatnich zmiennych błąd.

[PHP] pobierz, plaintext 
<?php
if($login !== 1 && $pass !== 1 && $pass2 !== 1 && $pytanie !== 1 && odpowiedz !== 1 && kolor !==1)
?>
[PHP] pobierz, plaintext 

Powinno być:

[PHP] pobierz, plaintext 
<?php
if($login !== 1 && $pass !== 1 && $pass2 !== 1 && $pytanie !== 1 && $odpowiedz !== 1 && $kolor !==1)
?>
[PHP] pobierz, plaintext 

Chyba, że ci chodziło o stałe Poza tym masz to troszkę niewygodnie. Jak chcesz sprawdzić błąd to musisz sprawdzać 6 zmiennych... Nie lepiej na początku zainicjować zmienną $blad z wartością 0, a potem w razie błędu przypisać jej 1? Potem wystarczy sprawdzić tylko tą zmienną.

Ten post edytował Ludvik 19.06.2004, 08:29:01

[invx]

nie moge uzyc jednej zmiennej, bo potem wykorzystuje je do okreslania, co jest zle, poprawilem te zmienne, ale nic to nie zmienilo dalej zle.

Ten post edytował invx 19.06.2004, 16:58:29

[stoprocent]

A jakbys poprostu zamiast tego :

[PHP] pobierz, plaintext 
<?php
 
if ( isset($_GET[m]) )
    {
         if( empty($_POST[login]) )
        {
            $login=1;
        }
        if( empty($_POST[pass]) )
        {
            $pass=1;
        }
        if( empty($_POST[pass2]) )
        {
            $pass2=1;
        }
        if( empty($_POST[pytanie]) )
        {
            $pytanie=1;
        }
        if( empty($_POST[odpowiedz]) )
        {
            $odpowiedz=1;
        }
        if( empty($_POST[kolor]) || $_POST[kolor] == wybierz )
        {
            $kolor=1;
        }
    }
 
?>
[PHP] pobierz, plaintext 

dal :

[PHP] pobierz, plaintext 
<?php
 
if ( isset($_GET['m']) )
    {
         if( empty($_POST['login']) )
        {
            $error=1;
        }
        if( empty($_POST['pass']) )
        {
            $error=1;
        }
        if( empty($_POST['pass2']) )
        {
            $error=1;
        }
        if( empty($_POST['pytanie']) )
        {
            $error=1;
        }
        if( empty($_POST['odpowiedz']) )
        {
            $error=1;
        }
        if( empty($_POST['kolor']) || $_POST['kolor'] == 'wybierz' )
        {
            $error=1;
        }
    }
 
//A pozniej tyklo
if($error!=1){}
 
?>
[PHP] pobierz, plaintext 

Bo tak czy tak nawet jak jedno pole nie bedzie wypelnione to ci wroci , a jak zadne nie bedzie puste to sie zapisze .

Ten post edytował nasto 19.06.2004, 17:10:36

[invx]

dalej to samo