Mam problem z użyciem if, elseif, else. Zresztą już nie pierwszy raz...

Skrypt służy do zmiany hasła.

[PHP] pobierz, plaintext 
<?php $pw=$_POST["pw"];
$pwn1=$_POST["pwn1"]
;$pwn2=$_POST["pwn2"];
 
$error=0;
if (empty($pwn1) or empty($pwn2) or empty($pw)) { $error++; }
elseif ($error==0) {
$sql1="SELECT id, password FROM users WHERE id='$myid'";
$data=mysql_query($sql1, $db);
while($mypw=mysql_fetch_object($data)) { $password=$mypw->password; }
$md5pw=md5($pw);
if ($password != $md5pw) { print "<br>Podałeś <b>złe</b> stare hasło!<br><br>"; }
elseif ($password == $md5pw) { print "<br>Nowe hasła nie pasują do siebie!<br><br>"; }
else ($pwn1 == $pwn2) { $md5pw2=md5($pwn2);
$update = "UPDATE users SET password='$md5pw2' WHERE id=$myid";
$sqlaction = mysql_query($update);
print "<br>Hasło pomyślnie zmienione<br><br><a href=\"index.php?id=news/index\">dalej</a><br><br>"; } }
else { print "<br>Przed zmianą hasła uzupełnij wszystkie pola!<br><br>"; } ?>
[PHP] pobierz, plaintext 

Błąd który się pojawia:


Oczywiście nawiązuje wcześniej połączenie do bazy... kiedyś ktoś mi to wypomniał .

A od kiedy to przy else podaje się warunki? I nie ma co się dziwić, że będą pojawiać się głupie błędy przy takim stylu pisania kodu...

Przy takim czymś wywala "Nowe hasła nie pasują do siebie!", po czym wysyła do bazy i zmienia...

