Michael Schröpl: Scripeinbau für Preloading!

Beitrag lesen

Hi,

<script language="JavaScript">

... (Millionen Zeilen ineffizienten Codes gelöscht)

bist Du schon mal auf die Idee gekommen, daß Du nicht tausendmal dasselbe
hinschreiben mußt, wenn Du immer dasselbe tun willst?

An vielen Stellen in Deinem Code könntest Du einfach entsprechende Variablen-
inhalte direkt ausgeben, statt sie abzufragen.

if (X == 1) document.writeln ("1")
     else if (X == 2) document.writeln ("2")

ist nämlich dasselbe wie

document.writeln (X)

was doch wesentlich übersichtlicher aussieht, nicht wahr?

An anderen Stellen brauchst Du ggf. eine Abbildung zwischen Zahlenwerten
und Strings - aber diese in einem array zu halten und diesen über integers
zu adressieren ist immer noch wesentlich effizienter als Deine "Lösung".

Mindestens 60% Deines Codes ist heiße Luft.
(Eher mehr, fürchte ich - und damit meine ich erst mal nur die Syntax ...)
Entferne diese, dann wird alles zumindest besser lesbar und wartbar.
Und dann wird es auch viel leichter, etwas einzubauen.

Abgesehen davon ist Dein JavaScript-Code ein ziemlich repräsentativer
Querschnitt all des Unfugs, den man als Anfänger aufgesammelt hat:
Viel Klick-Bunti, Ticker, Farben.
Also genau das, was die Besucher Deiner Seiten wie die Pest hassen werden.

Du hast viel zu lernen vor Dir.
Wenn Du guten Willens bist, dann bist Du dafür an einem ziemlich geeigneten
Ort. Trotzdem wirst Du auch in ein paar Monaten noch kein "Profi" sein.

Viele Grüße
      Michael