please correct the error in this code ?

Question

2.33/5 (3 votes)

See more:

HI Please see this code

Response.Write("<form name=forml action =die1.aspx method=get id=form1>")
Response.Write("<table>")
  Response.Write("<tr>")...
While (dr.Read())
Response.Write("<tr>")
Response.Write("<td>" + dr("XGENAME").ToString() + "</td><td>" + dr("EANAME").ToString() + "</td>")
Response.Write("<td>")
Response.Write("<input type='textbox' id ='fone'>")
Response.Write("</td>")
Response.Write("<td>")
Response.Write("<input type='textbox' id='fadd'>")
Response.Write("</td>")
Response.Write("<td>")
Response.Write("<input type =textbox id =flast>")
Response.Write("</td>")
Response.Write("<td>")
Response.Write("<input type =textbox id =dexp>")
Response.Write("</td>")
Response.Write("</tr>")
End While
orc.Dispose()
Response.Write("</table>")
Response.Write("<input type = submit value = submit>")
Response.Write("<center>")
Response.Write("</form>")
how to get the value of the textboxes in the action page

Posted 28-Jun-11 2:39am

hemantwithu

Add a Solution

5 solutions

Solution 3

If you are posting to another page, see http://msdn.microsoft.com/en-us/library/ms178139.aspx[^]

Posted 28-Jun-11 3:30am

MorganSimon

Solution 4

You have the id attribute but you are missing the name attribute (they can be the same). If you are posting to an action page, the values won't be in the form unless the form elements have the name attribute specified.

<input type="textbox" id="fone" name="fone"></input>

Cheers.

Posted 28-Jun-11 3:34am

TheyCallMeMrJames

Solution 5

First, as MrJames pointed out, to get values in a form postback you need to set the name attribute, not id. (id is used by JavaScript, if you have none of that, you don't need to set it at all. It is allowed to have id and name be the same.)

If there is more than one row, you will be duplicating names – you should include some unique key from dr in the name.

Posted 28-Jun-11 5:43am

BobJanova

Solution 1

single codes are missing in form tag.rectify that..

Posted 28-Jun-11 2:41am

IbrahimPoonawala

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)

#realJSOP · Accepted Answer · 2011-06-28T02:49:00

First, instead of doing all of those Response.Write calls, use a StringBuilder to build the entire output, and to set the text boxes, simply use the StringBuilder.AppendFormat() method, like so:

VB

Dim html As New StringBuilder("")
html.Append("<form name=\"forml\" action=\"die1.aspx\" method=\"get\" id=\"form1\">")
html.Append("<table>")
While (dr.Read())
    html.Append("<tr>")
    html.AppendFormat("<td>{0}</td><td>{1}", dr("XGENAME").ToString(), dr("EANAME").ToString())
    html.AppendFormat("<table><tbody><tr><td><input type=\"textbox\" id=\"fone\" text=\"{0}\" /></td></tr></tbody></table>", dr("FONE").ToString())
    html.AppendFormat("<table><tbody><tr><td><input type=\"textbox\" id=\"fadd\" text=\"{0}\" /></td></tr></tbody></table>", dr("FADD").ToString())
    html.AppendFormat("<table><tbody><tr><td><input type=\"textbox\" id=\"flast\" text=\"{0}\" /></td></tr></tbody></table>", dr("FLAST").ToString())
    html.AppendFormat("<table><tbody><tr><td><input type=\"textbox\" id=\"dexp\" text=\"{0}\" /></td></tr></tbody></table>", dr("DEXP").ToString())
    html.Append("</td></tr>")
End While
orc.Dispose()
html.Append("</table>")
html.Append("<input type=\"submit\" value=\"submit\">")
html.Append("</input></form>")
Response.Write(html)

That code is much cleaner and easier to maintain (not to mention much shorter).