✔ Codekorrektur

[Student123]

Hallo! Ich brauche dringend jemanden, der mir folgenden Code korrigieren könnte. Ich muss für die Schule eine Website schreiben & in dieser Website muss unter anderem ein JavaScript Programm laufen. Ich selber habe kleine Kenntnisse in Objective C & minimale Kenntnisse in JS. Wäre froh, wenn mir jemand den Code korrigieren könnte:

<script language="JavaScript">
<!--

<!-- Variabeln definieren -->

var geschlecht
var gewicht
var bauchumfang
var x
var y


<!-- Aufforderung -->
alert("Bitte geben Sie ihr Geschlecht an.");

<!-- Benutzer wird aufgefordert das Geschlecht einzugeben & die Variabeln werden dadurch definiert. -->
geschlecht = prompt("Sind sie ein Mann oder eine Frau?", "Mann/Frau");

<!-- Aufforderung -->
alert("Bitte geben Sie ihr Gewicht in kg an.");

<!-- Benutzer wird aufgefordert das Gewicht einzugeben & die Variabeln werden dadurch definiert. -->
gewicht = prompt("Bitte geben Sie hier ihr Gewicht in kg ein!", "Ihr Gewicht");
x = parseFloat(gewicht);

<!-- Aufforderung -->
alert("Bitte geben Sie ihren Bauchumfang in cm an.");

<!-- Benutzer wird aufgefordert den Bauchumfang einzugeben & die Variabeln werden dadurch definiert. -->
bauchumfang = prompt("Bitte geben Sie hier ihren Bauchumfang in cm ein!", "Ihr Bauchumfang");
x = parseFloat(bauchumfang);

switch (geschlecht) {
case "Mann":
var m = (-98.42+10.541*z-0.0371952*x)/x*0.4536);
alert("Ihr Körperfettanteil beträgt"+m+"%.");
break;
case "Frau":
var w = (-76.76+10.541*z-0.0371952*x)/x*0.4536);
alert("Ihr Körperfettanteil beträgt"+w+"%.");
break;
default:
document.write("Falsche Eingabe!");
break;
}
//-->
</script>
<noscript></noscript>

 

[timestamp]

<script language="JavaScript">
 <!--


 <!-- Variabeln definieren -->


//Semikolons haben gefehlt
 var geschlecht;
 var gewicht;
 var bauchumfang;
 var x;
 var y;

 alert("Bitte geben Sie ihr Geschlecht an.");

 geschlecht = prompt("Sind sie ein Mann oder eine Frau?", "Mann/Frau");

 alert("Bitte geben Sie ihr Gewicht in kg an.");

 gewicht = prompt("Bitte geben Sie hier ihr Gewicht in kg ein!", "Ihr Gewicht");
 x = parseFloat(gewicht);

 alert("Bitte geben Sie ihren Bauchumfang in cm an.");

 bauchumfang = prompt("Bitte geben Sie hier ihren Bauchumfang in cm ein!", "Ihr Bauchumfang");
 x = parseFloat(bauchumfang);

 switch (geschlecht) {
 case "Mann":
 var m = (-98.42+10.541*z-0.0371952*x)/x*0.4536);
 alert("Ihr Körperfettanteil beträgt"+m+"%.");
 break;
 case "Frau":
 var w = (-76.76+10.541*z-0.0371952*x)/x*0.4536);
 alert("Ihr Körperfettanteil beträgt"+w+"%.");
 break;
 default:
 document.write("Falsche Eingabe!");
 break;
 }
 //-->
 </script>

Die Kommentare halte ich für Überflüssig, die Funktionen alert und prompt sind selbsterklärend.
Außerdem würde ich die alerts weglassen, das ist mit dem prompt doppelt gemoppelt.
In deiner Formel berechnest du noch etwas mit einer Variablen z, die ist aber nirgendwo definiert.

 

[Student123]

Die Kommentare muss ich schreiben. Ist vom Lehrer so verlangt, da wir in einer 2er Gruppe arbeiten...

Ja, habs gefunden.
1. z Variabel
2. eine Klammer fehlt bei der Berechnung :/

 

[Parantatatam]

Die Kommentare, die du verwendest, sind außerdem HTML-Kommentare, die allerdings nicht in Javascript funktionieren. Kommentare in Javascript sind einmal die einzeiligen, die mit zwei Schrägstrichen beginnen, und die mehrzeiligen Kommentare, die so aussehen: /* ... */

 

[timestamp]

Ich hab mir schon gedacht, dass euer Lehrer Kommentare verlangt. Aber ich würde ihn mal Fragen, ob die bei selbsterklärendem Code nicht weggelassen werden können (dürfen wir in der Uni auch machen).
Weil die Kommentare sind in etwa so sinnvoll wie

i++; //inkrement i