[OOP] AUs einer Funktion mehrere Variablen rauslassen

Fipsi · 2. August 2012, 09:26

Morgen zusammen,

ich häng heut shcon wieder seit zwei Stunden an meinen Scripten rum, da muss endlich mal wieder was vorwärts gehn^^

So, zu meinem Problem:

Ich arbeite z. Z. daran ein paar Teile in OOP zu ändern. Ich bin jetzt gerade bei den News. Diese werden alle aus einer Datenbank gelesen, auf der news.php in einer Liste angezeigt, und dann per ?id= einzeln angezeigt.

In der Datei news.php steht lediglich das:

Quellcode

<?php
[...]
$news_id = $_GET['id'];
if (isset($news_id))
{
System\News::display_once_news ($mysqli, $access, $user5);
}
else
{
$seiten_inhalt = System\News::display_news_list($mysqli);
$seitentitel = "News";
$tab_titel = "";
}
user_last_aktion($mysqli, $user, $tab_titel);
System\HTML::printHead($access->zugang_check_tab_titel($user5, "news_see", $tab_titel."News"), $config, "");
System\HTML::printBody($config);
System\HTML::printNavigation($config, $user, $mysqli, $access, $user5);
System\HTML::printWerbung($config);
System\HTML::printHeadline($access->zugang_check_seitentitel($user5, "news_see", $seitentitel), $config);
System\HTML::printInhalt($access->zugang_check_seite($user5, "news_see", $seiten_inhalt), $config);
System\HTML::printFoot("standart", $config, $mysqli, $access, $user5);
?>

Alles anzeigen

Der else-Teil der des if-Blocks ist die Liste mit allen News, also die aufzurufende Funktion.
Der andere Teil ist die einzelne News, die angezeigt werden soll.
Jetzt habe ich folgendes Problem:
Ich kann nicht wie im else-Block einfach "$seiten_inhalt = System\News::display_once_news ($mysqli, $access, $user5);" machen, weil ich in der Funktion 3 Variablen hab, die ich benötige.
Geht das nicht irgendwie, dass die Variablen einfach zur Verfügung stehn, wie als würde die Funktion selbe rin der Datei stehn, anstatt der Aufruf zur Funktion?
Das nervt mich jetzt ehrlich gesagt langsam schon wieder...

LG

Fipsi

vince · 2. August 2012, 11:21

Bist du dir sicher das hat was mit OOP zu tun? Das schaut mehr nach einem auslagern in statische Funktionen aus. Nur weil alles in Klassen aufgeteilt ist, ist es nicht automatisch OOP!

So ganz kann ich dein Problem auch nicht nachvollziehen. Du möchtest nicht alles als Parameter übergeben?

ps.
Dein Denglisch ist gruselig

Fipsi · 2. August 2012, 11:29

vince schrieb:

ps.
Dein Denglisch ist gruselig

Sorry dafür^^

also ich hab ja schon 3 parameter übergeben, die der braucht umd zu wissen, was er aus der bd holen soll. macht er auch schön brav, bla, bla.
Der komplette Seiteninhalt wird in einer Variable $seiten_inhalt gespeichert. Der Seitentitel in $seitentitel, und das, wa sin der Tab-leiste steht in $tab_titel. Diese drei Variablen brauch ich jetzt aber außerhalb der Funktion, damit die Seite angezeigt werden kann. Mit return kann ich ja nur einen Teil zurückschmeißen, dannf helen ncoh die anderen beiden. Habs schon mti nem Array probiert, funktioniert genauso wenig.

Und zum OOP:
aso, ok... dachte jetzt nur, weils eig. aufgebaut is wie OOP... aber auch gut, dann halt nicht^^

vince · 2. August 2012, 13:29

Das benutzen von Klassen != OOP. Du arbeitest mehr in die prozedurale Richtung.

Deine ganzen Parameterübergaben sind alle redundant, warum nicht einmal als Klassenvariable?
Warum X verschiedene Printmethoden und nicht 1?

Quellcode

$html = new HTML($config, $mysql, ....);
$html->print(PrintEnum::Head, $access );
$html->print(PrintEnum::Body);
...

Anmerkung:
PHP kennt keine Enums, aber ein Workaround als Klasse mit const wäre durchaus möglich.

Ist der Ausschnitt den man dort sieht, der Inhalt einer Methode? Wenn ja, dann speicherst du $seiten_inhalt, $seitentitel und $tab_titel in Klassenvariablen und greifst einfach über die Klasse von aussen darauf zu.

Ich würde dir aber raten, dich wirklich mit OOP zu beschäftigen. Schau dir auch mal das MVC-Konzept an, das ist nicht unbedingt für den Einstieg geeignet, aber ein Blick lohnt sich.

