0

これが私のコードです。ドイツのエニグマ機の原案です。関数を2回目に実行するまでそのメッセージが作成されないことを除いて、実際にマシンを実行するというメッセージを作成しようとしています。奇妙なことは、関数の一部が実行されるのを見て、関数が実行されることを知っていることですが、コードに関する限りtoWorkWith、最初の実行では空で、2 番目の実行では満たされていますか?

 

function encode(){
        var alphabet = ["A","B","C","D","E","F","G","H","I","J","K","L","M","N","O","P","Q","R","S","T","U","V","W","X","Y","Z"]
        var rotor_1 = {"A":["E"],"B":["K"],"C":["M"],"D":["F"],"E":["L"],"F":["G"],"G":["D"],"H":["Q"],"I":["V"],"J":["Z"],"K":["N"],"L":["T"],"M":["O"],"N":["W"],"O":["Y"],"P":["H"],"Q":["X"],"R":["U"],"S":["S"],"T":["P"],"U":["A"],"V":["I"],"W":["B"],"X":["R"],"Y":["C"],"Z":["J"]};
        var rotor_2 = {"A":["A"],"B":["J"],"C":["D"],"D":["K"],"E":["S"],"F":["I"],"G":["R"],"H":["U"],"I":["X"],"J":["B"],"K":["L"],"L":["H"],"M":["W"],"N":["T"],"O":["M"],"P":["C"],"Q":["Q"],"R":["G"],"S":["Z"],"T":["N"],"U":["P"],"V":["Y"],"W":["F"],"X":["V"],"Y":["O"],"Z":["E"]};
        var rotor_3 = {"A":["B"],"B":["D"],"C":["F"],"D":["H"],"E":["J"],"F":["L"],"G":["C"],"H":["P"],"I":["R"],"J":["T"],"K":["X"],"L":["V"],"M":["Z"],"N":["N"],"O":["Y"],"P":["E"],"Q":["I"],"R":["W"],"S":["G"],"T":["A"],"U":["K"],"V":["M"],"W":["U"],"X":["S"],"Y":["Q"],"Z":["O"]};
        var reflector = {"A":["A"],"B":["B"],"C":["C"],"D":["D"],"E":["E"],"F":["F"],"G":["G"],"H":["H"],"I":["I"],"J":["J"],"K":["K"],"L":["L"],"M":["M"],"N":["N"],"O":["O"],"P":["P"],"Q":["Q"],"R":["R"],"S":["S"],"T":["T"],"U":["U"],"V":["V"],"W":["W"],"X":["X"],"Y":["Y"],"Z":["Z"]};
        document.simulator.encoder.value.toUpperCase();
        var message = document.simulator.encoder.value.trim();
        message.toUpperCase();
        document.simulator.encoder.value = message.toUpperCase();
        var code = []
        //Turns the rotors
        function updateRotorState(rotorNum){
                var rotor1state = document.simulator.rotor1.value.toUpperCase();
                var rotor2state = document.simulator.rotor2.value.toUpperCase();
                var rotor3state = document.simulator.rotor3.value.toUpperCase();
                if(rotorNum == 1){
                        var rotorPos = alphabet.indexOf(rotor1state);
                       
                        var newPos = rotorPos + 1;
                       
                        if(rotor1state == "V"){
                                document.simulator.rotor1.value=alphabet[newPos]
                                updateRotorState(2);
                        }
                        if(rotorPos == 25){
                                newPos = 0;
                        }
                        document.simulator.rotor1.value = alphabet[newPos];
                }
                if(rotorNum == 2){
                        var rotorPos = alphabet.indexOf(rotor2state);
                        var newPos = rotorPos + 1;
                        if(rotor2state == "E"){
                                document.simulator.rotor2.value = alphabet[newPos];
                                updateRotorState(3);
                        }
                        if(rotorPos == 25){
                                newPos = 0;    
                        }
                        document.simulator.rotor2.value = alphabet[newPos];
                }
                if(rotorNum == 3){
                        var rotorPos = alphabet.indexOf(rotor3state);
                        var newPos = rotorPos + 1;
                        if(rotorPos == 25){
                                newPos = 0;    
                        }
                        document.simulator.rotor3.value = alphabet[newPos]
                        //Eventually need to add code to make next rotor turn over
                }
        }
        //Turns the message into a stripped output. Removes all non letter characters including spaces
        function workingMessageGen(message){
                var workingMessage = ""
                var messageArray = message.split('');
                for(var char in messageArray){
                        for(var letter in alphabet){
                                if(messageArray[char] == alphabet[letter]){
                                        workingMessage += alphabet[letter];
                                }
                        }
                }
                return workingMessage;
        }
        toWorkWith = workingMessageGen(message);
        for(var letter in message){
                updateRotorState(1);
        }
        document.simulator.decoder.value=toWorkWith;
}
4

1 に答える 1

0

あなたが何を期待しているのかはわかりませんが、コードにはいくつかの欠陥があります:

  • JavaScript 文字列値は不変であり、オブジェクトではありません。のような関数を呼び出しtoUpperCaseても、変数は変更されませんが、新しい値が返されます。したがって、役に立たないdocument.simulator.encoder.value.toUpperCase()。message.toUpperCase()
  • 後にセミコロンvar code = []がありません (問題ではありませんが、確実ではありません)。
  • toWorkWithvar宣言がありません - 意図していないようです
  • for(var char in messageArray)- 配列プロパティを列挙しないでください! for ループを使用して、その indize を繰り返します ( Why is using "for...in" with array iteration a bad idea? を参照してください) 。
  • for(var letter in message)-messageは文字列であるため、そのプロパティを列挙しようとしないでください! a) これは IE では機能しませんか b) で列挙可能なプロパティをキャッチする可能性がありますString.prototype。代わりに、通常の for ループとmessage.length.
  • スクリプト全体で、コード ロジックと DOM アクセスが混在しています。DOM を使用して値を取得、表示、保存しますか? DOM からの入力の読み取り、ロジックの実行、出力の書き込みの 3 つの手順で行います。
于 2013-01-16T04:50:57.117 に答える