Allgemeiner Fragenthread II

[Drakes]

Wieso sollte man in PHP array() verwenden? Sollte man es der Lesbarkeit zu liebe tun oder kann ich mir die Linien sparen? (mal ausgenommen man gibt die Variable in einer Funktion zurück und die Array könnte leer bleiben)

[Xardas der Dunkle]

Ganz simpel, in erster Regel ist die "Funktion" (ist eig. keine richtige Funktion, sondern ein festes Element) dafür da um einer Variable einen Anfangswert zu geben, bzw. ein Array einer Funktion zu übergeben:

PHP-Code:

tuWas(Array('wert', 'wer2', 'wert3')); 

Auch wenn in PHP Variablen nicht direkt deklariert werden müssen, brauchen gerade wenn man einem Array in einer Schleife Werte hinzufügen möchte diese ein Anfangswert der PHP mitteilt das die Variable auch existiert und das es sich um ein Array handelt.

PHP-Code:

$tmp = Array();
while($row = mysql_fetch_assoc($result)) {
      $tmp[] = $row;
} 

[Drakes]

Ganz simpel, in erster Regel ist die "Funktion" (ist eig. keine richtige Funktion, sondern ein festes Element) dafür da um einer Variable einen Anfangswert zu geben, bzw. ein Array einer Funktion zu übergeben:

PHP-Code:

tuWas(Array('wert', 'wer2', 'wert3')); 

Auch wenn in PHP Variablen nicht direkt deklariert werden müssen, brauchen gerade wenn man einem Array in einer Schleife Werte hinzufügen möchte diese ein Anfangswert der PHP mitteilt das die Variable auch existiert und das es sich um ein Array handelt.

PHP-Code:

$tmp = Array();
while($row = mysql_fetch_assoc($result)) {
      $tmp[] = $row;
} 

...

Ich weiss nicht, aber dein Beispiel hat bei mir auch ohne $tmp = array(); funktioniert. (bzw. ein konvergentes Beispiel)

[Xardas der Dunkle]

Tut es nur augenscheinlich.
Schreib mal ein error_reporting(E_ALL); an den vor die Schleife, du wirst eine "Notice"-Fehlermeldung bekommen.

Desweiteren bei dem Beispiel oben werden ja Daten aus der db eingelesen, wenn nun, aber keine gefunden werden, würde das Array $tmp gar nicht erst angelegt werden und wenn du es später dann später z.B. in einer foreach schleife abarbeiten willst, würdest du nicht nur eine E_NOTICE bekommen sondern eine E_WARNING, die besagt das foreach eben ein Array erwartet und nicht eine Variable vom Typ null.

[Tessio]

Huhu.
Nun, ich habe eine einfache Frage zu einem Query, es geht um folgenden :

PHP-Code:

$sql = "INSERT INTO `spieler` SET `y` = '".$_SESSION['y']."' WHERE `id` = '".$_SESSION['spieler_id']."'"; 

Beim Ausführen sagt der Browser mir folgendes :
You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'WHERE `id` = '6'' at line 1

Ich frage mich, warum er das sagt.
Vielleicht bin ich gerade blind und sehe den einfachen Fehler nicht, aber meine Augen zeigen mir, dass ich nach dem Einbinden der ID in den Query ein ' stehen habe und die anderen " als Abgrenzungen zum String gebraucht werden.
Wieso kriege ich diesen Fehler?

EDIT : Jaaa. INSERT mit UPDATE vertauscht xD

[Xardas der Dunkle]

Zudem niemals Variablen (Vorallem Request-Variablen) ungeparsed an n' Query anhängen.
Desweiteren sind die ' bei numerischen Werten unnötig .

[dead_orc]

Zudem niemals Variablen (Vorallem Request-Variablen) ungeparsed an n' Query anhängen.
Desweiteren sind die ' bei numerischen Werten unnötig .

...

Request-Variablen sind (neben $_REQUEST natürlich) $_POST, $_GET, $_COOKIE und $_FILES (sowie deren $HTTP_*_VARS Pendants). $_SESSION besteht nicht aus Daten, die der User übermittelt hat, daher sollte es kein zu großes Problem sein, sie direkt zu benutzen. Natürlich wäre folgendes sauberer:

PHP-Code:

$sql = sprintf("UPDATE `spieler` SET `y` = '%s' WHERE `id` = %d", mysql_real_escape_string($_SESSION['y']), $_SESSION['spieler_id']); 

Hier wird sichergestellt, dass $_SESSION['spieler_id'] eine Zahl ist und $_SESSION['y'] wird vorher mit mysql_real_escape_string() behandelt, um ' zu escapen. Aber wenn man davon ausgeht, dass die Variablen bereits beim Setzen verifiziert wurden, ist das unnötig. Denn wenn jemand die Session-Variablen, die aufm Server gespeichert werden, verändern kann, hat man noch ganz andere Probleme.

[Xardas der Dunkle]

Da man die MySQL-Api mit neueren MySQL-Versionen eh nicht mehr benutzen sollte, sondern die MySQLi API oder PDO, kann man ggf. auch auf Statemants zurück greifen und somit auf escaping verzichten:
http://de3.php.net/manual/de/mysqli.prepare.php
http://de3.php.net/manual/de/pdo.prepare.php

