Witam, pisanie tych klas zaczęło się od pracy semestralnej gdzie miałem zaprojektować bazę danych biblioteki - pomyślałem: "a jakby tak w php napisać w ramach ćwiczeń całą obsługę biblioteki?"

I tak powstał mały projekt, plików jest kilka + plik z testem (index.php) dlatego wrzucam w formie paczki .zip
Biblioteka.zip

Proszę o ocenę, ew. pomysły jak coś lepiej rozwiązać

Kiedyś chciałem napisać skrypt biblioteki, jednak jak to bywa lepiej wydać 100 tyś na gów.. niż wcześniej pomyśleć. Co do kodu, ogólnie nie będę mocno oceniał, ale ładnie

W class_books:

[PHP] pobierz, plaintext 
if (empty($title)) {
				throw new BooksException('Book Title is Empty!');
			}
 
			$isbn = $obj->getISBN();
			if (empty($isbn)) {
				throw new BooksException('Books ISBN is Empty!');
			}
 
			$authors = $obj->getAuthors(false);
			if (count($authors) < 1) {
				throw new BooksException('Book Authors is Empty!');
			}
[PHP] pobierz, plaintext 

to raczej powinno być sprawdzane na poziomie obiektu Book.

A co powiesz na taką koncepcję tej metody?:

[PHP] pobierz, plaintext 
public function addBook(Book $obj) {   //zwraca TRUE jeśli dodano książkę
			$id = $obj->getID();
			$warunek = true; //boolean
 
			for($i = 0; $i<$this->_books.length&&warunek; $i++) {
				$b_obj = $this->_books[$i];
				$warunek = !($b_obj->getID() == $id);
			}
 
			if($warunek) {
				$this->_books[] = $obj;
			}
 
			return $warunek;
		}
[PHP] pobierz, plaintext 

//i analogicznie w klasie Authors

Poza tym: czemu w Book i Author nie ma konstruktorów, które przyjmują parametry? Chodzi mi o to, że łatwiej w jednym konstruktorze 'utworzyć książke' niż tworzyć 'pusty konstruktor' a potem wywoływać miliony setterów...

Tak samo w books i authors - moim zdaniem powinieneś przygotować konstruktor, który przyjmuje tablicę książek/autorów. Myślę, że w przyszłości może to czasem ułatwić sprawę.

A dodatkowo: czemu by nie napisać metody w Books: addBooks() przyjmującej kolekcję książek? => znowu moim zdaniem ułatwienie w przyszłości. A w niej pętlą wystarczyłoby lecieć i po kolei wywoływać addBook().

pozdro, Antoni