[PHP]elseif - czemu nie działa Opcje

[jacusek]

Witam.
Mam taki oto skrypcik:

[PHP] pobierz, plaintext 
<?php
include('connect2.php');
if(empty($_GET['nazwisko'])||empty($_GET['imie'])||empty($_GET['id_lek'])
||empty($_GET['ID_BAD'])||empty($_GET['termin'])){
    echo "<center><b>!!!!SPRAWDŹ CZY WYPEŁNIŁEŚ WSZYSTKIE OBOWIĄZKOWE POLA!!!!!</b></center>";
    echo "<center><h2><a href=\"rezwizyta.php\">Wróć do REZERWACJI</a></h2></center>";
}
else{
$pytanie='select max(id_pac) as id_pac from pacjenci';
if ($res=mysqli_query($mysqli,$pytanie)){
    while ($row=mysqli_fetch_row($res))
   $max=$row['id_pac'];
//jeżeli pacjent ma już id - przeniesione dane z wyszukiwarki
if (isset($_GET['id_pac'])){
        $query1 = 'insert into wizyta (id_pac,id_lek, id_bad, termin, new) values 
        ("'.$_GET['id_pac'].'", "'.$_GET['id_lek'].'","'.$_GET['ID_BAD'].'", "'.$_GET['termin'].'",now())';
        $result1=mysqli_query($mysqli,$query1) or
        die(mysqli_error($mysqli));}
} 
//pacjent z PESEL na nową wizytę 
elseif(empty($_GET['id_pac'])&&(isset($_GET['PESEL']))){
        $query = 'insert into pacjenci (PESEL, nazwisko, imie, miasto, kod, adres, tel, tel_kom, rok_ur) values 
        ("'.$_GET['PESEL'].'","'.$_GET['nazwisko'].'","'.$_GET['imie'].'",
        "'.$_GET['miasto'].'","'.$_GET['kod'].'",
        "'.$_GET['adres'].'","'.$_GET['tel'].'","'.$_GET['tel_kom'].'","'.$_GET['rok_ur'].'")';
        $result=mysqli_query($mysqli,$query)
        or die(mysqli_error($mysqli));
        $pytanie1='SELECT max(id_pac) as id_pac from pacjenci';
        $max_pac1=mysqli_query($mysqli,$pytanie1) or die(mysqli_error($mysqli));
        $pacjent = mysqli_fetch_assoc($max_pac1);
        $pac = $pacjent['id_pac'];}
           $query2 = 'insert into wizyta (id_pac, id_lek, id_bad, termin, new) values 
        ("'.$pac.'","'.$_GET['id_lek'].'","'.$_GET['ID_BAD'].'", "'.$_GET['termin'].'", now())';
        $result1=mysqli_query($mysqli,$query2) or
        die(mysqli_error($mysqli));
}
//pacjent bez PESEL na nową wizytę
elseif(empty($_GET['PESEL'])&&(empty($_GET['id_pac']))){
$query = 'insert into pacjenci (PESEL, nazwisko, imie, miasto, kod, adres, tel, tel_kom, rok_ur) values 
        (NULL,"'.$_GET['nazwisko'].'","'.$_GET['imie'].'",
        "'.$_GET['miasto'].'","'.$_GET['kod'].'",
        "'.$_GET['adres'].'","'.$_GET['tel'].'","'.$_GET['tel_kom'].'","'.$_GET['rok_ur'].'")';
        $result=mysqli_query($mysqli,$query)
        or die(mysqli_error($mysqli));}
}
 
    $query4= 'select nazwisko_imie from wizyta inner join lekarze1 using (id_lek) inner join pacjenci using (id_pac)';
    $rez4= mysqli_query($mysqli, $query4);
        while ($nowaTab=mysqli_fetch_array($rez4,MYSQLI_ASSOC))
        {
            $doktor=$nowaTab['nazwisko_imie'];
            $dr=$doktor;
        }
echo '<center>';
echo 'Gratujację właśnie umówiłeś wizytę </h4><br/>';
echo 'Nazwisko pacjecnta: <b>' .$_GET['nazwisko'].'</b><br/>';
echo 'Imię pacjecnta: <b>' .$_GET['imie'].'</b><br/>';
echo 'termin wizyty: <b>' .$_GET['termin'].'</b><br/>';
echo 'Lekarz: <b>' .$dr.'</b>';
echo '</center>';
echo "<center><h2><a href=\"menu.html\">Wróć do MENU</a></h2></center>";
echo "<center><h2><a href=\"rezwizyta.php\">Umów wizytę</a></h2></center>";}
?>
[PHP] pobierz, plaintext 

Jest to skrypt wysyłający dane z formularza. I tutaj pytanie - dlaczego nie działa mi elseif. Eclipse wyrzuca mi błąd na poziomie składni przy 38 linijce unexpected elseif. No właśnie nie rozumiem za bardzo dlaczego?
Z góry dzięki za pomoc.

