[didi]

Hi, ich komme mit dieser Funktion nicht klar. Die fehlermeldung die ich bekomme besagt soweit ich weiss, daß ich zwei "gleiche Aktion" mache...Ist es das "drawpoint" in der Funtion ? muss ich das eventuell umbenennen?

Fatal  error: Cannot redeclare drawpoint() in  /var/www/html/web410/html/lms/inc/map.func.php on line  93.

Der Code dazu wäre der hier. In Zeile 93 steht eine "}"
Danke schonmal Gruss Didi

PHP-Code:

<?php

// Funktionen zum laden und anzeigen der Karte
 
require_once('conf.php');
require_once('sql.class.php');
 
 
 
// Breitengrad in X-Pixelkoordinate umrechnen
function lon2px($lon) {
 global $City;
 return round((($lon - $City[0]['lon'])/LON_PER_PX + $City[0]['x'])/RASTER_SIZE);
}
 
// Längengrad in Y-Pixelkoordinate umrechnen
function lat2px($lat) {
 global $City;
 return round((($City[0]['lat'] - $lat)/LAT_PER_PX + $City[0]['y'])/RASTER_SIZE);
}
 
// Einträge aus der Datenbank laden und nach Pixelkoordinaten gruppieren
function getPoints($id=0) {
 // Suche ?
 $q = "SELECT u.name, g.lon, g.lat, g.ort FROM ".TABLE_USER." u LEFT JOIN ".TABLE_GEODB."  g ON u.geoid = g.id WHERE 1 ";
 if($id) $q .="AND u.id = '".mysql_real_escape_string($id)."' ";
 if($_GET['name']) $q .="AND u.name LIKE '%".mysql_real_escape_string($_GET['name'])."%' ";
 if($_GET['plz']) $q .="AND g.plz LIKE '".mysql_real_escape_string($_GET['plz'])."%' ";
 if($_GET['ort']) $q .="AND g.ort LIKE '".mysql_real_escape_string($_GET['ort'])."%' ";
 
 global $db;
 $res = $db->query($q);
 $p=array();
 
 // Gefundene Einträge durchgehen
 while($row = mysql_fetch_assoc($res)) {
 
  // Längen- und Breitengrad in Pixelkoordinaten umrechnen
  $x=lon2px($row['lon']);
  $y=lat2px($row['lat']);
 
  // Eintrag in das Array übergeben (X -> Y -> Ort -> N -> Username )
  // Durch das verwenden der X u. Y Pixelkoordinaten als Array-Keys,
  // werden mehrere Einträge auf einem Pixel automatisch zusammengefasst
  if($x <= MAP_WIDTH && $y <= MAP_HEIGHT && $x > 0 && $y > 0)   
   $p[$x][$y][$row['ort']][] = $row['name'];
 }
 return $p; 
}
 
// Liste durchgehen und ausgeben
function generatePoints(&$p) {
 foreach($p AS $x => $xs) {
  foreach($xs AS $y => $ys) {
   drawPoint($x, $y, $ys);
  }
 }
}
 
// Punkt zeichnen
function drawPoint($x, $y, &$p) {
 $s = '';
 $usercount = 0;
 
 // Inhalt des Mouse-Over-DIVs
 foreach($p AS $ort => $user) {
  // Nicht der erste Ort für diesen Punkt? -> Zeilenumbruch
  if($s) $s .= '<br/><br/>';
  
  // Ortsname
  $s .="<h2>$ort</h2>";
  
  // Usernamen
  foreach($user AS $key => $name) {
   $s.=($key?', ':' ').str_replace("'", "\\'",$name);
   $usercount++;
  }
 }
 
 // Größe des Punktes ( css-class) anhand der Anzahl der User auf diesem Punkt festlegen
 foreach($GLOBALS['UserCountClass'] as $dCount => $dClass) {
  if($usercount > $dCount || $_GET['m'])
   $cssClass = $dClass;
 }
  
 // Punkt ausgeben
 echo '<a href="?l='.$x.','.$y.'" onMouseOut="h();" onMouseOver="s(this, \''.$s.'\');" class="'.$cssClass.'" style="left:'.($x*RASTER_SIZE).'; top:'.($y*RASTER_SIZE).'"></a>'."\n";
}
 
?>

[Xean]

"Cannot redeclare drawpoint()" heißt, dass es die Methode drawpoint() schon gibt. D.h. in den dateien conf.php oder sql.class.php muss se entweder direkt oder durch include() diese methode schon geben.

wenn ich mich nicht irre, kannst du das Problem mit include_once() umgehen.
Ich hab aber keine ahnung, ob man das als "unschön" verpönt, oder ob das in ordnung ist....

[Ben]

Jau, die Funktion drawPoint() existiert bereits im Sichtbarkeitsbereich und kann somit kein zweites Mal deklariert werden.

Überprüfe mal sorgsam, ob du eventuell die Datei, die du oben gepostet hast (map.func.php), nicht eventuell mehrfach einbindest. Eventuell einfach mal ganz oben ein echo __FILE__ . '<br />'; einfügen.

Grüße, Ben.

