Forum PHP.pl

Witam serdecznie to mój pierwszy post. Uczę się projektowania obiektowego, dziś napisałem swój pierwszy skrypt jednak nie mogę ustawić przyczyny błędu bardzo proszę o pomoc (IMG:style_emoticons/default/smile.gif)

[PHP] pobierz, plaintext 
 
<?php
 
class book
{
public $Tytul;
public $Autor;
public $Ilosc_Stron;
public $Tematyka;
 
public function __construct($Tytul, $Autor)
{
	$this->Tytul=$Tytul;
	$this->Autor->$Autor;	
}
 
public function getTytul ()
{
	return $this->Tytul;
}
 
public function getAutor($Autor)
{
	$this->Autor = $Autor;
}
 
public function getIlosc_Stron()
 
{
	return $this->Ilosc_Stron;
}
 
public function getTematyka()
 
{
	return $this->Tematyka;
}
 
 
class fantasty_book extends book 
{
}
 
class criminal_book extends book 
{
}
 
$book1 = new fantasty_book('jakis tam tytul', 'jakis tam autor');
$book2 = new criminal_book('jakis tam tutul inny', 'jakis tam inny autor');
 
echo ($book1, $book2);
 
}
?>
 
[PHP] pobierz, plaintext 

a błąd mi wyskakuje taki : Parse error: syntax error, unexpected 'class' (T_CLASS), expecting function (T_FUNCTION) in C:\wamp\www\php\ksiazka.php on line 39

