[PHP][MYSQL] Singleton zadanie Opcje

[fantek]

Witam . Mam takie zadanie:

[PHP] pobierz, plaintext 
<?php
 
/*
 
  Task 2:
  Implement db handler
 
  Create database `adb2` and user `root` with pasword `` for your project;
  Create file ./lib/db.php and implement in it abstract class MjDb that extends MjSingleton;
  Class should use config from clas Config.
  
  Class MjDb should implements methods:
  
  Public
 
  > execute(string $query)
    (function that execute query and return true or return false in case failure)
    
  > get_row(string $query)
    (function that execute query and returns row from database 
    or return false in case failure)
    
  > execute_s(string $query)
    (function that execute query and returns array of rows from database 
    or return false in case failure)
 
*/
 
include('./data/config.php');
include('./lib/singleton.php');
include('./lib/db.php');
 
class task2 extends MjDb{
  public function run(){
    $q = "
    CREATE TABLE IF NOT EXISTS `a_user` (
      `id` int(10) unsigned NOT NULL AUTO_INCREMENT,
      `name` varchar(50) DEFAULT NULL,
      `pass` varchar(50) DEFAULT NULL,
      `salt` varchar(50) DEFAULT NULL,
      `type` TINYINT DEFAULT NULL,
      PRIMARY KEY (`id`),
      UNIQUE KEY `id` (`id`)
    ) ENGINE=InnoDB  DEFAULT CHARSET=latin2 AUTO_INCREMENT=1;";
 
    if(!$this->execute($q)){
      return __LINE__;
    }
 
    $q = "TRUNCATE a_user";
    if(!$this->execute($q)){
      return __LINE__;
    }
 
    $q = "
      INSERT INTO `a_user` (
        `id` , `name` , `pass` , `salt` 
      )
      VALUES (
        NULL , 'name', 'pass', 'salt'
      );
    ";
 
    for($i=0;$i<5;$i++){
      if(!$this->execute($q)){
        return __LINE__;
      }
    }
 
    $q = "SELECT * FROM `a_user`";
 
    $r = $this->get_row($q);
    if(!$r || count($r) != 4){
      return __LINE__;
    }
 
    $r = $this->execute_s($q);
 
    if(!$r || count($r) != 5){
      return __LINE__;
    }
 
    return null;
  }
 
}
 
$error = task2::get_instance()->run();
 
if($error == null){
  echo 'Test Completed';
}else{
  echo 'Test Failed on line '.$error;
}
[PHP] pobierz, plaintext 

Pomijając wszystkie szczegóły klas zastanawia mnie jedna rzecz:
Ten fragment kodu

[SQL] pobierz, plaintext 
    FOR($i=0;$i<5;$i++){
      IF(!$this->execute($q)){
        RETURN __LINE__;
      }
    }
[SQL] pobierz, plaintext 

utworzy mi w bazie 5 identycznych rekordów .

Następnie mamy:

[PHP] pobierz, plaintext 
 $q = "SELECT * FROM `a_user`";
 
    $r = $this->get_row($q);
    if(!$r || count($r) != 4){
      return __LINE__;
    }
[PHP] pobierz, plaintext 

Która to metoda zwracać będzie ilość wierszy w bazie (skoro wcześniej utworzyło mi 5 rekordów w bazie to wywali mi error __LINE__ bo jest 5 wierszy)

[PHP] pobierz, plaintext 
 
    $r = $this->execute_s($q);
 
    if(!$r || count($r) != 5){
      return __LINE__;
    }
[PHP] pobierz, plaintext 

Tutaj się zgodzę bo metoda zwróci mi tablice z 5 rekordami.


Dlatego się zastanawiam czy bodajże nie ma tu haczyka jakiegoś czy to po prostu jakiś błąd ?

[nospor]

Przeczytaj jeszcze raz co zwraca get_row()... przeciez masz wyraznie napisane a mimo to dodajesz jakąś wlasna teorie...

[fantek]

Nie czaje do tej pory .... o co chodzi ...

[nospor]

(IMG:style_emoticons/default/facepalmxd.gif)

Ty twierdzisz, ze get_row zwraca wszystkie wierwsze z zapytania:

Która to metoda zwracać będzie ilość wierszy

Zas w komentarzach jak wol masz napisane, ze get_row zwraca pojedynczy wiersz z zapytania

> get_row(string $query)

(function that execute query and returns row from database

or return false in case failure)

Jesli nadal nie widzisz roznicy to palka sie nalezy jak nic (IMG:style_emoticons/default/smile.gif)
Jesli nadal to naprawde dla Ciebie niepojete to wez sobie sprawdz co zawiera twoje $r to moze cie oswieci, jesli prosta teoria do ciebie nie przemawia
var_dump($r);

[fantek]

No ok ale i tak jak wróci pojedynczy wiersz to wyjdzie 5 bo wcześniej tworzymy 5 kolumn (`id``name``pass``salt``type`) .
Co do metody get_row($q) sam muszę napisać żeby właśnie zwracała wiersz z bazy i zrobiłem to tak:

[PHP] pobierz, plaintext 
<?php
 
abstract class MjDb extends MjSingleton{
 
		private $_query = null;
 
	public function execute($q, $params = array()) {
 
		$this->_error = false;
 
		if($this->_query = $this->_pdo->prepare($q)) {
			$x = 1;
			if(count($params)) {
				foreach($params as $param) {
					$this->_query->bindValue($x, $param);
					$x++;
				}
			}
 
			if($this->_query->execute()) {
				$this->_results = $this->_query->fetchAll(PDO::FETCH_OBJ);
				$this->_count = $this->_query->rowCount();
			} else {
				$this->_error = true;
			}
		}
 
		return $this;
	}
 
	public function get_row($q){
 
			$query = $this->_query = $this->_pdo->prepare($q);
			$query ->execute();
			$row = $query->fetch(PDO::FETCH_ASSOC);
			return $row;
 
 
 
	}
}
[PHP] pobierz, plaintext 

var_dump($r)

[PHP] pobierz, plaintext 
array (size=5)
  'id' => string '1' (length=1)
  'name' => string 'name' (length=4)
  'pass' => string 'pass' (length=4)
  'salt' => string 'salt' (length=4)
  'type' => null
[PHP] pobierz, plaintext 

Czyli 5 (IMG:style_emoticons/default/sad.gif)

[nospor]

Tak, 5 kolumn a nie 5 wierszy (IMG:style_emoticons/default/smile.gif)
Pewnie ktos sie rypnal, albo zmienil tabele user i zapomnial o zmianie liczby kolumn