I need help java isn't adding when its supposed to

[TheSealPortalTeam]

Code: Select all

  <script language="JavaScript" type="text/javascript">
    function AddStrength(args)
    {

        var value1 = 0;
        value1 = document.getElementById("ctl00_ContentPlaceHolder1_txtStrength").value;
        var value2 = 0;
        value2 = document.getElementById("ctl00_ContentPlaceHolder1_txtTotalPoints").value;
        document.getElementById("ctl00_ContentPlaceHolder1_txtStrength").value = value1 +1 ;
        document.getElementById("ctl00_ContentPlaceHolder1_txtTotalPoints").value = value2 -1 ;

    }
    </script>

I use this in aspx (as you can tell) and I'll hit the add button and it'll subtract but it won't add, It'll do 81 instead of 9, 811 instead of 10. HELP

http://thesealportal.no-ip.biz/tsp/p...harCreate.aspx
the url wasn't supposed to have the team part sorry

[spinal007]

Ok, remenber the following:

1. Java is very different from Javascript.
Java: Programming launguage, needs to be compiled, strict data types.
Javascript: Scriptting language, needs an interpreter (browser), generic data types.

2. You're adding a number and text. Javascript can't perform the mathematical operation on the text so it converts the number to text and creates a new text string. What you need to do is convert the text into a number, like this:

Code: Select all

  <script language="JavaScript" type="text/javascript">
    function AddStrength(args)
    {

        var value1 = 0;
        value1 = new Number(document.getElementById("ctl00_ContentPlaceHolder1_txtStrength").value);
        var value2 = 0;
        value2 = new Number(document.getElementById("ctl00_ContentPlaceHolder1_txtTotalPoints").value);
        document.getElementById("ctl00_ContentPlaceHolder1_txtStrength").value = value1 +1 ;
        document.getElementById("ctl00_ContentPlaceHolder1_txtTotalPoints").value = value2 -1 ;

    }
    </script>

[TheSealPortalTeam]

thanks, I can also do this

Code: Select all

  <script language="JavaScript" type="text/javascript">
    function AddStrength(args)
    {
        //Specifically add strength while decreasing total points
        var value1 = 0;
        value1 = document.getElementById("ctl00_ContentPlaceHolder1_txtStrength").value;
        var value2 = 0;
        value2 = document.getElementById("ctl00_ContentPlaceHolder1_txtTotalPoints").value;
        if (value2 == 0){
        alert("You have maxed out your total points");
        }
        else{
        document.getElementById("ctl00_ContentPlaceHolder1_txtStrength").value = value1 -=(-1) ;
        document.getElementById("ctl00_ContentPlaceHolder1_txtTotalPoints").value = value2 -=1 ;
        }

    }
    </script>

[spinal007]

I'm not sure that would work, but if you've tested it and it works, then it works!

[TheSealPortalTeam]

okay now java is just not working by putting a value deep into the hidden feilds (hid***)

