Der Martin: img src mit variable

Beitrag lesen

Hallo,

<div style="position: absolute;left: 00px;top: 10px">
     <script type="text/javascript">
          document.write("<img src ='"+ jpgdatei  +"' alt = '"+ort+"' width='396' height  ='550'>");
     </script>
</div>

bist du dir darüber im Klaren, was du tust? Absolute Positionierung hat viele Stolpersteine, und ich könnte mir vorstellen, dass das Ergebnis nur "zufällig" das ist, was du haben möchtest.
Außerdem: Wozu das div-Element, wenn sein einziger Inhalt ein Bild ist? Da könntest du ebensogut das Bild selbst positionieren und das Containerlement entsorgen. Es sei denn, es hätte noch einen Zweck, der aus diesem Ausschnitt nicht erkennbar ist.

<script type="text/javascript">

     var orte = new Array(40);
     for (var i = 0; i < orte.length; ++i)
     {orte[i] = new Array(3);
     }

orte[1][ 1]="06.08!3d 50.78 ";        orte[2][ 1 ]="aachen" ;          orte[3] [ 1 ]="Aachen";
orte[1][ 2]="12.44!3d 50.99 ";        orte[2][ 2 ]="bad hersfeld" ;    orte[3] [ 2 ]="Bad Hersfeld";

//   hier fehlen natürlich die Zeilen 3 bis 33

orte[1][34]="10.54!3d 52.16 ";        orte[2][34 ]="wolfenbüttel" ;    orte[3] [34 ]="Wolfenbüttel";

//   bei Erweiterung der Liste Tabellengröße prüfen

</script>

Hmm. Mysteriös.

  1. Das Array-Element mit dem Index 0 lässt du konsequent links liegen. Absicht?
  2. Du benutzt die Indexe genau verkehrt herum: Du erstellst orte[] als Array mit 41 Elementen (von 0 bis 40), benutzt dann aber immer nur Index 1 bis 3.
  3. Die Eigenschaften jedes Ortes würde ich nicht als untergeordnetes Array anlegen, sondern als Objekt mit benamsten Eigenschaften. Feldnamen verraten mehr über ihren Zweck als Array-Indexe.
  4. Ich sehe keinen Sinn darin, den Ortsnamen einmal groß und einmal kleingeschrieben zu speichern. Lass die Variante in Kleinschreibung weg, sie ist redundant. Sie kann jederzeit aus der normalen Schreibweise gewonnen werden.
  5. Die geheimnisvolle Angabe in der linken Spalte ist mir nicht ganz klar. Es könnte sich um die Längen- und Breitengrad handeln (das erklärt aber das "!3d" noch nicht), so dass getrennte Felder für Länge und Breite (dann als Zahlenwert, nicht als String) wohl sinnvoller wären.

[...] zumal es um eine einmalige umgrenzte Aufgabenstellung geht.

Dann nimm meine Vorschläge und Anregungen wenigstens als Denkanstoß für künftige Projekte.

Nun halte ich es für möglich, dass ein Experte über diesen Code die Nase rümpft

So hätte ich es nicht gesagt, aber man kann eben einiges "schöner" machen.

So long,
 Martin