[PHP MYSQL] Czy ten skrypt jest dobry? Opcje

[julek12]

Napisałem taki o to skrypt:

[PHP] pobierz, plaintext 
<?php
try
{
  $pdo = new PDO('mysql:host=localhost;dbname=xxx', 'xxx', 'xxx'); // Inicjacja bazy bez parametrów kodowania
  //$pdo = new PDO('mysql:host=hostBazyDanych;dbname=nazwaBazy', 'login', 'haslo', array(PDO::MYSQL_ATTR_INIT_COMMAND => 'SET NAMES utf8')); // Inicjacja bazy z kodowaniem UTF-8
  $pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION); // Ustawienia parametrów wyjątków PDO
 
  $stmt = $pdo -> query('SELECT `uid`, `username`, `vip`, `group_vip`, `ile_vip` FROM `mybb_users` WHERE `group_vip` != 0');
 
  echo '<ol>';
  foreach($stmt as $row)
  {
      $sql = $pdo->prepare('SELECT `username`, `vip`, `group_vip`, `ile_vip` FROM `mybb_users` WHERE `group_vip` != 0 AND `username` = :nick'); // Przygotowuanie zapytania
      $sql->bindValue(':nick', $row['username'], PDO::PARAM_STR); // Podpinanie jest odporne na SQL Injection ;], a na dodatek szybsze
      $sql->execute(); // Wykonanie
      $data = $sql->fetch(PDO::FETCH_ASSOC); // Pobranie tablicy asocjacyjnej
 
      $czas = $data['vip'] - time();
      $dni = floor($czas / 86400);
      $godzin = floor(($czas - floor($dni) * 86400) / 3600);
      $minut = floor(($czas - floor($dni) * 86400 - floor($godzin) * 3600) / 60);
 
      $d = 'dni';
      $g = 'godzin';
      $m = 'minut';
 
      if ($dni == 1)
      {
         $d = 'dzień';
      }
 
      if ($godzin == 1)
      {
         $g = 'godzina';
      }
      else if ($godzin == 2 || $godzin == 3 || $godzin == 4)
      {
         $g = 'godziny';
      }
 
      if ($minut == 1)
      {
         $m = 'minuta';
      }
      else if ($minut == 2 | $minut == 3 | $minut == 4)
      {
         $m = 'minuty';
      }
 
      switch($data['ile_vip'])
      {
        case 1:
           $ilosc = "raz";
           break;
        case !1:
           $ilosc = "razy";
           break;
      }
 
      switch($data['group_vip'])
      {
        case 2:
           echo '<li><a href="http://forum.tibiaserv.pl/'.$row['username'].'-u-'.$row['uid'].'.html"><span style="color: blue;"><strong>'.$row['username'].'</strong></span></a>: '.$row['ile_vip'].' '.$ilosc.' był VIPem. VIP skończy się za: '.$dni.' '.$d.', '.$godzin.' '.$g.', '.$minut.' '.$m.'.</li>';
           break;
      }
 
  }
  $stmt -> closeCursor();
 
  echo '</ol>';
 
}
catch(Exception $e) // łapiemy błędy
{
   echo $e->getMessage(); // Wyświetlamy treść błędu
}
?>
[PHP] pobierz, plaintext 

Co o nim sądzicie??

Ten post edytował julek12 13.03.2009, 15:24:57

[zegarek84]

przecierz widzę że tu:

Kod

$stmt = $pdo -> query('SELECT `uid`, `username`, `vip`, `group_vip`, `ile_vip` FROM `mybb_users` WHERE `group_vip` != 0');

pobrałeś już wszystkie potrzebne Ci elementy z bazy to po co pobierasz potem znowu w petli pojedyńczo??:

Kod

$sql = $pdo->prepare('SELECT `username`, `vip`, `group_vip`, `ile_vip` FROM `mybb_users` WHERE `group_vip` != 0 AND `username` = :nick'); // Przygotowuanie zapytania
     $sql->bindValue(':nick', $row['username'], PDO::PARAM_STR); // Podpinanie jest odporne na SQL Injection;], a na dodatek szybsze
     $sql->execute(); // Wykonanie
     $data = $sql->fetch(PDO::FETCH_ASSOC);

przecież jak patrzę na kod to nie odwołując się kolejny raz do bazy w petli to Twojemu $data['vip']<==>$row['vip']

[Mephistofeles]

Chyba możesz wywalić query, a zostawić samo drugie zapytanie.

[julek12]

no niestety nie moge usunąć ani pierwszego ani drugiego. Pierwszego poniewaz na jego podstawie jest wykonywana pętla foreach, a drugiego bo tam są informacje troche inne to znaczy ze jest dodany nick i musi być to w pętli foreach:P
a tak poza tym to dobrze??

Ten post edytował julek12 13.03.2009, 13:58:30

[zegarek84]

no niestety nie moge usunąć ani pierwszego ani drugiego. Pierwszego poniewaz na jego podstawie jest wykonywana pętla foreach, a drugiego bo tam są informacje troche inne to znaczy ze jest dodany nick i musi być to w pętli foreach:P
a tak poza tym to dobrze??

