Proszę o testy klasy, ...pakera/unpakera plików Opcje

[kiler129]

Witajcie! (IMG:style_emoticons/default/smile.gif)
Tym razem brak pytania ale za to prośba o testy klasy. Dopiero co ją ukończyłem więc nieco śmieci w kodzie jest (IMG:style_emoticons/default/winksmiley.jpg) Zostanie też podzielona na dwie osobne.
Klasa ma za zadanie spakować katalog oraz umieć go rozpakować.

Kod: http://wklej.org/id/414824/

Użycie:

[PHP] pobierz, plaintext 
<?php
require_once("packer.class.php");
$_packer = new packer;
 
/* PAKOWANIE */
$_packer->packTree("katalog_do_pakowania", "map.dat", "data.dat");
 
/* ROZPAKOWYWANIE */
$_packer->unpackTree("katalog_do_wypakowania", "map.dat", "data.dat");
?>
[PHP] pobierz, plaintext 

Będę wdzięczny za wszystkie konstruktywne opinie, komentarze i raporty błędów (IMG:style_emoticons/default/smile.gif)
Dodatkowo nie wiem jak rozawiązać problem z chmodami (stąd część kodu jest w komentarzu), jeśli mamy taką sytuację:

Kod

test/costam/ => chmod 0444
test/costam/nic => chmod 0777

kod wysypie się z pr. denied.

Ten post edytował kiler129 7.11.2010, 23:22:24

[Fifi209]

Pierwsze co mi się rzuca w oczy to konstruktor, rozwiązałbym to sprawdzanie funkcji troszkę inaczej.

Wrzuć ich nazwy do tablicy i komunikaty - potem pętlą.

Co do pakowania, czemu nie użyjesz zip ?

[kiler129]

Co do pakowania, czemu nie użyjesz zip ?

Z funkcji owszem wiem, zrobię to ale jutro (IMG:style_emoticons/default/winksmiley.jpg)
Co do zipa - paker ma tworzyć automatyczny instalator apliakcji php - nikt nie lubi dbać o poprawny przesył kilkuset plików przez ftp + chmody (IMG:style_emoticons/default/winksmiley.jpg)
Do tego zip na większości shared hostów jest unicestwiony (IMG:style_emoticons/default/winksmiley.jpg) Kolejna sprawa - czy zip zachowuje chmody?

Ten post edytował kiler129 8.11.2010, 01:01:49

[Fifi209]

Co do zipa - paker ma tworzyć automatyczny instalator apliakcji php - nikt nie lubi dbać o poprawny przesył kilkuset plików przez ftp + chmody (IMG:style_emoticons/default/winksmiley.jpg)

phar z tego co pamiętam.

Ten post edytował fifi209 8.11.2010, 15:50:40