WItam pisze sobie sterownik do bazy Mysql dla zapotrzebowan mojego cms'a i mysle tak sobie czy takie rozwiazane jest godne uzycia np:

[PHP] pobierz, plaintext 
<?php
function DbSqlSelect($columns = array('*'), $table = '', $wh = array(''), $other = '') {
 
 if(count($columns) == 1) $column = $columns[0];
 else if(count($columns) > 1) $column = implode(',', $columns);
 else $column = '*';
 
 if(count($wh) == 1) {
 
 $key_tab = array_keys($wh);
 $where = $key_tab[0];
 
  foreach($wh as $argv => $args) 
 
  $countArgs = count($args);
 
   for($i = 0; $i < $countArgs; $i++) {
 
    if(is_int($args[$i])) $where .= "(int)$args[$i]";
    else if(is_string($args[$i-1])) $where .= "mysql_real_escape_string('$args[$i]')";
    else $where .= $args[$i];
 
   }
 }
 
 else if(count($wh) > 1) {
 
 $where = '';
 
  foreach($wh as $key => $value) {
 
  $where .= $key;
  $countValue = count($value);
 
   for($i = 0; $i < $countValue; $i++) {
 
   if(is_int($value[$i])) $where .= "(int)$value[$i]";
   else if(is_string($value[$i-1])) $where .= "mysql_real_escape_string('$value[$i]')";
   else $where .= $value[$i];
 
     }
   }
 }
 else $where = '';
 
 if(count($wh) != 0 && !empty($other)) $SqlSelect = 'select '.$column.' from '.$table.' where '.$where.' '.$other.'';
 
  else if(count($wh) != 0 && empty($other)) $SqlSelect = 'select '.$column.' from '.$table.' where '.$where;
 
   else if(count($wh) == 0 && !empty($other)) $SqlSelect = 'select '.$column.' from '.$table.' '.$other;
 
    else $SqlSelect = 'select '.$column.' from '.$table;
 
 return $sql = mysql_query($SqlSelect);
 
}
?>
[PHP] pobierz, plaintext 

A tak np wywoluje zapytania:

[PHP] pobierz, plaintext 
<?php
/*Wiecej warunkow where*/
 
//Wynik: select * from m_users where id=(int)1 and login=mysql_real_escape_string('marcio')
$zawartosc .= DbSqlSelect(array('*'), 'm_users', array('id' => array('=', 1, ' and '), 'login' => array('=','marcio')));
 
/*Dla jednego warunku where*/
 
//Wynik: select * from m_users where login=mysql_real_escape_string('marcio')
$zawartosc .= DbSqlSelect(array('*'), 'm_users', array('login' => array('=', 'marcio')));
 
/*Przyklad dla Sys. news'ow*/
 
$ris = DbSqlSelect(array('*'), 'news', array(), "order by id desc limit ".$strona*$liczba.",".$liczba.""); 
$dane = DbSqlRecordsAssoc($ris);
$count_dane = count($dane);
for($i = 0; $i < $count_dane; $i++) {
 
  $wynik2 = DbSqlSelect(array('id'), 'komentarze', array('id_news' => array('=', $dane[$i]['id']))); //SPosob z uzyciem funk. DbSqlSelect
  $wynik2 = DbSqlQuery('select * from komentarze where id_news = '.(int)$dane[$i]['id'].''); //Zwykly select
  $ile_comm = DbSqlNumRows($wynik2); 
  $ile_comm = DbSqlCountRows('select count(id) from komentarze where id_news = '.$dane[$i]['id'].'');
}
?>
[PHP] pobierz, plaintext 

Ilosc komentarzy mozna utrzymac w 2 sposoby albo za pomoca DbSqlQuery() ktory pobiera resource zapytania lub za pomoca DbSqlCountRows() ktore liczy normalnie.

Nie musze filtrowac zmiennych ani nic wydaje mi sie tez to bardziej czytelne i w ogole.

Na podstawie funckji DbSqlSelect() chcialbym zrobic tez wlasna skladnie dla zapytan Update,Insert i Delete, oczywiscie DbSqlSelect() nie oferuje wszyskich przypadkow zapytan ale dla podstawowych wystarcza.

P.S pojawil sie problem mam funkcje:

[PHP] pobierz, plaintext 
<?php
function DbSqlNumRows($Sql) {
 
 return $sql = mysql_num_rows($Sql);
}
?>
[PHP] pobierz, plaintext 

