[PHP][MYSQL], Zapytania z książki Opcje

[miooodek]

Witam, napisałem taki oto skrypcik pobierający dane z bazy, działa bez zastrzezeń, ale jednak chciałbym żeby ktoś spojrzał na niego krytycznym okiem i doradził co i jak powinno wyglądac. Z góry dzięki, pozdrawiam

Kod

<?php    
            
        if( !isset($_POST['kontrahent_id']) && !isset($_POST['metoda_szukania']) && !isset($_POST['wyrazenie'])) // sprawdzenie czy została przesłana wcześniej jakaś zmienna, jesli tak przechodzi dalej, jesli nie wyswietla Napis Klient
        {
        
        echo '<ul><li></li><li>Klient</li></ul>';
            
        }
        
        if(!isset($_POST['kontrahent_id']))
        
{
    if(isset($_POST['metoda_szukania'])){
  

    $metoda_szukania=$_POST['metoda_szukania'];
    $wyrazenie=$_POST['wyrazenie'];
    $kontrahent_id=$_POST['kontrahent_id'];
    
    $wyrazenie = trim($wyrazenie);
    
    
    $metoda_szukania = addslashes($metoda_szukania);
    $wyrazenie = addslashes($wyrazenie);
    
    include("../data/mysql.inc");
    if($_POST['wyrazenie']==NULL)
    { echo '<ul id="blad"><li></li><li>Wpisz szukaną fraze</li></ul>';}
    else{
    
    mysql_select_db('transport');    
    $zapytanie = "select * from kontrahenci where ".$metoda_szukania." like '%".$wyrazenie."%'";
    $wynik = mysql_query($zapytanie);
    

    $ile_znalezionych = mysql_numrows($wynik);    
    if($ile_znalezionych==0)
    { echo '<ul id="blad"><li></li><li>Brak odpowiadających wpisów</li><li>w bazie danych</li></ul>';}
    
    if($ile_znalezionych==1)
    {
    for ($i=0; $i <$ile_znalezionych; $i++)

    {
    $wiersz=mysql_fetch_array($wynik);
    echo '<li>Nazwa: ';    
    echo stripslashes($wiersz['nazwa']);
    echo '</li><li>Miasto: ';
    echo stripslashes($wiersz['miasto']);
    echo '</li><li>NIP: ';
    echo stripslashes($wiersz['nip']);
    echo '</li>';    
    
  
    }}
    else
    {    
        for ($i=0; $i <$ile_znalezionych; $i++)
    {
    $wiersz=mysql_fetch_array($wynik);
    echo '<form action="#" method="POST">';
    echo '<div id="ramka">';    
    echo '<label>';    
    echo '<li>Nazwa: ';    
    echo stripslashes($wiersz['nazwa']);
    echo '</li><li>Miasto: ';
    echo stripslashes($wiersz['miasto']);
    echo '</li><li>NIP: ';
    echo stripslashes($wiersz['nip']);    
    echo '</li><li><div align=right><input type="hidden" name="kontrahent_id" value="' . stripslashes($wiersz['kontrahent_id']) . '"> <input type="submit" id="searchbutton" value="Wybieram" /></div>';
    echo '</div>';    
echo '</form>';
    }}    

}}
else
{

{
    $kontrahent_id=$_POST['kontrahent_id'];
    
    $wyrazenie = trim($wyrazenie);
    
    
    $metoda_szukania = addslashes($metoda_szukania);
    $wyrazenie = addslashes($wyrazenie);
    
    include("../data/mysql.inc");
    
    mysql_select_db('transport');    
    $zapytanie = "select * from kontrahenci where kontrahent_id like '%".$kontrahent_id."%'";
    $wynik = mysql_query($zapytanie);
    

    $ile_znalezionych = mysql_numrows($wynik);    
    
    if($ile_znalezionych==1)
    {
    for ($i=0; $i <$ile_znalezionych; $i++)

    {
    $wiersz=mysql_fetch_array($wynik);
    echo '<li>Nazwa: ';    
    echo stripslashes($wiersz['nazwa']);
    echo '</li><li>Miasto: ';
    echo stripslashes($wiersz['miasto']);
    echo '</li><li>NIP: ';
    echo stripslashes($wiersz['nip']);
    echo '</li>';    
    
  
    }
    }
}
}
}
?>

[melkorm]

[PHP] pobierz, plaintext 
<?php
if($_POST['wyrazenie']==NULL)
?>
[PHP] pobierz, plaintext 

mozna zastąpić przez empty

[PHP] pobierz, plaintext 
<?php
for ($i=0; $i <$ile_znalezionych; $i++)
?>
[PHP] pobierz, plaintext 

foreach będzie lepsze tak samo nie musisz sparwdzac ile znalazł wierszy tylko odrazu dać foreacha i znów użyć warunku z empty.

To tak na początek sądze że im dłużej w tym posiedzisz i podpatrzysz rozwiązania dojdziesz do najlepszych rozwiązań - oczywiście nie zapominając by dążyć do OOP .. a na początek to oddzielenie kodu php od HTML'a.