Kan ikke finde fejlene i min validering af et input felt.

Jeg ville nok bruge regexp i stedet, fx noget a la:

function tjekTal(str){
  var re1 = /^[0-9][,.]?[0-9]*$/;
  var re2 = /^-?[0-9][,.]?[0-9]*$/;
  if (re2.test(str))
    if (re1.test(str)){
      return str+" er ok"; 
    }
    else {
      return str+" er mindre end 0";
    }
  else {
    return str+" er ikke korrekt";
  }
}

Hov jeg kan se jeg glemte at skrive hvad der ikke virkede :( Jeg får ikke vist min confirm (vises i advar(m)) - jeg sidder på en Mac og bruger safari/mozilla, men det skal også virke i internet explore. Jeg har nu taget dit regexp i brug i stedet for, men får altså stadig ikke min alert:

function validate(){
          var bananBeloeb = konverterTilFloat(getE('bananBeloeb').value);
          if (!tjekPosTal(bananBeloeb)) {
            alert("Det indtastede beløb skal være et positivt tal!");       
            getE('bananBeloeb').focus();
            return false;
            }
          else{
            return advar(getE('indbetaler').value)
          }
        }

        function tjekPosTal(str)
        {
          var re1 = /^[0-9][,.]?[0-9]*$/;
          var re2 = /^-?[0-9][,.]?[0-9]*$/;
          if (re2.test(str))
            if (re1.test(str)){
              return true; //er korrekt
            }
            else {
              return false;//er mindre end 0
            }
          else {
            return false; //er ikke korrekt
          }
        }
       
        function advar(m)
        {
            if(confirm("Er du sikker på at du vil registere en indtægt betalt af "+m))
            {
            return true;
            }
            return false;
        }
       
        function getE(id)
        {
            return document.getElementById(id);
        }

Hvorfor ikke bare:

        function validate(){
          var bananBeloeb = getE('bananBeloeb').value;
          if (!bananBeloeb.match(/^\d+([.,]\d+)$/)) {
            alert("Det indtastede beløb skal være et positivt tal!");       
            getE('bananBeloeb').focus();
            return false;
            }
          else{
            return confirm("Er du sikker på at du vil registere en indtægt betalt af " + getE('indbetaler').value));
          }
        }

-- det klarer hele baduljen (getE-funktionen skal selvfølgelig stadig være der !-)

Hov, der manglede lige et spørgsmålstegn:

          if (!bananBeloeb.match(/^\d+([.,]\d+)?$/)) {

>>trp79
Du bør nok fjerne 'konverterTilFloat' i:
var bananBeloeb = konverterTilFloat(getE('bananBeloeb').value);

Jeg får stadig ingen pop-up når jeg gør følgende:

    <script type="text/javascript">
        function validate(){
          var bananBeloeb = getE('bananBeloeb').value;
          if(!bananBeloeb.match(/^\d+([.,]\d+)?$/)) {
            alert("Det indtastede beløb skal være et positivt tal!");       
            getE('bananBeloeb').focus();
            return false;
            }
          else{
            return confirm("Er du sikker på at du vil registere en indtægt betalt af " + getE('indbetaler').value));
          }
        }
       
        function getE(id)
        {
            return document.getElementById(id);
        }
    </script>

Måske det skyldes jeg sidder på en Mac ? Jeg har prøvet i safari og firefox, men ingen af dem giver en pop up - funktionen bliver altså blot gennemført.

Bliver validate() overhovedet kaldt? Den burde jo give en pop-up uanset hvad.

Hrm, jeg kan heller ikke finde grunden, men der bør faktisk forkortes en anelse mere:

    <script type="text/javascript">
        function validate(){
          var bananBeloeb = getE('bananBeloeb').value;
          if(!bananBeloeb.match(/^\d+([.,]\d+)?$/)) {
            alert("Det indtastede beløb skal være et positivt tal!");       
            getE('bananBeloeb').focus();
            return false;
          }
          return confirm("Er du sikker på at du vil registere en indtægt betalt af " + getE('indbetaler').value));
        }
       
        function getE(id){
            return document.getElementById(id);
        }
    </script>

Det er altså lidt underligt - jeg får ingen pop-up.
Min jsp side genererer denne kildekode:

