Re: Plz find out error in my code. - ASP.NET Discussion Boards

Sujit Gupta24-Apr-07 7:51

24-Apr-07 7:51

Call on page load even it will work properly

page load()
{
bt.Attributes.Add("OnClick", "return window.confirm('Are you sure want to remove this user?');");
}

Sujit

Re: window.confirm

badgrs24-Apr-07 5:51

badgrs

24-Apr-07 5:51

Why not just add it to the tag in the aspx page?

<asp:LinkButton OnClientClick="return window.confirm('Are you sure want to remove this user?');"></asp:LinkButton>

Plz find out error in my code.

Care Career24-Apr-07 2:27

Care Career

24-Apr-07 2:27

Hi plz help
This code is not working ,i m using asp.net 2.0 with C#

for (int i = 0; i < ds.Tables[0].Rows.Count; i++)
{
if ((txtUser.Text == ds.Tables["CreateAccount"][i]["User_Name"]) && (txtPassword.Text == ds.Tables["CreateAccount"][i]["Password"]))
Response.Write("You have succesfully logged in");
else
Response.Write("User Name Does Not Exist");
}

Thanks
CARE CAREER

Re: Plz find out error in my code.

Colin Angus Mackay24-Apr-07 2:32

Colin Angus Mackay

24-Apr-07 2:32

Care Career wrote:
This code is not working

That is vague. What do you mean by "not working"?

Upcoming events:
* Glasgow: Introduction to AJAX (2nd May), SQL Server 2005 - XML and XML Query Plans, Mock Objects, SQL Server Reporting Services...

Never write for other people. Write for yourself, because you have a passion for it. -- Marc Clifton

My website

Re: Plz find out error in my code.

Russell Jones24-Apr-07 2:42

Russell Jones

24-Apr-07 2:42

I'd probably use something like

