spl_autoload_register charge la classe deux fois

Question

Je continue d'essayer de savoir quel est le problème avec ce script de chargeur de classe très simple. Le chargeur de classe ressemble à ceci:

#src/vendors/Autoloading/lib/ClassLoader.php 

namespace App\Vendors\Autoloading; 

class ClassLoader 
{ 
    private $path; 

    function __construct($path) 
    { 
     $this->path   = $path; 
    } 

    public function load($class) 
    { 
     if(file_exists( $class = str_replace(array('\\', '_'), DIRECTORY_SEPARATOR, $this->path) . '.php')){ 
      require $class; 
      return true; 
     } 
    } 

    public function register() 
    { 
     return spl_autoload_register([$this, 'load']); 
    } 
} 

Le chargeur de classe initiale avait plus des méthodes et des fonctions pour valider les noms de fichiers ... mais, dans le processus de débogage je devais à cette étroite. Par conséquent, ce chargeur de classe est requis dans un fichier autoload.php, comme vous pouvez le voir ci-dessous.

#src/vendors/autoload.php 

namespace App\Vendors; 

require 'Autoloading/lib/ClassLoader.php'; 
$autoload = new Autoloading\ClassLoader('path/Foo/FooClass'); 
$autoload->register(); 

Le FooClass.php est situé dans src/Foo/FooClass.php

namespace App\Foo; 
class FooClass{} 

et il n'y a en réalité aucun problème avec la partie autoloading, la classe se charge très bien, mais il est fait deux fois qui me montre le dessous Erreur. Je l'appelle à partir d'un fichier index.php

<?php 

use \App\Foo\FooClass; 
FooClass::somefunction(); 

Juste en utilisant qui génère cette erreur.

Fatal error: Cannot redeclare class path\foo\FooClass in /path/to/index.php on line 4

Answer 1

Votre fonction autoloading est erroné:

public function load($class) 
{ 
    if (
     file_exists( 
      $class = str_replace(
       array('\\', '_'), 
       DIRECTORY_SEPARATOR, 
       $this->path) 
      . '.php') 
    ) { 
     require $class; 
     return true; 
    } 
} 

La fonction est calles avec le nom du classe à charger (si impossible, la fonction ne devrait rien faire).

Ce que vous faites est ignorer le nom de la classe, et en créer un nouveau basé sur le chemin avec lequel l'autoloader est créé. Cela chargera toujours le même fichier, avec la même classe, même s'il y a différentes classes à charger. Cela explique pourquoi vous obtenez l'erreur, car quel que soit le nom de classe passé, vous incluez toujours le fichier associé au chemin.

Vous souhaitez probablement utiliser un autochargeur PSR-0 ou PSR-4 approprié. Je vous recommande d'utiliser celui qui accompagne Composer, car vous utiliserez probablement Composer tôt ou tard. Pourquoi ne pas commencer aujourd'hui?

Answer 2

Essayez d'utiliser require_once au lieu de require. Votre autochargeur ne gardera aucune trace des fichiers qui ont été chargés jusqu'à présent.

Voici comment je mets le mien. Je n'utilise pas de cours. Peut-être que cela vous aidera

function PlatformAutoloader($classname) { 
    try { 
     // Change \ to/so namespacing will work 
     $classname = strtolower(str_replace('\\', DIRECTORY_SEPARATOR, $classname)); 

     if(!@include_once(DIR_CLASSES . DIRECTORY_SEPARATOR . $classname . '.php')) return false; 
    } catch(Exception $e) { 
     return false; 
    } 
} 
spl_autoload_register('PlatformAutoloader'); 

Answer 3

Vérifiez si la classe existe déjà en utilisant class_exists avant de vérifier le fichier