[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]

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
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

[nospor]

Tak, 5 kolumn a nie 5 wierszy
Pewnie ktos sie rypnal, albo zmienil tabele user i zapomnial o zmianie liczby kolumn