Gdy jej wynik przypisze jakiejs zmiennej to Mysql zwraca mi blad ze jest zly identyfikator polaczenia a jesli dam odrazu do echo DbSqlNumRows($id_zapytania) to jest good dlaczego tak sie dzieje, po modyfikacji funkcji DbSqlSelect() nastapil taki problem a przeciez jest return w funkcji moze juz jest pozno i nie kontaktuje.

P.S2 oczywiscie nie chce oceny co do kodu bo wiem ze ameryki nie odkrylem jednak nie bylem pewny gdzie umiescic temat jako ze jest troche dwuznaczny.

moim zdaniem te wszystkie funkcje nie mają sensu, jeśli używasz tylko MySQL, bo ani trochę nie ułatwiają pracy

[PHP] pobierz, plaintext 
<?php
$wynik2 = DbSqlSelect(array('id'), 'komentarze', array('id_news' => array('=', $dane[$i]['id'])));
$id = intval($dane[$i]['id']);
$wynik2 = CośDoQuery("SELECT `id` FROM `komentarze` WHERE `id_news` = '$id'");
?>
[PHP] pobierz, plaintext 

Jak widać, czyste rozwiązanie nie jest wiele dłuższe, a o wiele bardziej czytelne. A można zrobić jeszcze coś w tym stylu:

[PHP] pobierz, plaintext 
<?php
$wynik2 = CośDoQuery("SELECT `id` FROM `komentarze` WHERE `id_news` = '%1'", intval($dane[$i]['id']));
//albo
$wynik2 = CośDoQuery("SELECT `id` FROM `komentarze` WHERE `id_news` = '%i'", $dane[$i]['id']);
?>
[PHP] pobierz, plaintext 

No takie srednie te twoje funkcje. Może zamiast takie rzeczy pisać zainteresuj się PDO
http://www.php.net/pdo
http://pl.wikibooks.org/wiki/PHP/Biblioteka_PDO

Nie będziesz wymyślał koła na nowo.

Moim zdaniem pisanie takich "sterowników" mija się z celem. Dużo bardziej przydatnym narzędziem byłoby coś na wzór Kohanowego Query Buildera. Dzięki niemu można w bardzo prosty sposób dynamicznie budować zapytania (szczególnie przydatne przy SELECTach).

Phpion ma rację, też używam takiego bardzo prymitywnego "sklejacza" - kiedy używa się wielu podobnych zapytań. np. we wszelkiego rodzaju raportach, wydzielenie części wspólnej bardzo pomaga w utrzymaniu. Lepsze to, niż, z czym się spotkałem, dodawanie warunków do zapytania za pomocą preg_replace

Czyli nie jest tez takie zle ??


Nie pisze w OOP wiec nie chce mieszac kodu to raz a dwa nie chce mi sie czytac jakiejs dokumentacji wystarczy mi ta z php i sql


W sumie tu daje ci 70% racji ale kto wie moze projekt sie rozwinie i bede musial zrobic sterownik tez dla postgresql etc... wiec wtedy mozna edytowac ta funkcje i dodac np jako argument jakiej bazy sie uzywa i jest to porzyteczne, a dwa nie musze sie martwic co do filtrowania zmiennych ktore trafiaja do zapytania jak juz mi sie stalo w starym cms'ie ze zapomnialem rzytowac na int i byl Sql.Inj

Nie wiem jak dla mnie takie cos wydaje sie czytelniejsze niz pisanie za kazdym razem calego zapytania i bawienie sie w ' i " jak juz sie ktos przyzyczai to czemu nie

No ale PDO też ma sterownik dla innych baz danych, a to że nie piszesz w OOP to o niczym nie swiadczy w kodzie strukturalnym też można odwoływać się do klas. Jak będziesz chcial wygenerować sobie PDF to tez napiszesz sobie funkcje skoro masz gotowe klasy

( jakas mało składna ta wypowiedz )

No ale albo jest struktularny kod albo OOP.

Nie wiem wole pisac wlasne klasy nie lubie frameworkow ani gotowych klas dopoki dobrze php nie poznam wole uzywac wlasne skrypty


O co chodzi?

To może zamiast marnować czas na pisanie skryptów naucz się dobrze php?

To jak mam sie go uczyc jak nie piszac wlasne kody i analizowac kody innych czytac ksiazki ehh juz sie na takiej nauce przejechalem i wiem ze moja metoda ktora teraz uzywam jest super

Chcesz umieć jakikolwiek język dobrze? Naucz się korzystać z tego co już ktoś wymyślił, szczególnie gdy jest to lepsze, wygodniejsze i istneije spore prawdopodobieństwo, że inni prog. też będą to znać.