mehrere if = geht nich??

lippl

heyho
also ich hab n problem, ich hab für mein gbook mid mysql ich möchte für e-mail iicq... symbole ausgeben aber natürlich nur wenn auch eine e-mailadresse aneggeben is....nun habe ich mehrere ifs... wenn ich duiese weglasse wird alles ausgegeben wie ich möchte aber wenn ich die ifs reinsetzte wird nur 1 davon angezeigt??
ty für die hilfe schomma im vorraus

Code

if($row->email == "")
       {
       $email = "";
       }
    else
       {
       //Der User hat eine Email Adresse angegeben:
       $email = "<a href=\"mailto:$row->email\"><img src=\"images/email_icon.png\" width=\"19\" height=\"15\" border=\"0\"/></a>"; }
if($row->icq == "")
       {
       $icq = "";
       }
    else
       {
       //Der User hat eine ICQ Adresse angegeben:
       $icq = "<a href=\"http://www.icq.com/scripts/search.dll?to=$row->icq\" target=\"_blank\"><img src=\"images/icq_icon.png\" width=\"15\" height=\"15\" border=\"0\" /></a>"; }
if($row->hp == "")
       {
       $hp = "";
       }
    else
       {
       //Der User hat eine HP Adresse angegeben:
       $hp = "<a href=\"$row->hp\" target=\"_blank\"><img src=\"images/hp_icon.png\" width=\"17\" height=\"15\" border=\"0\" /></a>"; }