splasch · 3. August 2012, 17:49

Hmm also ich hab auch nicht den Eindruck das es nach OO ausieht eher nee samlung von Funktionen die Statisch aufgerufen werden.
Auch die Parameter Anzahl scheint mir unötig hoch zu sein. Du versucht eben alles in jeder Funktion zu übergeben.
Statt eben Methoden und Objekte zu nutzen.

Beschreibung was du nun von uns wissen willst ist auch sehr wage. Du versucht uns einen nicht nachvoll ziehbaren Lösung weg vorzugeben.
Und fragst dann wie mann das Umsetzen kann.

Sieht im ganzen eben aus als were da mehr duchreinander als sinvoll genutzt.

Mfg Splasch

Fipsi · 19. August 2012, 13:00

So...

also ich hab jez nach meinem Urlaub und ner kurzen keine-Lust-Phase angefangen, das in "$html = new HTML();" umzubauen.

Auf meinem localhost funktioniert das jetzt nach gut 3 Stunden umbau reibungslos (ja, ich hab mcih etwas angestellt, ich weiß), und hab das nun auf meinen Server hochgeladen. Ergebnis: weiße Seite. Danke.
Ich hab dann error_reporting(E_ALL); eingeschrieben, immernoch weis. Ich denke mal, das liegt an der php.ini. Ich hab die Datei gesucht und 3 oder 4 gefunden. Bei allen auch error_Reporting und alle error-einstellungen auf on gestellt: immernoch weis.
Kann mir jemand sagen, wie ich so ne Fehlermeldung krieg?
Danke schonmal

Jan Thurau · 19. August 2012, 16:50

ini_set('display_errors', true);

Fipsi · 19. August 2012, 17:15

Klasse, hat funktioniert.
Hatte ich selber schon oft benutzt, nur mal wieder nciht drangedacht *facepalm*.
Vielen Dank

Fipsi · 26. August 2012, 08:23

ich könnt grad echt heuln. Ich versuch gerade, das wie von vince vorgeschlagen:

$html = new HTML($config, $mysql, ....);
$html->print(PrintEnum::Head, $access );
$html->print(PrintEnum::Body);

einzubinden.. funktioniert soweit, nur das mit "$html = new HTML($config, $mysql, ....);" will nicht. Wenn ich die Variablen in die klammern schreib, bei den Objectaufrufen rauslösch, in der Klassen-datei bei den Objecten ebenfalls diese Parameterangaben rauslösch, hauts mir mein ganzes Skript zusammen. Funktioniert garnichts mehr. Wenn ich in der Klassendatei die Parameterangaben drin lass, meckert er, weil sie beim aufruf nicht vorhanden sind. Kann mir bitte jemand sagen, wie das dann funktioniert? hab meinen php-wälzer grad danach durchgesucht, nichts gefunden.

Vielen Dank schonmal.

vince · 27. August 2012, 09:27

Fipsi schrieb:

einzubinden.. funktioniert soweit, nur das mit "$html = new HTML($config, $mysql, ....);" will nicht. Wenn ich die Variablen in die klammern schreib, bei den Objectaufrufen rauslösch, in der Klassen-datei bei den Objecten ebenfalls diese Parameterangaben rauslösch, hauts mir mein ganzes Skript zusammen. Funktioniert garnichts mehr. Wenn ich in der Klassendatei die Parameterangaben drin lass, meckert er, weil sie beim aufruf nicht vorhanden sind. Kann mir bitte jemand sagen, wie das dann funktioniert? hab meinen php-wälzer grad danach durchgesucht, nichts gefunden.

hi,

könntest du Auszüge aus deinem Code posten? So ganz kann ich dir nicht folgen, was du genau machst. Interessant sind deine Konstuktoren, Klassenvariablen und deine Aufrufe die du tätigst.

Fipsi · 27. August 2012, 20:50

ok..des sind n paar zeilen

also ich nehm jez mal als beispiel einfach die index-seite:

Quellcode

