Auch das funktioniert bei mir.
Wie dem auch sei - um das Skript vernünftig zu testen könnte ich es auch grad neuschreiben. Und dafür hab ich weder Lust noch Zeit.
Ich empfehle dir irgendeine Version die ich da bereits gepostet habe zu verwenden oder etwas PHP zu lernen und es selber versuchen anzupassen.
Gruss
Er will das dann ausgeben. Aber die Lösung für sein Problem steht ja schon auf der ersten Seite also lassen wir den Thread doch ruhen.
Wie wärs mit
<?php
// vor dem login
session_start();
$_SESSION['get_vars'] = $_GET;
// nach dem login - url wieder aufbauen
$url = '';
$i = 1;
foreach ($_SESSION['get_vars'] as $key => $value)
{
$separator = ($i == 1) ? '?' : '&';
$url.= $separator.$key.'='.$value;
$i++;
}
header('Location: '.$_SERVER['SERVER_NAME'].$url);
?>Gruss
Keine Ahnung. Das Skript ist eh komisch.
Du hast ja gesagt, dass der Fehler nicht kommt, wenn mehrere Seiten da sind. Als Schnell-Lösung habe ich jetzt eingefügt, dass wenn [anzahlBilderProSeite] kleiner ist als [anzahlBilderÜberhaupt], dass dann [anzahlBilderProSeite] halbiert wird, damit es weitere Seiten gibt und der Fehler nicht mehr kommt. Und, dass wen ?page nicht gesetzt ist, dass es automatisch 1 wird.
Klappt das so?
<?php
/*
THEPICS lite - a picture gallery generator
Copyright (C) 2001-2003
Website: http://d0x.codesociety.com/
Email: d0x@codesociety.com
This program is free software; you can redistribute it and/or modify
it under the terms of the GNU General Public License as published by
the Free Software Foundation; either version 2 of the License, or
(at your option) any later version.
This program is distributed in the hope that it will be useful, but
WITHOUT ANY WARRANTY; without even the implied warranty of
MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
General Public License for more details.
*/
/************ THEPICS lite CONFIGURATION ************/
$pathToImagesDir = "images/";
// Name of your gallery
$subject = "Gallery";
// Short description of your gallery.
$description = "Übersicht";
// How many images to output per row
$imagesPerRow = 4;
// How many images to output per page
$imagesPerPage = 5;
// Gallery sort order [ alpha | date ]
$sort = "alpha";
// Height of each thumbnail
$thumbHeight = 50;
// Width of each thumbnail. NOTE: Width takes precedence
$thumbWidth = 50;
if (!$_GET['page'])
$_GET['page'] = 1;
// Debugging
$showdebug = "0"; //Set to "1" to show the page generation time
error_reporting(E_ALL); //Uncomment this to show all warnings and errors.
//phpinfo(); //Uncomment this to display PHP info.
/************ END THEPICS lite CONFIGURATION ************/
$mtime = explode(" ", microtime()); //debug timer
$starttime = $mtime[1] + $mtime[0];
//Styles
echo "<!DOCTYPE HTML PUBLIC '-//W3C//DTD HTML 4.01 Transitional//EN'>
<html xmlns='http://www.w3.org/1999/xhtml'>
<head>
<meta http-equiv='Content-Type' content='text/html; charset=utf-8' />
<title>Depo-Fit-Entspannung</title>
<link rel='stylesheet' type='text/css' media='screen, projection' href='../stylesheet.css'>
<link rel='shortcut icon' href='../favicon.ico>
<style type='text/css'>
</head>
<div id='galerie'><body>
<div align='center'><FIELDSET style='width: 680px'><LEGEND><STRONG>$subject</STRONG></LEGEND>";
//Code
if (isset($_GET['pic'])) {
$picsdir = $pathToImagesDir;
$dir = opendir($picsdir);
while ($dirdata = readdir($dir)) {
if (eregi("\.jpg", $dirdata)) {
$main[] = $dirdata;
$picdates[] = filemtime($dirdata);
}
}
closedir($dir);
if($sort == "date") {
arsort($picdates, SORT_NUMERIC);
}
else {
natcasesort($main); //Sort pictures alphabetically
}
$pickey = ($sort == "date") ? array_keys($picdates) : array_keys($main);
$curPic = array_search($_GET['pic'], $main);
$nextkey = $pickey[0];
while ($nextkey != $curPic) {
$nextkey = next($pickey);
}
$nextkey = next($pickey);
$page_next = ($pickey[$curPic] != count($main)-1) ? " <A href='gallery.php?pic={$main[$nextkey]}' title='nächstes Foto'><IMG src='grafik/showfoto_next.gif' border='0'></A>" : "";
prev($pickey);
$prevkey = prev($pickey);
$page_prev = ($curPic != $pickey[0]) ? "<A href='gallery.php?pic={$main[$prevkey]}' title='vorheriges Foto'><IMG src='grafik/showfoto_prev.gif' border='0'></A> " : "";
// Make sure the file exists
if (file_exists($picsdir.$_GET['pic'])) {
$desc = stripslashes(preg_replace("/.jpg/", "", $_GET['pic']));
echo "<TABLE id='Table_Fotoalbum_ShowFoto_Nav' width='640' align='center' border='0'>
<TR>
<TD align='center'><FONT size='3'>[ <A href='gallery.php'>$description</A> ]</FONT></TD>
<TD align='right' width='200'><FONT size='2'>{$page_prev}{$page_next}</FONT></TD>
</TR>
</TABLE>
<IMG src=\"{$picsdir}{$_GET['pic']}\" border=\"1\" alt=\"$desc\" width=\"600\" />";
}
} else {
$rootdir = $pathToImagesDir;
$picsdir = $rootdir;
$thumbsdir = $rootdir . 'thumbs/';
$thumbpicssitedir = 'thumbs/';
$numberofmainpics = 0;
$dir = opendir($picsdir);
while ($dirdata = readdir($dir)) {
if (eregi("\.jpg", $dirdata)) {
$main[] = $dirdata;
$picdates[] = filemtime($dirdata);
$numberofmainpics++;
}
}
closedir($dir);
$actionlistfile = $main; //temp variables
$numberofactionpics = $numberofmainpics;
// Get mainthumblistfile
$numberofmainpics = 0;
if (file_exists($thumbsdir)) {
$dir = opendir($thumbsdir);
while ($dirdata = readdir($dir)) {
if (eregi("\.jpg", $dirdata)) {
$mainthumbs[] = $dirdata;
}
}
closedir($dir);
} else {
if (!@mkdir($thumbsdir, 0777)) {
echo "Error: Did you CHMOD the installation directory to 777 ?<BR />";
}
}
$counter = 0;
$createdhowmanymain = 0;
$createdhowmanyaction = 0;
foreach($main as $mainpic) {
if (!@in_array($mainpic, $mainthumbs)) {
$createmainthumbs[$counter] = $mainpic;
$createdhowmanymain++;
$counter++;
}
}
// Find out if we need to create thumbnails or not
if ($createdhowmanymain > 0) {
foreach($createmainthumbs as $createmain) {
makethumb($picsdir . $createmain, $thumbsdir . $createmain, $thumbWidth, $thumbHeight);
}
}
if ($description != "") {
$showDescription = "<FONT size='3'><STRONG>$description</STRONG> ( <FONT color='#800000'>$numberofactionpics</FONT> Fotos )</FONT><BR /><BR />";
}
// The follow lines control the Gallery Header.
echo "$showDescription
<FONT size='16'></FONT>";
if ($numofactionpics < $imagesPerPage)
$imagesPerPage = (round($imagesPerPage/2));
// Set inital row and column counts to 1
$actionrow = 1;
$actioncol = 0;
if($sort == "date") {
arsort($picdates, SORT_NUMERIC);
}
else {
natcasesort($actionlistfile); //Sort pictures alphabetically
}
$pickey = ($sort == "date") ? array_keys($picdates) : array_keys($actionlistfile);
echo '<TABLE>';
$multiPage = "";
$totalpages = ceil($numberofactionpics / $imagesPerPage);
$_GET['page'] = isset($_GET['page']) ? $_GET['page'] : 0;
if ($totalpages > 1) {
if (($numberofactionpics - $_GET['page']) <= $imagesPerPage) { // no next page
$page_next = "<IMG src='grafik/showfoto_next.gif' border='0'>";
} else {
$nextpage = $_GET['page'] + $imagesPerPage;
$page_next = "<A href='gallery.php?page={$nextpage}' title='nächste Seite'><IMG src='grafik/showfoto_next.gif' border='0'></A>";
}
$thepages = "";
for($temp = 0; $temp < $totalpages; $temp++) {
$gotopage = $temp * $imagesPerPage;
$temp_pagenum = $temp + 1;
$thepages .= ($_GET['page'] == $gotopage) ? "<STRONG>{$temp_pagenum}</STRONG> " : "<a href='gallery.php?page={$gotopage}'>{$temp_pagenum}</a> ";
}
if ($_GET['page'] == 0) { // no prev page
$page_previous = "<grafik/showfoto_prev.gif' border='0'>";
} else {
$previouspage = $_GET['page'] - $imagesPerPage;
$page_previous = "<A href='gallery.php?page={$previouspage}' title='vorherige Seite'><IMG src='grafik/showfoto_prev.gif' border='0'></A>";
}
$multiPage = "<FONT size='8'>{$page_previous} {$thepages} {$page_next}</FONT>";
}
if (($_GET['page'] + $imagesPerPage) > $numberofactionpics) {
$imagesPerPage = $numberofactionpics % $_GET['page'];
$imagesPerPage = $imagesPerPage ? $imagesPerPage : $numberofactionpics;
}
// Output the actual images
for($x = $_GET['page']; $x < ($_GET['page'] + $imagesPerPage) ; $x++) {
if ($actioncol == 0) {
echo '<TR>';
}
$sizetemp = preg_replace("/ /", "%20", $actionlistfile[$pickey[$x]]);
$picname = addslashes($sizetemp);
$desc = preg_replace("/.jpg/i", "", $actionlistfile[$pickey[$x]]);
$sizer = $picsdir . $sizetemp;
$openPic = "<a href=\"gallery.php?pic=$picname\"";
echo '<TD align="center" style="width: 150px">
<A ', $openPic , '><img src="'.$picsdir.'thumbs/', $sizetemp , '"border="1" alt="', $desc, '" /></A></TD>';
$actioncol++;
if ($actioncol == $imagesPerRow) {
$actioncol = 0;
echo '</TR>';
}
}
echo "<TR><TD colspan='{$imagesPerRow}' align='center'>$multiPage</TD></TR></TABLE>";
$mtime = explode(" ", microtime());
$totaltime = round((($mtime[1] + $mtime[0]) - $starttime), 5);
$debug = "";
if ($showdebug == 1) {
$debug = "Diese Thumbnailgallery wurde in {$totaltime} Sekunden erzeugt.";
}
}
// Version info
$debug = isset($debug) ? $debug : "";
echo '<FONT size="2">', $debug, '</FONT>
</FIELDSET>
</div>
</body></div>
</html>';
function makethumb($image, $filename, $newwidth, $newheight)
{
$imagequality = 70;
$size = getimagesize($image);
$ratio = $newwidth / $newheight;
if ($im = @imagecreatefromjpeg($image)) {
if ($newheight && ($size[0] < $size[1])) {
$newwidth = ($newheight / $size[1]) * $size[0];
} else {
$newheight = ($newwidth / $size[0]) * $size[1];
}
$im2 = imagecreatetruecolor($newwidth, $newheight);
// Version Dependant setting
// imageantialias($im2, TRUE); //for PHP 4.3.2 and up only
imagecopyresized($im2, $im, 0, 0, 0, 0, $newwidth, $newheight, $size[0], $size[1]);
if (@imagejpeg($im2, $filename, $imagequality)) {
return true;
} else {
echo "Error: Did you CHMOD the installation directory to 777 ?<BR />";
}
}
echo "Error: Thumbnail creation failed for <STRONG>{$filename}</STRONG>. Is the original picture corrupted?<BR />";
return false;
}
?>Ja, klar kann man das.
Erklär mir nochmal wo da welcher Name drin stehen soll.. Ich weiss nicht was du meinst.
Ach ja - wenn du einem deiner Posts hier etwas zuzufügen hast und seither niemand geantwortet hat, brauchst du nicht einen neuen Post zu machen, editier einfach deinen Letzten.
Ne bekomm ich nicht. Hast du den vorher auch gesehen, wenn du das Script im gleichen Ordner wie die Bilder hast laufen lassen?
Da musst du wohl ein Cookie setzen wenn du das "Login" speichern willst.
Naja ich krieg keinen Fehler bei mir da kann ich das schlecht ändern damit es passt..?
Dann würd ich sowas machen:
[angezeigte Seite] -> Suchaufruf -> [verarbeitungs- / steuerungs-Datei] -> Ausgabe -> [include eines HTML Grundgerüstes mit Variablen]
v
Daten holen
v
[Datenbankabfragen Handler](Die [] umklammerten Wörter entsprechen realer Dateien)
Das ist eine Art MVC Abwicklung (wikipedia) kombiniert mit dem Vorschlag von SinlosS.
In der "verarbeitungs- / steuerungs-Datei" führst du allgemeine Variablen, wie $content oder $search_results, halt je nachdem was du brauchst und nimmst Suchanfragen entgegen (if $_POST['search_str'] usw.). Dann erstellst du eine Instanz der DB-Handler Klasse, und hast dort pro Suchfunktion eine Funktion, die genau das aus der DB holt was du brauchst und die Daten so aufbereitet wie du sie brauchst. Diese Daten holst du dir mit sowas wie $results = $db_handler->search_xyz($search_str); in die Verarbeitungsdatei, und gibst sie am Schluss über die "Template" Datei aus (.php Datei mit HTML Grundgerüst die an den entsprechenden Stellen print()'s von den Variablen, die du in der verarbeitungs-Datei setzst, macht).
Alles klar soweit? 
Das von SinlosS genannte Tutorial ist in der Tat gut, lies dich da mal etwas durch wenn du Zeit hast.
Kannst das auf Zeile 101 vom obigen Skript anpassen (<A href='gallery.php?page=1).
Erzähl mir doch etwas mehr über deine Applikation dann überlegen wir uns was.
Da, bitte:
<?php
/*
THEPICS lite - a picture gallery generator
Copyright (C) 2001-2003
Website: http://d0x.codesociety.com/
Email: d0x@codesociety.com
This program is free software; you can redistribute it and/or modify
it under the terms of the GNU General Public License as published by
the Free Software Foundation; either version 2 of the License, or
(at your option) any later version.
This program is distributed in the hope that it will be useful, but
WITHOUT ANY WARRANTY; without even the implied warranty of
MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
General Public License for more details.
*/
/************ THEPICS lite CONFIGURATION ************/
$pathToImagesDir = "images/";
// Name of your gallery
$subject = "Gallery";
// Short description of your gallery.
$description = "Übersicht";
// How many images to output per row
$imagesPerRow = 4;
// How many images to output per page
$imagesPerPage = 32;
// Gallery sort order [ alpha | date ]
$sort = "alpha";
// Height of each thumbnail
$thumbHeight = 50;
// Width of each thumbnail. NOTE: Width takes precedence
$thumbWidth = 50;
// Debugging
$showdebug = "0"; //Set to "1" to show the page generation time
//error_reporting(E_ALL); //Uncomment this to show all warnings and errors.
//phpinfo(); //Uncomment this to display PHP info.
/************ END THEPICS lite CONFIGURATION ************/
$mtime = explode(" ", microtime()); //debug timer
$starttime = $mtime[1] + $mtime[0];
//Styles
echo "<!DOCTYPE HTML PUBLIC '-//W3C//DTD HTML 4.01 Transitional//EN'>
<html xmlns='http://www.w3.org/1999/xhtml'>
<head>
<meta http-equiv='Content-Type' content='text/html; charset=utf-8' />
<title>Depo-Fit-Entspannung</title>
<link rel='stylesheet' type='text/css' media='screen, projection' href='../stylesheet.css'>
<link rel='shortcut icon' href='../favicon.ico>
<style type='text/css'>
</head>
<div id='galerie'><body>
<div align='center'><FIELDSET style='width: 680px'><LEGEND><STRONG>$subject</STRONG></LEGEND>";
//Code
if (isset($_GET['pic'])) {
$picsdir = $pathToImagesDir;
$dir = opendir($picsdir);
while ($dirdata = readdir($dir)) {
if (eregi("\.jpg", $dirdata)) {
$main[] = $dirdata;
//$picdates[] = filemtime($dirdata);
}
}
closedir($dir);
if($sort == "date") {
arsort($picdates, SORT_NUMERIC);
}
else {
natcasesort($main); //Sort pictures alphabetically
}
$pickey = ($sort == "date") ? array_keys($picdates) : array_keys($main);
$curPic = array_search($_GET['pic'], $main);
$nextkey = $pickey[0];
while ($nextkey != $curPic) {
$nextkey = next($pickey);
}
$nextkey = next($pickey);
$page_next = ($pickey[$curPic] != count($main)-1) ? " <A href='gallery.php?pic={$main[$nextkey]}' title='nächstes Foto'><IMG src='grafik/showfoto_next.gif' border='0'></A>" : "";
prev($pickey);
$prevkey = prev($pickey);
$page_prev = ($curPic != $pickey[0]) ? "<A href='gallery.php?pic={$main[$prevkey]}' title='vorheriges Foto'><IMG src='grafik/showfoto_prev.gif' border='0'></A> " : "";
// Make sure the file exists
if (file_exists($picsdir.$_GET['pic'])) {
$desc = stripslashes(preg_replace("/.jpg/", "", $_GET['pic']));
echo "<TABLE id='Table_Fotoalbum_ShowFoto_Nav' width='640' align='center' border='0'>
<TR>
<TD align='center'><FONT size='3'>[ <A href='gallery.php?page=1'>$description</A> ]</FONT></TD>
<TD align='right' width='200'><FONT size='2'>{$page_prev}{$page_next}</FONT></TD>
</TR>
</TABLE>
<IMG src=\"{$picsdir}{$_GET['pic']}\" border=\"1\" alt=\"$desc\" width=\"600\" />";
}
} else {
$rootdir = $pathToImagesDir;
$picsdir = $rootdir;
$thumbsdir = $rootdir . 'thumbs/';
$thumbpicssitedir = 'thumbs/';
$numberofmainpics = 0;
$dir = opendir($picsdir);
while ($dirdata = readdir($dir)) {
if (eregi("\.jpg", $dirdata)) {
$main[] = $dirdata;
//$picdates[] = filemtime($dirdata);
$numberofmainpics++;
}
}
closedir($dir);
$actionlistfile = $main; //temp variables
$numberofactionpics = $numberofmainpics;
// Get mainthumblistfile
$numberofmainpics = 0;
if (file_exists($thumbsdir)) {
$dir = opendir($thumbsdir);
while ($dirdata = readdir($dir)) {
if (eregi("\.jpg", $dirdata)) {
$mainthumbs[] = $dirdata;
}
}
closedir($dir);
} else {
if (!@mkdir($thumbsdir, 0777)) {
echo "Error: Did you CHMOD the installation directory to 777 ?<BR />";
}
}
$counter = 0;
$createdhowmanymain = 0;
$createdhowmanyaction = 0;
foreach($main as $mainpic) {
if (!@in_array($mainpic, $mainthumbs)) {
$createmainthumbs[$counter] = $mainpic;
$createdhowmanymain++;
$counter++;
}
}
// Find out if we need to create thumbnails or not
if ($createdhowmanymain > 0) {
foreach($createmainthumbs as $createmain) {
makethumb($picsdir . $createmain, $thumbsdir . $createmain, $thumbWidth, $thumbHeight);
}
}
if ($description != "") {
$showDescription = "<FONT size='3'><STRONG>$description</STRONG> ( <FONT color='#800000'>$numberofactionpics</FONT> Fotos )</FONT><BR /><BR />";
}
// The follow lines control the Gallery Header.
echo "$showDescription
<FONT size='16'></FONT>";
// Set inital row and column counts to 1
$actionrow = 1;
$actioncol = 0;
if($sort == "date") {
arsort($picdates, SORT_NUMERIC);
}
else {
natcasesort($actionlistfile); //Sort pictures alphabetically
}
$pickey = ($sort == "date") ? array_keys($picdates) : array_keys($actionlistfile);
echo '<TABLE>';
$multiPage = "";
$totalpages = ceil($numberofactionpics / $imagesPerPage);
$_GET['page'] = isset($_GET['page']) ? $_GET['page'] : 0;
if ($totalpages > 1) {
if (($numberofactionpics - $_GET['page']) <= $imagesPerPage) { // no next page
$page_next = "<IMG src='grafik/showfoto_next.gif' border='0'>";
} else {
$nextpage = $_GET['page'] + $imagesPerPage;
$page_next = "<A href='gallery.php?page={$nextpage}' title='nächste Seite'><IMG src='grafik/showfoto_next.gif' border='0'></A>";
}
$thepages = "";
for($temp = 0; $temp < $totalpages; $temp++) {
$gotopage = $temp * $imagesPerPage;
$temp_pagenum = $temp + 1;
$thepages .= ($_GET['page'] == $gotopage) ? "<STRONG>{$temp_pagenum}</STRONG> " : "<a href='gallery.php?page={$gotopage}'>{$temp_pagenum}</a> ";
}
if ($_GET['page'] == 0) { // no prev page
$page_previous = "<grafik/showfoto_prev.gif' border='0'>";
} else {
$previouspage = $_GET['page'] - $imagesPerPage;
$page_previous = "<A href='gallery.php?page={$previouspage}' title='vorherige Seite'><IMG src='grafik/showfoto_prev.gif' border='0'></A>";
}
$multiPage = "<FONT size='8'>{$page_previous} {$thepages} {$page_next}</FONT>";
}
if (($_GET['page'] + $imagesPerPage) > $numberofactionpics) {
$imagesPerPage = $numberofactionpics % $_GET['page'];
$imagesPerPage = $imagesPerPage ? $imagesPerPage : $numberofactionpics;
}
// Output the actual images
for($x = $_GET['page']; $x < ($_GET['page'] + $imagesPerPage) ; $x++) {
if ($actioncol == 0) {
echo '<TR>';
}
$sizetemp = preg_replace("/ /", "%20", $actionlistfile[$pickey[$x]]);
$picname = addslashes($sizetemp);
$desc = preg_replace("/.jpg/i", "", $actionlistfile[$pickey[$x]]);
$sizer = $picsdir . $sizetemp;
$openPic = "<a href=\"gallery.php?pic=$picname\"";
echo '<TD align="center" style="width: 150px">
<A ', $openPic , '><img src="'.$picsdir.'thumbs/', $sizetemp , '"border="1" alt="', $desc, '" /></A></TD>';
$actioncol++;
if ($actioncol == $imagesPerRow) {
$actioncol = 0;
echo '</TR>';
}
}
echo "<TR><TD colspan='{$imagesPerRow}' align='center'>$multiPage</TD></TR></TABLE>";
$mtime = explode(" ", microtime());
$totaltime = round((($mtime[1] + $mtime[0]) - $starttime), 5);
$debug = "";
if ($showdebug == 1) {
$debug = "Diese Thumbnailgallery wurde in {$totaltime} Sekunden erzeugt.";
}
}
// Version info
$debug = isset($debug) ? $debug : "";
echo '<FONT size="2">', $debug, '</FONT>
</FIELDSET>
</div>
</body></div>
</html>';
function makethumb($image, $filename, $newwidth, $newheight)
{
$imagequality = 70;
$size = getimagesize($image);
$ratio = $newwidth / $newheight;
if ($im = @imagecreatefromjpeg($image)) {
if ($newheight && ($size[0] < $size[1])) {
$newwidth = ($newheight / $size[1]) * $size[0];
} else {
$newheight = ($newwidth / $size[0]) * $size[1];
}
$im2 = imagecreatetruecolor($newwidth, $newheight);
// Version Dependant setting
// imageantialias($im2, TRUE); //for PHP 4.3.2 and up only
imagecopyresized($im2, $im, 0, 0, 0, 0, $newwidth, $newheight, $size[0], $size[1]);
if (@imagejpeg($im2, $filename, $imagequality)) {
return true;
} else {
echo "Error: Did you CHMOD the installation directory to 777 ?<BR />";
}
}
echo "Error: Thumbnail creation failed for <STRONG>{$filename}</STRONG>. Is the original picture corrupted?<BR />";
return false;
}
?>Versprich dir nicht zuviel von Template-Lösungen. Für die meisten Projekte ist das übers Ziel hinaus geschossen. Eine wirklich sinnvolle und Projekt-gerechte Lösung bei der Code und Layout weitgehend getrennt sind erfordert Erfahrung. Oftmals reicht auch ein sauberes include-System, oder dass man Layoutcode in einzelne PHP Dateien auslagert und bei den dynamischen Teilen dann einfach eine Variable ausgibt, die eine übergeordnete, einfache Klasse abfüllt.
Am besten ist es, wenn du dich etwas umschaust und dir verschiedene Lösungsansätze ansiehst und dich dann anhand deiner Projektgrösse und der dir zur Verfügung stehenden Entwicklungszeit entscheidest.
Wenn du objektorientiertes Programmieren beherrschst und es Sinn macht, die Applikation objektorientiert zu programmieren, dann ist das die beste Lösung: ja.
Ich kenne deine Anwendung jedoch zu wenig um das zu beurteilen, deshalb kann ich es dir nicht sagen.
Wenn du aber von oop nichts verstehst, kannst du das auch so machen wie ich es in meiner ersten Antwort gesagt habe. Zum Teil sind solche Lösungen sogar besser weil sie das Ziel am besten erfüllen (weniger Code, weniger Dateien, weniger Entwicklungsaufwand).
Also als Antwort: du kannst das durchaus so machen. So schlecht ist das nicht wie es vielleicht manche sagen.
Wenn du jedoch mal Zeit und Lust hast, oop zu lernen, dann ist das eine sinnvolle Investition. Google kann dir dabei weiterhelfen.
Du kannst die beiden Zeilen ruhig auskommentieren (// Zeilencode), dann werden sie nicht berücksichtigt. Sie dienen nur dazu, das Änderungsdatum der Datei zu ermitteln um die Bilder zu sortieren.
Um das zweite Problem zu lösen, kannst du ja die Anzahl Bilder / Seite oben in der Konfiguration des Skripts anpassen.
Was willst du mir damit sagen?
Das ist absoluter Unsinn - wenn du kein PHP zur Verfügung hast, dann besorg dir entweder ein PHP fähiger Webspace, oder mach kein Login, denn das was du da hast ist sowieso kein Login. Als Alternative kannst du ja eine Webseite machen, die du nirgends auf der Webseite verlinkst, und den Link gibst du nur denen, die ihn bekommen sollen.
Um dennoch deine Frage zu beantworten, der Fehler liegt vermutlich hier
parent.frames['inhalt'].location.href=htms[user];Bist du sicher, dass das array htms[] existiert und dass es für den User ein entsprechender Arrayeintrag hat?
P.S. Ich rate dir ab mit frames zu arbeiten, macht man heute nicht so.
Code und Layout trennen ist aber fast immer eine gute und praktische Lösung, es wird aber auch ohne gehen.
Als Alternativ-Lösung kannst du die PHP Teile in include-Dateien auslagern. Also sowas:
index.php
<html>
<head></head>
<body>
<?php include_once('user_begruessung.php'); ?>
<div>
<h1>Mitgliederstatistiken 2009/05</h1>
<?php include_once('statistik.php'); ?>
</div>
<?php include_once('seitenwechsler.php'); ?>
</body>
</html>Achte bei einer solchen Lösung darauf, dass HTML Code, der aus den include Dateien generiert wird so geschrieben ist das man ihn einfach anpassen kann. Also schön mit <div> <span> & co. arbeiten und allen eine class="" verpassen, damit du sie von einem Stylesheet aus positionieren und stylen kannst.
Da haben wirs. Liegen die gallerys auf unterschiedlichen Servern oder haben sie abweichende PHP Installationen?
Warum versuchst du Login-sachen mit Javascript zu realisieren? Kannst es auch grad sein lassen, denn jeder kann dein Javascript lesen und somit kann sich jeder versierter Benutzer auch ohne Anmeldung deine "geschützten" Seiten sehen..
Ein vernünftiges Login kannst du z.B. mit PHP realisieren.