OOP rechnen

Stef

Hey,

ich habe mit OOP jetzt begonnen. Ist etwas kompliziert, und ich bekomm da irgendwie diese Rechnung nicht hin. Ich möchte die anzahl (10) +1 nehmen und dies dann der var ergebnis übergeben.

Mein momentaner Aufbau:

PHP

<?php




class Tiere{




	//Sichtbarkeit einstellen (public,protected,private) - Eigenschaft
	public $tieranzahl = '10';
	public $tierart = 'Hunde';
	protected $neueHunde;







	//functionen  (Methoden) erstellen
	public function zeigeanzahl(){
		echo $this->tieranzahl;
		//mit $this greift man in ner methode auf die eigenschaft zu
	}




	public function tierart(){
		echo $this->tierart;
	}




	public function neueHunde($anzahl = 10, $ergebnis ){
		$this->neueHunde = $anzahl;
		$this->neueHunde + 1 = $ergebnis; // <--- hier




		echo $this->neueHunde;

	}




	







}




$anzahltiere = new Tiere; //Instanz setzen
$anzahltiere->zeigeanzahl();
echo '<br>';
$art = new Tiere;
$art->tierart();
echo '<br>';
$rechne = new Tiere;
$rechne->neueHunde();







?>

Alles anzeigen

Dabei gibt der Browser mit immer den Fehler aus :
Parse error: syntax error, unexpected '=' in E:\Web\XAMPP\htdocs\OOP\grundlagen.php on line 23

Warum denn dass ? Ich versteh auch nicht warum es nicht rechnet... OOP erscheint mir etwas kompliziert.

Weitere Frage:

zu dem hier: protected $neueHunde;
Hier habe ich ja die eigenschaft neue hunde als geschützt gesetzt.
Ist die methode dafür auch geschützt oder weil ich public gesetzt habe frei zugänglich ?

Freue mich auf eure Hilfe.

Gruß,
Stef

cottton

Die Fehlermeldung sagt es doch: das "=" ist unerwartet.

PHP

$this->neueHunde + 1 = $ergebnis;
// sollte sicher das hier sein:
$this->neueHunde = $ergebnis + 1;

Da ist min noch ein Fehler:

PHP

public function neueHunde($anzahl = 10, $ergebnis )

1. optionale Parameter sollten immer NACH nicht optionalen Parametern kommen.
2. Du gibts kein Wert für $ergebnis rein (Zeile 41)

BTW: würde empfehlen im Code alles Englisch zu halten. Kommentare wie Variablen und Funktionen ect.
Ich meine, wenn Du schon mit OOP anfängst, kann man sich gleich an "andere Sachen" gewöhnen =).

Hast Du hier schon mal reingesehen: http://www.php-kurs.com/objektorientie…rung-in-php.htm

Zu protected:
Die Property (Eigenschaft) $neueHunde ist protected. Nicht die Methode.

Hier mal ein Bsp am Hund:
// -Kommentare sind natürlich nur zur Erklärung

PHP

/**
 * Class Dog
 *
 * A dog is a single entity.
 */
class Dog
{
    /**
     * The name of the dog.
     *
     * @var string
     */
    protected $name; // default null




    /**
     * The age of the dog.
     *
     * @var int
     */
    protected $age; // default null




    /**
     * Returns the name of the dog.
     *
     * @return string
     */
    public function getName()
    {
        return (string)$this->name; // always as string
    }




    /**
     * Returns the age of the dog.
     *
     * @return int
     */
    public function getAge()
    {
        return (int)$this->age; // always as int
    }




    /**
     * Sets the name of the dog.
     *
     * @param string $name
     *
     * @return $this
     */
    public function setName($name)
    {
        $this->name = (string)$name; // always as string
        return $this; // return $this to be able to use method-chaining
    }




    /**
     * Sets the age of the dog.
     *
     * @param int $age
     *
     * @return $this
     */
    public function setAge($age)
    {
        $this->age = (int)$age;
        return $this; // return $this to be able to use method-chaining
    }




    /**
     * Adds given amount of years to the dogs age.
     *
     * @param int $amountOfYears
     *
     * @return $this
     */
    public function addAge($amountOfYears)
    {
        $this->age += (int)$amountOfYears;
        return $this; // return $this to be able to use method-chaining
    }
}




$dog = new Dog();
// $dog->name = 'fetzer'; // error: cannot access protected property
// $dog->age = 1; // error: cannot access protected property
// so we need to use the public "getter" and "setter" (f.e. ::getName(), ::setName())
$dog->setName('fetzer');
$dog->setAge(1);




// method chaining:
$dog->setName('fetzer')// returns $this so we still got "this dog" and can call the next method
    ->setAge(1);




echo $dog->getName(); // out: "fetzer"
echo $dog->getAge(); // out: 1




$dog->addAge(2);
echo $dog->getAge(); // out: 3

Alles anzeigen

Bei mehreren Tieren (Animals) würde man dann eine Class Animals erstellen, die eine Sammlung (collection) von Tieren hält.
Aber das evtl später.

