data reader

Question

plz tell me about this    i removed some extra code


 private void save_Click(object sender, EventArgs e)


   {

            SqlCommand cmd = new SqlCommand("SELECT INVESTOR_CNIC,INVESTOR_ID  FROM add_investor", my);
               my.Open();
           SqlDataReader dr = cmd.ExecuteReader();
                dr.Read();

                if (dr.HasRows)

               { if ((dr["INVESTOR_CNIC"].ToString() == investorcnic.Text) )........the CNIC is same as  investorcnic.Text  but control skip this portion and exicute nest protion why
                   {
                       MessageBox.Show("found");

                        dr.Close();
                        SqlCommand cmdd = new SqlCommand("INSERT INTO investor_detail (investor_id,investor_amount,date,pay_amount,remaining_amount  ) VALUES ('" + investorid.Text + "','" + investoramount.Text + "','" + dateTimePicker.Value + "','" + payamount.Text + "','" + remainnigamount.Text + "')", my);
                        cmdd.ExecuteNonQuery();

                   }
               else                   .................................this portion is exicuted
               {
                   dr.Close();    

                          SqlCommand cmds = new SqlCommand("INSERT INTO add_investor (investor_name,investore_address,investor_cnic,investor_contact) VALUES ('" + investorname.Text + "','" + investoraddress.Text + "','" + investorcnic.Text + "','" + investorno.Text + "')", my);
                          cmds.ExecuteNonQuery();
                         SqlCommand cmdda = new SqlCommand("INSERT INTO investor_detail (investor_id,investor_amount,date,pay_amount,remaining_amount) VALUES ('" + investorid.Text + "','" + investoramount.Text + "','" + dateTimePicker.Value + "','" + payamount.Text + "','" + remainnigamount.Text + "')", my);
                          cmdda.ExecuteNonQuery();
                        my.Close();
                        MessageBox.Show("Investor added  dddfdsfdfdf");
                  }  }

                                                                                                                          

Answer 1

Are you getting an exception.  Can you post the line that is highlighted when the exception occurs and the details of the error?

---

jdweng

Answer 2

thanks for your reply i realy need to solve this problem plz help

exception is in portion with which i write  "this portion is exicuted " i know the reason due to primery ki duplication but th condition    if ((dr["INVESTOR_CNIC"].ToString() == investorcnic.Text) )  is true

then why control skip its body part if this part will be exicuted next els part is skiped and error will be gone

is the way in which i am comparing reader value is not ok ?

Answer 3

Use dr["INVESTOR_CNIC"].Value or dr["INVESTOR_CNIC"].Value.ToString().

You don't have any exception handlers in the code which sometimes will skip around the code.  VS automatically puts an exception handler in an application before main is called which is the default exception handler.  When an exception occurs the code moves up the execution stack until the first exception handler is found.  If you don't have any in the code the default handler is found.  If a function doesn't have an exception handler then it move into the parent function looking for the exception handler and continues until an exception handler is found.  Some Net Library function have a built in exception handler (like Linq functionality) which may catch the exception.  by not having exception handlers the code is not stopping where the error is really occurring.

---

jdweng

Answer 4

only need to change is    dr.read()  to  while(dr.read())    every thing is ok

Answer 5

The reader is an enumerator object and you have to perform a read to get data.  Make one change to read only one row.

From :  if (dr.HasRows)

To :  if (dr.Read()))

---

jdweng

Answer 6

Hi mughal,

add where condition to select query

SqlCommand cmd = new SqlCommand("SELECT INVESTOR_CNIC,INVESTOR_ID  
FROM add_investor Where INVESTOR_CNIC='" + investorcnic.Text  + "'", my);

because datareader may check only the first record of the resultset

---

Happy Coding, RDRaja

Answer 7

Hi,

What ever code you wrote is correct better do trim for both sides.

if ((dr["INVESTOR_CNIC"].ToString().Trim() == investorcnic.Text.Trim()) ).

Answer 8

Hello,

Since the program executed dr.Read() once, it would get the first record from the collection generated by the sql statement.

You can set a temporary variable to accept the value of (dr["INVESTOR_CNIC"].ToString() to check whether its value equals with the value of payamount.Text.

Regards.

---

We are trying to better understand customer views on social support experience, so your participation in this interview project would be greatly appreciated if you have time. Thanks for helping make community forums a great place.
Click HERE to participate the survey.

Answer 9

I know this is the DataSet forum, but I don't see any dataset or related problem in your question.

Because I use Chrome and the forum software of Microsoft has to many bugs I cannot see what forum is the best for you.

But try to find the AdoNet Data forum. 

---

Success
Cor

Answer 10

Found it in a strange way

http://social.msdn.microsoft.com/Forums/en-US/home?category=dataplatformdev

---

Success
Cor