Hi,

ich habe eine Klasse für Produkte angelegt. Im Konstruktor wird die ID des Produktes übergeben.

Jetzt möchte ich eine Übersicht aller Produkte darstellen.

Müsste man nun für jedes Produkt eine neues Objekt erzeugen? Man könnte ja auch eine Methode zur Anzeige aller Produkte anlegen, aber dazu brauchte ich ja nicht die ID im Konstruktor.

Was wäre denn das beste Vorgehen für sowas? Die Klasse kann natürlich auch noch umgestellt werden.

Müsste man nun für jedes Produkt eine neues Objekt erzeugen?Japp, zumindest eine neue Instanz.

Man könnte ja auch eine Methode zur Anzeige aller Produkte anlegen Im Produkt Objekt, welches EIN Produkt repräsentiert???

aber dazu brauchte ich ja nicht die ID im Konstruktor. Da siehst du es. ;)

Was wäre denn das beste Vorgehen für sowas? Eine Klasse ProductList o.ä.

Hi,

danke für die Antwort.
An ne neue Klasse hatte ich gar nicht gedacht.

Wäre es denn auch ok wenn man eine Methode getProduct($id) und eine Methode getAllProducts() in die Klasse aufnimmt? Also ohne im Konstruktor schon die Id zu übergeben?

Da du ja geschrieben hast das die Klasse ja EIN Produkt darstellt (darstellen sollte?) ist es wohl eher nicht so gut oder?

Wäre es denn auch ok wenn man eine Methode getProduct($id) und eine Methode getAllProducts() in die Klasse aufnimmt? Also ohne im Konstruktor schon die Id zu übergeben? Von welcher Klasse sprichst du?

Original geschrieben von TobiaZ
Von welcher Klasse sprichst du?

von der Klasse, die ich am Anfang meinte, also die Produktklasse.

dann: NEIN!

gut ok...dann werde ich das aufteilen auf 2 klassen :-) du hattest ja noch geschrieben o.ä. ...gibts denn noch andere (bessere) möglichkeiten?

nein

Hi,

hab zu dem Thema mal noch ne Frage.

Jetzt gibt es eine Produktklasse und verschiedene Produktarten erben von dieser Klasse.

In der URL steht als get-Parameter die Produktart und das objekt erzeuge ich dann


$cat = strip_tags(stripslashes($_GET['category']));
$obj = new $cat();


