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

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

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ą.

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

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 .

dalej to samo