Code: Select all

  <script language="JavaScript" type="text/javascript">
    //Strength
    function AddStrength(args)
    {
        //Specifically add strength while decreasing total points
        var value1 = 0;
        value1 = document.getElementById("ctl00_ContentPlaceHolder1_txtStrength").value;
        var value2 = 0;
        value2 = document.getElementById("ctl00_ContentPlaceHolder1_txtTotalPoints").value;
        if (value2 == 0){
        alert("You have maxed out your total points");
        }
        else{
        document.getElementById("ctl00_ContentPlaceHolder1_txtStrength").value = value1 -=(-1) ;
        document.getElementById("ctl00_ContentPlaceHolder1_txtTotalPoints").value = value2 -=1 ;
        document.getElementById("ctl00_ContentPlaceHolder1_hidTotalPoints").value = document.getElementById("ctl00_ContentPlaceHolder1_txtTotalPoints").value
        document.getElementById("ctl00_ContentPlaceHolder1_hidStr").value = document.getElementById("ctl00_ContentPlaceHolder1_txtStrength").value 
        }

    }
        function SubbStrength(args)
    {
        //Specifically subtract strength while increasing total points
        var value1 = 0;
        value1 = document.getElementById("ctl00_ContentPlaceHolder1_txtStrength").value;
        var value2 = 0;
        value2 = document.getElementById("ctl00_ContentPlaceHolder1_txtTotalPoints").value;
        if (value1 == 8){
        alert("You CAN NOT lower a skill past 8");
        }
        else{
        document.getElementById("ctl00_ContentPlaceHolder1_txtStrength").value = value1 -=1 ;
        document.getElementById("ctl00_ContentPlaceHolder1_txtTotalPoints").value = value2 -=(-1) ;
        document.getElementById("ctl00_ContentPlaceHolder1_hidTotalPoints").value = document.getElementById("ctl00_ContentPlaceHolder1_txtTotalPoints").value
        document.getElementById("ctl00_ContentPlaceHolder1_hidStr").value = document.getElementById("ctl00_ContentPlaceHolder1_txtStrength").value 
        }

    }
        //Wisdom
    function AddWisdom(args)
    {
        //Specifically add wisdom while decreasing total points
        var value1 = 0;
        value1 = document.getElementById("ctl00_ContentPlaceHolder1_txtWisdom").value;
        var value2 = 0;
        value2 = document.getElementById("ctl00_ContentPlaceHolder1_txtTotalPoints").value;
        if (value2 == 0){
        alert("You have maxed out your total points");
        }
        else{
        document.getElementById("ctl00_ContentPlaceHolder1_txtWisdom").value = value1 -=(-1) ;
        document.getElementById("ctl00_ContentPlaceHolder1_txtTotalPoints").value = value2 -=1 ;
        document.getElementById("ctl00_ContentPlaceHolder1_hidTotalPoints").value = document.getElementById("ctl00_ContentPlaceHolder1_txtTotalPoints").value
        document.getElementById("ctl00_ContentPlaceHolder1_hidWis").value = document.getElementById("ctl00_ContentPlaceHolder1_txtWisdom").value 
        }

    }
        function SubbWisdom(args)
    {
        //Specifically subtract wisodm while increasing total points
        var value1 = 0;
        value1 = document.getElementById("ctl00_ContentPlaceHolder1_txtWisdom").value;
        var value2 = 0;
        value2 = document.getElementById("ctl00_ContentPlaceHolder1_txtTotalPoints").value;
        if (value1 == 8){
        alert("You CAN NOT lower a skill past 8");
        }
        else{
        document.getElementById("ctl00_ContentPlaceHolder1_txtWisdom").value = value1 -=1 ;
        document.getElementById("ctl00_ContentPlaceHolder1_txtTotalPoints").value = value2 -=(-1) ;
        document.getElementById("ctl00_ContentPlaceHolder1_hidTotalPoints").value = document.getElementById("ctl00_ContentPlaceHolder1_txtTotalPoints").value
        document.getElementById("ctl00_ContentPlaceHolder1_hidWis").value = document.getElementById("ctl00_ContentPlaceHolder1_txtWisdom").value 
        }

    }
    //Wisdom
    function AddDefence(args)
    {
        //Specifically add wisdom while decreasing total points
        var value1 = 0;
        value1 = document.getElementById("ctl00_ContentPlaceHolder1_txtDefence").value;
        var value2 = 0;
        value2 = document.getElementById("ctl00_ContentPlaceHolder1_txtTotalPoints").value;
        if (value2 == 0){
        alert("You have maxed out your total points");
        }
        else{
        document.getElementById("ctl00_ContentPlaceHolder1_txtDefence").value = value1 -=(-1) ;
        document.getElementById("ctl00_ContentPlaceHolder1_txtTotalPoints").value = value2 -=1 ;
        document.getElementById("ctl00_ContentPlaceHolder1_hidTotalPoints").value = document.getElementById("ctl00_ContentPlaceHolder1_txtTotalPoints").value
        document.getElementById("ctl00_ContentPlaceHolder1_hidDef").value = document.getElementById("ctl00_ContentPlaceHolder1_txtDefence").value 

        }

    }
        function SubbDefence(args)
    {
        //Specifically subtract wisodm while increasing total points
        var value1 = 0;
        value1 = document.getElementById("ctl00_ContentPlaceHolder1_txtDefence").value;
        var value2 = 0;
        value2 = document.getElementById("ctl00_ContentPlaceHolder1_txtTotalPoints").value;
        if (value1 == 8){
        alert("You CAN NOT lower a skill past 8");
        }
        else{
        document.getElementById("ctl00_ContentPlaceHolder1_txtDefence").value = value1 -=1 ;
        document.getElementById("ctl00_ContentPlaceHolder1_txtTotalPoints").value = value2 -=(-1) ;
        document.getElementById("ctl00_ContentPlaceHolder1_hidTotalPoints").value = document.getElementById("ctl00_ContentPlaceHolder1_txtTotalPoints").value
        document.getElementById("ctl00_ContentPlaceHolder1_hidDef").value = document.getElementById("ctl00_ContentPlaceHolder1_txtDefence").value 
       
        }

    }
    </script>

I just want it to copy the value it makes into both the text boxes (txt***) and the hidden fields(hid***)