przechwycenie wyjątku przy tworzeniu pliku, try catch fopen Opcje

[damianooo]

Witam,

Mam problem z zapisem pliku w miejscu sieciowym na serwerze i chciałbym teraz poprzez przechwycenie wyjątku dowiedzieć się jaki jest problem, tzn. : czy jest problem z uprawnieniami , czy z błędną ścieżką czy może jeszcze coś innego ...

zrobiłem tak :

[PHP] pobierz, plaintext 
	try
	{
		$dane = "jakiś tam tekst\n";
		$file = "/zap/hurt/test.txt";
		$fp = fopen($file, "a");
		flock($fp, 2);
		fwrite($fp, $dane);
		flock($fp, 3);
		fclose($fp); 
	}
	catch( Exception $e ) 
	{
		$danes = "Komunikat: ".$e->getMessage().". Plik: ".$e->getFile().". Wiersz: ".$e->getLine()."\r\n";
		$files = "info_log.txt";
		$fps = fopen($files, "a");
		flock($fps, 2);
		fwrite($fps, $danes);
		flock($fps, 3);
		fclose($fps);
	}
[PHP] pobierz, plaintext 

jednak nie uzyskuję żadnej informacji w pliku info_log.txt , a plik test.txt nie zostaje utworzony

Proszę o pomoc czy dobrze próbuję przechwycić wyjątek czy może powinienem to zrobić w inny sposób ...

dziękuję

Ten post edytował damianooo 29.11.2011, 09:52:24

[Crozin]

Standardowa biblioteka PHP praktycznie w ogóle nie wykorzystuje wyjątków. Będziesz musiał to samemu sprawdzić (is_writable itp.) i ręcznie rzucić wyjątek. Ewentualnie możesz poszukać jakieś biblioteki-nakładki na funkcje f*(), która robiłaby to już za Ciebie.

[damianooo]

ale to jak? proponujesz tak:

[PHP] pobierz, plaintext 
$filename = '/zap/hurt/test.txt';
if (is_writable($filename)) {
    echo 'The file is writable';
} else {
    echo 'The file is not writable';
}
[PHP] pobierz, plaintext 

zrobiłem tak i tylko zauważyłem że długo się skrypt wykonywał, tak jakby próbował parę razy wykonać tą operację zapisu albo jakby nie mógł znaleźć ścieżki do zapisu ..


czy czegoś mi tu brakuje ?

bo myślałem że jakieś warningi dostanę czy coś ... ale nic ...

[Ilware]

spróbuj utworzyć plik, i nadać mu uprawnienia readonly,wywołaj następnie sam zapis i zobaczysz jaki błąd dostaniesz.

[damianooo]

ok zrobiłem najpierw tak:

[PHP] pobierz, plaintext 
$nazwa_pliku = '/zap/hurt/test.txt';
if (file_exists($nazwa_pliku)) {
    echo "Plik $nazwa_pliku istnieje";
} else {
    echo "Plik $nazwa_pliku nie istnieje";
}
[PHP] pobierz, plaintext 

i dostałem tak:

"Plik /zap/hurt/test.txt nie istnieje"

następnie zrobiłem tak:

[PHP] pobierz, plaintext 
$filename = '/zap/hurt/test.txt';
if (is_writable($filename)) {
    echo 'The file is writable';
} else {
    echo 'The file is not writable';
}
[PHP] pobierz, plaintext 

i otrzymałem tak:

"The file is not writable"


a więc wyraźnie widać że nie może znaleźć katalogu i nie może zapisać tam tego pliku ...

wniosek ? nie mam uprawnień ? katalog na pewno istnieje ponieważ admin twierdzi że go utworzył ...

skoro tak dużo się dowiedziałem ... można jeszcze coś wymyśleć aby sprawdzić dlaczego są problemy .? ... czy pewne że nie mam uprawnień (IMG:style_emoticons/default/questionmark.gif) ?

proszę o propozycje

dzięki

Ten post edytował damianooo 29.11.2011, 10:26:55

[Crozin]

No widzisz, niestety PHP od 7 lat (od czasu wprowadzenia wyjątków) nie doczekało się żadnych istotnych zmian w jego bibliotece standardowej i nadal jest na tym z poziomu roku 1999 (chyba). Nie pojawiła się też żadna wiodąca biblioteka zastępująca standardową, tak więc nadal jest się zdanym samym na siebie.

I tak, niestety będziesz musiał zrobić coś w stylu:

[PHP] pobierz, plaintext 
$path = '...';
 
try {
  if (!file_exists($path)) {
    throw ...;
  }
 
  if (!is_writable($path)) {
    throw ...;
  }
 
  if (($fh = fopen(...)) === false) {
    throw ...;
  }
 
  if (!fwrite(...)) {
    throw ...;
  }
 
  if (!fclose(...)) {
    throw ...;
  }
} catch (MyIOException $mioe) {
  ...
}
[PHP] pobierz, plaintext 

