[MySQL][PHP]Poprawność funkcji łączących z bazą, Prośba o schematowe "dobre rady". Opcje

[ixpack]

Proszę kogoś elokwentnego o pomoc w poprawieniu niżej podanych funkcji (mój 1szy wątek i trochę się boję "was", ale musi być kiedyś ten 1szy raz (IMG:style_emoticons/default/winksmiley.jpg) ):

[PHP] pobierz, plaintext 
function polaczenie()
{
   $wynik = @new mysqli('ip.ip.ip.ip', 'login', 'haslo', 'baza'); 
   if (!$wynik) throw new Exception('Error with connection, please try later');
   else $wynik->autocommit(TRUE);
   return $wynik;
}
[PHP] pobierz, plaintext 

To jedna z 2ch funkcji - ustanawiająca połączenie... Co powinienem w niej zmienić?

[PHP] pobierz, plaintext 
function pobierz_nazwe_czegos($idczegos)
{
   $idczegos = intval($idczegos);
   $lacz = polaczenie();
   $zapytanie = "select nazweczegos from tabelaczegos where idczegos = $idczegos";
   $wynik = @$lacz->query($zapytanie);
   if (!$wynik)
     return false;
   $ilosc_nazw = @$wynik->num_rows;
   if ($ilosc_nazw == 0)
      return false;  
   $nazwa = $wynik->fetch_object();
   $lacz->close();
   return $nazwa->nazwaczegos;
}
 
$nazwa = pobierz_nazwe_czegos($idczegos);
[PHP] pobierz, plaintext 

Tu pobieram tylko jedną nazwę elementu, gdzie znam konkretny id elementu - Jestem "samoukiem", gdzieś czytałem, gdzieś podpatrzyłem, sam napisałem... Dlatego "pytam", bo chyba sam lepiej nie wymyślę.

Ps. Czy muszę zamykać połączenie "$lacz->close();" jeżeli w funkcji 1 mam "autocommit(TRUE);"?

Dzięki za odpowiedź (IMG:style_emoticons/default/smile.gif)


ixpack
-------------
Łatwo jest być odważnym za murami własnego zamku...

[ixpack]

Nie wiem, czy dobrze przejrzałem manual, ale czy to wygląda lepiej?:

[PHP] pobierz, plaintext 
function lacz_bd()
{
   $wynik = new mysqli('ip.ip.ip.ip', 'login', 'pass', 'baza'); 
   if (!$wynik)
    throw new Exception(' no 1');
   else
   {
      $wynik->autocommit(TRUE);
      return $wynik;
   }
}
 
function pobierz_nazwe_czegos($idczegos)
{
   $idczegos = intval($idczegos);
   $lacz = polaczenie();
   $zapytanie = "select nazweczegos from tabelaczegos where idczegos = $idczegos";
   $wynik = $lacz->query($zapytanie);
   if (!$wynik)
    throw new Exception(' no 2');
   else 
   {
      $ilosc_nazw = $wynik->num_rows;
      if ($ilosc_nazw == 0) return false;  
      $nazwa = $wynik->fetch_object();
      $lacz->close();
      return $nazwa->nazwaczegos;
   }
}
 
try 
{
   $nazwa = pobierz_nazwe_czegos($idczegos);   
} 
catch (Exception $e) 
{
   echo 'Error: ',  $e->getMessage(), "\n";
}
[PHP] pobierz, plaintext 

I tu nasuwa się kolejne pytanie (IMG:style_emoticons/default/smile.gif) : Jak odbije się wstawienie całego skryptu w blok "try" na działanie i szybkość strony? Czy powinienem członkować kod i odpowiednie "kawałki" wstawiać w kolejne bloki "try" i czy powinienem nadawać inne zmienne ($e, $f itd.)?

Ten post edytował ixpack 1.06.2010, 13:05:29