[Manni]

Request-Variablen sind (neben $_REQUEST natürlich) $_POST, $_GET, $_COOKIE und $_FILES (sowie deren $HTTP_*_VARS Pendants). $_SESSION besteht nicht aus Daten, die der User übermittelt hat, daher sollte es kein zu großes Problem sein, sie direkt zu benutzen.

...

Da sollte man sich nicht vertun. Ich hatte schon mal Probleme, weil ich die Request URI in der Datenbank speichern wollte, und die halt ein ' enthielt. Lieber absichern, als nachher Lücken offen haben

[WeTa]

Wie definiere ich in meiner .css Link-Farben (Hover, Active, Visited...) für einzelne DIVs? Die in #navi sollen andere Farben als die in #main haben.
Hab mal ein bisschen rumprobiert, bin aber nicht wirklich zu einem Ergebnis gekommen.

[Xardas der Dunkle]

Code:

#navi a:link {
    color:....;
}

#navi a:visited {
    ....
}

[...]

#main a:link {
    ....
}

[...]

Einfacher geht es eig. nicht^^. Nur meine ich das du bei den pseudo-classen eine feste Reihenfolge einhalten musst, wenn nicht habe ich Blödsinn gelernt xD.

/edit: *zurück edetier*

[Drakes]

Wie definiere ich in meiner .css Link-Farben (Hover, Active, Visited...) für einzelne DIVs? Die in #navi sollen andere Farben als die in #main haben.
Hab mal ein bisschen rumprobiert, bin aber nicht wirklich zu einem Ergebnis gekommen.

...

a:hover meint Xardas 1mal noch.

Edit: sorry, steht ja #main.

[DFYX]

a:hover meint Xardas 1mal noch.

...

Ich editier das mal schnell rein...

[Xardas der Dunkle]

*verwirrt bin*

Die in #navi sollen andere Farben als die in #main haben.

...

Fragt er, wieso ändert ihr dann das #main a:link auf #main a:hover o_Ô?
Das war doch nur ein Beispiel.

Wenn ihr es so genau haben wollt:

Code:

#navi a:link {
    ...
}

#navi a:active {
   ...
}

#navi a:visited {
   ...
}

#navi a:hover {
   ...
}

#main a:link {
    ...
}

#main a:active {
   ...
}

#main a:visited {
   ...
}

#main a:hover {
   ...
}

[DFYX]

Sorry, ich schätze, wir haben uns beide verlesen

[WeTa]

Vielen Dank

[Mugen]

Hab da noch mal 'ne Frage, die nicht unbedingt einen eigenen Thread benötigt, imo.

Hat vllt. irgendwer 'nen Plan, warum der IE (7. 6, denke ich, dann wohl auch) diese Seite falscht anzeigt?
Also der BG wird etwas nach unten verschoben, 1, 2 Pixel.

[Whiz-zarD]

erstmal:
Du arbeitest mit tranparenten png Dateien.
Der IE unterstützt sowas nicht.

Und wenn nur das Deisgn um 1,2 Pixel nach unten verschoben ist, solltest du dich freuen und dies so lassen.
Es ist allgemein bekannt, dass der IE bei CSS Mist baut.

Ansonsten versuch mal im Body
padding:0px;
margin:0px;
einzutragen. AFAIR benutze der IE einen Standardrahmen um die Webseite.

[Drakes]

erstmal:
Du arbeitest mit tranparenten png Dateien.
Der IE unterstützt sowas nicht.

Und wenn nur das Deisgn um 1,2 Pixel nach unten verschoben ist, solltest du dich freuen und dies so lassen.
Es ist allgemein bekannt, dass der IE bei CSS Mist baut.

Ansonsten versuch mal im Body
padding:0px;
margin:0px;
einzutragen. AFAIR benutze der IE einen Standardrahmen um die Webseite.

...

IE 7 unterstützt zum Glück transparente png-Bilder.

Code:

* {
   margin: 0;
   padding: 0;
   }

Ist meinerseits eigentlich immer empfohlen. Manche Browser setzen zudem automatisch Rahmen um <img>'s.(Das ist aber OT)

Und eigentlich ist der Allgemeine Fragethread dafür gedacht nicht extra ein Thema eröffnen zu müssen.

Edit:
probiers doch bei body mit

Code:

background: url(../images/bg.jpg) top left repeat;

Edit2:
War das falsche Hintergrundbild, müsste wohl eher bei .header_left sein:

Code:

background: url(../images/bg.jpg) top;

Aber ich hab nicht das Gefühl, dass das wirklich das Problem ist.

[Drakes]

Gibt es etwas zu beachten, damit sich dynamisch generierte div's im IE voll ausbreiten? Das obere Fenster ist statisch und wird ja eigentlich auch richtig angezeigt. Ich hab mal Rahmen rein gemacht zur Veranschauung:
http://drakes.dr.ohost.de/pz/ (man muss die Werbung wegklicken und dann neuladen -.-)
Zudem wär es toll wenn mir jemand sagen könnte was der Opera zu meckern hat, ich verstehs nicht.