2016-04-29 7 views
0

Ich versuche, einen Entschlüsseler & Entschlüsseler für meine Uni-Zuweisung basierend auf einem Caesar-Chiffre zu bauen.Caesar Cipher String Länge Ausgabe

Ich habe einen ordentlichen Start gemacht, aber immer wenn ich versuche, die String-Länge zurückzugeben, so dass ich den Verschlüssler für jedes Zeichen loopen kann, gibt es es als null zurück.

Hier ist mein Code

EDIT: Ich habe den fertigen Code enthalten, die

<!DOCTYPE html> 
<html> 
<head> 
<meta charset="utf-8"> 
<title>Encryption version 0</title> 
<style> 

body { 
margin-left: 15%; 
margin-right: 15%; 
} 

#description { 
font-style: italic; 
font-size: 18px; 
} 

#instruction { 
font-weight: bold; 
color: red; 
font-size: 18px; 
} 
</style> 

</head> 
<body> 

<h1>Welcome to the encryptor</h1> 
<p id="description">Please type below the text you wish to encript</p> 
    <input name="txt" type="text" maxlength="512" id="txt" /> 
    <p>And pick an encryption key</p> 
     <input name="key1" type="text" maxlength="512" id="key1" /> 
    <button onclick= "enrypt()">Submit</button> 

    <p id="word"></p> 


<h1>Welcome to the decrypter</h1> 
<p id="description">Please type below the text you wish to decrypt</p> 
    <input name="txt2" type="text" maxlength="512" id="txt2" /> 
    <p>And your encryption key</p> 
     <input name="key" type="text" maxlength="512" id="key" /> 

    <button onclick= "decrypt()">Submit</button> 
    <p id="word2"></p> 
    <script type="text/javascript"> 

     //Script 1 

     function enrypt() { 
    var text, additon, encrypted, numb; 
    text, encrypted = ""; 

    text  = document.getElementById('txt').value; 
    additon = document.getElementById('key1').value; 
    for (i = 0; i < text.length; i++) { 
    numb = text.charCodeAt(i); 
    key = parseFloat(additon) + parseFloat(numb); 
    encrypted += String.fromCharCode(key) 
    } 

    document.getElementById("word").innerHTML = 'Your encrypted word is: ' + encrypted; 
} 
     //Script 2 

     function decrypt() { 
    var text2, additon2, encrypted2, numb2; 
    text2, encrypted2 = ""; 

    text2  = document.getElementById('txt2').value; 
    addition2 = document.getElementById('key').value; 


    for (i = 0; i < text2.length; i++) { 
    numb2 = text2.charCodeAt(i); 
    key = parseFloat(numb2) - parseFloat(addition2); 
    encrypted2 += String.fromCharCode(key) 
    } 

    document.getElementById("word2").innerHTML = 'Your encrypted word is: ' + encrypted2; 
} 

</script> 
</body> 
</html> 

arbeitet Ich weiß, ich Inline-CSS bin mit, das ist nur vorübergehend

Antwort

0

Hier ist Ihre feste Funktion:

function enrypt() { 
    var text, additon, encrypted, numb; 
    text, encrypted = ""; 

    text  = document.getElementById('txt').value; 
    addition = Math.floor(Math.random() * (122 - 97 + 1)) + 97; 

    window.alert('Your encryption key is: ' + addition); 
    window.alert('Your original word is: ' + text); 

    for (i = 0; i < text.length; i++) { 
    numb = text.charCodeAt(i); 
    key = addition + numb; 
    encrypted += String.fromCharCode(key) 
    } 

    window.alert('Your encrypted word is: ' + encrypted); 
} 

Alte Antwort:

Sie versuchen, den Wert des nicht initialisierten Variablen txt zu lesen:

letter = document.getElementById(txt.value); 

Sie wollen wahrscheinlich um es initialisieren zuerst:

txt = document.getElementById("txt"); 
letter = document.getElementById(txt.value); 

Überprüfen Sie auch für die Länge der Eingabetextlänge, nicht das DOM-Element:

while (i < txt.value.length) { 
+0

Danke ich hätte daran gedacht. bekomme immer noch die String-Länge von undefined – jkb114

+0

Ich sehe, dann in Ihrem Code weiter, bemerkte ich, dass Sie nicht die Länge der Zeichenfolge, sondern das DOM-Element überprüfen. Überprüfen Sie nach 'txt.value.length'. Meine Antwort wurde aktualisiert. – Uzbekjon

+0

Danke nochmal. jetzt heißt es Kann Eigenschaft 'charCodeAt' von null nicht lesen. Ich verstehe nicht, wie ich die Variable "i" mit einem Wert von 0 festlegen, so dass es an Position 0 gelesen werden sollte, die die erste ist. Recht? – jkb114