nicht angemeldet
hi, noch mal ich :)
mein kleiner charakter-generator ist nun etwas fortgeschritten und unter http://www.acridaudio.de/vg/index.htm zur anschauung freigegeben.
allerdings gibt es da noch ein paar kleine probleme, die ich nicht in den griff bekomme.
ohje, ich hab grad keine grosse Lust alles durchzuschauen, da einiges im argen liegt. Aber was mir auffällt:
http://www.acridaudio.de/vg/classen.js
du definierst überall globale Variabeln, das ist extrem schlecht für dich, da du grosse Schwierigkeiten kriegen wirst (oder schon hast?) dies zu kontrollieren. Arbeite mit Strukturen, Klassen o.ä. und benutze globale Variabeln nur in Ausnahemfällen.
http://www.acridaudio.de/vg/berechnen.js
Du solltest überprüfen, ob das was du verwenden möchtest auch wirklich existent ist.
z.b.
function berechnen()
{
document.chrbogen.Rasse.value = Rasse;
document.chrbogen.Punkte.value = Freie_punkte;
...
woher bist du so sicher, das document.chrbogen existiert?
und du würdest dir Tipparbeit ersparen, wenn du hier mit Variabeln arbeiten würdest.
function berechnen()
{
if(typeof document.chrbogen == 'undefined') return alert('Es trat ein Fehler auf!......');
var my_form = document.chrbogen;
....
diese sehr unsauberen Programmiertechnik zieht sich durch alle Skript, insofern ist es immer schwer Fehler zu finden und deine Skripte zu debuggen (du programmierst nicht zufällig normalerweise mit VB?)
Die einzige Fehlermeldung die ich sehe ist:
Error: Rasse is not defined
Source File: http://www.acridaudio.de/vg/berechnen.js
Line: 3
Aber woran das liegt ist halt schwer nachzuvollziehen.
Struppi.