PHP: Geht das noch besser?

Donkey

Hallo zusammen!

Nur ne kurze Frage. Hab eine Seite erstellt auf der man Internetradio hören kann - nichts Aufregendes eigentlich.

Da soll halt auch angezeigt werden welchen Sender man zuletzt gehört hat - das funzt auch.
Wollte jetzt nur wissen ob man das noch ein bissl besser/kompakter schreiben kann.

Hier gehts zur Testseite: http://kendo-bunker.de/donkey/demo/music.php

und das is der php-Code:

PHP

<?php
$referrer = $HTTP_REFERER;
$pfad = 'http://'.$_SERVER['HTTP_HOST'];
$sender = 'kein Sender ermittelt';
if ($referrer == $pfad.$_SERVER['PHP_SELF'].'?trance'){
$sender = 'Trance';
}
if ($referrer == $pfad.$_SERVER['PHP_SELF'].'?eurodance'){
$sender = 'Eurodance';
}
if ($referrer == $pfad.$_SERVER['PHP_SELF'].'?house'){
$sender = 'House';
}
if ($referrer == $pfad.$_SERVER['PHP_SELF'].'?harddance'){
$sender = 'Harddance';
}
if ($referrer == $pfad.$_SERVER['PHP_SELF'].'?goa'){
$sender = 'Goa/Psy';
}
 
 
...
...
...
 
 
Du hast zuletzt gehört: <b>'.$sender.'</b><a href="'.$referrer.'" class="navi">Wieder anhören</a><br>
 
...
...
...
 
?>

Alles anzeigen

Gruß, Donkey

Dragon-D

hast du schonmal von elseif gehört?
wenn du nur if verwendest prüft er jede anweisung...

gruß

Bandit

Warum ist es schlecht, mit dem Referer zu arbeiten?
http://www.php-faq.de/q-http-referer.html

Dieser Thread gehört doch wohl eher nach "Forum für PHP - MySQL - CGI - Perl - JavaScript und Co"

pApAnoAh

Du könntest doch einen Cookie setzen?
http://tut.php-quake.net/de/cookies.html

d4f

Ich würds spontan mit switch machen.
break; nicht vergessen, da er ansonsten alle weiteren Cases abklappern geht wenn er schon eine Lösung gefunden hat.

Code

$path = $pfad[COLOR=#000000][COLOR=#006600].[/COLOR][COLOR=#0000cc]$_SERVER[/COLOR][COLOR=#006600][[/COLOR][COLOR=#cc0000]'PHP_SELF'[/COLOR][COLOR=#006600]][/COLOR][/COLOR];
switch($_SERVER['HTTP_REFERER']) {
  case $path.'?trance':
    $sender = 'Trance',
    break;
  case $path.'?eurotrance':
     [usw....]
  default:
     $sender = 'Kein Radio';
}

So oder so muss er auch bei Cookies usw einen Check machen, wenn er Defacements ausschliessen will. Nur $_SESSION-Variablen könnten diese Kontrolle abschaffen, allerdings funzt das nur wenn der RadioSender respektiv die Lade-Seite bei ihm auf dem Server liegt.

Bandit

Was soll der ganze Quatsch mit $path und Referer????

PHP

<?php
   switch($_SERVER['QUERY_STRING'])
   {
      case 'trance'      : $sender = "Trance"; break;
      case 'eurotrance': $sender = "Eurotrance"; break;
      default: $sender =""; break;
   }
?>

d4f

Ohne dich kritisieren zu wollen, aber du versuchst den aktuellen QueryString auszulesen.
Er will aber den aus seinem Referer

Ein Aufputzen des $pfad ist da nur mit schnippeln vor dem switch() möglich

Donkey

Vielen Dank für die Antworten.
Das hat mir alles sehr geholfen

Ich bin den Tipp von d4f gefolgt

PHP

$referrer = $_SERVER['HTTP_REFERER'];
$path = 'http://'.$_SERVER['HTTP_HOST'].$_SERVER['PHP_SELF'];
switch($referrer) {
case $path.'?trance':
$sender = 'Trance';
break;
case $path.'?eurodance':
$sender = 'Eurodance';
break;
case $path.'?house':
$sender = 'House';
break;
case $path.'?harddance':
$sender = 'Harddance';
break;
case $path.'?goa':
$sender = 'Goa/Psy';
break;
default:
$sender = 'Kein Sender';
}

Alles anzeigen

Kann die "zuletzt gehört" Zeile ja ausblenden lassen wenn kein Referrer ermittelt werden kann oder wenns einer is der nichts mit der Seite zu tun hat...

So in der Art vielleicht

PHP

$last = 'Du hast zuletzt gehört: <b>'.$sender.'</b><a href="'.$referrer.'" class="navi">Wieder anhören</a><br>';
if ($referrer == NULL){
echo '&nbsp;';
}
else{
echo $last;
}

oder so ähnlich....

Mit der Materie der Cookies hab ich mich noch garnicht beschäfftigt und hab dem zufolge kein Plan wie man das macht

Danke nochmal
Gruß, Donkey

EDIT:

So habs jetzt endgültig gelösst - manchmal wirkt logisches Denken Wunder

Da ja ohne den entsprechenden Referrer kein Sendername angezeigt wird
Hab ich einfach

PHP

default:
$sender = 'Kein Sender';

entfernt und dann $sender als Bedingung für die Anzeige genommen

PHP

if ($sender == NULL){
echo '&nbsp;';
}
else{
echo 'Du hast zuletzt gehört: <b>'.$sender.'</b><a href="'.$referrer.'" class="navi">Wieder anhören</a><br>';
}

Bandit

Zitat von Donkey

Ich bin den Tipp von d4f gefolgt

Gratulation! Damit hast du wesentliche Verbesserungsvorschläge in den Wind geschrieben und hast weiterhin ein Script, das nicht unbedingt richtig funktionieren muss. Aber was soll's, wenn du dir Mühe gibst, wirst du dein Script bestimmt noch schlechter hinbekommen.

d4f

Zitat

Damit hast du wesentliche Verbesserungsvorschläge in den Wind geschrieben und hast weiterhin ein Script, das nicht unbedingt richtig funktionieren muss.

*Troll* Und das kommt von jemandem der in seinem Beispiel Referer mit Query-String verwechselt.

Ich habe ihm lediglich die Frage beantwortet, wie er das Script, das Referer übeprüft optimieren kann.
Auf die obenstehende Vorschläge (die übrigens nicht von dir kommen) bez. Cookies und Session bin ich nicht eingegangen, da dies ein völlig weiteres Thema ist.

Wie man es mit Sessions löst:

a) auf jeder RadioSeite folgenden Code unterbringen