[jasin]

Bo "rąbnąłeś" się w klamrach ... Ten elsif odnosi się do: (wyciąłem środek bloku by było "jaśniej")

[PHP] pobierz, plaintext 
if(empty($_GET['nazwisko'])||empty($_GET['imie'])||empty($_GET['id_lek'])
||empty($_GET['ID_BAD'])||empty($_GET['termin'])){
 
}
else{
 
}
//pacjent bez PESEL na nową wizytę
elseif(empty($_GET['PESEL'])&&(empty($_GET['id_pac']))){
[PHP] pobierz, plaintext 

a w php nie ma kontrukcji if ... else ... elseif

w linii 18 masz zbędną klamrę.

Ten post edytował jasin 15.12.2009, 19:10:01

[jacusek]

Dzięki za pomoc składniowo pomogło, ale mam jednak teraz problem merytoryczny

[PHP] pobierz, plaintext 
//jeżeli pacjent ma już id - przeniesione dane z wyszukiwarki
if(isset($_GET['id_pac'])){
        $query1 = 'insert into wizyta (id_pac,id_lek, id_bad, termin, new) values 
        ("'.$_GET['id_pac'].'", "'.$_GET['id_lek'].'","'.$_GET['ID_BAD'].'", "'.$_GET['termin'].'",now())';
        $result1=mysqli_query($mysqli,$query1) or
        die(mysqli_error($mysqli));}
 
//pacjent z PESEL na nową wizytę 
elseif(empty($_GET['id_pac'])&&(isset($_GET['PESEL']))){
        $query = 'insert into pacjenci (PESEL, nazwisko, imie, miasto, kod, adres, tel, tel_kom, rok_ur) values 
        ("'.$_GET['PESEL'].'","'.$_GET['nazwisko'].'","'.$_GET['imie'].'",
        "'.$_GET['miasto'].'","'.$_GET['kod'].'",
        "'.$_GET['adres'].'","'.$_GET['tel'].'","'.$_GET['tel_kom'].'","'.$_GET['rok_ur'].'")';
        $result=mysqli_query($mysqli,$query)
        or die(mysqli_error($mysqli));
if(isset($result)){
        $pytanie1='SELECT max(id_pac) as id_pac from pacjenci';
        $max_pac1=mysqli_query($mysqli,$pytanie1) or die(mysqli_error($mysqli));
        $pacjent = mysqli_fetch_assoc($max_pac1);
        $pac = $pacjent['id_pac'];
           $query2 = 'insert into wizyta (id_pac, id_lek, id_bad, termin, new) values 
        ("'.$pac.'","'.$_GET['id_lek'].'","'.$_GET['ID_BAD'].'", "'.$_GET['termin'].'", now())';
        $result1=mysqli_query($mysqli,$query2) or
        die(mysqli_error($mysqli));}
}
 
//pacjent bez PESEL na nową wizytę
elseif(empty($_GET['PESEL'])&&(empty($_GET['id_pac']))){
$query = 'insert into pacjenci (PESEL, nazwisko, imie, miasto, kod, adres, tel, tel_kom, rok_ur) values 
        (NULL,"'.$_GET['nazwisko'].'","'.$_GET['imie'].'",
        "'.$_GET['miasto'].'","'.$_GET['kod'].'",
        "'.$_GET['adres'].'","'.$_GET['tel'].'","'.$_GET['tel_kom'].'","'.$_GET['rok_ur'].'")';
        $result=mysqli_query($mysqli,$query)
        or die(mysqli_error($mysqli));
if(isset($result)){
$pytanie1='SELECT max(id_pac) as id_pac from pacjenci';
        $max_pac1=mysqli_query($mysqli,$pytanie1) or die(mysqli_error($mysqli));
        $pacjent = mysqli_fetch_assoc($max_pac1);
        $pac = $pacjent['id_pac'];
           $query2 = 'insert into wizyta (id_pac, id_lek, id_bad, termin, new) values 
        ("'.$pac.'","'.$_GET['id_lek'].'","'.$_GET['ID_BAD'].'", "'.$_GET['termin'].'", now())';
        $result1=mysqli_query($mysqli,$query2) or
        die(mysqli_error($mysqli));
}
}
[PHP] pobierz, plaintext 

Ech no właśnie między linikami 9-25 i 28-45 wykonuje mi się tak jakby połowa funkcji. Co ciekawe wydaje mi się że część drugiej części (tylko i wyłącznie $query2). Wbijają wszystkie informacje do tab. wizyta, ale przy id_pac podają 0. No i nie tworzą rekordu i tabeli pacjenci.

[Pilsener]

Niepotrzebnie komplikujesz kod, nie używaj else if, ja tego nie używam i jakoś żyję, po prostu rozpisuje skrypt na etapy i algorytm działa np. tak:

1. Wykonuje się etap 1
2. Sprawdzmy, czy wykonał się etap 1, jeśli tak to robimy etap 2
3. Sprawdzamy, czy wykonał się etap 2, jeśli tak to robimy etap 3

W ten sposób rozbijasz algorytm na "drzewo" (polecam case, by podzielić kod na wygodne mniejsze fragmenty):

etap1
--etap 1.1
----etap 1.1.1
----etap 1.1.2
-- etap 1.2
etap2

Kod który napisałeś jest dla mnie nieczytelny i niezrozumiały, skąd mam wiedzieć jak to ma działać i jaki efekt chcesz osiągnąć? Twój opis problemu dużo nie pomaga, jeśli chcesz zrobić jakąś akcję dla wybranych warunków to wystarczą same ify:
1. Jeśli id_pac i pesel nie istnieją to coś tam
2. Jeśli id_pac nie istnieje ale pesel istnieje to coś tam
3. Analogicznie

Jak patrzę na stado else ifów to nie ogarniam tego.

[piotrooo89]

tak to jest najlepszy sposób, później wszystko wynika jedno z drugiego.

[jacusek]

Dzięki za rady. Na razie zostałem przy elseif. W sumie w tym momencie działa. Próbowałem to robić na takim case:

[PHP] pobierz, plaintext 
switch($_GET["action"]){
    case 'id':
if (isset($_GET['id_pac'])){
        $query1 = 'insert into wizyta_test (id_pac,id_lek, id_bad, termin, new) values 
        ( "'.$_GET['id_pac'].'", "'.$_GET['id_lek'].'","'.$_GET['ID_BAD'].'", "'.$_GET['termin'].'",now())';
        $result1=mysqli_query($mysqli,$query1) or
        die(mysqli_error($mysqli));}else{
        break;
        }
    case "PESEL": //nowy pacjent z PESEL
        if  (!isset($_GET['id_pac'])){
        $query = 'insert into pacjenci (PESEL, nazwisko, imie, miasto, kod, adres, tel, tel_kom, rok_ur) values 
        ("'.$_GET['PESEL'].'","'.$_GET['nazwisko'].'","'.$_GET['imie'].'",
        "'.$_GET['miasto'].'","'.$_GET['kod'].'",
        "'.$_GET['adres'].'","'.$_GET['tel'].'","'.$_GET['tel_kom'].'","'.$_GET['rok_ur'].'")';
        $result=mysqli_query($mysqli,$query)
        or die(mysqli_error($mysqli));
        $pytanie1='select max(id_pac) from pacjenci';
        $max_pac1=mysqli_query($mysqli,$pytanie) or die(mysqli_error($mysqli));
        $pacjent = mysqli_fetch_assoc($max_pac1);
        $pac = $pacjent["id_pac"];
           $query2 = 'insert into wizyta_test (id_pac, id_lek, id_bad, termin, new) values 
        ("'.$pac.'","'.$_GET['id_lek'].'","'.$_GET['id'].'", "'.$_GET['data'].'", now())';
        $result1=mysqli_query($mysqli,$query2) or
        die(mysqli_error($mysqli));}else{
        break;
        }
    default://nowy pacjent bez PESEL
        if ((!isset($_GET['id_pac']))&&((!isset($_GET['id_pac'])))){
    $query = 'insert into pacjenci (PESEL, nazwisko, imie, miasto, kod, adres, tel, tel_kom, rok_ur) values 
        ("'.$_GET['PESEL'].'","'.$_GET['nazwisko'].'","'.$_GET['imie'].'",
        "'.$_GET['miasto'].'","'.$_GET['kod'].'",
        "'.$_GET['adres'].'","'.$_GET['tel'].'","'.$_GET['tel_kom'].'","'.$_GET['rok_ur'].'")';
        $result=mysqli_query($mysqli,$query)
        or die(mysqli_error($mysqli));
        $pytanie1='SELECT max(id_pac) as id_pac from pacjenci';
        $max_pac1=mysqli_query($mysqli,$pytanie1) or die(mysqli_error($mysqli));
        $pacjent = mysqli_fetch_assoc($max_pac1);
        $pac = $pacjent['id_pac'];
        $query2 = 'insert into wizyta (id_pac, id_lek, id_bad, termin, new) values 
        ("'.$pac.'","'.$_GET['id_lek'].'","'.$_GET['ID_BAD'].'", "'.$_GET['termin'].'", now())';
        $result1=mysqli_query($mysqli,$query2) or
        die(mysqli_error($mysqli));}else{
        break;
        }
}
?>
[PHP] pobierz, plaintext 

No właśnie taki case nie pomógł - w ogóle się nic nie działa.
Jestem dość początkujący więc stąd moje pytania.