Forum PHP.pl

Tak jak w temacie mam problem gdyż dane else nie działa. .... (tutaj)

[PHP] pobierz, plaintext 
$lvl = mysql_fetch_row(mysql_query("SELECT lvl FROM user WHERE user='$user'"));
$ile = ($lvl[0] * 10) * 2;
if(isSet($_POST['odpocznij'])){
    $godziny = (int)round($_POST['godziny'], '0');
    if($godziny > 8){
        $godziny = 8;
    }
    elseif($godziny < 1){
        $godziny = 1;
    }
............................................................... tutaj
  else {
    $zaplata = $ile * $godziny;
    $kasa = mysql_fetch_row(mysql_query("SELECT kasa FROM user WHERE user='$user'"));
    if($zaplata > $kasa['kasa']){
echo "Masz za mało kasy na koncie.";
}
}
.................................................................tutaj
  $zaplata = $ile * $godziny;
    $konczy = mktime() + (($godziny * 60) * 60);
    $kasa = mysql_fetch_row(mysql_query("SELECT kasa FROM user WHERE user='$user'"));
    mysql_query("UPDATE user SET konczy2='$konczy', godzin2='".$godziny."', zaplata2='$zaplata' WHERE user='$user'");
    header("Location: lokal.php");
 
}
[PHP] pobierz, plaintext 

no u mnie działa
chyba że czegoś nie zrozumiałem
opisz dokładniej

edit:
ten elese jest w ogóle nie potrzebny
to co jest w jego wnętrzu wstaw zamiast linijki 20

Ten post edytował sazian 30.09.2010, 19:33:38

[PHP] pobierz, plaintext 
if($godziny > 8){
      $godziny = 8;
   }
   elseif($godziny < 1){
      $godziny = 1;
}
[PHP] pobierz, plaintext 

- po co tworzyć takie struktury? Radzę w ogóle nie używać elseifa, bo kod staje się mało czytelny. Tak samo radzę nie używać echo, tylko pracować na zmiennych, zmienne można dowolnie preparować a to co wysłałeś do browsera nie.

Jeśli potrzebujemy podzielić kod w pliku na funkcjonalne części lepiej użyć instrukcji switch.

A przy skomplikowanych i długich warunkach (np. chcemy podzielić partię produkcyjną na 5 przedziałów wg jakości od - do dla każdego przedziału) lepiej użyć tablicy.

Działa lecz gdy na koncie użytkownika jest 5 euro a musi zapłacić 30euro to i tak leci dalej a pozniej pokazuje w banku kase na - bo odjelo sobie to. Więc tak jak by ten warunek w ogole nie działał .