[PHP] Bannerrotation

[Berti]

Hi wieso erhalte ich bei folgendem Scirpt eine Fehlermeldung?

 /**
 * @author     Adalbert Rohr <ad.rohr@schulbuch-und-co.de>
 * @copyright  2006-2007 Wirtschaftsprojekt JTG
 * @desc       Erstellt eine Bannerrotation
 * @uses       config.php
 */ 
function banner($position)
{
    require("config.php");
    $link = mysql_connect($mysql_host, $mysql_user, $mysql_pass);
    mysql_select_db($mysql_data, $link);
    $sql = "SELECT
               `id`,
               `clickurl`,
               `imageurl`,
               `name`,
               `banner_count`,
               `banner_max`
           FROM
               `bero_banner`
           WHERE
               `check` = '1'
           AND
               `banner_count` <= `banner_max`
           AND
               `position` = '". $position ."'";
    $result = mysql_query($sql);

    while($row = mysql_fetch_assoc($result))
        $banner[] = $row;

    $rand = rand (0, mysql_num_rows($result)-1);
    $url  = $banner[$rand]['clickurl'];
    $id   = $banner[$rand]['id'];
    $name = $banner[$rand]['imageurl'];
    $path = $banner[$rand]['id'];

    print "<a href=\"". $url ."\" target=\"_blank\">\n";
    print "<div id=\"promotion-field-right\">\n";
    print "<img alt=\"". $name ."\" height=\"600\" border=\"0\" src=\"". $path ."\" width=\"160\">\n";
    print "</div>\n";
    print "</a>\n";

    $sql2 = "UPDATE
               `bero_banner`
           SET
               `banner_count` = banner_count+1
           WHERE
               `id` = '". $id ."'";
    mysql_query($sql2);
} 
  

Meldung:

Notice:  Undefined variable:  banner in /home/schulbuc/public_html/includes/functions.php on line 66

Notice:  Undefined variable:  banner in /home/schulbuc/public_html/includes/functions.php on line 67 

Diese Meldung kommt, sobald das Feld banner_count größer gleich 2 ist.

[ljungi]

Hi wieso erhalte ich bei folgendem Scirpt eine Fehlermeldung?

 /**
 * @author     Adalbert Rohr <ad.rohr@schulbuch-und-co.de>
 * @copyright  2006-2007 Wirtschaftsprojekt JTG
 * @desc       Erstellt eine Bannerrotation
 * @uses       config.php
 */ 
function banner($position)
{
    require("config.php");
    $link = mysql_connect($mysql_host, $mysql_user, $mysql_pass);
    mysql_select_db($mysql_data, $link);
    $sql = "SELECT
               `id`,
               `clickurl`,
               `imageurl`,
               `name`,
               `banner_count`,
               `banner_max`
           FROM
               `bero_banner`
           WHERE
               `check` = '1'
           AND
               `banner_count` <= `banner_max`
           AND
               `position` = '". $position ."'";
    $result = mysql_query($sql);

    while($row = mysql_fetch_assoc($result))
        $banner[] = $row;

    $rand = rand (0, mysql_num_rows($result)-1);
    $url  = $banner[$rand]['clickurl'];
    $id   = $banner[$rand]['id'];
    $name = $banner[$rand]['imageurl'];
    $path = $banner[$rand]['id'];

    print "<a href=\"". $url ."\" target=\"_blank\">\n";
    print "<div id=\"promotion-field-right\">\n";
    print "<img alt=\"". $name ."\" height=\"600\" border=\"0\" src=\"". $path ."\" width=\"160\">\n";
    print "</div>\n";
    print "</a>\n";

    $sql2 = "UPDATE
               `bero_banner`
           SET
               `banner_count` = banner_count+1
           WHERE
               `id` = '". $id ."'";
    mysql_query($sql2);
} 
  

Meldung:

Notice:  Undefined variable:  banner in /home/schulbuc/public_html/includes/functions.php on line 66

Notice:  Undefined variable:  banner in /home/schulbuc/public_html/includes/functions.php on line 67 

Diese Meldung kommt, sobald das Feld banner_count größer gleich 2 ist.

Das ist kein Fehler, sonder nur eine Notice.. Also ein Hinweis auf einen nicht "sauberen" Code.

Stell mal vor die whileschleife ...

  while($row = mysql_fetch_assoc($result))
        $banner[] = $row; 
  

... folgendes

 $banner = array(); 
  

[Berti]

Ne, die Meldung kommt leider weiterhin Jetzt kommt auch nicht mehr

Notice: Undefined variable: banner in /home/schulbuc/public_html/includes/functions.php on line 66
Notice: Undefined variable: banner in /home/schulbuc/public_html/includes/functions.php on line 67

sondern

Notice: Undefined offset: 0 in /home/schulbuc/public_html/includes/functions.php on line 90
Notice: Undefined offset: 0 in /home/schulbuc/public_html/includes/functions.php on line 92
Notice: Undefined offset: 0 in /home/schulbuc/public_html/includes/functions.php on line 92
Notice: Undefined offset: 0 in /home/schulbuc/public_html/includes/functions.php on line 101

Nur noch kurz zur Erklärung des Script.
Mit

     $sql = "SELECT
           `id`,
           `clickurl`,
           `imageurl`,
           `name`,
           `banner_count`,
           `banner_max`
       FROM
           `bero_banner`
       WHERE
           `check` = '1'
       AND
           `banner_count` <= `banner_max`
       AND
           `position` = '". $position ."'";
$result = mysql_query($sql);
 
while($row = mysql_fetch_assoc($result))
    $banner[] = $row; 
  

Rufe ich die vorhandenen Banner aus der Datenbank ab und speichere sie in einem Array.

     $rand = rand (0, mysql_num_rows($result)-1);
$url  = $banner[$rand]['clickurl'];
$id   = $banner[$rand]['id'];
$name = $banner[$rand]['imageurl'];
$path = $banner[$rand]['id']; 
  

Dann wähle ich mit rand() ein zufälliges Element aus dem Array aus.

 print "<a href=\"". $url ."\" target=\"_blank\">\n";
    print "<div id=\"promotion-field-right\">\n";
    print "<img alt=\"". $name ."\" height=\"600\" border=\"0\" src=\"". $path ."\" width=\"160\">\n";
    print "</div>\n";
    print "</a>\n"; 
  

Hier soll nun der Banner ausgegeben werden.

[Byrel]

Mach doch mal:
<?php
assert_options(ASSERT_ACTIVE, 1);
assert_options(ASSERT_WARNING, 0);
assert (isset ($banner[$rand]));
?>

$rand verweist glaub ich auf einen index, der nicht existiert.

Btw.:
Verwende statt rand () mt_rand ()
Statt mysql_num_rows COUNT(*)

MfG Byrel

[Berti]

Im Moment benutze ich für die Ausgabe
$banner[$rand]['name']
Mit mt_rand funzt das so nicht mehr. Wie gebe ich es dann aus?

[Byrel]

http://at.php.net/mt_rand

Überprüf einfach ob der Index $rand existiert.