Forum PHP.pl

Witam wszystkich,
mam pewien problem z dosyć prostym kodem - jest funkcja, która dostaje jako argument string i załącza plik np. z tablicą do przyłączenia do już istniejącej, w mega-skrócie tak:

[PHP] pobierz, plaintext 
function add_file($filename = '')
{
  $file = DIR_MAIN . "/files/{$filename}.php";
 
  if (file_exists($file))
  {
    require_once $file;
  }
}
[PHP] pobierz, plaintext 

Problem pojawa się, gdy jako argument podam coś z wielkimi znakami - np. plik nazywa się testFile.php ale podanie jako argumentu testFile nie zadziała - wywala błąd o tym, że nie znaleziono testfile (tak, bez dużej litery).
Z czego to wynika? Zakładam, że chodzi o konfigurację PHP, bo problem występuje na 1-2 z 6-7 współdzielonych kontach hostingowych.

W unixach ważna jest wielkość liter plików, W Windowsach nie.

W manualu funkcji readdir jest napisana bardzo elegancka funkcja, która moim zdaniem mogłaby się do tego nadać.

[PHP] pobierz, plaintext 
if ($handle = opendir('.')) {
    while (false !== ($entry = readdir($handle))) {
        if ($entry != "." && $entry != "..") {
            echo "$entry\n";
        }
    }
    closedir($handle);
}
[PHP] pobierz, plaintext 

@Mackos chyba nie wiesz o czym mówisz...

@wookieb:
Wszystkie testowane przeze mnie konta działały na linuksach (czasem Debian, czasem CentOS, chyba był też jeden Ubuntu). Oczywiście wiem, że wielkość znaków ma znaczenie, ale dziwi mnie zachowanie PHP - zupełnie jakby "wymuszał" aby ten argument miał same małe litery, lub też w takiej formie używał go podczas sprawdzania i załączania pliku.
Po drodze nie ma nic, co modyfikowałoby ten argument, a jednak do file_exists oraz require trafia on jakby bo strtolower...

No to spróbuj nie udziwniać składni, zamiast:

[PHP] pobierz, plaintext 
$file = DIR_MAIN . "/files/{$filename}.php";
[PHP] pobierz, plaintext 

zrób to normalnie:

[PHP] pobierz, plaintext 
$file = DIR_MAIN.'/files/'.$filename.'.php';
[PHP] pobierz, plaintext 

I wyświetl sobie tą ścieżkę i sprawdź czy jest różnica. Dodatkowo, do funkcji która raczej wymaga podania argumentu, nie nadawaj pustego argumentu, nie wiem jaki to ma cel?

W pierwszym poście napisałem, że to mega-skrót a nie kod - chodzi o sam fakt zmniejszania liter, który bierze się znikąd.
A co do składni - nie ma znaczenia jak by nie było napisane, poza tym ta zaprezentowana wyżej również jest jak najbardziej poprawna.

W takim wypadku pokaż cały kod.

BTW używanie eregi też jest poprawne, a czy jest właściwe to jest inna kwestia.

Ten post edytował by_ikar 23.12.2011, 17:41:38

Cały kod:

[PHP] pobierz, plaintext 
	/**
	 * Load the language variables for a section.
	 *
	 * @param string The section name.
	 * @param boolean Is this a datahandler?
	 * @param boolean supress the error if the file doesn't exist?
	 */
	function load($section, $isdatahandler=false, $supress_error=false)
	{
		// Assign language variables.
		// Datahandlers are never in admin lang directory.
		if($isdatahandler === true)
		{
			$this->language = str_replace('/admin', '', $this->language);
		}
		$lfile = $this->path."/".$this->language."/".$section.".lang.php";
 
		if(file_exists($lfile))
		{
			require_once $lfile;
		}
		elseif(file_exists($this->path."/english/".$section.".lang.php"))
		{
			require_once $this->path."/english/".$section.".lang.php";
		}
		else
		{
			if($supress_error != true)
			{
				die("$lfile does not exist");
			}
		}
 
		// We must unite and protect our language variables!
		$lang_keys_ignore = array('language', 'path', 'settings');
 
		if(is_array($l))
		{
			foreach($l as $key => $val)
			{
				if((empty($this->$key) || $this->$key != $val) && !in_array($key, $lang_keys_ignore))
				{
					$this->$key = $val;
				}
			}
		}
	}
[PHP] pobierz, plaintext 

A co jest niewłaściwego w używaniu {} do wyświetlania zmiennych?
Pytam szczerze, bo jeżeli coś jest nie tak, to wolałbym wiedzieć

Nie ma nic złego w klamrach, ale używanie ich wszędzie, czy używanie wszędzie cudzysłowia to jest moim zdaniem, jest niepotrzebnym narzucaniem na wydajność. Żeby w cudzysłowiu php znalazło zmienną, musi całą jego zawartość przeparsować. W proceduralnym stylu, promowanym przez wiele cms'ów praktycznie każda linijka to jest echo z cudzysłowiem. Niby nic, ale kilkaset takich linijek i tworzy się niepotrzebny narzut. Klamry słóżą do innego celu niż ładnego wyglądania i również stosuje się je tam gdzie trzeba, przykładowo w sytuacjach jak w tym temacie: http://forum.php.pl/index.php?showtopic=190084 a nie za każdym razem. W cudzysłowiu dodatkowo chyba większość IDE nie "odnajduje" naszych smiennych. Czyli w takiej liście zmiennych, podczas refaktoryzacji zmieni się tylko część zmiennych, które "widzi" nasze IDE. kolorowanie składni również leży. Przypomina mi to właśnie proceduralny styl..

Po kodzie niczego nie widzę, ale całkowicie całego kotu też nie widzę. Wrzuć w tą funkcję wszędzie tam gdzie używasz zmiennej $section, tuż przed użyciem var_dumpa. Albo jeżeli używasz jakiegoś IDE porób sobie breakpointy i sprawdź co jest przekazywane do metody.

Ten post edytował by_ikar 24.12.2011, 14:03:42