Schön, das Du OOP nutzen möchstest. Macht sehr viel Spass und Sinn.
Es sieht zwar viel aus, kommt aber nach und nach - irgendwann ist das alles logisch

Stef

Hey,

warum ist es da unerwartet? Versteh ich nicht..

Ich wollte die rechnung aus $this->neueHunde + 1 in $ergebnis speichern.
Habe es dann geändert in :

PHP

$ergebnis = $this->neueHunde + 1;

. Und es klappt auch.

Wenn ich z.b. jetzt ein Rechner habe der nur + rechnen kann mit benutzereingabe kann es so dann aussehen ?

PHP

class calculator{




	public $result;




		function calculate($zahl1 = null, $zahl2 = null)
		{
			$zahl1 =  isset($_POST['zahl1']) ? $_POST['zahl1'] : null ;
			$zahl2 = isset($_POST['zahl2']) ?  $_POST['zahl2'] : null;




			$this->result = $zahl1 + $zahl2;




			echo $this->result;
		}
}




?>

Alles anzeigen

Könnte man bei dieser function auch mittels return es ausgeben ?

Ok. Ja , hab ich, etwas veraltet manches. Schau mit dazu das PHP-Handbuch an (lerne aus dem). Dabei ´lernt man dann wieder etwas englisch

Danke fürs beispiel.

Wenn man die eigenschaften auf protected setzt dann sind die functionen in welche diese aufgerufen werden sozusagen nur mit set oder get ausführbar sonst kommt ein error ?
Wenn ich jetzt z.b. ein datenbankzugriff da habe denn muss ich ja auf private/protected setzen. Dies kann man dann mittels set oder get ausführen ?

Ja am anfang, als ichs mir das erste mal angeschaun habe hab ich nicht viel kappiert. Aber langsam versteh ich es.

Gruß, Stef

cottton

Dein

PHP

$this->neueHunde + 1 = $ergebnis;

Ist halt eben nicht korrekt.
Was Du geschrieben hattest, ist "spiegelverkehrt".

PHP

/ FALSCH
// $this->neueHunde + 1 = $ergebnis;
// $a + 1 = $b;




// RICHTIG
$ergebnis = $this->neueHunde + 1;;
$b = $a + 1;

Dein calc Bsp macht irgendwie keinen Sinn
Du nutzt 2 Parameter ($zahl1 und $zahl2), nutzt diese aber nicht,
sondern überschreibst sie mit $_POST-Daten.

Ich denke Du musst Dir nochmal Funktionen ansehen.

am Bsp - einfache Funktion - zwei Zahlen zusammen zählen.
Die Werte kommen über $_POST:

PHP

// die funktion sollte "sum" heißen, denn sie addiert.
// calculate w+rde bedeuten, dass diese funktion "alles" kann.




// die parameter sollten keine default werte besitzen,
// denn beim addieren braucht man nun mal immer 2 werte.




// die funktion sollte kein echo oder print (...) nutzen,
// denn die funktion soll rechnen (summieren) und nichts ausgeben
// (zurückgeben statt ausgeben)




function sum($value1, $value2)
{
    return $value1 + $value2;
}




// die funktion ist bewusst "dumm".
// ihr ist es egal, wo die daten herkommen.
// sie bekommt einfach 2 werte, und gibt die summe der beiden zurück.
// daher rufen wir die funktion folgendermaßen auf:




$result = sum($_POST['zahl1'], $_POST['zahl2']);




// die wiederverwendbarkeit ist also gegeben:
$priceTotal = sum($price, $tax);

Alles anzeigen

Stef

Hi,

ja hab es auch gemerkt.

Aso. Habe mit functionen lange nix mehr gemacht. Muss ich mir nochmal anschauen.

Ich brauche doch 2 Zahlen die muss ich doch als parameter mitgeben ? Und dann habe ich gedacht das diese 2 Zahlen die value besitzen sollen von der Benutzereingabe. Daher habe ich das so gemacht. Mein Fehler. Aber irgendwie mach ich das was ich so nicht machen würde bei oop.. keine ahnung warum..

Ok. Dann würde bei oop die 2 values in die public variable abgelegt und dann wird diese returnt. Und dann außerhalb auf diese public variable zugreifen und die Parameter geben welche diese berechnen soll?

PHP

class calculator{
  public $ergebnis;

 	function sum($value1, $value2){
	$this->ergebnis = $value1 + $value2;
	return $this->ergebnis;
  }
}




$sum = new calculator;
$sum->ergebnis($_POST['zahl1'], $_POST['zahl2']); //dies gibt dann das ergebnis aus .

Alles anzeigen

Stef

cottton

Die Methode ::ergebnis gibt es nicht.
Du musst $sum->sum(...) aufrufen.

In ergebnis wird doch das Ergebnis abgelegt.Also der Wert.

Stef

Aso.. stimmt ja. Mein Fehler.

Danke

Jetzt mitmachen!