Anfängerproblem

akiii

Hallo Community,

ich sitze jetzt schon 2 Stunden an diesem Problem und komme nich weiter -_-

Vll ist einer von euch so nett und hilft mir mein Problem zu lösen...

Ich weiss nicht ob dass hier erlaubt ist einfach den code reinzuposten..
falls es ein Vergehen ist bitte ich um Sorry

Es ist ein php dokument und 2 css .. im php dokument ist eine selectbox wo man zwischen den 2 css dateien wechseln kann...

Zitat

<?php
$link = $_GET['style'];
if ($link == Yellow)

{
echo "<link rel='stylesheet' type='text/css' href='red.css'>";
}
else
{
echo "<link rel='stylesheet' type='text/css' href='yellow.css'>";
}

echo"
<html>
<head>";

echo "
<title>Beispiel Dunic</title>";

echo "
<script type='text/javascript'>

</script>

</head>";
echo "<body>";

echo "
<div align='center'>
<h1>Sammelbeispiel</h1>
</div>
";

echo "
<div id='main_menu'>
<form name='formular' method='get' action='$PHP_SELF'>
<p>
<br>
Choose your style:
<br>
<select name='style' size='1' >
<option name='style2'>YELLOW</option>
<option name='style3'>RED</option>

</select>
<br>

<input type='button' value='GO!' name='submity' onclick='send()'>
<br/>
</p>
</form>
</div>";

echo "<div class='inhalt'>";
echo "<p><img src='1.jpg' width='200' height='200'></p>";
echo "
ausgewaehltes style:<br><b>
";

if ($_GET['style'] != "")
echo $_GET['style'];

echo "</b></p>";
echo "</div>";

echo "</body>";
echo "</html>";

?>

Alles anzeigen

css1:

Zitat

body
{
background-color: red;
}

#main_menu
{
background-color: red;
float: left; width: 10em; height: 22em;
margin: 0; padding: 0;
border: 1px dashed silver;
text-align: center;
}

.inhalt {
margin-left: 10em;
padding: 0 1em;
border: 1px solid silver;
}

p
{
font-family: "Arial";
font-size: 20px;
}

Alles anzeigen

css2

Zitat

body
{
background-color: yellow;
}

#main_menu
{
background-color: yellow;
float: left; width: 10em; height: 22em;
margin: 0; padding: 0;
dashed silver;
text-align: center;
position: static;
}

.inhalt {
margin-left: 10em;
padding: 23 ;
border: 55px solid black;
}

p
{
font-family: "Arial";
font-size: 40px;
}

Alles anzeigen

threadi

Da du es nicht geschrieben hast was das Problem ist, lass mich raten: der Style-Wechsel funktioniert nicht und es wird immer yellow.css genutzt oder ein php-Fehler erzeugt.

Der Grund:
Du vergleichst falsch in PHP:

Code

if ($link == Yellow)

Da Du einen Stringvergleich machst musst Du den Wert mit Anführungszeichen umgeben

Code

if ($link == "Yellow")

oder (besser noch) gleich einen Stringvergleich korrekt implementieren:

Code

if (strcmp($link,"Yellow") == 0)

akiii

Hallo,

if ($link == "Yellow")

hab das jetzt so gemacht und es geht noch immer nicht.. es bleibt einfach beim yellow style und will nicht auf red switchen :&

driver

mach des echo mal im head, und nicht vorm html-tag

Bandit

Anmerkungen:
Nur weil man kilometerlangen HTML-Code per echo ausgibt, wird dieser nicht schöner
An den Anfang des Scriptes diese Zeilen einbauen:

PHP

error_reporting(E_ALL);
ini_set("display_errors", true);

$PHP_SELF ist veraltet

Dieser Thread gehört eigentlich in den PHP-Bereich

Pion

Zitat von bandit600

$PHP_SELF ist veraltet

Anmerkung2 :

Zudem unsicher also hier ganz auf PHP_SELF verzichten bzw sichern

mfg

Afrael

Zitat

if ($_GET['style'] != "")
echo $_GET['style'];

Selbstverständlich hast du vorher sichergestellt, dass $_GET['style'] nur das ist, was man auswählen kann, und garantiert keinen HTML-Code enthält, oder?

Dodo

Zitat von Afrael

Selbstverständlich hast du vorher sichergestellt, dass $_GET['style'] nur das ist, was man auswählen kann, und garantiert keinen HTML-Code enthält, oder?

Das Ganze wurde von einem Afänger - wie der Thread schon sagt - geschrieben.
Außerdem denke ich, dass solche Sicherheitsüberprüfungen in diesem Code nicht notwendig sind...

Ich denke diese Hilfe wird ihn - als Anfänger - nur verwirren...

crAzywuLf

Hi akii,
wie schon gesagt wurde scheint das Problem an den Anführungszeichen oder der Implementierung überhalb des <html> Tags zu liegen. Ich habe dir deine Version mal ein wenig verändert, da es auch nicht nötig ist, dass du die ganze Datei in PHP ausgibst

PHP

<html>
<head>
<title>Beispiel Dunic</title>
<script type="text/javascript">
<!--
function send()
{
    document.formular.submit();
}
//-->
</script>
<?php
$link = $_GET['style'];
if ($link == "Yellow")
{
    echo '<link rel="stylesheet" type="text/css" href="red.css">';
}
else
{
    echo '<link rel="stylesheet" type="text/css" href="yellow.css">';
}
?>
</head>
<body>


<div align="center">
    <h1>Sammelbeispiel</h1>
</div>


<div id="main_menu">
    <form name="formular" method="get" action="<?php echo $_SERVER['PHP_SELF']; // nicht sinnvoll, schreib hier lieber den Dateinamen hin ?>">
    <p>
        <br>
        Choose your style:
        <br>
        <select name="style" size="1" >
            <option name="style2">YELLOW</option>
            <option name="style3">RED</option>
        </select>
        <br>
        <input type="button" value="GO!" name="submity" onclick="send()">
        <br/>
    </p>
    </form>
</div>




<div class="inhalt">
    <p><img src="1.jpg" width="200" height="200"></p>


    ausgewaehltes style:<br>
    <b>
<?php
if ($_GET['style'] != "") echo $_GET['style']; // Sicherheitslücke, hier kann der Benutzer mit HTML oder javascript die Seite manipulieren
?>
    </b>
</div>


</body>
</html>

Alles anzeigen

Mich wundert an dem Code nur, dass wenn man Yellow auswählt, eine red.css geladen wird? Ist das so richtig?
Wie du siehst sind auch Einrückungen ganz sinnvoll
Und wenn du echo's mit dem Hochkomma startest brauchst du innerhalb des Codes die Anführungszeichen nicht in Hochkommas zu verändern (oder escapen) und falls doch mal ein Hochkomma vorkommt dann escapst du es einfach so \'

Tags

Benutzer online in diesem Thema