-1

皆さん、これは私の初めての jquery スクリプトであり、エラーや悪い癖が見られるかどうか疑問に思っていました。コードのシマティックスまたは一般的なもので、早期に失う必要があります..そして、検証を実装した後に注意してください誰かのウェブサイトで本物になります。また、検証の例をどのように提案しますか

<script type="text/javascript" src="jquery.js">
</script>
<script type="text/javascript">
$(document).ready(function(e) {
$(".calculatebutton").on("click",function()
{
//perform a input type check when i learn rregex
var item_cost=$("#itemCost").val();
if($(this).val()=="Cash Payment")

{
    var answer=item_cost*0.10;
}
else
{
    var answer=item_cost*0.12;
}
answer+=parseInt(item_cost);
$("#total").val(answer);
 })
});
</script>
4

1 に答える 1

1

コメント付きの小さな例

http://jsfiddle.net/blackjim/JktYg/2/

$(document).ready(function (e) {
    $(".calculatebutton").on("click", function () {
        //perform a input type check when i learn rregex
        var answer = 0,
            item_cost = $("#itemCost").val(); // this is a string, NOT a number

        // check if you got a number, with your way or criteria
        if(isNaN(item_cost) || item_cost === true || item_cost === ''){  
            $("#total").val("Not a Number here");
            return false; // you didn't
        }
        item_cost = parseFloat(item_cost); // or use parseInt if you want integer

        if ($('.calculatebutton').text() === 'Cash Payment'){ // use always === for equality
            answer = item_cost * 0.10;
        } else {
            answer = item_cost * 0.12;
        }
        answer += item_cost;
        $("#total").val(answer);
    })
});
于 2013-07-03T08:48:44.193 に答える