if ((ds.Tables["CreateAccount"].Select(string.format("User_Name = '{0}' AND Password = '{1}'", new object []{txtUser.Text,txtPassword.text})).length ==1)
//login message
else
//failed message

except i wouldn't process the message like that, i'd do a query such as:

Select count(*) from users where User_Name = <username> AND Password = <password>

and check that the return value was 1. The danger would be that someone will pick up your asp.net login function and deploy it in winforms where the client would have a list of all the username / password entries in the database.

Russell

Re: Plz find out error in my code.

Md. Ismail24-Apr-07 2:50

Md. Ismail

24-Apr-07 2:50

The code will produce output for all row of the table. So user never understand that he successfully logged in or not.

Re: Plz find out error in my code.

Dave Sexton24-Apr-07 3:16

Dave Sexton

24-Apr-07 3:16

Well firstly you can't apply indexing with [] to a DataTable - it should be more like this (note bold text)

for (int i = 0; i < ds.Tables[0].Rows.Count; i++)
            {
                if ((txtUser.Text == ds.Tables["CreateAccount"].Rows[i]["User_Name"]) &&
                    (txtPassword.Text == ds.Tables["CreateAccount"].Rows[i]["Password"]))
                {
                    Response.Write("You have succesfully logged in");
                }
                else
                {
                    Response.Write("User Name Does Not Exist");
                }
            }

Also - I'm no C# guru but I find using the curly braces helps improve readability - one of the experts can correct me if I'm wrong but I don't see any harm in using curly braces in your "if" statement in this context.

Secondly, how do you know that the username doesn't exist if the password is wrong? I'd suggest either breaking it down into 2 separate helper functions to determine where the problem is or (easier) changing the response to "Either the username or password is incorrect".

Thirdly, you are passing the password as plaintext, which leads me to assume you're saving the password as plain text - bad idea. Rather use a hash to encrypt the password before processing or saving. Take a look at this[^]

Re: Plz find out error in my code.

DeepToot24-Apr-07 5:11

DeepToot

24-Apr-07 5:11

Have you tried putting a .ToString() at the end of the DataSet?

<br />
for (int i = 0; i < ds.Tables[0].Rows.Count; i++)<br />
{<br />
    if ((txtUser.Text == ds.Tables["CreateAccount"].Rows[i]["User_Name"].ToString()) &&<br />
       (txtPassword.Text == ds.Tables["CreateAccount"].Rows[i]["Password"].ToString()))<br />
     {<br />
                    Response.Write("You have succesfully logged in");<br />
     }<br />
     else<br />
     {<br />
                    Response.Write("User Name Does Not Exist");<br />
     }<br />
}<br />

Steve Welborn
Software Engineer
BitWise Solutions

Re: Plz find out error in my code.

DeepToot24-Apr-07 5:13

DeepToot

24-Apr-07 5:13

And to add to that in your for loop, its zero based..you need to make it like this:

<br />
for (int i = 0; i < ds.Tables[0].Rows.Count - 1; i++)<br />
{<br />
<br />
}<br />

Steve Welborn
Software Engineer
BitWise Solutions

Re: Plz find out error in my code.

Dave Sexton24-Apr-07 10:36

Dave Sexton

24-Apr-07 10:36

I did it "off-the-cuff" & missed that one (oops! & thanks), what did you think of the rest or the advice?

Re: Plz find out error in my code.

DeepToot24-Apr-07 5:21

DeepToot

24-Apr-07 5:21

ismailiiuc is right. This will loop through the whole DataSet.

what you should do is take it out of a loop all together. Its not needed.

There should only be one username and password in there.

after you fill your DataSet just do this:

<br />
 if (txtUser.Text == ds.Tables["CreateAccount"][0]["User_Name"].ToString() &&  txtPassword.Text == ds.Tables["CreateAccount"][0]["Password"].ToString())<br />
{<br />
  Response.Write("You have succesfully logged in");<br />
}<br />
else<br />
{<br />
  Response.Write("User Name Does Not Exist");<br />
}<br />

Steve Welborn
Software Engineer
BitWise Solutions

Re: Plz find out error in my code.

Sujit Gupta24-Apr-07 7:57

Sujit Gupta

24-Apr-07 7:57

use this to find data from table

ds.Tables["CreateAccount"].rows[i]["User_Name"].tostring();
or
ds.Tables[0].rows[i]["User_Name"].tostring();

Sujit

print xsl

TheEagle24-Apr-07 2:13

TheEagle

24-Apr-07 2:13

I have a webform that includs the xml web server control which uses the xml file and xsl file to show my CV data(doesn't use ado.net) So how could I get a printable format like PDF from this page?.

I am too late but i will never give up

Configurationsetting Class?

Care Career24-Apr-07 0:25

Care Career

24-Apr-07 0:25

Hi
Tell me In asp.net CofigurationSetting class from which Namespace is derived.
& also tell me Wht is the diff. b/w Covert & cast in SQL SERVER.

Thanks & Regards

CARE CAREER

Re: Configurationsetting Class?

Colin Angus Mackay24-Apr-07 0:36

Colin Angus Mackay

24-Apr-07 0:36

Care Career wrote:
Tell me In asp.net CofigurationSetting class from which Namespace is derived.

It isn't in ASP.NET framework. It is in the main body of the .NET framework in the system.Configuration namespace.

Care Career wrote:
tell me Wht is the diff. b/w Covert & cast in SQL SERVER

Now this really ought to be in a different forum. But here's the documentation for CAST and CONVERT[^]

Re: Configurationsetting Class?

Sandeep Kumar24-Apr-07 0:50

Sandeep Kumar

24-Apr-07 0:50

Hi,

Diff. between CAST and CONVERT

If you're just changing data from one form to another for use, such as changing a "31" from a varchar to an int, use CAST; Convert offers more options than cast which may lead to more processing time. And by this, I of course mean MINIMAL, not noticeable processing time. Convert allows you to add a format style to your convertion, if you want it to. For example, if you're changing a date to a string, you can say which part of the date to return instead of returning the whole thing and making .NET format it.

CONVERT is specific to SQL Server, and allows for a greater breadth of flexibility when converting between date and time values, fractional numbers, and monetary signifiers.

Regards,
Sandeep Kumar.V

Re: Configurationsetting Class?

Sathesh Sakthivel24-Apr-07 1:40

Sathesh Sakthivel

24-Apr-07 1:40

Explicitly converts an expression of one data type to another. CAST and CONVERT provide similar functionality.

Syntax
Using CAST:

CAST ( expression AS data_type )

Using CONVERT:

CONVERT ( data_type [ ( length ) ] , expression [ , style ] )

Regards,

Satips.

Adding a row to ASP.NET table programatically

Rohde24-Apr-07 0:01

Rohde

24-Apr-07 0:01

Hi,

I have an .aspx page with an asp:Table and a button. When the button is clicked I want the clik event handler to add a row to the table programatically.

The code in the event handler is like so:

<br />
        // Create a new row.<br />
        TableRow row = new TableRow();<br />
<br />
        TableCell cell = new TableCell();<br />
        cell.BorderColor = Color.Black;<br />
        cell.BorderStyle = BorderStyle.Dashed;<br />
        cell.BorderWidth = new Unit("1");<br />
<br />
        <br />
        cell.Text = "test";<br />
<br />
        row.Cells.Add(cell);<br />
        <br />
<br />
        tblEval.Rows.Add(row);<br />
<br />

Unfortuantely I can only see the first row getting added on the page - subsequent button clicks have no effect; although when stepping through in the debugger I can see that the asp:table actually gets the new rows added to it, I just can't see it.

What am I doing wrong? Any help is very much appreciated.

"When you have made evil the means of survival, do not expect men to remain good. Do not expect them to stay moral and lose their lives for the purpose of becoming the fodder of the immoral. Do not expect them to produce, when production is punished and looting rewarded. Do not ask, `Who is destroying the world?' You are."

-Atlas Shrugged, Ayn Rand

Re: Adding a row to ASP.NET table programatically

Spunky Coder24-Apr-07 0:48

Spunky Coder

24-Apr-07 0:48

Are you binding the table to datagrid or datalist to display it...if so update the datagrid by binding it again in the buttonclick event.

Koushik

Re: Adding a row to ASP.NET table programatically

Rohde24-Apr-07 1:06

Rohde

24-Apr-07 1:06

No, I'm not binding it to anything at all. The table is declared in the .aspx table and the idea is that every button click adds a new row to it which the user can then fill out.

Re: Adding a row to ASP.NET table programatically

enjoycrack24-Apr-07 1:12

enjoycrack

24-Apr-07 1:12

As the web is stateless env...that's why you dont know how many times the user clicks on this button...So just an idea..store the number of clicks into session variable

<< >>

8x Solutions Ltd

Re: Adding a row to ASP.NET table programatically

Rohde24-Apr-07 1:58

Rohde

24-Apr-07 1:58

Thank you. I tried to do this and now I can get it to works. D

ynamically created tables isn't part of the view state, and apparently that also pertains to the table structure itself.

Anyway, now I can get the table constructed as I wanted, now I just need to save the table contents proper.

Re: Adding a row to ASP.NET table programatically

Nouman Bhatti24-Apr-07 1:17

Nouman Bhatti

24-Apr-07 1:17

you should do it like this

row = tblEval.newRow()
tblEval.Rows.Add(row)

Asp.net page cannot be displayed

chater buster24-Apr-07 0:00

chater buster

24-Apr-07 0:00

Dear All

My application folder is C:\Inetpub\wwwroot\webapp

I gave full permission to the folder for the aspnet user = IUSR_MYSYSNAME

My application is available on local machine with http://localhost/webapp or http://IPADDRESS/webapp but when i access from LAN its NOT available and page error = "Page cannot be displayed"

Pls. Help me whats wrong. I have a deadline to launch
Thanks
MY

Re: Asp.net page cannot be displayed

Sandeep Kumar24-Apr-07 0:29

Sandeep Kumar

24-Apr-07 0:29

Hi,

Due to Windows Security errors in WindowsXP SP2 u will be unable to acces the page.

so inorder to acces the application you need to instal windowsXP sp1 or windowsXP operating system insted of SP2....

Regards,
Sandeep Kumar.V

Use Ctrl+Left/Right to switch messages, Ctrl+Up/Down to switch threads, Ctrl+Shift+Left/Right to switch pages.