peddel Posted August 11, 2008 Share Posted August 11, 2008 Im getting items from some cells im using for a intranet form I got 12 cells. When i use the first cell the form works perfectly, when code for all 1é cells the form doesnt function good anymore. Anyone sees problems in this ? Working code : if(waarde_21M1_1 == ""){alert("Gelieve een waarde voor condensor 1 op te geven. (21M1 1)"); return false;} Need to find problem in this part : if(waarde_21M1_2 == ""){alert("Gelieve een waarde voor condensor 1 op te geven. (21M1 2)"); return false;} if(waarde_21M1_3 == ""){alert("Gelieve een waarde voor condensor 1 op te geven. (21M1 3)"); return false;} if(waarde_21M1_4 == ""){alert("Gelieve een waarde voor condensor 2 op te geven. (21M1 4)"); return false;} if(waarde_21M1_5 == ""){alert("Gelieve een waarde voor condensor 2 op te geven. (21M1 5)"); return false;} if(waarde_21M1_6 == ""){alert("Gelieve een waarde voor condensor 2 op te geven. (21M1 6)"); return false;} if(waarde_21M1_7 == ""){alert("Gelieve een waarde voor condensor 3 op te geven. (21M1 7)"); return false;} if(waarde_21M1_8 == ""){alert("Gelieve een waarde voor condensor 3 op te geven. (21M1 "); return false;} if(waarde_21M1_9 == ""){alert("Gelieve een waarde voor condensor 3 op te geven. (21M1 9)"); return false;} if(waarde_21M1_10 == ""){alert("Gelieve een waarde voor condensor 4 op te geven. (21M1 10)"); return false;} if(waarde_21M1_11 == ""){alert("Gelieve een waarde voor condensor 4 op te geven. (21M1 11)"); return false;} if(waarde_21M1_12 == ""){alert("Gelieve een waarde voor condensor 4 op te geven. (21M1 12)"); return false;} Im double and tripple checked it but cant find the error, its not a problem with getting value out of the form, cuz i get all values after testing with alert Quote Link to comment Share on other sites More sharing options...
peddel Posted August 11, 2008 Author Share Posted August 11, 2008 i solved it, had a typo in my form I hate javascript when it comes to erroring and not telling me why Quote Link to comment Share on other sites More sharing options...
peddel Posted August 11, 2008 Author Share Posted August 11, 2008 Fek put my hopes up too soon : This is my full code for this part to work, i selected it from my original code. var waarde1 = document.forms[0].waarde1.value; var waarde2 = document.forms[0].waarde2.value; var waarde3 = document.forms[0].waarde3.value; var waarde4 = document.forms[0].waarde4.value; var waarde5 = document.forms[0].waarde5.value; var waarde6 = document.forms[0].waarde6.value; var waarde7 = document.forms[0].waarde7.value; var waarde8 = document.forms[0].waarde8.value; var waarde9 = document.forms[0].waarde9.value; var waarde10 = document.forms[0].waarde10.value; var waarde11 = document.forms[0].waarde11.value; var waarde12 = document.forms[0].waarde12.value; if(waarde1 == ""){alert("Gelieve een waarde voor condensor 1 op te geven. (21M1 1)"); return false;} if(waarde2 == ""){alert("Gelieve een waarde voor condensor 1 op te geven. (21M1 2)"); return false;} if(waarde3 == ""){alert("Gelieve een waarde voor condensor 1 op te geven. (21M1 3)"); return false;} if(waarde4 == ""){alert("Gelieve een waarde voor condensor 2 op te geven. (21M1 4)"); return false;} if(waarde5 == ""){alert("Gelieve een waarde voor condensor 2 op te geven. (21M1 5)"); return false;} if(waarde6 == ""){alert("Gelieve een waarde voor condensor 2 op te geven. (21M1 6)"); return false;} if(waarde7 == ""){alert("Gelieve een waarde voor condensor 3 op te geven. (21M1 7)"); return false;} if(waarde8 == ""){alert("Gelieve een waarde voor condensor 3 op te geven. (21M1 "); return false;} if(waarde9 == ""){alert("Gelieve een waarde voor condensor 3 op te geven. (21M1 9)"); return false;} if(waarde10 == ""){alert("Gelieve een waarde voor condensor 4 op te geven. (21M1 10)"); return false;} if(waarde11 == ""){alert("Gelieve een waarde voor condensor 4 op te geven. (21M1 11)"); return false;} if(waarde12 == ""){alert("Gelieve een waarde voor condensor 4 op te geven. (21M1 12)"); return false;} <tr> <td class="kortvak" align="center">21M1 1</td> <td class="kortvak" align="center"><input type="text" name="waarde1" size="10" /></td> <td class="kortvak" align="center">21M1 4</td> <td class="kortvak" align="center"><input type="text" name="waarde4" size="10" /></td> <td class="kortvak" align="center">21M1 7</td> <td class="kortvak" align="center"><input type="text" name="waarde7" size="10" /></td> <td class="kortvak" align="center">21M1 10</td> <td class="kortvak" align="center"><input type="text" name="waarde9" size="10" /></td> </tr> <tr> <td class="kortvak" align="center">21M1 2</td> <td class="kortvak" align="center"><input type="text" name="waarde2" size="10" /></td> <td class="kortvak" align="center">21M1 5</td> <td class="kortvak" align="center"><input type="text" name="waarde5" size="10" /></td> <td class="kortvak" align="center">21M1 8</td> <td class="kortvak" align="center"><input type="text" name="waarde8" size="10" /></td> <td class="kortvak" align="center">21M1 11</td> <td class="kortvak" align="center"><input type="text" name="waarde11" size="10" /></td> </tr> <tr> <td class="kortvak" align="center">21M1 3</td> <td class="kortvak" align="center"><input type="text" name="waarde3" size="10" /></td> <td class="kortvak" align="center">21M1 6</td> <td class="kortvak" align="center"><input type="text" name="waarde6" size="10" /></td> <td class="kortvak" align="center">21M1 9</td> <td class="kortvak" align="center"><input type="text" name="waarde9" size="10" /></td> <td class="kortvak" align="center">21M1 12</td> <td class="kortvak" align="center"><input type="text" name="waarde12" size="10" /></td> </tr> ANyone sees a flaw inhere? Quote Link to comment Share on other sites More sharing options...
peddel Posted August 11, 2008 Author Share Posted August 11, 2008 friend found it i used 2 times waarde9 and didnt use waarde10 so he errored on the last Quote Link to comment Share on other sites More sharing options...
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.