Najoptymalniejszy sposób na __autoload z wielu katalogów Opcje

[Ultear]

Witam, zrobiłem __autoload, który wygląda tak o:

[PHP] pobierz, plaintext 
$classes_dirs = array('libs/', 'application/controllers/', 'application/models/');
 
function __autoload($class){
 
	global $classes_dirs;
 
 
 
	try{
		require_once(autoload_class_existance($class, $classes_dirs));
	} catch(Exception $e){
		echo 'Caught exception'.$e->getMessage()."\n";
	}
}
[PHP] pobierz, plaintext 

funkcja autoload_class_existance:

[PHP] pobierz, plaintext 
function autoload_class_existance($class, $dirs){
 
	foreach($dirs as $dir){
 
		$path = APP_PATH.$dir.$class.'.php';
 
		if(file_exists($path)){
			return $path;
			break;
		} else{
			continue;
		}
 
		return false;
	}
 
}
[PHP] pobierz, plaintext 

Czy jest to dość optymalny sposób, czy da radę zrobić to lepiej?

[irekk]

A nie lepiej oprzeć to o PSR-0?

[pedro84]

Zasadnicze pytanie: po co potrzebujesz bawić się w pisanie swojej funkcji ładującej?

[Ultear]

Nie potrzebuje się bawić, po prostu zastanawiam się na nad najmniejszym i najoptymalniejszym wyjściem (IMG:style_emoticons/default/smile.gif)

[pedro84]

Poczytaj sobie o klasie ładującej z PSR-0, jak Ci zaproponował @irekk. Z całym szacunkiem, ale wątpię, żebyś coś lepszego wymyślił (IMG:style_emoticons/default/wink.gif)

[by_ikar]

Taa zwłaszcza że jest już kilkanaście gotowych klas do autoładowania, które istnieją już od jakiegoś czasu, są przetestowane przez dziesiątki ludzi którzy nie tylko piszą w php, ale i go tworzą. Zawsze możesz użyć include_path, który wraca ostatnimi dniami jak bumerang (IMG:style_emoticons/default/wink.gif)

[Pyton_000]

Proszę bardzo. 10 stron postów. Same dobre rzeczy
Temat: Wlaczanie plikow autoloader

[Ultear]

Ok, zastosowałem klasę SplClassLoader, teraz tylko jest jeden problem:

Stuktura wygląda tak o:

-root
--application
-----models
-----views
-----controllers
--libs
-----Bootstrap.php



No i sprawa wygląda tak, że:

[PHP] pobierz, plaintext 
require_once APP_PATH.'libs/SplClassLoader.php';
 
$loader = new SplClassLoader('Application', __DIR__);
$loader->register();
 
 
$Bootstrap = new Application\Libs\Bootstrap(); //Nie do konca wiem jak z tym postapic, z kolei, bez Application nie moze znaleźć klasy Libs\Bootstrap
 
[PHP] pobierz, plaintext 

Bo jezeli chce autoloadowac biblioteke, ktora jest poza katalogiem application, to jest mały problem, bo:


Warning: require(X:\Server\root\Application\Libs\Bootstrap.php) [function.require]: failed to open stream: No such file or directory in X:\Server\blog\libs\SplClassLoader.php on line 152

Co w takim wypadku zrobic?

Ten post edytował Ultear 8.09.2014, 21:27:38

[Pyton_000]

libs wpakuj do aplications.

[Ultear]

Chciałbym później użyć libs w aplikacji backendowej, więc odpada

[pedro84]

Chciałbym później użyć libs w aplikacji backendowej, więc odpada

Dlaczego? Nie możesz zastosować normalnej struktury?

Kod

src
src/Application

[Ultear]

Mógłbyś mi rozpisać prawidłową strukture MVC?
Byłbym wdzięczny (IMG:style_emoticons/default/smile.gif)

[pedro84]

Mógłbyś mi rozpisać prawidłową strukture MVC?

Nie chcę Ci zmartwić, ale... nie ma czegoś takiego (IMG:style_emoticons/default/smile.gif) Obecnie spora część projektów, korzysta z loadera z PSR-0, co wiąże się z pewnymi wymaganiami odnośnie struktury. Zobacz na ten projekt na Githubie: https://github.com/thephpleague/flysystem.

Masz główny katalog src (ewentualnie lib - jest to główny katalog Twojej aplikacji), katalog tests (gdzie siedzą testy). Przykładowa struktura biblioteki z namespacem Foo:

Kod

src/
src/Foo
src/Foo/Parsers
src/Foo/Parsers/HtmlParser
src/Foo/Builder
src/Foo/MenuBuilder

Coś takiego. Mam nadzieję, że załapiesz. Poczytaj sobie poza tym, PSR - warto wiedzieć i wdrażać.

[Crozin]