PIERWSZE:

[SQL] pobierz, plaintext 
$stmt = $pdo -> query('SELECT `uid`, `username`, `vip`, `group_vip`, `ile_vip` FROM `mybb_users` WHERE `group_vip` != 0');
[SQL] pobierz, plaintext 

DRUGIE:

[SQL] pobierz, plaintext 
$sql = $pdo->prepare('SELECT `username`, `vip`, `group_vip`, `ile_vip` FROM `mybb_users` WHERE `group_vip` != 0 AND `username` = :nick');
[SQL] pobierz, plaintext 

GDZIE :nick <==> $row['username'] GDZIE $row['username'] to `username` z pierwszego zapytania, pierwsze i drugie operuje na tej samej tablicy więc w koło macieju poruwnujesz `username`=`username`

w pierwszym zapytaniu pobierasz nawet więcej kolumn i wszystkie z drugiego zapytania są w pierwszym, nawet warunek WHERE widzę ten sam - czyli jednym słowem jak pisałem wcześniej nie potrzebne jest to drugie zapytanie - w pierwszym dodatkowo masz więcej o 'uid'

całego kodu nie analizuję gdyż nie mam czasu ;/

[julek12]

To jak ma to wyglądać bo mi nie chce działać;/ Napiszcie prosze.

[Spawnm]

co ci nie chce działać, pokaż co masz teraz.

[julek12]

Na guże jest to co mam i to działa ale to co proponują inni to nie moge dać bo nie chce działać;/

[Mephistofeles]

Mniej więcej tak:

[PHP] pobierz, plaintext 
<?php
$stmt = $pdo -> query('SELECT `uid`, `username`, `vip`, `group_vip`, `ile_vip` FROM `mybb_users` WHERE `group_vip` != 0');
$table = $stmt -> fetch(PDO::FETCH_ASSOC);
foreach ($table as $row)
{
   np. $data = $row['vip'];
   itd.
}
?>
[PHP] pobierz, plaintext 

Po prostu niepotrzebnie wykonujesz drugie zapytanie, bo i tak w pierwszym pobrałeś to samo.

Ten post edytował Mephistofeles 13.03.2009, 15:51:43

[julek12]

Teraz skrypt wygląda tak:

[PHP] pobierz, plaintext 
<?php
try
{
  $pdo = new PDO('mysql:host=localhost;dbname=xx', 'xx', 'xx'); // Inicjacja bazy bez parametrów kodowania
  //$pdo = new PDO('mysql:host=hostBazyDanych;dbname=nazwaBazy', 'login', 'haslo', array(PDO::MYSQL_ATTR_INIT_COMMAND => 'SET NAMES utf8')); // Inicjacja bazy z kodowaniem UTF-8
  $pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION); // Ustawienia parametrów wyjątków PDO
 
  $stmt = $pdo->query('SELECT `uid`, `username`, `vip`, `ile_vip` FROM `xx` WHERE `group_vip` = 2 ORDER BY `vip` ASC');
  $count = $stmt->rowCount();
  if ($count == 0)
  {
  throw new Exception('<b>Brak!</b><br />');
  $stmt->closeCursor();
  }
 
  echo '<ol>';
  foreach($stmt as $row)
  {
      $czas = $row['vip'] - time();
      $dni = floor($czas / 86400);
      $godzin = floor(($czas - floor($dni) * 86400) / 3600);
      $minut = floor(($czas - floor($dni) * 86400 - floor($godzin) * 3600) / 60);
 
      $d = 'dni';
      $g = 'godzin';
      $m = 'minut';
 
      if ($dni == 1)
      {
         $d = 'dzień';
      }
 
      if ($godzin == 1)
      {
         $g = 'godzina';
      }
      else if ($godzin == 2 || $godzin == 3 || $godzin == 4)
      {
         $g = 'godziny';
      }
 
      if ($minut == 1)
      {
         $m = 'minuta';
      }
      else if ($minut == 2 || $minut == 3 || $minut == 4)
      {
         $m = 'minuty';
      }
 
      switch($row['ile_vip'])
      {
        case 1:
           $ilosc = "raz";
           break;
        case !1:
           $ilosc = "razy";
           break;
      }
 
      echo '<li><a href="http://forum.tibiaserv.pl/'.$row['username'].'-u-'.$row['uid'].'.html"><span style="color: blue;"><strong>'.$row['username'].'</strong></span></a>: '.$row['ile_vip'].' '.$ilosc.' jest VIPem. VIP skończy się za: '.$dni.' '.$d.', '.$godzin.' '.$g.' i '.$minut.' '.$m.'.</li>';
  }
  $stmt -> closeCursor();
 
  echo '</ol>';
 
}
catch(Exception $e) // łapiemy błędy
{
   echo $e->getMessage(); // Wyświetlamy treść błędu
}
?>
[PHP] pobierz, plaintext 

Lepiej??

Ten post edytował julek12 13.03.2009, 22:25:08