[Iso]

Hi Leute, so langsam bin ich am verzweifeln, da ich den Fehler einfach nicht finde. Entweder geht es so gar nicht oder ich seh den Wald vor lauter Bäumen nicht.

Hier erstmal der code:

<?
session_start();

class winkelwagentje
{
	private $artikel = array();
	
	public function setArtikel($artnr, $aantal) {
		$this->artikel[$artnr] = $aantal ;
	}
	
	public function getArtikelen() {
		return $artikel;
	}
	
	public function getKosten() {
		foreach($artikel as $wert){
			$totaal += $wert * 10;
		}
		return $totaal; 
	}
	
	public function verwijderArtikel($artnr){
		unset($artikel[$artnr]);
	}
	
}

$winkelwagen = new winkelwagentje;

$winkelwagen->setArtikel('5',2);
$winkelwagen->setArtikel('10',5);

echo $winkelwagen->getKosten();

$_SESSION['winkelwagen'] = $winkelwagen;

?>

Ist leider das meiste auf Niederländisch da ich in Holland studiere, es soll aber eine Einkaufswagen ( winkelwagen ) -Klasse wiederspiegln, jedoch funktioniert diese nicht, da immer folgender Fehler erscheint:

Zitat

Warning: Invalid argument supplied for foreach() in C:\XAMPP\xampp\htdocs\ecommerce\winkelwagen.php on line 18

Was ja soviel wohl heißt, dass $artikel nicht gesetzt ist, ich verstehe aber nicht warum das so sein sollte...

[Fabi]

müsste die klammer hinter dem arry nicht so ausshen

[]

?

ich kann zwar kein php aber in C und C++ ist das so...

mfg
Fabi

[Iso]

hinter welchem Array meinste genau? Oben bei der Konstruktion wird ja ein neues Array-Object erzeugt, deswegen müssen da ja () hin, ist halt nunmal anders als bei C :-/

[Fabi]

jap, da hab ich gemeint...naja ein versuch wars wert...

also dann geh ich mal schlafen...

gn8
Fabi

[Yalamand]

Was du durchaus einmal richtig gemacht hast, kannst du auch weiterhin richtig machen.

$this->artikel[$artnr] = $aantal;

ist doch vollkommen okay, warum dann gleich wieder alles kaputt machen mit

return $artikel;

.

$artikel wird immer leer sein, wenn nicht vorher deklariert. $this->artikel wird sich immer auf $artikel innerhalb des aktuellen objektes beziehen. Dann weiss die foreach auch endlich was $artikel überhaupt sein soll folglich wäre

foreach($this->artikel as $wert)

richtig.

[Witi]

[off]Endlich einer der versucht ordentlich in PHP zu entwickeln [/off]

Yalamand hat vollkommen recht, da muss ich nichts mehr zu sagen.

Ich hätte jedoch noch einige Tipps, die mir beim Überfliegen deines Codes aufgefallen sind:

Klassennamen bitte immer groß schreiben, also Winkelwagentje

public function getKosten() {
foreach($artikel as $wert) {
$totaal += $wert * 10;
}
return $totaal;
}

$totaal bitte oben deklarieren. Du hast Glück, das PHP extremst hässlich ist und es überhaupt zulässt. In ordentlichen Sprachen wird das nämlich nicht gehen. Mach lieber sowas, erhöht zudem die Lesbarkeit:

$totaal = 0;

foreach($artikel as $wert ) {
	$totaal += $wert * 10;
}

return $totaal;

Bei der Erzeugung einer Instanz, also dem Aufrufen des Konstruktors sollte man auch immer Klammern setzen:

$winkelwagen = new Winkelwagentje();

Und diesen Code sollte man zudem nicht in der eigentlichen PHP-Datei der Klasse (wahrscheinlich Winkelwagentje.php), sondern an einer anderen Stelle haben.

[Iso]

Ok danke für die zahlreichen Tipps. Werde es gleich mal ausprobieren,


Edit:
Ok funktioniert, danke erstmal.

Falls ich noch weitere Fragen spöter haben sollte, werde ich es hier posten.

Dieser Beitrag wurde von Iso bearbeitet: 15. Mai 2007 - 11:56