1. MVC to jedynie wzorzec architektoniczny i nie ma on ścisłej implementacji - w przeciwieństwie do samej jego definicji.
2. Jeżeli chodzi o strukturę plików w projekcie to właściwie nie ma o czym mówić: PSR-0 oraz jego następna PSR-4 właściwie kończą temat.
3. Jeżeli chodzi o implementację autoloadera to rzuć okiem na tego udostępnianego przez Composera (zresztą w ogóle rzuć okiem na to narzędzie). W środowisku deweloperskim właściwie w pełni przejmuje to zadanie od programisty (można by powiedzieć, że jest to najoptymalniejszy autoloader względem czasu programisty), a w środowisku produkcyjnym wygeneruje Ci zoptymalizowany pod względem czasu działania autoloader.

Ten post edytował Crozin 8.09.2014, 22:14:13

[Ultear]

Dziękuje wszystkim za udzielanie się tutaj, będę praktykował standardy narzucane przez PSR, a także zobaczę narzędzie wspomnianie przez @Crozin.

Witam ponownie, by nie zaśmiecać forum piszę tutaj. Mianowicie mam jeszcze jeden problem, napotkałem go przy ładowaniu PDO z SplClassLoader

Kod

Warning: require(C:\WebServ\httpd\mvc\Application\Libs\PDO.php) [function.require]: failed to open stream: No such file or directory in C:\WebServ\httpd\mvc\Application\Libs\SplClassLoader.php on line 152

[PHP] pobierz, plaintext 
function __construct(){
		$dsn = 'mysql:host=' . $this->host . ';dbname=' . $this->dbname;
		try{
			$this->db = new PDO($dsn, $this->user, $this->pass, $this->options);
		} catch(PDOException $e){
			$this->error = $e->getMessage();
		}
	}
[PHP] pobierz, plaintext 

[php][/php]

Ten post edytował Ultear 10.09.2014, 12:45:01

[pedro84]

Msz zagadkę ma dziś (IMG:style_emoticons/default/smile.gif) Poprawiłem Twój kod, ale nie wyjaśnię, spróbuj dojść do tego sam, jak będziesz miał problemy, to pisz, ale spróbuj chociaż.

[PHP] pobierz, plaintext 
        function __construct(){
		$dsn = 'mysql:host=' . $this->host . ';dbname=' . $this->dbname;
		try{
			$this->db = new \PDO($dsn, $this->user, $this->pass, $this->options);
		} catch(\PDOException $e){
			$this->error = $e->getMessage();
		}
	}
[PHP] pobierz, plaintext 

[Ultear]

Już widzę (IMG:style_emoticons/default/wink.gif)

Chodzi o \ przy PDO, dziękuje zap moc (IMG:style_emoticons/default/smile.gif)

Z tego co się domyślam jest to nadpisanie przestrzeni nazw, tak?

Niestety wciąż uzyskuje błąd :c

Kod

Warning: require(X:\Server\mvc\Application\Libs\PDO.php) [function.require]: failed to open stream: No such file or directory in X:\Server\blog\Application\Libs\SplClassLoader.php on line 152

Fatal error: require() [function.require]: Failed opening required 'X:\Server\mvc\Application\Libs\PDO.php' (include_path='.;C:\php\pear') in X:\Server\mvc\Application\Libs\SplClassLoader.php on line 152

[PHP] pobierz, plaintext 
	function __construct(){
    	$dsn = 'mysql:host=' . $this->host . ';dbname=' . $this->dbname;
    	try{
   			 $this->db = new \PDO($dsn, $this->user, $this->pass, $this->options);
    	} catch(\PDOException $e){
   			 $this->error = $e->getMessage();
    	}
    }
[PHP] pobierz, plaintext 

Ten post edytował Ultear 10.09.2014, 13:45:04

[pedro84]

Nie nadpisanie, ale przestrzeń globalna: http://php.net/manual/en/language.namespaces.global.php.

[Ultear]

Wciąż mam problem, pomimo dodania \ wciąż autoloader chce mi czytać z Libsów

[pedro84]

A Ty coś ten autoloader modyfikowałeś? Jeśli tak, wklej go tutaj, aczkolwiek wszystko powinno działać.

[Ultear]

Nie modyfikowąłem, wygląda standardowo, czyli tak o:

[PHP] pobierz, plaintext 
<?php
 
/*
 * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
 * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
 * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
 * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
 * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
 * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
 * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
 * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
 * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
 * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
 * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 *
 * This software consists of voluntary contributions made by many individuals
 * and is licensed under the MIT license. For more information, see
 * <http://www.doctrine-project.org>.
 */
 
