[JavaScript] Ist der Code gut?
lima-city → Forum → Die eigene Homepage → HTML, CSS & Javascript
abfangen
anspruch
ausprobieren
code
diversen kommentaren
ergebnis
gerade profi
http
nachladen
paste
punkt
sinnvolle antwort
situation
switch
tolle idee
unbekannter quelle
url
verwendeten bilder
vorhandene ersetzen
zuviel
-
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 -
Diskutiere mit und stelle Fragen: Jetzt kostenlos anmelden!
lima-city: Gratis werbefreier Webspace für deine eigene Homepage
-
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 -
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 -
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 -
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 -
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 -
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. -
Diskutiere mit und stelle Fragen: Jetzt kostenlos anmelden!
lima-city: Gratis werbefreier Webspace für deine eigene Homepage
