why the form cannot be submit?

Question

0.00/5 (No votes)

See more:

my code is here,when the submit button clicked ,the form cannot be post.and i add some code in js

C#

else if (oCheck[i].type == 'button') {
               oCheck[i].onclick = function () {
                   alert("ddddd");
               }
           }

to see what happened with the button ,but the expected messagebox "ddddd" was not even popup.what's wrong with my code?

HTML

@using (Html.BeginForm())
{
    @Html.ValidationSummary(true)
    <fieldset>
        
       <div id="somecheckbox">
       <input type="checkbox" value="somevalue" />
       <input type="checkbox" value="somevalue" />
       <input type="checkbox" value="somevalue" />
       </div>

	<input type="text" id="edittobeset" />
        <p>
            <input type="submit" value="submit" />
            <input type="reset"  value="reset" />
        </p>
    </fieldset>
}

<script type="text/javascript">

    window.onload = function () {
        var divCheck = document.getElementById("somecheckbox");
        var oCheck = divCheck.getElementsByTagName("input");
        var oSet = document.getElementById("edittobeset");
        
        for (var i = 0; i < oCheck.length; i++) {
            if (oCheck[i].type == 'checkbox') {
                oCheck[i].onclick = function () {

                    if (this.checked == true) {
                        oSet.value += this.value + ".";
                    }
                }
            }
            else if (oCheck[i].type == 'button') {
                oCheck[i]. önclick = function () {
                    alert("ddddd");
                }
            }
        }
   }
</script>

Posted 6-Jun-12 8:13am

lamparded

Updated 6-Jun-12 8:58am

Sandeep Mewara

v5

Add a Solution

2 solutions

Solution 1

The problem you have is the control you're looking for is not in the collection your iterating.

The for loop:

JavaScript

for (var i = 0; i < oCheck.length; i++)

Is checking the oCheck object for controls. oCheck is:

JavaScript

var oCheck = divCheck.getElementsByTagName("input");

divCheck is:

JavaScript

var divCheck = document.getElementById("somecheckbox");

This means, in your for loop, you checking the contents of 'somecheckbox' for input elements. The input element you're trying to find isn't a child to 'somecheckbox' and as such would not part of the oCheck collection.

As you're only targeting a single submit button I'd change the code to the following:

JavaScript

window.onload = function () {
       var divCheck = document.getElementById("somecheckbox");
       var oCheck = divCheck.getElementsByTagName("input");
       var oSet = document.getElementById("edittobeset");

       for (var i = 0; i < oCheck.length; i++) {
           if (oCheck[i].type == 'checkbox') {
               oCheck[i].onclick = function () {

                   if (this.checked == true) {
                       oSet.value += this.value + ".";
                   }
               }
           }
       }

       var oSubmit = document.getElementsByTagName('submit')[0];
       oSubmit. önclick = function () {
                   alert("ddddd");
               }

  }

Posted 6-Jun-12 8:23am

Stephen Hewison

Updated 6-Jun-12 11:12am

v3

Comments

lamparded 6-Jun-12 14:37pm

even i add a <input type="button" /> in the form. there is no chanage .the ddddd messagebox not displayed .

taha bahraminezhad Jooneghani 6-Jun-12 16:48pm

change button to submit!

Stephen Hewison 6-Jun-12 17:04pm

Sorry, just found the problem. Going to rewrite the solution!

Wonde Tadesse 6-Jun-12 20:35pm

You don't need to evaluate "if (this.checked == true){ ... }" Instead use "if(this.checked) { ...}"

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

lamparded · Accepted Answer · 2012-06-06T14:32:00

Solution 2

i made a mistake. type a wrong letter in my action function name.

Posted 6-Jun-12 14:32pm

lamparded

Comments

lamparded 6-Jun-12 20:41pm

public ActionResult RestaurantAdd()
[HttpPost]
public ActionResult RestaurandAdd()

here ,the Restaurand <--> Restaurant