/**
 * SplClassLoader implementation that implements the technical interoperability
 * standards for PHP 5.3 namespaces and class names.
 *
 * <a href="http://groups.google.com/group/php-standards/web/psr-0-final-proposal?pli=1" target="_blank">http://groups.google.com/group/php-standar...-proposal?pli=1</a>
 *
 *     // Example which loads classes for the Doctrine Common package in the
 *     // Doctrine\Common namespace.
 *     $classLoader = new SplClassLoader('Doctrine\Common', '/path/to/doctrine');
 *     $classLoader->register();
 *
 * @license <a href="http://www.opensource.org/licenses/mit-license.html" target="_blank">http://www.opensource.org/licenses/mit-license.html</a>  MIT License
 * @author Jonathan H. Wage <jonwage@gmail.com>
 * @author Roman S. Borschel <roman@code-factory.org>
 * @author Matthew Weier O'Phinney <matthew@zend.com>
 * @author Kris Wallsmith <kris.wallsmith@gmail.com>
 * @author Fabien Potencier <fabien.potencier@symfony-project.org>
 */
class SplClassLoader
{
    private $_fileExtension = '.php';
    private $_namespace;
    private $_includePath;
    private $_namespaceSeparator = '\\';
 
    /**
     * Creates a new <tt>SplClassLoader</tt> that loads classes of the
     * specified namespace.
     * 
     * @param string $ns The namespace to use.
     */
    public function __construct($ns = null, $includePath = null)
    {
        $this->_namespace = $ns;
        $this->_includePath = $includePath;
    }
 
    /**
     * Sets the namespace separator used by classes in the namespace of this class loader.
     * 
     * @param string $sep The separator to use.
     */
    public function setNamespaceSeparator($sep)
    {
        $this->_namespaceSeparator = $sep;
    }
 
    /**
     * Gets the namespace seperator used by classes in the namespace of this class loader.
     *
     * @return void
     */
    public function getNamespaceSeparator()
    {
        return $this->_namespaceSeparator;
    }
 
    /**
     * Sets the base include path for all class files in the namespace of this class loader.
     * 
     * @param string $includePath
     */
    public function setIncludePath($includePath)
    {
        $this->_includePath = $includePath;
    }
 
    /**
     * Gets the base include path for all class files in the namespace of this class loader.
     *
     * @return string $includePath
     */
    public function getIncludePath()
    {
        return $this->_includePath;
    }
 
    /**
     * Sets the file extension of class files in the namespace of this class loader.
     * 
     * @param string $fileExtension
     */
    public function setFileExtension($fileExtension)
    {
        $this->_fileExtension = $fileExtension;
    }
 
    /**
     * Gets the file extension of class files in the namespace of this class loader.
     *
     * @return string $fileExtension
     */
    public function getFileExtension()
    {
        return $this->_fileExtension;
    }
 
    /**
     * Installs this class loader on the SPL autoload stack.
     */
    public function register()
    {
        spl_autoload_register(array($this, 'loadClass'));
    }
 
    /**
     * Uninstalls this class loader from the SPL autoloader stack.
     */
    public function unregister()
    {
        spl_autoload_unregister(array($this, 'loadClass'));
    }
 
    /**
     * Loads the given class or interface.
     *
     * @param string $className The name of the class to load.
     * @return void
     */
    public function loadClass($className)
    {
        if (null === $this->_namespace || $this->_namespace.$this->_namespaceSeparator === substr($className, 0, strlen($this->_namespace.$this->_namespaceSeparator))) {
            $fileName = '';
            $namespace = '';
            if (false !== ($lastNsPos = strripos($className, $this->_namespaceSeparator))) {
                $namespace = substr($className, 0, $lastNsPos);
                $className = substr($className, $lastNsPos + 1);
                $fileName = str_replace($this->_namespaceSeparator, DIRECTORY_SEPARATOR, $namespace) . DIRECTORY_SEPARATOR;
            }
            $fileName .= str_replace('_', DIRECTORY_SEPARATOR, $className) . $this->_fileExtension;
 
            require ($this->_includePath !== null ? $this->_includePath . DIRECTORY_SEPARATOR : '') . $fileName;
        }
    }
}
[PHP] pobierz, plaintext 

[pedro84]

Pokaż swoją strukturę katalogów oraz w którym miejscu masz rzucany ten błąd.

[Ultear]

(IMG:http://iv.pl/images/28113811220350395835.png)

[pedro84]

Mały hint: stałe w tablicy w zmiennej $options.

[Ultear]

[PHP] pobierz, plaintext 
public $options = array(
		\PDO::ATTR_PERSISTENT => true,
		\PDO::ATR_ERRMODE => \PDO::ERRMODE_EXCEPTION
	);
[PHP] pobierz, plaintext 

Zapomniałem o tamtym (IMG:style_emoticons/default/wink.gif)

Dziękuje za pomoc haha (IMG:style_emoticons/default/biggrin.gif)

[pedro84]

Będziesz pamiętał na przyszłość (mam nadzieję (IMG:style_emoticons/default/smile.gif) ). Aż się chce takim ludziom pomagać, gdzie mały hint wystarczy, żeby sami doszli do rozwiązania.

[Ultear]

Dopiero zaczynam tematy z przestrzeniami nazw, dlatego tak (IMG:style_emoticons/default/smile.gif) Jeszcze raz dziękuje za pomoc