Discussion :

[moimp]

Bonjour,
Je crée un fichier de test d'une classe et je n'arrive pas à m'affranchir des erreurs (et j'en ai plusieurs à faire sur le même modèle).
Mon problème est qu'avant de tester ma classe, je veux vérifier que les fichiers nécessaires existent et je coince sur l'emplacement des namespaces et du mot clef 'use'.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
//use OptListSelect\OptListSelect;
 
$erreur = array();
require_once("../config.php");		// Dans lequel est la définition de URL_SITE
 
define('TEST_FILE', URL_SITE."classes/moimp/OptListSelect.php");
if (! file_exists(TEST_FILE)) {
	echo sprintf("Le fichier '%s' n'existe pas.", TEST_FILE);
}
else {
	require_once(TEST_FILE);
	//use OptListSelect\OptListSelect;
}

[Pytet]

Bonjour,

Difficile de comprendre le problème sans le message d'erreur ni le code de la classe.

Pour importer la classe avec require_once, il faut renseigner le chemin relatif ou absolu de la classe via le système de fichier et pas via le protocole HTTP : par exemple require_once("./classes/moimp/OptListSelect.php") ou require_once("/var/www/monprojet/classes/moimp/OptListSelect.php");
Si la constante URL_SITE contient bien une URL, ton require_once est surement incorrect : require_once('http://localhost/classes/moimp/OptListSelect.php'); via HTTP ne retourne pas le même résultat.

[cavo789]

Bonjour

Il s'agit de la fonctionnalité première de composer qui a été développé exactement pour rencontrer ce besoin : https://getcomposer.org/

Le logo est môche et semble dater du siècle dernier mais il ne faut pas s'arrêter à ça, composer est indispensable quand on code en php et qu'on travaille, comme tu le fais, avec des classes.

Tu devras définir ton projet dans un fichier nommé composer.json et y définir le chemin vers tes classes.

Ensuite tu lanceras "composer update" dans ta console pour générer le dossier vendor qui contiendra alors un fichier nommé autoload.php.

Plus d'explications sur le site de composer.

[moimp]

Pour importer la classe avec require_once, il faut renseigner le chemin relatif ou absolu de la classe via le système de fichier et pas via le protocole HTTP : par exemple require_once("./classes/moimp/OptListSelect.php") ou require_once("/var/www/monprojet/classes/moimp/OptListSelect.php");
Si la constante URL_SITE contient bien une URL, ton require_once est surement incorrect : require_once('http://localhost/classes/moimp/OptListSelect.php'); via HTTP ne retourne pas le même résultat.

Merci pour cette remarque qui résout une partie du problème.
Mon code devient:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
define('TEST_FILE', substr(__DIR__, 0, strlen(__DIR__)-5)."classes/moimp/OptListSelect.php");
if (! file_exists(TEST_FILE)) {
	echo sprintf("Le fichier '%s' n'existe pas.", TEST_FILE);
}
else {
	require_once(TEST_FILE);
	use OptListSelect\OptListSelect;		// génère une erreur unexpected token "use"
}

Il me semble que le mot clef use doit être placé en début de code mais je ne vois pas comment faire si je veux tester l'existence du fichier contenant la classe au préalable.

[ddaweb]

use OptListSelect\OptListSelect; // génère une erreur unexpected token "use"

J'ai eu la même erreur en utilisant PHPMailer et la fonction "use", j'ai su solutionner en ne l'utilisant plus (intégré directement dans le code) : une solution à l'erreur de use m'intéresse donc également

[Pytet]

Le mot clé use doit effectivement être utilisé dans le contexte global : https://www.php.net/manual/fr/langua...mporting.scope
Il ne faut pas confondre les espaces de noms et le chargement automatique des classes, les deux sont souvent liés mais c'est deux notions différentes. Faire un require et use de toutes tes classes dans le point d'entrée de ton application n'est pas la bonne approche.

Comme indiqué par Cavo, le mieux est surement de suivre le standard PSR-4 et d'utiliser l'autoload de composer.

[laurentSc]

Bonsoir,

Bonjour
Il s'agit de la fonctionnalité première de composer qui a été développé exactement pour rencontrer ce besoin : https://getcomposer.org/

Comme indiqué par Cavo, le mieux est surement de suivre le standard PSR-4 et d'utiliser l'autoload de composer.

A noter que j'ai d'abord essayé d'utiliser l'autoloader de Composer mais que celui-ci ne trouvait pas une de mes classes. Je l'ai remplacé par un autoloader de rawsrc (ex-modo de DVP et très fort) et le problème a été résolu. Du coup, je mets en doute la qualité de l'autoloader généré par Composer.

[Séb.]

L'autoloader de Composer ayant fait ses preuves depuis des années sur des millions de projets, perso j'ai plutôt tendance à mettre en doute la manière dont tu l'as utilisé

[cavo789]

Du coup, je mets en doute la qualité de l'autoloader généré par Composer.

Composer est très largement éprouvé. Il fonctionne, cela ne fait strictement aucun doute.

Ton souci était chez toi, par exemple dans le nommage de ta classe ou la localisation de ton fichier versus la configuration que tu as faite dans composer.json.

[laurentSc]

OK ; en effet, le problème ne venait pas de composer. J'ai un peu modifié la structure de mon projet (j'ai créé un répertoire src et mis les fichiers sources dedans) puis adapté composer.json :

Code json :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
{
  "name": "schmitt/pasdpanique",
  "description": "",
  "type": "project",
  "license": "MIT",
  "authors": [
    {
      "name": "Laurent Schmitt",
      "email": "xxx@gmail.com"
    }
  ],
  "keywords": [
    "php"
  ],
  "minimum-stability": "dev",
  "require": {
    "php": ">=8.0",
    "phpmailer/phpmailer": "dev-master",
    "rawsrc/phpecho": "dev-master"
  },
  "autoload": {
    "psr-4": {
      "PasDePanique\\": "src/"
    }
  }
}

puis composer update puis remis le bon include dans mon index.php (point d'entrée unique) : require DIR_ROOT . 'vendor' . DIRECTORY_SEPARATOR . 'autoload.php'; //autoload de composer et c'est bon (du moins pour ça car du coup, je n'ai plus que cet autre problème : https://www.developpez.net/forums/d2...-cette-classe/)