Oczywiście jeżeli w wyjątkach chciałbyś przemycić coś więcej niż tylko "nie udało się zapisać", np. konkretny powód takiej niemożliwości, musiałbyś kombinować ze zmienną $php_errormsg, bo API funkcji f*() nie udostępnia żadnego normalnego sposobu do dobrania się do treści błędu - ewentualnie użyć error_handlera, ale to mocno ograniczające podejście.

Oczywiście radziłbym napisać jakąś nakładę na powyższy kod, bo coś Cię trafi, jeżeli będziesz za każdym razem pisać taką obsługę błędów - pomijając fakt, że przy takiej formie niewiele się z wyjątków zyskuje.

[damianooo]

apropo tego co napisał CROZIN ... chyba szkoda zachodu .... zrobiłem tak jak wcześniej napisałem .. powiedz mi co z tego wynika .. jeśli nie działa is_writeable oraz is_exists no to czy na pewno jest już peewne że nie mam uprawnień ? ... co może być jeszcze ?

wychodzi na to że muszę męczyć admina w firmie , bo to on pewnie stoi za tym i powinien wszystko sprawdzić jeszcze dokładnie

[melkorm]

a nie lepiej po prostu skorzystać z splFileObject ?

[damianooo]

melkorm ...

czyli że tak:

[PHP] pobierz, plaintext 
$file = new SplFileObject("/zap/hurt/test.txt", "w");
if ($file->flock(LOCK_EX)) { // do an exclusive lock
    $file->ftruncate(0);     // truncate file
    $file->fwrite("Write something here\n");
    $file->flock(LOCK_UN);   // release the lock    
} else {
    echo "Couldn't get the lock!";
}
[PHP] pobierz, plaintext 

albo tak:

[PHP] pobierz, plaintext 
<?php
$file = new SplFileObject("/zap/hurt/test.txt", "w");
$written = $file->fwrite("12345");
echo "Wrote $written bytes to file";
?>
[PHP] pobierz, plaintext 

tylko czy coś więcej informacji wyłuskam wprost z serwera (IMG:style_emoticons/default/questionmark.gif) ?

Ten post edytował damianooo 29.11.2011, 10:49:53

[Crozin]

a nie lepiej po prostu skorzystać z splFileObject ?

Już myślałem, że właśnie popisałem się nieznajomością tak starej części SPL-a, która dawałaby namiastkę sensownego API ale... deweloperzy PHP jak zwykle dali dupy i nie potrafili wprowadzić normalnej obsługi błędów. Czyli na dobrą sprawę pozostaje się z dokładnie tym samym problemem co przy funkcjach f*().

[melkorm]

Fakt - sam wyjątek jest tylko rzucany przy tworzeniu obiektu, właśnie zobaczyłem że dla fwrite zwraca NULL w przypadku błędu, eh.

Czyli kontrolować na poziomie klasy możesz tylko tworzenie obiektu, czyli:

Throws a RuntimeException if the filename cannot be opened.

PS. Zawsze można dopisać jakąś sensowną nakładkę.

Ten post edytował melkorm 29.11.2011, 11:04:43

[by_ikar]

Nie no to co chcesz osiągnąć można osiągnąć, chociażby w ten sposób:

[PHP] pobierz, plaintext 
<?php
 
if(is_readable($path))
{
    if(is_writable($path))
    {
        file_put_contents($path, $dane);
    } else
    {
        throw new RuntimeException(sprintf('Brak uprawnień zapisu dla pliku: %s ', $path));
    }
} else
{
    throw new RuntimeException(sprintf('Podana ścieżka %s nie istnie, lub nie posiadasz uprawnień odczytu.', $path));
}
[PHP] pobierz, plaintext 

Możesz jeszcze dodatkowo do tego dołożyć zamiast is_readable sprawdzanie czy to plik, czy to katalog, w sumie co chcesz. Męczące jest robienie tego często tam gdzie się chce coś zapisać/odczytać i fajnie by było gdyby php rzucało w takich przypadkach wyjątki jeżeli masz zdefiniowanego swojego handlera wyjątków, no ale jak już wyżej napisano, tak nie jest (IMG:style_emoticons/default/wink.gif)

Nie mniej, dokładnie więcej informacji się nie dowiesz, to że nie możesz zapisać tych danych to raczej najprawdopodobniej problem z uprawnieniami zapisu.

[damianooo]

ok admin naprawił swój błąd ...

a żeby w php nie było fajnej obsługi wyjątków to skandal (IMG:style_emoticons/default/smile.gif) .. delikatnie mówiąc .. pozostaje czekać na nową wersję PHPa - pewnie tam już to będzie ...

dzięki wszystkim za usiłowanie pomocy ...

za pomoc najbardziej dziękuję CROZINowi ... bo w sumie wystarczyło sprawdzić tylko is_writable lub file_exists aby wiedzieć czemu nie chce się plik utworzyć w podanym katalogu

temat zamknięty

pozdrawiam

[melkorm]

Obsługa wyjątków jest, tylko że 99% metod została stworzona gdy jeszcze obsługi wyjątków i w ogóle OOP w PHP nie było.
I nie zapowiada się by cokolwiek w tej materii się zmieniło.

Ewentualnym rozwiązaniem jest set_error_handler.