php.de

Zurück   php.de > Webentwicklung > PHP Einsteiger > PHP Tipps 2007

 
 
LinkBack Themen-Optionen Thema bewerten
Alt 24.12.2005, 09:27  
Gast
 
Beiträge: n/a
Standard [Erledigt] Zuviele Seiten werden angezeigt

Hi,

ich bin dabei gerade ein Forum zu programmieren und habe nun ein kleines Problem entdeckt.
Es werden mir bei der Posts ausgabe einfach zu viele Seiten angezeigt und ich weiß nicht warum.
Um es anschaulicher zumachen, habe ich das Forum mal hochgeladen:

Ihr könnt euch mit folgenden Daten Einloggen:

http://newwarrior.milten.lima-city.d...p?action=login

Login: test
Passwort: test

Und dann klickt ihr auch den Link:

http://newwarrior.milten.lima-city.d...=1&thread_id=1

Dort könnt ihr euch ansehen was ich meine. Auf drei Seiten steht nur ein Beitrag, und es werden aber 5 Seiten angezeigt und ich weiß nicht warum.

Auf der dritten Seite ist der Code meiner beiden PHP Dateien gepostet, womit ich die Posts in Seiten einteile.
Ich kann mir aber nicht erklären warum mir zuviele Seiten angezeigt werden.
Wäre nett wenn ihr mir da helfen könntet.

Danke für die Hilfe.
 
Sponsor Mitteilung
PHP Code Flüsterer

Registriert seit: 21.08.2005
Beiträge: 4682
PHP-Kenntnisse:
Fortgeschritten

Alt 24.12.2005, 10:41  
Gast
 
Beiträge: n/a
Standard

Zitat:
$prev_site = $site-1;
$next_site = $site+1;
Vermutlich meinst Du "page". site heißt Standort, Schauplatz u.ä.

Zitat:
$addy = "posts.php?forum_id=".$_GET["forum_id"]."&thread_id=".$_GET["thread_id"]."";
if($site > 1)echo '«« «';
Damit kann Code aus den url Parametern in den html Code der Seite einfließen; nicht gut.
Wenn forum_id und thread_id Zahlen sein sollen, stelle sicher, dass sie es sind.
Wozu .""? Eine leere Zeichenkette anhängen ist überflüssig.

Zitat:
$sql_sites = "SELECT * FROM user";

$result_sites = mysql_query($sql_sites);

$num_sites = mysql_num_rows($result_sites);
Damit werden alle Datensätze von mysql in den Speicher von php kopiert.
Wozu? Du willst ja nur die Anzahl haben.
Wesentlich schneller ist daher SELECT Count(*) FROM user

Und was hat die Tabelle user mit der Anzahl der Seiten (oder doch Standorte?) zu tun?

Konsequenter einrücken. Das schließt auch einheitliches Setzen/Einrücken von Klammern mit ein.
Bezeichnungen/Variablennamen überdenken. Sprechende, treffende Namen verwenden. Englische Bezeichnungen im Zweifelsfall bei http://dict.leo.org/ nachschlagen.
 
Alt 24.12.2005, 11:14  
Gast
 
Beiträge: n/a
Standard

Danke,

dies war wirklich der Fehler...
 
Alt 24.12.2005, 11:55  
Erfahrener Benutzer
 
Registriert seit: 02.04.2008
Beiträge: 2.603
Corvin befindet sich auf einem aufstrebenden Ast
Standard

Mal so nebenbei:
Zitat:
PHP-Code:
$sql_sites "SELECT * FROM user"
Sowas solltest du vermeiden, gerade bei einem Forum.
http://php-faq.de/q/q-sql-select.html

Und ein paar mehr Kommentare wären vielleicht auch keine schlechte Idee.
Corvin ist offline  
Alt 24.12.2005, 12:40  
Gast
 
Beiträge: n/a
Standard

Wie genau meinst du das mit den Kommentaren?
 
Alt 24.12.2005, 12:49  
Erfahrener Benutzer
 
Registriert seit: 02.04.2008
Beiträge: 2.603
Corvin befindet sich auf einem aufstrebenden Ast
Standard

Du hast da fast 60 Zeilen Code und die enthalten nur einen Kommentar. Zur späteren Wartung des Codes ist es sicher hilfreich (und zeitsparend), wenn du ein paar mehr Kommentare an entsprechenden Stellen verwendest.

Was mir eben noch aufgefallen ist:
Zitat:
echo '«« «';
[...]
echo '» »»

';
Hier solltest du lieber "«" statt "«" schreiben. Und natürlich "»" statt "»".

Und das solltest du dir auch mal zu Herzen nehmen:
http://tut.php-q.net/einruecken.html
Corvin ist offline  
 


Themen-Optionen
Thema bewerten
Thema bewerten:

Forumregeln
Es ist dir nicht erlaubt, neue Themen zu verfassen.
Es ist dir nicht erlaubt, auf Beiträge zu antworten.
Es ist dir nicht erlaubt, Anhänge hochzuladen.
Es ist dir nicht erlaubt, deine Beiträge zu bearbeiten.

BB-Code ist an.
Smileys sind an.
[IMG] Code ist an.
HTML-Code ist aus.
Trackbacks are an
Pingbacks are an
Refbacks are an
Gehe zu

Ähnliche Themen
Thema Autor Forum Antworten Letzter Beitrag
Frage zu Query Results auf mehreren Seiten verteilen whiggy Datenbanken 2 13.06.2006 22:05
Seiten sichern!? c01001 PHP Tipps 2006 7 20.01.2006 09:13
[Erledigt] Seiten werden fehlerhaft angezeigt wie z.B. <TABLE width= PHP-Fortgeschrittene 1 13.09.2005 19:18
[Erledigt] Seiten in PHP mit Datenbank PHP Tipps 2005-2 3 24.07.2005 09:07
php5 Seiten in htdocs werden nicht angezeigt. PHP Tipps 2005-2 26 25.06.2005 18:30
HTML Seiten generieren obi PHP Tipps 2005-2 5 05.06.2005 03:21
seiten werden nicht angezeigt Server, Hosting und Workstations 1 24.05.2005 11:41
Seiten werden nicht angezeigt PHP Tipps 2005 4 11.05.2005 11:59
[Erledigt] Bilder werden nicht richtig angezeigt PHP Tipps 2005 31 30.04.2005 18:29
3 Seiten durch Passwort schützen mit einmaliger PW-Eingabe? PHP Tipps 2005 12 19.04.2005 22:41
wml oder php/html Seiten automatisch erkennen Soese PHP Tipps 2005 4 13.01.2005 22:57
[Erledigt] div auf mehreren Seiten HTML, Usability und Barrierefreiheit 0 25.11.2004 19:10
News Rubrik auf mehrere Seiten splitten PHP Tipps 2004 7 07.10.2004 12:29
[Erledigt] Bild wird nicht angezeigt Server, Hosting und Workstations 7 03.09.2004 16:08
[Erledigt] Beim Blättern werden keine weiteren Seiten angezeigt Datenbanken 1 17.06.2004 16:12


Alle Zeitangaben in WEZ +2. Es ist jetzt 15:34 Uhr.




Powered by vBulletin® Version 3.7.2 (Deutsch)
Copyright ©2000 - 2012, Jelsoft Enterprises Ltd.
Search Engine Optimization by vBSEO 3.2.0
Aprilia-Forum, Aquaristik-Forum, Liebeskummer-Forum, Zierfisch-Forum, Geizkragen-Forum

Creative Commons License
Dieser Inhalt ist unter einer Creative Commons-Lizenz lizenziert.