//Ausgabe
    {echo "
   <center><table cellpadding=\"0\" cellspacing=\"0\" width=\"100%\">
  <tr class=\"guestbooktitle\">
    <td>Von $row->name</td>
	<td>$email $icq $hp</td>
	<td align=\"right\">am $row->datum</td>
  </tr>
    <td align=\"left\" valign=\"top\">$row->text</td>
  </tr>
</table></center>";}

Alles anzeigen

Gast

ich hab zwar noch nciht so die ahnung von PHP, aber ich sehe, dass du if ja garnicht definierst. An deiner Stelle würde ich mit CASE arbeiten:

Zitat

<?php
switch($site)
{
case news:
$content = "news.php";
break;
case about:
$content = "about.php";
break;
case links:
$content = "links.php";
break;
case kontakt:
$content = "kontakt.php";
break;
case imprint:
$content = "imprint.php";
break;
case guestbook:
$content = "gb.php";
break;
default:
$content = "news.php";
break;
}
?>

Alles anzeigen

CASE ist zwar nur für Content gedacht, aber dafür lässt es sich bestimmt auch missbrauchen^^
$row->email = $site (kannste natürlich nach belieben ändern - ich hatte nur gerade keine lust site in email umzutippen

Ich hoffe ich hab dir geholfen...

driver

doch, macht er. schaumal genau hin !

The User

Marcel, die breaks müssen weg, sonst macht er nur das erste.
Am Ende dann default: statt case "xy": und nur da break;.

Viele liebe Grüße
The User

Bandit

switch ist in dem falle sinnlos, genauso wie das weglassen von breaks beim switch. Ich habe das chaos-script mal etwas formatiert, damit der quellcode lesbar wird:

PHP

<?php
  if($row->email == "") 
  { 
     $email = ""; 
  } 
  else 
  { 
     //Der User hat eine Email Adresse angegeben: 
     $email = "<a href='mailto:$row->email'>
                [img]images/email_icon.png[/img]
              </a>"; 
  } 

  if($row->icq == "") 
  { 
    $icq = ""; 
  } 
  else 
  { 
     //Der User hat eine ICQ Adresse angegeben: 
     $icq = "<a href='http://www.icq.com/scripts/search.dll?to=$row->icq' target='_blank'>
              [img]images/icq_icon.png[/img]
             </a>"; 
  } 

  if($row->hp == "") 
  { 
     $hp = ""; 
  } 
  else 
  { 
    //Der User hat eine HP Adresse angegeben: 
    $hp = "<a href='$row->hp' target='_blank'>
            [img]images/hp_icon.png[/img]
           </a>"; 
  } 

  //Ausgabe 
  echo "<center>
          <table cellpadding='0' cellspacing='0' width='100%'> 
            <tr class='guestbooktitle'> 
              <td>Von $row->name</td> 
              <td>$email $icq $hp</td> 
              <td align='right'>am $row->datum</td> 
            </tr> 
             <td align='left' valign='top'>$row->text</td> 
            </tr> 
          </table>
        </center>";
?>

Alles anzeigen

da ist nichts falsch dran.

Gruß
bandit

also auch die geordnete version hat nichts gebracht=(
un sry dass mid dem case zeugs hab ich nich geplant?? aber driver meint ja das brauch ich nich..zumindest liegt dort nicht der fehler=)

Bandit

An diesem teil des scriptes liegt es nicht.

Gruß
bandit

lippl...

na dann bekommd ihr ebn das ganze:

PHP

<!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">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1" />
<title>News</title>
</head>


<body>
<table cellpadding="0" cellspacing="0" align="center" width="600">
  <tr>
    <td><div style="margin-left:150px;"><form method="post" action="gbsend.php">



Name*:


          <input name="name" size="40" type="text" />



        E-Mail:


        <input name="email" type="text" size="40" value="" />


		Homepage:


        <input name="hp" type="text" size="40" value="" />



        ICQ
        :


        <input name="icq" type="text" size="40" value="" />
      </p>



 Text*:


          <textarea name="text" cols="40"
 rows="12" wrap="physical" type="text"></textarea>



          <input name="submit" type="submit" value="Absenden" />
          <input name="reset" type="reset" value="Zurücksetzen" />


      </p>
      </form></div></td>
  </tr>
  <tr>
    <td><?php
$seite = $_GET["seite"];  //Abfrage auf welcher Seite man ist


//Wenn man keine Seite angegeben hat, ist man automatisch auf Seite 1
if(!isset($seite))
   {
   $seite = 1;
   }


//Verbindung zu Datenbank aufbauen


$link = mysql_connect("localhost","lippl1","passwort^^") or die ("Keine Verbindung moeglich");
mysql_select_db("lippl1") or die ("Die Datenbank existiert nicht");




//Eintr&auml;ge pro Seite: Hier 15 pro Seite
$eintraege_pro_seite = 5;


//Ausrechen welche Spalte man zuerst ausgeben muss:


$start = $seite * $eintraege_pro_seite - $eintraege_pro_seite;


//Tabelle Abfragen
//Tabelle hei&szlig;t hier einfach: Tabelle
$abfrage = "SELECT * FROM guestbook ORDER BY datum DESC LIMIT $start, $eintraege_pro_seite";
$ergebnis = mysql_query($abfrage);
while($row = mysql_fetch_object($ergebnis))


//Überprüfung


if(empty($row->email))
  {
     $email = "";
  }
  else
  {
     //Der User hat eine Email Adresse angegeben:
     $email = "<a href='mailto:$row->email'>
                [img]images/email_icon.png[/img]
              </a>";
  }
 
  if(empty($row->icq))
  {
    $icq = "";
  }
  else
  {
     //Der User hat eine ICQ Adresse angegeben:
     $icq = "<a href='http://www.icq.com/scripts/search.dll?to=$row->icq' target='_blank'>
              [img]images/icq_icon.png[/img]
             </a>";
  }
 
  if(empty($row->hp))
  {
     $hp = "";
  }
  else
  {
    //Der User hat eine HP Adresse angegeben:
    $hp = "<a href='$row->hp' target='_blank'>
            [img]images/hp_icon.png[/img]
           </a>";
  }
 
  //Ausgabe
  echo "<center>
          <table cellpadding='0' cellspacing='0' width='100%'>
            <tr class='guestbooktitle'>
              <td>Von $row->name</td>
              <td>$email $icq $hp</td>
              <td align='right'>am $row->datum</td>
            </tr>
             <td align='left' valign='top'>$row->text</td>
            </tr>
          </table>
        </center>";




//Jetzt kommt das "Inhaltsverzeichnis",
//sprich dort steht jetzt: Seite: 1 2 3 4 5




//Wieviele Eintr&auml;ge gibt es &uuml;berhaupt


//Wichtig! Hier muss die gleiche Abfrage sein, wie bei der Ausgabe der Daten
//also der gleiche Text wie in der Variable $abfrage, blo&szlig; das hier das LIMIT fehlt
//Sonst funktioniert die Bl&auml;tterfunktion nicht richtig,
//und hier kann nur 1 Feld abgefragt werden, also id


$result = mysql_query("SELECT id FROM news");
$menge = mysql_num_rows($result);


//Errechnen wieviele Seiten es geben wird
$wieviel_seiten = $menge / $eintraege_pro_seite;


//Ausgabe der Seitenlinks:
echo "<div align=\"center\">";
echo "[b]Seite:[/b] ";




//Ausgabe der Links zu den Seiten
for($a=0; $a < $wieviel_seiten; $a++)
   {
   $b = $a + 1;


   //Wenn der User sich auf dieser Seite befindet, keinen Link ausgeben
   if($seite == $b)
      {
      echo "  [b]$b[/b] ";
      }


   //Aus dieser Seite ist der User nicht, also einen Link ausgeben
   else
      {
      echo "  <a href=\"?seite=$b\">$b</a> ";
      }




   }
echo "</div>";
?></td>
  </tr>
</table>
</body>
</html>

Alles anzeigen

un die gbsend.php:

PHP

<?php
$link = mysql_connect("localhost","lippl1","passwort^^") or die ("Keine Verbindung moeglich");
mysql_select_db("lippl1") or die ("Die Datenbank existiert nicht");
?>
          <?php 
$name = $_POST["name"];
$text = $_POST["text"];
$email = $_POST["email"];
$icq = $_POST["icq"];
$hp = $_POST["hp"];


$datum = date("d.m.Y H:i");


if($name == "" OR $text == "")
    {
   echo "* die Felder \"Name\" und \"Text\" sind Pflicht
";
   exit; //Script Ablauf wird unterbrochen, Eintrag wird nicht gespeichert
   }


$eintrag = "INSERT INTO guestbook (name, text, email, hp, icq, datum) VALUES ('$name', '$text', '$email', '$hp', '$icq', '$datum')";
$eintragen = mysql_query($eintrag);
?>
          <?php
if($eintragen == true)
   {
   echo "<a href=\"index.php?site=guestbook\">Beitrag erfolgreich gespeichert</a>";
   }
else
   {
   echo "<a href=\"javascript:history.back()\">Fehler beim Speichern</a>";
   }


 ?>

Alles anzeigen

Bandit

Da fehlen klammern bei der while-schleife. Wenn man ordentlich einrücken würde, würde man das selber erkennen.

Gruß
bandit

wohoo yeahh THX das war wirklich der fehler..

echd nochmal vieln dank

Tags