Forum PHP.pl

Witam!
Piszę frameworka i zabrałem się za sterownik baz danych. Niestety jest problem, bo wyświetla tylko pierwszy znak wyniku . Podaję listingi:
Kontroler

[PHP] pobierz, plaintext 
<?php
class welcome extends Controller
{
var $test;
    function index()
    {
        $this->modelik = $this->load->model('modelik');
        $query = $this->modelik->test();
        function szescian($n)
{
    return $n*$n*$n;
}
 
$b = array_map("szescian", $query);
print_r($b);
        foreach($query as $this->test)
        {
            print $this->test['tekst'];
            echo $this->test[tekst];
        }
    
    }
    
    function elo()
    {
        echo('to elo ;p');
    }
 
}
?>
[PHP] pobierz, plaintext 

Model:

[PHP] pobierz, plaintext 
<?php
class modelik extends Model
{
    function test()
    {
        return $this->db->query("SELECT * FROM `test` WHERE 1");
    }
}
?>
[PHP] pobierz, plaintext 

Sterownik db(tylko funkcja query):

[PHP] pobierz, plaintext 
<?php
function query($query, $typ = assoc)
    {
    
        self::$counter++;
        
        /* Jeżeli wystąpiły błędy w połączeniu anulujemy wykonywanie zapytania */
        if($dberrors == true)
        {
            error('[lib db] Nie można wykonać zapytania, gdyż połączenie z bazą danych nie zostało poprawnie nawiązane');
        }
        
        /* Podmieniamy tag _PREFIX_ na prefix zapisany w ustawieniach skryptu */
        str_replace('_PREFIX_', PREFIX, $query);
        
        $zapytanie = mysql_query($query);
        
        /* Jeżeli zapytanie zwróciło błąd, wyświetlamy je */
        if(!$zapytanie)
        {
            error('Zapytanie mysql błędne! Kod błędu <br />' . mysql_errno() . ': ' . mysql_error());
        }
        
        /*
        
        Wybór typu tablicy, którą ma zwrócić metoda
        assoc - mysql_fetch_assoc
        row - mysql_fetch_row
        query - zwraca samo mysql_query
        
        */
        
        switch($typ)
        {
            case 'assoc':
            return mysql_fetch_assoc($zapytanie);
            break;
            
            case 'row':
            return mysql_fetch_row($zapytanie);
            break;
            
            case 'query':
            return $zapytanie;
            break;
        }
?>
[PHP] pobierz, plaintext 

Loader(tylko funkcja ładująca model)

[PHP] pobierz, plaintext 
<?php
function model($path)
    {
    
        if(!file_exists(APP_PATH . 'models/' . $path . '.php'))
        {
            error('[lib loader] Nie mogę znaleść modelu ' . $path . '! Prawdopodobnie nie ustawiłeś ścieżki dostępu do aplikacji w ustawieniach systemowych (configs/system.php)');
        }
        
        include(APP_PATH . 'models/' . $path . '.php');
        $model = new $path();
        return $model;
    
    }
?>
[PHP] pobierz, plaintext 

No i zamiast otrzymać wynik (z kolumny test) l-testowanie, to otrzymuję:

Funkcje mysql_fetch_xxx() pobierają tylko pojedynczy wiersz ze zbioru wyników. Swój sterownik zakodowałeś tak, że zwraca Ci tylko pierwszy wiersz, więc foreach iteruje Ci po jego elementach. Jak jeszcze do tego dodasz indeks tablicy, PHP traktuje to jako odwołanie do znaku tekstu i wychodzą takie oto głupoty. Przed zabieraniem się za pisanie frameworka wypadałoby znać takie rzeczy, gdyż są to podstawy.

Jeszcze kilka uwag odnośnie samego frameworka. Kod jest lekko masakryczny - nie trzymasz się jednej konwencji, tylko raz robisz tak, raz inaczej. Jeśli piszesz pod PHP4, to sam skazujesz się na problemy, a jeśli pod PHP5, to czemu nie korzystasz z tego, co ta wersja oferuje? Wziąłbyś takie PDO i nie miałbyś żadnych problemów ze sterownikiem, a co najwyżej byś go musiał rozbudować o jakieś opcje profilowania itd.

Ludzie, troszkę wyrozumiałości ja mam 14 LAT i mimo swojego młodego wieku już wyciągam pieniądze z webdeveloperki. Czy to źle, że chłopak w moim wieku zamiast siedzieć na TIBII cały dzień chce zdobyć doświadczenie pisząc frameworka?

W pierwszym poście nie było ani słowa o wieku. Pokazałeś kod, wystawiając go na ostrzał publiczny i mnie niespecjalnie obchodzi, ile kto ma lat (sam programowałem, gdy miałem 14) - mam kod i go analizuję. Dostałeś rozwiązanie problemu, kilka dodatkowych porad gratis odnośnie tego, jak pisać, by uniknąć później choć kilku dodatkowych problemów, a zamiast się ucieszyć i spróbować wcielić je w życie, by poprawić jakość kodu (tym bardziej gdy już ciągniesz z tego jakiś dochód), to złościsz się bez sensu.

A co ma piernik do wiatraka?! Ja mam 16 i jakoś nie płaczę, że nic mi nie wychodzi w programowaniu. Nie szukaj sobie wymówki. Jak będziesz miał 20 napiszesz, że jesteś studentem itp.? 14 lat to nie jest mało.
Możesz zasugerować, że jesteś młody i niedoświadczony. Nie może to być wymówka od pracy. Jeżeli delikatnie zasugerujesz to oczywiście inni użytkownicy popatrzą na Ciebie innym okiem i postarają zaprowadzić Cię na dobrą drogę, abyś za kilka lat mógł innym pomagać.

Popracuj nad tabulacją. echo nie jest funkcją! Używaj angielskich nazw funkcji, klas itp. Jeżeli komentarz jest jedno liniowy, to nie jest dwu. Zacznij używasz skróconych warunków.

Ogólnie kod jest ładny. Nie denerwuj się. Stres nic dobrego nie daje.

IMO bierzesz się za pisanie FW nie mając tak naprawdę większego pojęcia jak on powinien wyglądać. Na Twoim miejscu radziłbym Ci zacząć pisać w jakimś już istniejącym, porządnym FW (np. Symfony czy ZF). Jednak zanim to zrobisz: naucz się porządnie tego języka. Jak zauważył Zyx naprawdę błahy błąd.

Nie rozumiem też po co robisz funkcję szescian() skoro istnieje już pow()?

ps: Włącz sobie raportowanie błędów (E_ALL | E_STRICT)