Hallo zusammen,
ich erstelle diesen Vorschlag, als Vorschlag doch auch zeitgleich als Frage.
Wieso ist der Coding Style des WCFs an PSR-2 angelehnt, aber folgt dem Standard nicht ganz? Um mal ein aktuelles BSP zu zeigen was die Änderung erzeugen würde:
/**
* Includes the required util or exception classes automatically.
*
* @param string $className
* @see spl_autoload_register()
*/
public static final function autoload($className) {
$namespaces = explode('\\', $className);
if (count($namespaces) > 1) {
$applicationPrefix = array_shift($namespaces);
if ($applicationPrefix === '') {
$applicationPrefix = array_shift($namespaces);
}
if (isset(self::$autoloadDirectories[$applicationPrefix])) {
$classPath = self::$autoloadDirectories[$applicationPrefix] . implode('/', $namespaces) . '.class.php';
if (file_exists($classPath)) {
require_once($classPath);
}
else {
// If File doesn´t exists with .class.php try to find it without .class
$classPathWithoutClass = self::$autoloadDirectories[$applicationPrefix] . implode('/', $namespaces) . '.php';
if (file_exists($classPathWithoutClass)) {
require_once($classPathWithoutClass);
}
}
}
}
}würde zu
/**
* Includes the required util or exception classes automatically.
*
* @param string $className
*
* @see spl_autoload_register()
*/
public static final function autoload($className)
{
$namespaces = explode('\\', $className);
if (count($namespaces) > 1) {
$applicationPrefix = array_shift($namespaces);
if ($applicationPrefix === '') {
$applicationPrefix = array_shift($namespaces);
}
if (isset(self::$autoloadDirectories[$applicationPrefix])) {
$classPath = self::$autoloadDirectories[$applicationPrefix] . implode('/', $namespaces) . '.class.php';
if (file_exists($classPath)) {
require_once($classPath);
} else {
// If File doesn´t exists with .class.php try to find it without .class
$classPathWithoutClass = self::$autoloadDirectories[$applicationPrefix] . implode('/', $namespaces) . '.php';
if (file_exists($classPathWithoutClass)) {
require_once($classPathWithoutClass);
}
}
}
}
}Die Umsetzung sollte auch nicht allzu große Probleme bereiten, da es meist nur eine IDE Einstellung ist.
Liebe Grüße
WCF orientiert sich aus historischen Gründen (*) an den Java Code Conventions. Grundsätzlich ist die strikte Befolgung der PSR langfristig sinnvoll und von uns angestrebt, dies wäre aber frühestens bei einer vollständigen Neuentwicklung anwendbar.
(*) WCF ist sehr viel älter als PSR und wurde seit WCF 1.0 relativ konsequent beibehalten.
Ich persönlich finde Leerzeichen zur Einrückung übrigens doof.
Don’t have an account yet? Register yourself now and be a part of our community!