<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN"
"http://www.w3.org/TR/html4/loose.dtd">
<html xmlns="http://www.w3.org/1999/xhtml" sml:lang="da" lang="da">
  <head>
    <title>Banan Indtægt</title>
    <script type="text/javascript">
        function validate(){
          var bananBeloeb = getE('bananBeloeb').value;
          if(!bananBeloeb.match(/^\d+([.,]\d+)?$/)) {
            alert("Det indtastede beløb skal være et positivt tal!");       
            getE('bananBeloeb').focus();
            return false;
          }
          return confirm("Er du sikker på at du vil registere en indtægt betalt af " + getE('indbetaler').value));
        }
       
        function getE(id){
            return document.getElementById(id);
        }
    </script>
    <link rel="stylesheet" type="text/css"
    href="../css/common.css" media="screen" />
  </head>
  <body>
   


<div id="rightcol">
    <ul id="menu">
      <li><a href=""></a></li>
      <li><a href="/Banankasse-context-root/legaliseret/oversigt.jsp">&raquo;&nbsp;Min oversigt</a></li>
      <li><a href="/Banankasse-context-root/legaliseret/minebananer.jsp">&raquo;&nbsp;Mine bananer</a></li>
      <li><a href="/Banankasse-context-root/legaliseret/minebananIndb.jsp">&raquo;&nbsp;Mine indbetalinger</a></li>
      <li><a href="/Banankasse-context-root/legaliseret/holdmedlemmer.jsp">&raquo;&nbsp;Hold medlemmer</a></li>
      <li><a href="/Banankasse-context-root/legaliseret/holdforumkategorier.jsp">&raquo;&nbsp;Hold forums</a></li>
      <li><a href="/Banankasse-context-root/legaliseret/holdbananer.jsp">&raquo;&nbsp;Hold bananer</a></li>
      <li><a href="/Banankasse-context-root/legaliseret/holdbanantyper.jsp">&raquo;&nbsp;Hold banantyper</a></li>
      <!--<li><a href="na.jsp">&raquo;&nbsp;Hold lister</a></li>-->
      <li><a href="/Banankasse-context-root/legaliseret/tilmeld_til_hold.jsp">&raquo;&nbsp;Holdtilmelding</a></li>
      <li><a href="/Banankasse-context-root/legaliseret/rediger_bruger.jsp">&raquo;&nbsp;Mine oplysninger</a></li>
      <li><a href="/Banankasse-context-root/index.jsp?handling=log_ud">&raquo;&nbsp;Log ud</a></li>
      <!--<li><a href="templateL.jsp">&raquo;&nbsp;Template</a></li>-->
    </ul>
</div>
    <div id="banner">
  <img src="../gif/banana.gif" height="75" width="73"></img>[Banankassen]
</div>
    <div id="indhold">
      <h3 class="obs advarsel">Registrer en indtægt til medlemmer af Amager SK - Herre</h3>
     
      <form action="bananIndtaegter.jsp" method="post" onsubmit="return validate()">
      <table>
        <tr>
          <td>Beløb</td>
          <td>Indbetaler</td>
          <td>Kommentar</td>
        </tr>
        <tr>
          <td>
            <input type="HIDDEN" name="hold" value="Amager SK - Herre">
            <input type="TEXT" name="bananBeloeb" value="12"></input>
          </td>
          <td>
            <select name="indbetaler" id="indbetaler">
           
                  <option value="Torben"> Torben </option>
                  <option value="Torben2"> Torben2 </option>
                </td>
             
          <td>
            <input type="TEXT" name="kommentar" value="Betaling var i banankassen besiddelse den " maxlength="60" size="45"></input>
          </td>
        </tr>
        <tr>
          <td colspan="4">Vær opmærksom på, at det kan tager lidt tid at registrere indbetalingen</td>
        </tr>
      </table>
        <input type="SUBMIT" name="handling" value="registrer_indbetaling">
      </form>
     
      <br/>
      <h3 class="obs advarsel">
      Der blev succesfuldt registreret en indbetaling på kr.12.0 på Torben
      </h3>
    </div>
  </body>
</html>

Nogen der har nogle ideer?

jeg får en js fejl om at den mangler ; Men det kan jeg da ikke lige se?

Arh, der var en slut parantes for meget i confirm :)

Tak for hjælpen, det virker bare som det skal nu.

Er i ikke venlige at smide et svar hver især...

Velbekomme... ;-)

>>tjp

-- den skal du vist prøve 'en gång mer' !-)

Velbekomme '-)

tjp vil du prøve at smide et svar igen....

;O)

time out tjp....

Tak for point ;~}