MVC framework #7 – Config(2)

[metost]

** Тази тема дискутира:: MVC framework #7 – Config(2)**
Автор: gatakka

[borisov87]

Не мога да разбера тосзи фрагмент

Код: PHP

1. public function run() {
2. //if config folder is not set, use defaultone
3. if ($this->_config->getConfigFolder() == null) {
4. $this->setConfigFolder(‘../config’);
5. }
6. }

защо логиката е вътре в метода run(), смисъл нали код който е в тялото на даден метод се изпълнява само когато викаме този метод? Та как тази проверка ще стане автоматично като никъде не викаме метода run

[borisov87]

Оф грешка сега видях че ние метода run си го викаме в индекс файла

[borisov87]

Отново въпрос
в клипа ванката вика метода
$app->getConfig()->app
 и работи преди да е извикал метода run()
при мен работи само след като извикам метода run и е логично щото нали метода run държи пътя към default foldera config, а гледам че при него метода run() е след
$app->getConfig()->app и кода пак си работи ? Е как става тази магия

[borisov87]

Код: [Маркирай кода]

<?php
error_reporting(E_ALL ^ E_NOTICE);
include '../../bf/App.php';
$app= \BF\App::getInstance();
echo $app->getConfig()->app;
привет защо ехото ми връща цифрата 1 вместо array?

Код: [Маркирай кода]

$config= \BF\Config::getInstance();
echo $config->app;
с това няма проблем връща ми array но с функцията по-горе не сработва

[relax4o]

Отново въпрос
в клипа ванката вика метода
$app->getConfig()->app
 и работи преди да е извикал метода run()
при мен работи само след като извикам метода run и е логично щото нали метода run държи пътя към default foldera config, а гледам че при него метода run() е след
$app->getConfig()->app и кода пак си работи ? Е как става тази магия

Не се заблуждавай. Забелязах, че докато обясняваше защо не е хубаво тази логика да е в конструктора, то той именно там беше сложил веднъж тази логика. След това я копира в run(), но забрави да я премахне от конструктора. Така че не се заблуждавай. Логиката трябва да си стои в run(), защото преди да извикаш run() може да искаш да сетнеш нов конфигурационен файл и чак след това да стартираш приложението.

[opelcrumper]

Здравейте,

Имам малък проблем със сработването на този фрагмент и това видео. Правя всичко 1:1 спрямо видеото, но когато се опитам да изпълня метода "run" получавам няколко грешки.

Първо да покажа, в index.php - тестовия файл имам следния код:

Код: [Маркирай кода]

include '../../framework/App.php';

$app = \framework\App::getInstance();
echo $app->getConfig()->app;
$app->run();
Спрямо видеото - оставям в конструктора на класа App.php следните неща:

Код: [Маркирай кода]

\framework\Loader::registerNamespace('framework', dirname(__FILE__) . DIRECTORY_SEPARATOR);
\framework\Loader::registerAutoload();
$this->_config = \framework\Config::getInstance();
if($this->_config->getConfigFolder() == null){
    $this->setConfigFolder('../config');
}
Грешките са:
Notice: Trying to get property of non-object in C:\wamp\www\test\public\index.php on line 7
където кода е:

Код: [Маркирай кода]

echo $app->getConfig()->app;

И следващата грешка е: Fatal error: Using $this when not in object context in ....
където кода е:

Код: [Маркирай кода]

public static function run(){
        if($this->_config->getConfigFolder() == null){
            $this->setConfigFolder('../config');
        }
}


И в метода run() това, което написах по-горе.

Пробвах да проследя логиката, всичко изглежда нормално и трябва да работи... Спрямо видеото и коментарите тук от форума не мога да се ориентирам - какво трябва да остане от логиката в конструктора на App.php и какво трябва да се изнесе в метода run()?
Може ли някой да "удари" едно рамо, че това ме спира за нормалното продължаване към следващите видео уроци? 

[Avalanche]

Здрасти,

подозирам че си с проблема на 90% от хората, дето разглежда въпросната серия уроци и то е версията ти на PHP. С коя точно си ти? А за фаталната грешка то си пише, ползваш $this в обхвата на статична функция. Трябват ти малко основи по OOP, но за статичните методи се ползва self::, вместо $this->. В последното видео Иван е публикувал крайния код и гледам, че функцията в последствие е променена да не е статична.

[opelcrumper]

Току що свалих source кода и сравних - и двата файла са 1:1 

Версията ми на php e 5.4.3. 

EDIT:
Промених метода run() да не е статичен и... (не знам дали фикснах проблема), но ми излезнаха други нотиси.  но си е напредък  Мерси колега!

[Avalanche]

И аз седнах да гледам видеото чак толкова да се учудя. Гледам че си избрал друго име за фрейма - няма нищо лошо, внимавай само да ги оплетеш някъде. Ако не изчезнат и новоизлезлите предупреждения, пиши пак.

[opelcrumper]

Няма проблем, с промяната на името на framework-a. Нямам нито 1 ред copy/paste. Целта ми е да го разбера и съм започнал уроците от самото начало, 1 по 1.

Не знам само защо се скриват нотисите (notice) за видеата, чрез error_reporting. Но засега продължавам към следващите уроци... 

[HD]

Каква машина, каква ос, версия на php ?  В php.ini пуснал ли си display_errors?

[Avalanche]

@HD Човекът искаше да каже, че не разбира защо в проекта е сложено:

Код: PHP

1. error_reporting(E_ALL ^ E_NOTICE);

Което напрактика показва всички грешки, освен предупрежденията (notice). Иван така е преценил, може би го е описал в някои от най-първите уроци.