Problem z if, elseif, else Opcje

[neoski]

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:

Parse error: syntax error, unexpected '{' in /cms1/modules/profile/actionpassword.php on line 23

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

[crash]

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

[neoski]

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 

A od kiedy to przy else podaje się warunki?

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

I nie ma co się dziwić, że będą pojawiać się głupie błędy przy takim stylu pisania kodu...

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.

[Speedy]

Hm, od kiedy zwraca się tak do użytkowników

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

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

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.

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

Ten post edytował Speedy 6.02.2006, 00:27:37

[cadavre]

[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? ^

Ten post edytował cadavre 6.02.2006, 00:38:25

[neoski]

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

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.

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

Dziękuję.

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

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.

Wygląda lepiej?

Masz rację bardziej przejrzyście .

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

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

Parse error: syntax error, unexpected '{' in /cms1/modules/profile/actionpassword.php on line 31

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

[cadavre]

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.

[neoski]

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.

[crash]

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?