nie wiem co z tym zrobić, proszę o pomoc ;(

Klamrę zamykającą z końca pliku przenieś nad klasę: fantasty_book

Klamra z linii 53 (na forum) co tam robi?

W konstruktorze przypisujesz tylko wartości do odpowiednich pól, więc sama klasa Ci nic nie zwróci.

[PHP] pobierz, plaintext 
$book = new Book('Testowy', 'Jadzia');
echo $book->getTytul(); // out Testowy
[PHP] pobierz, plaintext 

Dodatkowo nie wiem po co masz metody do pobrania wartości pola z ilością stron, skoro ani nigdzie ich nie przypisujesz, a nie nie masz metod żeby je przypisać.

Ten post edytował g2g3 6.09.2013, 10:22:11

[PHP] pobierz, plaintext 
<?php
 
class book
{
public $Tytul;
public $Autor;
public $Ilosc_Stron;
public $Tematyka;
 
public function __construct($Tytul, $Autor)
{
	$this->Tytul=$Tytul;
	$this->Autor->$Autor;	
}
 
public function getTytul ()
{
	return $this->Tytul;
}
 
public function getAutor($Autor)
{
	$this->Autor = $Autor;
}
 
public function getIlosc_Stron()
 
{
	return $this->Ilosc_Stron;
}
 
public function getTematyka()
 
{
	return $this->Tematyka;
}
 
}
class fantasty_book extends book 
{
}
 
class criminal_book extends book 
{
}
 
 
$book1 = new fantasty_book('Fantazy', 'Maciek');
$book2 = new criminal_book('Kryrymynał', 'Mietek');
 
echo $book1;
echo $book2; 
 
?>
[PHP] pobierz, plaintext 

(IMG:http://img571.imageshack.us/img571/1152/rfzj.jpg)

Ten post edytował Johnny_Smithh 6.09.2013, 10:21:31

Witam również, ale na przyszłość proszę nadać sensowne tematy swoim wątkom.

Ok, przepraszam.


Napisałem teraz tak, jednakże wyświetla mi tylko autora a tytułu już nie. Mogę prosić o jeszcze jakieś wskazówki ? tak jak napisałem na początku to mój pierwszy skrypt więc proszę o trochę zrozumienia i cierpliwości (IMG:style_emoticons/default/smile.gif)

[PHP] pobierz, plaintext 
 
<?php
 
class book
{
public $Tytul;
public $Autor;
public $Ilosc_Stron;
public $Tematyka;
 
public function __construct($Tytul, $Autor )
{
	$this->Tytul=$Tytul;
	$this->Autor=$Autor;
}
 
public function getTytul ()
{
	return $this->Tytul;
}
 
public function getAutor($Autor)
{
	$this->Autor = $Autor;
}
 
public function getIlosc_Stron()
 
{
	return $this->Ilosc_Stron;
}
 
public function getTematyka()
 
{
	return $this->Tematyka;
}
 
 public function __toString()
 
 {
	return $this->Autor;
	return $this->Tytul;
	return $this->Tematyka;
 }
 
}
 
 
class fantasty_book extends book 
{
}
 
class criminal_book extends book 
{
}
 
$book1 = new fantasty_book('Ogiem i mieczem', 'Henryk Sienkiewicz', 6000, 'historyczna'); 
 
$book2 = new criminal_book('Wladca pierscieni', 'John Tolkien', 600, 'fantazy');
 
echo $book1;
echo ("<br>");
echo $book2; 
 
?>
[PHP] pobierz, plaintext 

(IMG:http://img600.imageshack.us/img600/3648/x7vg.jpg)

Ten post edytował Johnny_Smithh 6.09.2013, 11:18:53

Polecenie return zwraca wartość z funkcji i kończy jej działanie, więc nie ma siły, żeby pozostałe 3 linijki się wykonały. Zamiast return zapisz wartości do jednej zmiennej i to ją zwróć jako wynik funkcji:

[PHP] pobierz, plaintext 
public function __toString()
{
$wynik = '';
$wynik .= $this->Autor;
$wynik .= $this->Tytul;
$wynik .= $this->Ilosc_Stron;
$wynik .= $this->Tematyka;
 
return $wynik;
}
[PHP] pobierz, plaintext 

Uporałem się z tym pozmieniałem trochę konstruktor ale teraz działa. Dzięki wszystkim za pomoc. Ma ktoś może jakieś uwagi jeszcze?

[PHP] pobierz, plaintext 
 
<?php
 
class book
{
public $Tytul;
public $Autor;
public $Ilosc_Stron;
public $Tematyka;
 
public function __construct($Tytul, $Autor, $Ilosc_Stron, $Tematyka )
{
	$this->Tytul=$Tytul;
	$this->Autor=$Autor;
	$this->Ilosc_Stron=$Ilosc_Stron;
	$this->Tematyka=$Tematyka;
 
}
 
public function setTytul ()
{
	return $this->Tytul;
}
 
public function getAutor($Autor)
{
	$this->Autor = $Autor;
}
 
public function setIlosc_Stron()
 
{
	return $this->Ilosc_Stron;
}
 
public function setTematyka()
 
{
	return $this->Tematyka;
}
 
public function __toString()
{
	$wynik = '';
	$wynik .= $this->Autor;
	$wynik .= $this->Tytul;
	$wynik .= $this->Ilosc_Stron;
	$wynik .= $this->Tematyka;
 
return $wynik;
}
 
}
 
 
class fantasty_book extends book 
{
}
 
class criminal_book extends book 
{
}
 
$book1 = new fantasty_book('Ogiem i mieczem', 'Henryk Sienkiewicz', 6000, 'historyczna'); 
 
$book2 = new criminal_book('Wladca pierscieni', 'John Tolkien', 600, 'fantazy');
 
 
echo $book1;
echo ('</br>');
echo $book2;
 
 
?>
 
 
[PHP] pobierz, plaintext 

zamień setIlosc na getIlosc i inne sety też pozamieniaj.

Już napisałem coś nowego o co chyba nie ma się przyczepić jednak jedynej rzeczy której nie rozumiem to o co chodzi z tą konwersją na Stringi

[PHP] pobierz, plaintext 
<?php
 
class Komputer 
{
	public $Dysk;
	public $RAM;
	public $Procesor;
 
	public function __construct ($Dysk, $RAM, $Procesor)
 
	{	$this->Dysk=$Dysk;
		$this->RAM=$RAM;
		$this->Procesor=$Procesor;
	}	
 
	public function setDysk ($Dysk)
	{
 
		$this->Dysk=$Dysk;
	}
 
	public function getDysk() {
 
		return $this->Tytul;
	}
 
	public function setRAM() {
 
		$this->RAM=$RAM;
	}
	public function getRAM() {
 
		return $this->RAM;
	}
 
	public function setProcesor() {
 
		$this->Procesor=$Procesor;
	}
 
	public function getProcesor() {
 
		return $this->Procesor;
	}
 
public function __toString()
{
	$wynik = '';
	$wynik .= $this->Dysk;
	$wynik .= $this->RAM;
	$wynik .= $this->Procesor;
 
return $wynik;
}
}
class NowyKomputer extends Komputer
{
}
 
$Komputer1 = new NowyKomputer ('sata', ' intel', ' atlon');
 
echo $Komputer1;	
 
 
?>
[PHP] pobierz, plaintext 

Chodzi o to, że klasa może zawierać różne dane. Metoda __toString pozwala na wyświetlenie tych elementów klasy, które chcemy i w takiej formie jak nam jest wygodniej. Poczytaj w manualu o metodach magicznych.