Cześć,

Chciałbym się z wami podzielić moją biblioteką do obsługi bazy danych (mały system ORM).

http://ttmdear.github.io/jub-orm

Powstała ona na potrzeby mojej pracy inżynierskiej

Opis wygląda świetnie ale już kod nie koniecznie

- Assercje są bez sensu. dla czego is_array zwraca tablicę zamiast true/false? Tak samo inne.
- hasIndex - isset zwróci false dla $tab['ddd'] = null, powinno być array_key_exists
- !is_null($msg) - lepiej by było !empty()

https://github.com/ttmdear/jub-orm/blob/mas...lder/Select.php
- rozbij to na metody
- kolejne:

jak będzie '' to błąd

Duużo kodu jest tam i sporo do poprawki takich pierdół.

- Brak composera mega minus jak dla mnie
- jak już robisz:
to wypadalo by łapać ten wyjątek.

Dalej nie chce mi się To tak wyrywkowo.

@Pyton_000

Hej, dzięki za uwagi

Te asercje wykorzystuje jako funkcje sprawdzające, wiec jeśli asercja się nie spełnia to
od razu wywalam wyjątek ponieważ dalsze działanie jest niemożliwe.

https://github.com/ttmdear/jub-orm/blob/mas.../src/Entity.php

[PHP] pobierz, plaintext 
    public static function table()
    {
        $assert = static::service('assert');
        $assert->notNull(static::$table, 'Each entity should have defined table.');
        return static::$table;
    }
[PHP] pobierz, plaintext 

Ogólnie cały czas pracuje na rozbudową i poprawą jakości.

No ja rozumiem do czego jest assercia, ale bez sensu jest zwracać wartość przekazaną. Lepiej już nic nie zwracaj i tylko rzucaj Exception jeśli nie spełnia assercji.

Zerknij na to: https://github.com/beberlei/assert/blob/mas...t/Assertion.php

Ok już rozumiem

Jest już composer

$sql = "INSERT INTO $table (";

powinno byc

$sql = "INSERT INTO `$table` (";
(lub cos innego w zaleznosci od bazy)

bo jakis geniusz moze uzyc zastrzezonego slowa dla nazwy tabeli.

No i skoro uzywasz PDO z tego co zrozumialem to powinienies jechac na bindowaniu a nie recznym wkladaniu danych

edit: podobna z sytuacja z select i kolumnami

edit: no i takie pierdy jak to:
$where = "$expr";
gdzie powinno byc poprostu:
$where = $expr;

edit:
nie: strpos($column, '.') != false
a: strpos($column, '.') !== false
Przy strpos niesprawdzanie typow jest szczegolnie niebezpieczne

edit:
kody jak ten

[PHP] pobierz, plaintext 
 private function isSimpleString($string)
    {
        $re = "/^[a-zA-Z,_,0-9]*$/";
        $match = preg_match($re, $string);
        if (!empty($match)) {
            return true;
        }
        return false;
    }
[PHP] pobierz, plaintext 

powinno sie raczej skracac do takiej postaci:

[PHP] pobierz, plaintext 
 private function isSimpleString($string)
    {
      return (bool) preg_match('/^[a-zA-Z,_,0-9]*$/', $string);
    }
[PHP] pobierz, plaintext