{<?PHP}Notice:Undefined Index

Es ist ein leichter Fehler, eben eine Notiz, welche dadurch hervorgerufen wird,
weil er versucht die $color in Zeile 2
zu interpretieren. Was aber nicht machbar ist,
da hier $_POST['color'] noch gar nicht gesetzt ist.

Daher solltest du am Anfang deine $color-Variable mit einem Leerstring definieren.
//Virtual

<?php 
$color = '';
$farbe = '';
if (isset( $_POST['color'])) 
{ 
	$color = htmlspecialchars($_POST['color']);
    switch($color) 
    { 
        case '1': 
        $farbe = '#FFE500'; 
        break; 


        case '2': 
        $farbe = '#00FF00'; 
        break; 


        default: 
        $farbe = '#FFFFFF'; 
        break; 
    }
	
} 






 ?> 
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd"> 
<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="de" lang="de"> 


<head> 
    <title>First</title> 


    <meta http-equiv="content-type" content="text/html; charset=ISO-8859-1" /> 
    <meta name="description" content="" /> 
    <meta name="author" content="" /> 
    <meta name="keywords" content="" /> 
    <meta name="generator" content="Webocton - Scriptly (www.scriptly.de)" /> 


    <link href="styles/style.css" type="text/css" rel="stylesheet" /> 
    <link href="favicon.ico" type="image/x-icon" rel="shortcut icon" /> 
</head> 


<body bgcolor="<?php echo $farbe; ?>"> 
<div id="website"> 
        <div id="header"> <h1>Website</h1></div> 


        <div id="main"> 
<div id="left"> 
<form action="lallo.php" method="post" > 
 Hintergrundfarbe: 
 <select name="color"> 
    <option value="1">Gelb</option> 
    <option value="2">Gr&uuml;n</option> 
    <option value="" selected="selected">Keine Auswahl</option> </select> 
    <input type="submit" value="Farbe wechseln" /> 
</form> 
</div> 
            <div id="right"></div> 
            <div id="footer"> 
            © - by Lalox3, All rights reserved.</div> 
        </div> 


    </div> 
</body> 
</html>

Erklärung: Zu erst solltest du beide Variablen $color und $farbe deklarieren.
Dann solltest du nicht abfragen, ob $color, sondern ob $_POST['color'] gesetzt ist, was der Fall ist, wenn du auf Farbe wechseln drückst.

Hier wird der Wert an $color erst übergeben und die Farbe kann ruhig gewechselt werden.

Auch dann erhält $farbe einen Wert, zuvor war diese Variable undeklariert, daher $farbe = '' am Anfang.

Mit freundlichen Grüßen
Virtual

Versuche dir im Allgemeinen immer zu überlegen:
Welche Variablen habe ich in Nutzung und welche Variablen haben in welchem Zustand welchen Wert?

Dein Beispiel

Zustand 1: Seite wird erst aufgerufen. //Zustand 1 trifft immer zu
-> $color = ''; //Wären diese Variablen nicht gesetzt, ergibt es immer eine Notiz. 
                //(sofern error_reporting nicht anders gesetzt.)


-> $farbe = '';






Zustand 2: Formular wird gesendet. //Zustand trifft erst nach Interaktion durch den User zu.
-> $color = $_POST['color'];
-> $farbe = HEX-Wert;

Es kommt auf das Gleiche dabei heraus, ja.
Schön, dass dir ein Beitrag dazu eingefallen ist.

Aus Gewohnheit, auch später in der objektorientierten Variante,
deklariere ich meine Variablen mit "null" [sofern kein Wert benötigt wird.].
=>

public $variable = null;

Typensicherheit ist in PHP meiner Meinung nach weniger von Bedeutung,
da der Typ sich jederzeit beliebig ändern kann.
In anderen Sprachen, wie (zumindest glaube ich es) C++ oder Java, ist es doch schon sinnvoller,
doch ist dies ein anderes Thema.
Wie gesagt, es ist reine Gewohnheit.
Ich habe es gerne einheitlicher.
Es sieht für mich komisch aus, wenn 1-2 Variablen mit Werten deklariert sind
und andere "nackt" dastehen. ;)

Damit auch jeder weiß, was du mit deinem ersten Beitrag gemeint hast,
habe ich hierzu ein Beispiel ohne die Switch-Methode.

<?php
$color = '';
if(isset($_POST['color']))
{
	$color = $_POST['color'];
}
?>


<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"  "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd"> 
<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="de" lang="de"> 
<head> 
    <title>First</title>  
</head> 
<body bgcolor="<?php echo $color; ?>"> 
	<form action="background.php" method="post" > 
		Hintergrundfarbe: 
		<select name="color"> 
			<option value="#FF0000">Gelb</option> 
			<option value="#00FF00">Gr&uuml;n</option> 
			<option value="" selected="selected">Keine Auswahl</option>
		</select> 
		<input type="submit" value="Farbe wechseln" /> 
	</form> 
</body> 
</html>

Ich habe nicht behauptet, es hätte keine Bedeutung, sondern eine eher geringe Bedeutung.
Sofern du nicht vorher den Typ bestimmst (, welcher auch durch Funktionen vorgegeben werden kann ),
wird auch keine "Notice" oder sogar ein "Fatal Error" ausgeworfen.
Ich arbeite stetig mit "E_ALL und STRICT" ;).

Doch wir kommen vom Thema ab.
Nicht, dass wir beide noch als Spammer gezählt werden :).

//Virtual

Ulle,
die Übung ist hier nicht gewesen, dass es auch über weiteres hinweg gespeichert wird,
sondern, dass es überhaupt funktioniert.

Wir wollen nicht direkt am Anfang mit der Tür ins Haus fallen.
Selbstverständlich könnte man noch einiges erweitern, wenn du schon diesen Einwand hast.
Ob nun Cookie,Session, oder gar in der Datenbank speichern.
Alles wäre möglich, doch zu viel und es würde das Anliegen des Threaderstellers übersteigen :).

Wenn du schon soweit gehst, könnten wir direkt mit Sicherheit und anderen Erweiterungen fortfahren,
bezüglich einer Sitzungsverwaltung, was jedoch zu weit führen würde.
Daher doch einfach nur dem einfachen Wunsch von Lalox3 nachkommen ;).

//Virtual

Alles kein Ding,
immerhin hast du ihn wahrscheinlich auf den Gedanken gebracht,
als nächstes mit $_SESSION zu üben, worauf er eventuell erst viel später gestoßen wäre.

Somit brauchst du dich keineswegs entschuldigen. Es ist ein Hinweis auf die nächsten Kapitel gewesen. :D

// Virtual