load for autoload

[jgetner]

this is my loader class and load function i 'am trying to use for my autoloading but currently i keep getting my exception thrown for file not existing but the file does exists and is writeable it also is the correct path to the file?.

<?php
if(!defined('OBJECTIVE')){exit();}

class Loader
{
    private $loaded = array();

private static $instance;

public static function Instance()
{
	if(!(self::$instance instanceof self))
	{
		self::$instance = new self;
	}

	return self::$instance;
}

private static function SetLoaded($id  , $data)
{
	if(!isset(self::$instance->loaded[$id]))
	{
		self::$instance->loaded[$id]=$data;

		return self::$instance->loaded[$id];
	}
	return(false);
}

private static function GetLoaded($id)
{
	if(!isset(self::$instance->loaded[$id]))
	{
		return(false);
	}

	elseif(isset(self::$instance->loaded[$id]))
	{
		return self::$instance->loaded[$id];
	}

	return(false);
}

public static function load($classname , $external = null)
{
    $path;

	if(class_exists($classname , FALSE) && isset(self::$instance->loaded[$classname]))
	{
		return(true);
	}

	elseif(class_exists($classname , FALSE) && !isset(self::$instance->loaded[$classname]))
	{
	    throw new SystemException('Fatal Error: The Current Class Request Is Outside The Scope Of Objective');
	}

	elseif(!class_exists($classname))
	{
		if(!isset(self::$instance->loaded[$classname]) && $external === false)
		{
		    echo $path = 'library/' . $classname . EXT;

			if(file_exists($path))
			{
				require($path);
			}

			elseif(!file_exists($path))
			{
				throw new SystemException('Fatal Error The Requested Class Was Not Found');
			}
		}

		elseif(!isset(self::$instance->loaded[$classname]) && $external === true)
		{
			$path = $classname . EXT;

			if(file_exists($path))
			{
				require($path);
			}

			else
			{
				throw new SystemException('The External Path File Could Not Be Located');
			}
		}

		elseif(isset(self::$instance->loaded[$classname]))
		{
			return(true);
		}
	}


}

}

?>

[NogDog]

Where does EXT get defined?

You might want to add some debug info to the exception, e.g. including $path in the error text.

[jgetner]

EXT along with the rest of the constants get defined in the bootstrapper as well that is where i define the spl_autoload_regsiter(array('Loader' , 'load'));

i have gotten the paths problem fixed.... But why does this not autoload my classes?

[NogDog]

How are you using/calling it? Is this intended to be used with __autoload()?

[jgetner]

from what i understand you shouldn't have to call it... as your defining your own __autoload() with spl_autoload_register().

here is my bootstrapper that defines the spl_autoload_register().

define('BASEPATH' , realpath(dirname(__FILE__)));

define('SYSTEM' , BASEPATH .'/');

define('APPLICATION' , BASEPATH . '/application/');

define('LIB' , SYSTEM. '/library/');

define('OBJECTIVE' , true);

define('EXT' , '.php');

require_once(SYSTEM . 'loader' . EXT);

spl_autoload_register('Loader::load');


// i should not have to call this?
$load = new Loader;
$load->load('test' , false);

// test case...
$test = new test();
$test->ObjectiveInfo();

[NogDog]

Simply reading through the code is not telling me why it's not working, but it does seem unnecessarily complex to me. By definition, an __autoload function should only be called if the class in question is not yet defined, so all the checks to see if the class exists and keeping track of them in the $loaded array seems superfluous to me at best, and a possible bug area at worst. If you have some functional requirement to keep track of loaded classes, I might suggest first implementing it without any of that part, thus making sure it will simply require() the class file when needed, then adding that additional functionality in once you know it works.

[jgetner]

well that's the point to making your own autoloader as you can enhance or limit what can be done with loading of classes..... but you are right on that version and the reason it wasn't working any registered autoloader has to have only 1 parameter then i added in some path clean up aswell....

<?php if(!defined('OBJECTIVE')){exit();}

//this is an auotloader for autoloading classes for the objective framework. 
//created by Joshua Getner.

class Loader
{
    private static $loaded = array();

private static function SetLoaded($id  , $data)
{
	if(!isset(self::$loaded[$id]))
	{
		self::$loaded[$id]=$data;

		return self::$loaded[$id];
	}
	return(false);
}

private static function GetLoaded($id)
{
	if(!isset(self::$loaded[$id]))
	{
		return(false);
	}

	elseif(isset(self::$loaded[$id]))
	{
		return self::$loaded[$id];
	}

	return(false);
}

public static function load($classname)
{
    $path;

	if(class_exists($classname , FALSE) && isset(self::$loaded[$classname]))
	{
		require(self::$loaded[strtolower($classname)]);
	}

	elseif(class_exists($classname , FALSE) && !isset(self::$loaded[$classname]))
	{
	    throw new SystemException('Fatal Error:' . $classname .  'Is not a valid framework class.' . $path .'Occurance:'. __FILE__);
	}

	elseif(!class_exists($classname) && !isset(self::$loaded[$classname]))
	{
		if(!isset(self::$loaded[$classname]))
		{
		    $path = LIB . $classname . EXT;
			 str_replace('\\' , '/' , $path);

			if(file_exists($path) && is_file($path))
			{
				require(strtolower($path));
				self::$loaded[$classname] = $path;
			}

			elseif(!file_exists($path))
			{
				$path = SYSTEM . $classname . EXT;
				str_replace('\\' , '/' , $path);

				if(file_exists($path) && is_file($path))
				{
					require($path);
					self::$loaded[$classname] = $path;
				}

				else
				{
					throw new SystemException('Fatal Error:' . $classname . ' Could Not Be Located at this path:' . $path . 'Error Occured Here'. __FILE__ );
				}
			}
		}

		elseif(isset(self::$loaded[$classname]) && !class_exists($classname))
		{
			require(self::$loaded[$classname]);
		}
	}


}

}