<?php
error_reporting(0);
session_start();
define('IN_FISW',True);
define('ROOT_PATH', "./");
require(ROOT_PATH."data/templates/includes.php");
$html = new HTML();
$link_index_edit = "<div id=\"rechts\"><a href=\"adm/edit.php?do=index\">Startseite ändern</a></div>\n";
$seiten_inhalt = $access->zugang_check_link($user5, "index_edit", $link_index_edit);
$datei = fopen(ROOT_PATH."data/templates/txt/index.txt", "r");
while ($inhalt = fgets($datei))
{
$seiten_inhalt .= nl2br($inhalt);
}
fclose($datei);
$seitentitel = "Herzlich Willkommen";
$tab_titel = "Startseite";
user_last_aktion($mysqli, $user, $tab_titel);
$html->header ($config, $access->zugang_check_tab_titel($user5, "index_read", $tab_titel), "");
$html->body ($config);
$html->navigation ($config, $user, $mysqli, $access, $user5);
$html->werbung ($config);
$html->headline ($config, $access->zugang_check_seitentitel($user5, "index_read", $seitentitel));
$html->inhalt ($config, $access->zugang_check_seite($user5, "index_read", $seiten_inhalt));
$html->footer ("standart", $config, $mysqli, $access, $user5);
?>

Alles anzeigen

das is dann die klasse dafür:

Quellcode

class HTML
{
/**
* Kontrolliert, ob die Website aktiv ist
*
*/
public static function website_aktiv ()
{
return true;
}
/**
* Erstellt den Kopf des HTML-Documents
*
*/
public static function header ($config, $titel_seite, $css)
{
header("Content-Type: text/html; charset=utf-8");
echo "<!DOCTYPE html PUBLIC \"-//W3C//DTD HTML 4.01 Transitional//EN\">"."\r\n";
echo "<html>\r\n";
echo "<head>\r\n";
if (self::website_aktiv() == true)
{
echo "<title>".bbcode($titel_seite)." - ".bbcode($config["website_titel"])."</title>\r\n";
}
else
{
echo "<title>Diese Seite ist deaktiviert - ".bbcode($config["website_titel"])."</title>\r\n";
}
echo "<meta http-equiv=\"content-type\" content=\"text/html; charset=utf-8\" />\r\n";
echo "<meta name=\"content-language\" content=\"de\" />\r\n";
echo "<meta name=\"author\" content=\"Phillip Seelig\" />\r\n";
echo "<meta name=\"publisher\" content=\"Phillip Seelig\" />\r\n";
echo "<meta name=\"copyright\" content=\"Phillip Seelig\" />\r\n";
echo "<meta name=\"keywords\" content=\"\" />\r\n";
echo "<meta name=\"description\" content=\"".$config['website_beschreibung']."\" />\r\n";
echo "<meta name=\"language\" content=\"Deutsch\" />\r\n";
echo "<meta name=\"revisit\" content=\"After 30 days\" />\r\n";
echo "<link rel=\"stylesheet\" type=\"text/css\" href=\"".ROOT_PATH."data/templates/css/main.css\" />\r\n";
echo "<link rel=\"stylesheet\" type=\"text/css\" href=\"".ROOT_PATH."data/templates/css/weiss.css\" />\r\n";
if (!empty($css))
{
echo "<link rel=\"srylesheet\" type=\"text/css\" href=\"".ROOT_PATH."data/templates/css/".$css.".css\" />\r\n";
}
}
/**
* Erstellt den Körper des HTML-Dokuments
*
*/
public static function body ($config)
{
echo "</head>\r\n";
echo "<body>\r\n";
echo "<div id=\"header\">\r\n";
if (!empty($config["name_header_pic"]))
{
echo "<div id=\"header_pic\"><img href=\"".ROOT_PATH."\" src=\"data/style/pic/templates/".$config["name_header_pic"]."></div>\r\n";
}
echo "<div id=\"header_titel\">".bbcode($config["website_titel"])."<br />\r\n";
echo bbcode($config["website_beschreibung"])."</div>\r\n";
echo "</div>\r\n";
echo "<div id=\"main_part\">\r\n";
}
/**
* Erstellt die Überschrift
*
*/
public static function headline ($config, $headline)
{
echo "<div id=\"inhalt_background\">\r\n";
echo "<div id=\"inhalt_text\">\r\n";
echo "<br />\r\n";
echo "<div id=\"headline\">";
if (self::website_aktiv() == TRUE)
{
echo bbcode($headline);
}
else
{
echo "Diese Website ist deaktiviert.";
}
echo "</div><br /><br />\r\n";
}

Alles anzeigen

[...]
bis halt alles durch is.

Das war jez alles, was du sehn wolltest, oder...?^^

P.S.: Warum sind die Syntax-bb-codes nimmer da..?

evilgoto · 1. September 2012, 15:11

Schau dir das Thema OOP nochmal gut an. Dir fehlen offenbare etliche Grundlagen.

[OOP] AUs einer Funktion mehrere Variablen rauslassen

[OOP] AUs einer Funktion mehrere Variablen rauslassen

Quellcode

vince schrieb:

Quellcode

Fipsi schrieb:

Quellcode

Quellcode

Teilen

Benutzer online 1

Tags