[PHP] pobierz, plaintext 
<?php ...
if (empty($pwn1) or empty($pwn2) or empty($pw)) { $error++; }
if ($error==0) {
$sql1="SELECT id, password FROM users WHERE id='$myid'";$data=mysql_query($sql1, $db);
while($mypw=mysql_fetch_object($data)) { $password=$mypw->password; }
$md5pw=md5($pw);
if ($password != $md5pw) { print "<br>Podałeś <b>złe</b> stare hasło!<br><br>"; }
if ($password == $md5pw) { print "<br>Nowe hasła nie pasują do siebie!<br><br>"; }
if ($pwn1 == $pwn2) { $md5pw2=md5($pwn2);
... ?>
[PHP] pobierz, plaintext 

Hm, od kiedy zwraca się tak do użytkowników? Tym bardziej, że jest to dział dla początkujących :/.

To może zamiast gadać, że źle, źle pokażecie w końcu jak jest dobrze?

Odczuwam bardzo nie miłą atmosferę na tym forum. To już nie pierwszy raz kiedy zwracają się do mnie osoby o parę książek do przodu w ten sposób.

Hm... a od kiedy tak się zwraca do użytkowników?
Jeśli już krytykujesz czyjeś wypowiedzi i uważasz, że są one niemiłe, to przynajmniej sam staraj się zachować jak należy, bo wygląda to tragicznie...

http://phppl.ezpublish.no/phppl/wortal/art...ch_programistow



Więc może warto trochę poczytać ...

[PHP] pobierz, plaintext 
<?php
(...)
if (empty($pwn1) or empty($pwn2) or empty($pw)) { 
  $error++; 
}
if ($error==0) {
  $sql1="SELECT id, password FROM users WHERE id='$myid'";
  $data=mysql_query($sql1, $db);
  while($mypw=mysql_fetch_object($data)) {
	$password=$mypw->password; 
  }
  $md5pw=md5($pw);
  if ($password != $md5pw) { 
	print "<br>Podałeś <b>złe</b> stare hasło!<br><br>"; 
  }
  elseif ($password == $md5pw) { 
	print "<br>Nowe hasła nie pasują do siebie!<br><br>";
  }
  elseif ($pwn1 == $pwn2) { 
	$md5pw2=md5($pwn2);
  }
}
(...)
?>
[PHP] pobierz, plaintext 

Wygląda lepiej?

BTW Nie wypada dodać else przed niektóre ify? ^

Po raz kolejny słuchając tylko wytyczek niestety nerwy puściły. Oczywiście jeżeli wygląda to tragicznie to tragiczne są też wypowiedzi poprzedników.

Dziękuję.

Niektórzy nie mają czasu szperać w paru książkach. Właśnie do tego między innymi są fora na których inni ludzie wiedzą i pomagają w zdecydowanie szybszy sposób.

Masz rację bardziej przejrzyście .

Właśnie dodałem i mam ten efekt co przed tem...



A jak mam if'y to ten else na samym dole krzyczy, że jest coś też nie tak.

To może to jakiś błąd w pliku, który include'ujesz do skryptu? Napisz która linijka to 31 w tym co wkleiłeś, bo widzę (...) tutaj.

Tak dokładnie wygląda cały plik:

[PHP] pobierz, plaintext 
<?php if($access<1) { print "$noacc"; } else { ?>
 
<TABLE width="<?php print "$width0" ?>" bgcolor="<?php print "$bgcolor0" ?>" border="0" cellspacing="1" cellpadding="3">
	<TR>
		<TD background="<?php print "$background0" ?>" align="center" colspan="2"><span class="content-title">Profil - zmiana hasła</span></TD>
	</TR>
 
	<TR bgcolor="<?php print "$bgcolor1" ?>">
		<TD>
		<?php $pw=$_POST["pw"];
		$pwn1=$_POST["pwn1"];
		$pwn2=$_POST["pwn2"];
		$error=0;
 
		if (empty($pwn1) or empty($pwn2) or empty($pw)) { 
		$error++; 
		}
		if ($error==0) {
		  $sql1="SELECT id, password FROM users WHERE id='$myid'";
		  $data=mysql_query($sql1, $db);
		  while($mypw=mysql_fetch_object($data)) {
			$password=$mypw->password; 
		  }
		  $md5pw=md5($pw);
		  if ($password != $md5pw) { 
			print "<br>Podałeś <b>złe</b> stare hasło!<br><br>"; 
		  }
		  elseif ($password == $md5pw) { 
			print "<br>Nowe hasła nie pasują do siebie!<br><br>";
		  }
		  else ($pwn1 == $pwn2) { 
			$md5pw2=md5($pwn2);
		  }
 
		$update = "UPDATE users SET password='$md5pw2' WHERE id=$myid";
		$sqlaction = mysql_query($update);
		print "<br>Hasło pomyślnie zmienione<br><br><a href=\"index.php?id=news/index\">dalej</a><br><br>"; } ?>
		</TD>
	</TR>
</TABLE>
 
<?php } ?>
[PHP] pobierz, plaintext 

W tym przypadku to będzie linijka 32.

To, że uznałeś moją wypowiedź za atak na Twoją osobę nie znaczy, że nie napisałem tam co masz źle... I dalej masz ten sam bląd.

[PHP] pobierz, plaintext 
<?php
if( $access < 1 )
{
   echo $noacc;
}
else
{
 
?>
<TABLE width="<?= $width0; ?>" bgcolor="<?= $bgcolor0; ?>" border="0" cellspacing="1" cellpadding="3">
   <TR>
       <TD background="<?= $background0; ?>" align="center" colspan="2"><span class="content-title">Profil - zmiana hasła</span></TD>
   </TR>
   <TR bgcolor="<?= $bgcolor1; ?>">
      <TD>
         <?php
         
         $pw   = $_POST[ 'pw' ];
         $pwn1 = $_POST[ 'pwn1' ];
         $pwn2 = $_POST[ 'pwn2' ];
 
         $error = 0;
 
         if( empty( $pwn1 ) or empty( $pwn2 ) or empty( $pw ) )
         { 
            $error++; 
         }
 
         if( $error == 0 )
         {
            $sql1 = 'SELECT id, password FROM users WHERE id= ' . $myid; // zakładając, że pole id jest liczbą
            $data = mysql_query( $sql1, $db );
            
            while( $mypw = mysql_fetch_object( $data ) )
            {
               $password = $mypw->password; 
            }
            
            $md5pw = md5( $pw );
            
            if( $password != $md5pw )
            { 
               echo '<br>Podałeś <b>złe</b> stare hasło!<br><br>'; 
            }
            elseif( $pwn1 != $pwn2 )
            { 
               echo '<br>Nowe hasła nie pasują do siebie!<br><br>'; // jako to nie pasują?! Zły warunek miałeś...
            }
            else
            { 
               $md5pw2 = md5( $pwn2 );
            }
        
            $update = 'UPDATE users SET password = "' . $md5pw2 . '" WHERE id= ' . $myid;
            $sqlaction = mysql_query( $update );
            
            echo '<br>Hasło pomyślnie zmienione<br><br><a href="index.php?id=news/index">dalej</a><br><br>';
        }
        else
        {
           echo 'Wypełnij wszystkie pola!';
        }
        
        ?>
        </TD>
    </TR>
</TABLE>
 
<?php }
[PHP] pobierz, plaintext 

Nie wygląda to czytelniej?