Jede erbende Klasse hat ja spezielle Eigenschaften (Bücher: Seitenanzahl, CD's: Spielzeit)

Jetzt weiß ich ja nicht ob es eine Instanz von Bücher oder CD ist.
Wie bekommt man das raus, da ich ja in den Büchern nicht auf $obj->getSpielzeit() zugreifen kann.

Jedesmla prüfen in welcher Kategorie ich mich befinde? Bei 2 mag es ja nch gehen aber bei 15 Kategorien nicht mehr so toll.

Original geschrieben von lx-club

$cat = strip_tags(stripslashes($_GET['category']));
$obj = new $cat();


Du lässt den Besucher mit GET entscheiden, welche Objekte angelegt werden? Mutig, mutig, ...

Original geschrieben von lx-club
...
Jede erbende Klasse hat ja spezielle Eigenschaften (Bücher: Seitenanzahl, CD's: Spielzeit)

Jetzt weiß ich ja nicht ob es eine Instanz von Bücher oder CD ist.
Wie bekommt man das raus, da ich ja in den Büchern nicht auf $obj->getSpielzeit() zugreifen kann.

Um herauszubekommen, ob ein Objekt "eine Instanz einer bestimmten Klasse" ist, gibts (seit PHP5) den instanceof-Operator (php.net/manual/en/language.operators.type.php).


$cat = strip_tags(stripslashes($_GET['category']));
$obj = new $cat();


Was soll der Unfug? Wovor genau sollen dich strip_tags() und strip_slashes() bewahren?
Und nein: Sie tun es nicht -- dein Script vor bösen oder dummen Eingaben beschützen. Dafür sind beide auch nicht gedacht.

Du solltest besser eine Liste erlaubter Klassen (http://de.wikipedia.org/wiki/Whitelist#Software) anlegen und dann vor der "Instanziierung" prüfen, ob die vom Benutzer angegebene Klasse in der Liste enthalten ist.

Original geschrieben von h3ll
Du lässt den Besucher mit GET entscheiden, welche Objekte angelegt werden? Mutig, mutig, ...
:D

Da es sich ja um Produkte handelt, würde ich in dem Fall gleich die passende Factory (http://en.wikipedia.org/wiki/Acme_Corporation) vorschlagen:


class acme {
public static function shipping(
$product = NULL
) {
if (NULL === $product) {
$product = $_GET['category'];
}
if (!class_exists($product, FALSE)) {
return FALSE;
}
return new $product;
}
}

$obj = acme::shipping();
echo '*meep_meep*';

Original geschrieben von h3ll
Du lässt den Besucher mit GET entscheiden, welche Objekte angelegt werden? Mutig, mutig, ...

darum ging es doch in meinem Posting überhaupt nicht... es sollte nur zeigen wie die objekte zustande kommen. hab dazu ne Factory-Klasse (wie hier auch schon vorgeschlagen) verwendet.

Original geschrieben von fireweasel

Was soll der Unfug? Wovor genau sollen dich strip_tags() und strip_slashes() bewahren?
Und nein: Sie tun es nicht -- dein Script vor bösen oder dummen Eingaben beschützen. Dafür sind beide auch nicht gedacht.



Wieso werden damit böswillige Eingaben nicht abgefangen?

Original geschrieben von lx-club
Wieso werden damit böswillige Eingaben nicht abgefangen?

Angenommen $_GET['category'] enthält den String "mysqli", schon wird ein neues mysqli-Objekt erzeugt, obwohl du das gar nicht vorgesehen hast. Und statt mysqli kann auch jede andere Klasse verwendet werden (zB. eine Admin-Klasse von dir).

Original geschrieben von lx-club
Wieso werden damit böswillige Eingaben nicht abgefangen?

Weil beide Funktionen nicht für diesen Zweck gedacht sind.

Mit strip_tags() (http://php.net/manual/en/function.strip-tags.php) entfernst du "HTML-Tags" (oder das was diese Funktion dafür hält). Es zerstört Benutzereingaben, macht sie aber nicht sicher. Alles was zwischen den "Tags" steht, bleibt da und kann weiter Schaden anrichten.

Und stripslashes() (http://php.net/manual/en/function.stripslashes.php) ist nichts weiter als die Umkehrfunktion zu addslashes() (http://php.net/manual/en/function.addslashes.php). Damit kannst du per addslashes() eingefügte "\"-Zeichen wieder entfernen. Es wird auch gerne benutzt, um die Veränderungen des Magic-Quote-Effekts rückgängig zu machen. Dafür ist es aber nicht (http://php.net/manual/en/function.stripslashes.php#45556) geeignet.

Du musst für jeden Anwendungsfall entscheiden, in welchen "Kontext" die die Daten eingebracht werden sollen. Dafür gibts Beispiele, aber keine Eine-für-Alle-Regel:

Sollen die Eingaben als HTML ausgegeben werden, hilft meist htmlspecialchars().

Fürs Einfügen in Datenbanken gibts passende Funktionen, wie bspw. mysqli::real_escape_string() (http://php.net/manual/de/mysqli.real-escape-string.php).

In deinem Fall kann die schon erwähnte "Whitelist" die Lösung sein:

// define a list of allowed classes|categories
$allowed_categories = array (
'my_book',
'my_compactdisc',
);

if (!in_array($_GET['category'], $allowed_categories, TRUE))) {
// Fehlerbehandlung
die('Ungültige Eingabe!');
}
if (!class_exists($_GET['category'])) {
// Fehlerbehandlung
die('Die Klasse gibts ja gar nicht!');
}

$obj = new $_GET['category'];


Wenn du wirklich ALLE in deinem Script existierenden Klassen (http://php.net/manual/en/function.get-declared-classes.php) erlauben willst, reicht es, wennn du die Eingabe auf erlaubte Zeichenmuster prüfst:

// Alle Zeichenkombinationen, die für Klassen-Namen erlaubt sind
if (!preg_match('/\A[a-z][a-z0-9_]+\z/i', $_GET['category'])) {
// Fehlerbehandlung
die('Ungültige Eingabe!');
}
if (!class_exists($_GET['category'])) {
// Fehlerbehandlung
die('Die Klasse gibts ja gar nicht!');
}

$obj = new $_GET['category'];

Danke für deine Hilfe. Mit der Factory klappt super :-)

Aber wenn man nun sehr viele Kategorien hat, dann wird es mit dem überprüfen von instanceof schon etwas aufwendiger :-) aber es klappt alles :-)

Original geschrieben von lx-club
Aber wenn man nun sehr viele Kategorien hat, dann wird es mit dem überprüfen von instanceof schon etwas aufwendiger :-)

Nicht wenn du mit Vererbungen arbeitest.

ich arbeite ja mit Vererbungen

class DVD extends Product usw....

Ich hab es so verstanden:


if ($obj instanceof DVD){
$obj->getPlaytime();
}

if ($obj instanceof Buch){
$obj->getPagenum();
}



Wenn man nun nicht nur DVD als eine Produktart hat sondern noch 10 ander muss man doch für jede Klasse prüfen oder?

Die Frage ist, was du machen willst.

Wo ist denn sowas erforderlich?if ($obj instanceof DVD){
$obj->getPlaytime();
}

if ($obj instanceof Buch){
$obj->getPagenum();
}

Das eine ist eine Zeit, das andere eine Nummer. Was willst du jetzt wirklich? Du solltest dich vielleicht auf eine Schnittstelle einigen, damit du bei jeder Klasse andere Funktionsnamen hast.

interface Identifiable { ... }
interface Listable { ... }
abstract class Factory { ... }
abstract class List implements ArrayIterator { ... }
class ProductFactory extends Factory { ... }
class ProductList extends List { ... }
class Product implements Identifiable, Listable { ... }
class CD extends Product { ... }
class DVD extends Product { ... }

So in etwa.

Zum Anzeigen der Liste über ihren Iterator laufen und an jedem Element getDetails() oder sowas aufrufen. Das liefert dann eben ein Array, in dem bei CDs eben die Playtime und bei DVDs wasweisich drinsteht.

Original geschrieben von h3ll
Die Frage ist, was du machen willst.

Wo ist denn sowas erforderlich?if ($obj instanceof DVD){
$obj->getPlaytime();
}

if ($obj instanceof Buch){
$obj->getPagenum();
}

Das eine ist eine Zeit, das andere eine Nummer. Was willst du jetzt wirklich? Du solltest dich vielleicht auf eine Schnittstelle einigen, damit du bei jeder Klasse andere Funktionsnamen hast.

das ist doch grad der sinn der vererbung, das ich spezielle eigenschaften hinzufügen kann. ich will bei ner dvd die spiellänge wissen und bei nem buch die anzahl der seiten...genau das will ich anzeigen.

Original geschrieben von onemorenerd

interface Identifiable { ... }
interface Listable { ... }
abstract class Factory { ... }
abstract class List implements ArrayIterator { ... }
class ProductFactory extends Factory { ... }
class ProductList extends List { ... }
class Product implements Identifiable, Listable { ... }
class CD extends Product { ... }
class DVD extends Product { ... }

So in etwa.

Zum Anzeigen der Liste über ihren Iterator laufen und an jedem Element getDetails() oder sowas aufrufen. Das liefert dann eben ein Array, in dem bei CDs eben die Playtime und bei DVDs wasweisich drinsteht.

ich glaub hab noch einiges zu lernen was oop betrifft :-), aber so ähnlich wie du beschreiben hast hab ich es gemacht... in ner methode showSpecifications() wird nen array zurückgegeben mit den jeweiligen daten

Original geschrieben von lx-club
das ist doch grad der sinn der vererbung, das ich spezielle eigenschaften hinzufügen kann. ich will bei ner dvd die spiellänge wissen und bei nem buch die anzahl der seiten...genau das will ich anzeigen.

Was willst du anzeigen? Antworte jetzt nicht mit "spiellänge" und "anzahl der seiten", sondern finde einen Oberbegriff dafür, denn genau das willst du anzeigen. Jetzt brauchst du nur noch Funktionen nach diesem Oberbegriff benennen und schon hast du überall eine Funktion, die gleich heißt, aber was unterschiedliches liefert.

Original geschrieben von h3ll
Jetzt brauchst du nur noch Funktionen nach diesem Oberbegriff benennen und schon hast du überall eine Funktion, die gleich heißt, aber was unterschiedliches liefert.

ich möchte alle daten zum produkt ausgeben.... da zählt jan icht nur spiellänge oder seitenzahl dazu sondern auch andere daten.

du meinst sowas hier?


public function getData(){
return $this->playtime; // in der DVD-Klasse
}

bzw.
public function getData(){
return $this->pagenum; // in der Buch-Klasse
}


Da ich aber mehrere Eigenschaften ausgeben will gebe ich ein Array zurück, wie es hier auch vorgeschlagen wurde.

Man kann onemorenerd's Klassen auch noch weiter stricken, dann löst sich auch dein Problem. Ich nehme mal an, du willst die Produkte ausgeben.

abstract class ProductWebControl extends WebControl {
public function __construct(Product $p) { ... }
abstract public function render();
}
class ProductWebControlFactory extends Factory {
public function fabricate(Product $p) { ... }
}
class CDWebControl extends ProductWebControl {
public function render() {
echo $this->product->getPlaytime();
}
}
class BuchWebControl extends ProductWebControl {
public function render() {
echo $this->product->getPageCount();
}
}