PHP

<?php 
session_start();
$_SESSION['radio'] = 'DIESE_SEITE';
?>

Dann auf der Hauptseite folgenden Code:

PHP

<?php
session_start();
if(isset($_SESSION['radio'])) {
  echo "Sie haben als letzes ".$_SESSION['radio']." gehört.";
}
else {
 echo "Sie haben kein Radio gehört.";
}

Jetzt zufrieden? *grummel*

Bandit

Zitat von d4f

*Troll* Und das kommt von jemandem der in seinem Beispiel Referer mit Query-String verwechselt.

Genau, ich bin der Obertroll, dem allerdings klar ist, dass man mit dem Referer nicht verlässlich arbeiten kann!

Zitat von d4f

Ich habe ihm lediglich die Frage beantwortet, wie er das Script, das Referer übeprüft optimieren kann.

Falls es dir nicht aufgefallen ist: ich habe nicht dich kritisiert, sondern nur auf die Problematik bzgl. Referer hingewiesen!!!

Zitat von d4f

Auf die obenstehende Vorschläge (die übrigens nicht von dir kommen) bez. Cookies und Session bin ich nicht eingegangen, da dies ein völlig weiteres Thema ist.

Korrekt, ich bin ebenfalls nicht auf Cookies und Sessions eingegangen. Warum auch?

Zitat von d4f

Jetzt zufrieden?

War ich immer! Wie schon erwähnt, ich habe nur darauf hingewiesen, dass mit dem Referer zu arbeiten einfach Mist ist!! Also erst die Antworten richtig lesen und dann (falls nötig) meckern!

d4f

Ich meckere nicht
Troll war auf mich bezogen, nicht dich

Jetzt geh ich aber ins Brett. Schlaf gut

Donkey

Hallo bandit600,

ich hab natürlich auch Deinen Hinweis zum Referrer gelesen, aber da in meinem Fall keine wirklich wichtige Funktion vom Referrer abhängig is passt das schon so.
Und ein konkreter Vorschlag is von Dir ja wirklich nicht gekommen - warum also der Wirbel jetzt?

Naja is ja auch egal jetzt...

Gruß Donkey

Bandit

Zitat von Donkey

Und ein konkreter Vorschlag is von Dir ja wirklich nicht gekommen - warum also der Wirbel jetzt?

Ohh, anscheinend hat mein Versuch geklappt, ein Script ins vBulletin zu schmuggeln:

PHP

<?php
   if ($nickName != 'bandit600')
      hidePosting('#6');
?>

DINGDONG. jemand zu Hause???

d4f

PHP

<?php
   if ($nickName == 'bandit600')
      hidePosting('#7');
?>

Bandit

Warum ist es schlecht, mit dem Referer zu arbeiten?
http://www.php-faq.de/q-http-referer.html

Tags