hamburg

hallo,
nach meinem posting und hinweis eines benutzers(danke) kam dabei dieses script heraus.

function in_wartung($seite){
$test= 0;
if (($sql_id = sql_connect())){
$sql="SELECT * FROM wartung WHRERE aktiv=1";
if ($result = mysql_query($sql,$sql_id))
{
if (mysql_num_rows($result) > 0)
{
while($seiten = mysql_fetch_array($result))
{
if ($seiten[0] == $seite)
{
$test = $seiten[1];
break;
}
}
}
}
}
mysql_free_result();
return $test;
}
ist das noch optimierbar??

CU hamburg


zu nikosch wächtercode??
wenn ich $sql ="select ´".$seite"´,aktiv from wartung where aktiv=1"; Php strike
mysql_num_rows wurde entfernt (doppelte abfragung) danke

zu wolla

es snd 15 seiten zb main, kontakt usw
alle seiten sind gleich aufgebaut

beispiel: news.php

include_once("../Inc/install.inc");

dieses segment soll eingebaut werden
>>$referer = explode('/',$_SERVER['HTTP_REFERER']);
if(in_wartung($referer[3])==0){
echo "<script>window.location.href=\"../system/index.php?sys=sys&seite=$referer[3]\";</SCRIPT>)";
} <<

define("SEITE","News"); // steuerung von Seitenzähler
include_once("Inc/funktion.inc");
lade_session();lade_statistik();
links(false);
include_once("../Inc/anzeige.inc.php");
include("news.php");//heute("news");
rechts("news");
include_once("../tpl/unten.tpl");
?>

danke

Mister Ad

phpdummi

Wenn du den Code in [php] Tags setzt und richtig einrückst kann man das eventuell beurteilen.

__________________
"Nobody is as smart as everybody" - Kevin Kelly
— The best things in life aren't things

nikosch

Sicher.
- Vom Stil her wäre Wächter-Code übersichtlicher.
- mysql_num_rows kannst Du Dir sparen. Für 0 DS wird die Schleife ohnehin nicht durchlaufen.

Ganz sicher
- Ein Datenvergleich macht man durch WHERE in der Datenbank, nicht durch Durchlaufen aller Datensätze

__________________
--
One pixel is still too big. Please make it smaller. ASAP.

Initiative Mittelstand.
Die wichtigste Gestaltungsregel im Screendesign ist Pi mal Daumen des Arbeitgebers.
--

Wolla

Du machst

SELECT *

nebst

$seiten = mysql_fetch_array()

und prüfst

$seiten[0]

Das funktioniert so lange, bis du deine Tabelle umbaust und im Null-ten Element was anderes steht. Besser selektiert man nur die Felder, die man grade wirjklich braucht und spricht diese mit mysql_fetch_assoc() und Index an:
SELECT id,name
... $seiten['id'] ...
... $seiten['name'] ...

Beim Nullten-Element wie hier ist die Wahrscheinlichkeit, dass du mal eine Spalte ganz vorne einfügst vergleichsweise gering. Bei großen Tabellen kommst du aber ins Schwimmen, z.B.

SELECT *
mysql_fetch_array()

if($row[23] == $eingabe AND $row[74] != $eingabe2) ...
Da blickst du nicht mehr durch.

__________________
Warum denkt mein Hund eigentlich immer dann, wenn es an der Tür klingelt, es sei für ihn?