[didi]

Hi, in der config habe ich diese methode nochmal... Das heisst ich muss statt

require_once('conf.php');
require_once('sql.class.php');

---

include_once('conf.php');
require_once('sql.class.php');

schreiben ?
@Ben danke das mache ich erstmal..

Gruss Didi

[Ben]

Hallo,
nein. include_once verhält sich von der Funktionalität her genau wie require_once.

Dadurch, dass du die Funktion schon in der conf.php schon mal hast, ist es klar, dass die Meldung kommt. Du musst dir ein include/require so vorstellen, dass der Inhalt der Datei, den du da einbindest, genau an der Stelle wo die Anweisung, in deinem Fall require_once, steht eingefügt wird.

Heißt also, dass du oben den Inhalt von conf.php einfügst, inkl. der Funktion drawPoint() und unten steht diese Funktion dann nochmal. Das kann ja nicht gehen.

Das Problem ist aber recht einfach zu lösen.
Sollten die Funktionen das gleiche machen, so entscheide dich für eine Datei. Haben sie unterschiedliche Aufgaben und heißen nur gleich, dann benenne mindestens eine der beiden Funktionen um.

Grüße, Ben.

[didi]

Zitat von Ben

Hallo,
nein. include_once verhält sich von der Funktionalität her genau wie require_once.

Dadurch, dass du die Funktion schon in der conf.php schon mal hast, ist es klar, dass die Meldung kommt. Du musst dir ein include/require so vorstellen, dass der Inhalt der Datei, den du da einbindest, genau an der Stelle wo die Anweisung, in deinem Fall require_once, steht eingefügt wird.

Heißt also, dass du oben den Inhalt von conf.php einfügst, inkl. der Funktion drawPoint() und unten steht diese Funktion dann nochmal. Das kann ja nicht gehen.

Das Problem ist aber recht einfach zu lösen.
Sollten die Funktionen das gleiche machen, so entscheide dich für eine Datei. Haben sie unterschiedliche Aufgaben und heißen nur gleich, dann benenne mindestens eine der beiden Funktionen um.

Grüße, Ben.

Hi,
Jau sie machen verschiedenes...ich habe jetzt die eine einfach mal drawpoint (ohne _ den hier genannt) und siehe da
Danke nochmal Ben

Gruss didi

[Ben]

Dann hoffen wir mal, dass du der einzigste bist, der jemals mit dem Code arbeiten muss.

Und warum ist überhaupt in der Konfiguration eine Funktion namens drawPoint()? Nunja, ..

[didi]

Zitat von Ben

Dann hoffen wir mal, dass du der einzigste bist, der jemals mit dem Code arbeiten muss.

Und warum ist überhaupt in der Konfiguration eine Funktion namens drawPoint()? Nunja, ..

Ja danke !! Jetzt bin wieder da wo ich nicht hinwollte. Warum ich die da habe ? Keine Ahnung. Die war da schon.

Ich hoffe das ich der einzige bin der damit arbeitet.. das wird ne Art Userkarte für ne kleine Com.
Was genau schwebt dir vor ? Bitte dann mal um Aufklärung

Gruss didi

[Ben]

Hallo,
ähm. Ich kenne jetzt ja die Struktur des Quelltextes nicht, aber es ist ja so, dass der Name conf.php impliziert, dass dort die Konfiguration geladen, gespeichert, was auch immer wird. Auf jeden Fall was mit der Konfiguration!

Ich würde dort niemals nach der Funktion drawPoint() suchen. Warum auch? Die hat da nichts zu suchen.


Wie könnte man so etwas strukturieren? Pseudocode:

file.php

PHP-Code:

<?php

// Die Datei enthält Standardsachen, wie das Starten der Session, Fehlerlevel setzen, Konfiguration laden, ..
require_once('prepend.inc.php');


// Einbinden spezieller Funktionen
require_once(special_functions_for_this_file.inc.php);


// [..] Quelltext, der für diese Datei wichtig und speziell ist.
?>

prepend.inc.php

PHP-Code:

<?php

error_reporting(E_ALL);

session_start();

// [..]

// Laden der Konfiguration
require_once('config.php');

// Einbinden "globaler" Funktionen, die eh überall gebraucht werden
require_once(functions_for_this_file.inc.php);

?>

Verstehste was ich meine?

[didi]

Verstehste was ich meine?

Jepp, ich werde mir die "drei" nochmal zur Brust nehmen..

Gruss didi

[Ben]

Bei Fragen .. is klar, ne?

[Bleistift]

Zitat von Ben

nein. include_once verhält sich von der Funktionalität her genau wie require_once.

Das stimmt nicht (ganz).
Wenn ein Script, das mit require(_once) nicht geladen werden kann, bricht das Script mit einem "Fatal error" ab. Bei inlcude(_once) wird lediglich eine Warnung ausgegeben.

[Ben]

Jau, die Funktionalität ist aber die gleiche!

[didi]

Zitat von Ben

Jau, die Funktionalität ist aber die gleiche!

Hi Hi die Funktionalität ist ja im Mom nicht gegeben

Gruss didi