2016-05-18 14 views
0

ユーザー入力を文字列の形式で受け取る関数を作成する必要があります。これまでのところ、ボタンとユーザ入力フィールドを逆にする機能があります。しかし、私はそれを前後に情報を渡すために苦労しています。JavaScriptボタン/入力/関数、文字列反転

Html: 

<!DOCTYPE html> 
<html lang="en"> 
<head> 
    <meta charset="UTF-8"> 
    <title>String reverser</title> 
<!-- imports script containing function--> 

    <script type="text/javascript" src="problem23.js"></script> 
</head> 
<body> 

<form> 
<!--defines user input field--> 
    <input title="userinput" type="text" id="inputValueId" name="inputId"/> 

<!--defines button which calls on the reverse function in javascript file--> 

    <button name="reverseButton" type="submit" id="btn1" onclick="reverse()">Click me!</button> 
</form> 

</body> 
</html> 

function reserve(conversionHolder){ 
    //picks up value 
    var userInput= document.getelementById("inputValueId").value; 
    //converts value 
    var conversionHolder= userInput.split('').reverse().join(''); 
    //passes value outside of function? 
    return conversionHolder; 
}//end reverse 
//sets userinput field to the reversed string 
document.getElementById('inputValueId').value = reverse(conversionHolder); 
+4

あなたの関数呼び出しは 'reserve'と指定されていますが、あなたの関数は' reserve'と綴られています。 – Quantastical

+0

'convertHolder'をパラメータとして宣言する必要はなく、 –

+0

の関数を呼び出すときに渡すこともありません。これらの変更を行いました。また、逆関数の内部にアラート(「テスト」)を追加しました。ボタンをクリックすると警告が表示されません。だから私は結ばれていないと思いますか? – nick

答えて

0

パラメータを渡さないと、あなたと未定義の値が渡されますので、必要ありません。

document.getElementById('inputValueId').value = reserve(); 

か、mistypを修正する場合

document.getElementById('inputValueId').value = reverse(); 

十分なはずです。

1

それはこのようにする必要があります:それはあるよう

function reserve(){ 
    //picks up value 
    var userInput= document.getElementById("inputValueId").value; 
    //convert and sets value 
    document.getElementById('inputValueId').value = userInput.split('').reverse().join(''); 
}//end reverse 

はあなたのhtmlをMantain。 js部分のみを上記に変更してください。

+0

私は関数内でそれを処理する方法を混乱しました、ありがとう! – nick

+0

いくつかの回答が役に立ったら、それを受け入れてください。ありがとう。 –