kostenloser Webspace werbefrei: lima-city


[JavaScript] Ist der Code gut?

lima-cityForumDie eigene HomepageHTML, CSS & Javascript

  1. Autor dieses Themas

    infinitysounds

    Kostenloser Webspace von infinitysounds

    infinitysounds hat kostenlosen Webspace.

    Hi Leute,

    Ich bin absoluter Neuling auf dem Gebiet JavaScript und brauche eure Meinung zu einem Script das ich geschrieben habe, ich hoffe,
    jemand kann dazu negative Kritik in Form von Änderungsvorschlägen abgeben.
    Hier ist das Link zum Download von script und drumherum.. Downloadlink für die ZIP-Datei: http://www.fileuploadx.de/18578
    Grüße,

    InfinitySounds

    PS: Ich habe nicht erklärt, was es machen soll, da es funktioniert und die Erklärung mehr zeit in anspruch nimmt, als wenn man es ausprobiert.



    Beitrag zuletzt geändert: 24.7.2011 15:21:05 von infinitysounds
  2. Diskutiere mit und stelle Fragen: Jetzt kostenlos anmelden!

    lima-city: Gratis werbefreier Webspace für deine eigene Homepage

  3. infinitysounds schrieb:
    PS: Ich habe nicht erklärt, was es machen soll, da es funktioniert und die Erklärung mehr zeit in anspruch nimmt, als wenn man es ausprobiert.

    Tolle Idee, aber wie soll man das ausprobieren? Copy&paste reicht nicht. Da müßte man schon alle verwendeten Bilder aus unbekannter Quelle nachladen oder durch vorhandene ersetzen. Das wäre mir dann, nur um dir eine sinnvolle Antwort geben zu können, doch etwas zuviel Arbeit. ;)

    Wie wäre es mit einem Link zu einer funktionierenden Version?

    FF
  4. Autor dieses Themas

    infinitysounds

    Kostenloser Webspace von infinitysounds

    infinitysounds hat kostenlosen Webspace.

    fatfreddy schrieb:
    infinitysounds schrieb:
    PS: Ich habe nicht erklärt, was es machen soll, da es funktioniert und die Erklärung mehr zeit in anspruch nimmt, als wenn man es ausprobiert.

    Tolle Idee, aber wie soll man das ausprobieren? Copy&paste reicht nicht. Da müßte man schon alle verwendeten Bilder aus unbekannter Quelle nachladen oder durch vorhandene ersetzen. Das wäre mir dann, nur um dir eine sinnvolle Antwort geben zu können, doch etwas zuviel Arbeit. ;)

    Wie wäre es mit einem Link zu einer funktionierenden Version?

    FF

    Damnit!!!

    Hatte ich total vergessen.
    Das Problem ist, ich habs nicht hochgeladen, aber ich guck mal was ich amchen kann, wartet eine Minute oder etwas mehr

    EDIT: Aber wie? Der Webspace ist voll

    EDIT2: Downloadlink für die ZIP-Datei: http://www.fileuploadx.de/18578

    Beitrag zuletzt geändert: 24.7.2011 2:39:38 von infinitysounds
  5. Hi infinitysounds,

    ich selber bin auh nicht gerade Profi was JavaScript angeht, aber ich wundere mich etwas darüber, dass du den intervall (move) leerst
    und ihn danach gleich neu definierst....
    Erklärung?

    Greets Persysele
  6. Autor dieses Themas

    infinitysounds

    Kostenloser Webspace von infinitysounds

    infinitysounds hat kostenlosen Webspace.

    persysele schrieb:
    Hi infinitysounds,

    ich selber bin auh nicht gerade Profi was JavaScript angeht, aber ich wundere mich etwas darüber, dass du den intervall (move) leerst
    und ihn danach gleich neu definierst....
    Erklärung?

    Greets Persysele

    Hi Persysele,

    Dafür habe ich einen ganz einfachen grund: Täte ich das nicht, würde der Bär nicht nur nach 735px nach unten gehen, wieder 5px pro 40ms, sondern ebenfalls 5px pro 40ms nach rechts, da moveIt() und moveIt2() dann gleichzeitig ausgeführt würden.
    Ich weiß allerdings nicht, ob das die beste Methode ist...

    Grüße,

    InfinitySounds
  7. Hallo,

    du wolltest ja wissen, ob dein javascript "gut" ist, bzw. ob man es verbessern sollte, dazu habe ich 2 Punkte.

    Zum einen ist es nicht sehr zielführend, wenn du für eine Sache (die bewegung des Bildes) vier Funktionen brauchst, das solltest du besser mit if-else oder mittels eines switch abfangen, außerdem ist dein HTML viel zu umfangreich, nur um so eine Situation abzubilden…

    Zum anderen solltest du dir eine saubere Schreibweise schon von anfang an angewöhnen, sonst kannst du deinen code später nicht mehr lesen, geschweige denn verstehen!

    Um dir zu helfen habe ich dein Script mal neu geschrieben und das HTML angepasst, das Ergebnis kannst du hier bewundern. Der dunkle Hintergrund war für Testzwecke und ist schlicht egal.

    Den Code kannst du hier einsehen, ich habe ihn extra durch meine Highlightscripte laufen lassen, außerdem ist das Javascript mit diversen Kommentaren versehen:
    Das HTML (inkl. CSS)
    Das Javascript

    Mit freundlichen Grüßen
  8. Autor dieses Themas

    infinitysounds

    Kostenloser Webspace von infinitysounds

    infinitysounds hat kostenlosen Webspace.

    nemoinho schrieb:
    Hallo,

    du wolltest ja wissen, ob dein javascript "gut" ist, bzw. ob man es verbessern sollte, dazu habe ich 2 Punkte.

    Zum einen ist es nicht sehr zielführend, wenn du für eine Sache (die bewegung des Bildes) vier Funktionen brauchst, das solltest du besser mit if-else oder mittels eines switch abfangen, außerdem ist dein HTML viel zu umfangreich, nur um so eine Situation abzubilden?

    Zum anderen solltest du dir eine saubere Schreibweise schon von anfang an angewöhnen, sonst kannst du deinen code später nicht mehr lesen, geschweige denn verstehen!

    Um dir zu helfen habe ich dein Script mal neu geschrieben und das HTML angepasst, das Ergebnis kannst du hier bewundern. Der dunkle Hintergrund war für Testzwecke und ist schlicht egal.

    Den Code kannst du hier einsehen, ich habe ihn extra durch meine Highlightscripte laufen lassen, außerdem ist das Javascript mit diversen Kommentaren versehen:
    Das HTML (inkl. CSS)
    Das Javascript

    Mit freundlichen Grüßen


    Dankeschön für den Aufwand den du dir gemacht hast;
    Leider ist die Darstellungsweise die du gewählt hast etwas nervig, da ich die Zahlen aus dem Text rausnehmen muss und dann die einzelnen Zeilen von einander trennen muss...

    ansonsten danke, ich muss dein script noch ein wenig anpassen , aber ich denke dass ist ganz gut.
  9. Diskutiere mit und stelle Fragen: Jetzt kostenlos anmelden!

    lima-city: Gratis werbefreier Webspace für deine eigene Homepage

Dir gefällt dieses Thema?

Über lima-city

Login zum Webhosting ohne Werbung!