Witam,

Z racji tego, że raczkuję dopiero w temacie programowania proszę o opinię na temat tego co wykonałem oraz o wskazówki jak to ewentualnie wykonać lepiej.

Mam tabelę 'osoba', która zawiera kolumny: imie, nazwisko, kod. Do tabeli głównej o nazwie 'glowna' wpisów dokonuje się za pomocą formularza, w którym oprócz pól tekstowych jest lista wybieralna pozwalająca wybrać dane osoby.

Na stronie www dane mają się wyświetlać w dwóch linijkach tj. imię i nazwisko w jednej, a kod w drugiej.

Część kodu odpowiedzialna za wyświetlanie listy osób i zapis do bazy wygląda tak:

[PHP] pobierz, plaintext 
$pobierzdane="SELECT imie, nazwisko, kod FROM osoba";
$wynik = mysql_query ($pobierzdane);
 
echo "<select name=osoba>";
while($nt=mysql_fetch_array($wynik))
{
echo "<option value='$nt[imie] $nt[nazwisko] <br /> kod: $nt[kod]'>$nt[imie] $nt[nazwisko] kod: $nt[kod]</option>";
}
echo "</select>";
[PHP] pobierz, plaintext 

Dane w tabeli głównej bazy danych zapisują się jako np. Jak Kowalski <br /> AB123456.
Wyświetlanie na stronie realizowane jest tak:

[PHP] pobierz, plaintext 
$wybor = "SELECT * FROM glowna WHERE id=$id";
$wynik = mysqli_query($db, $wybor);
$row = mysqli_fetch_assoc($wynik);
 
<div>'.$row['osoba'].'</div>
[PHP] pobierz, plaintext 

Zrobiłem to chyba najprościej jak można i działa. Pytanie tylko czy jest to poprawnie wykonane i jeśli nie to jak to zrobić w inny sposób? Prośba o wskazówki.

Pozdrawiam,
Zbyszek

1. Indeksy tablic zapisujemy w apostrofach $tablica['indeks']
2. Brak cudzysłowia w name=osoba i pozostałych atrybutach
3. Lepiej zapisywać string do zmiennej i potem go echować
4. Raz używasz mysql a innym razem mysqli - zdecyduj się
5. Po co w value opcji wypisujesz tyle danych?? gdy wystarczy podać ID rekordu
6. Kod html lepiej wstawiać za pomocą apostrofu dzięki czemu nie musisz używać \

[PHP] pobierz, plaintext 
$write = '';
$write .= '<select name="osoba">';
 
while($nt=mysql_fetch_array($wynik))
{
    $write .= '<option value="'.$nt['id'].'">'.$nt['imie'].$nt['nazwisko'].' kod: '.$nt['kod'].'</option>';
}
$write .= '</select>';
 
echo $write;
[PHP] pobierz, plaintext