Why when I use choose gv_RowCommand, it display the same information

Question

0.00/5 (No votes)

See more:

SQL-Server-2008R2

This is my gv_RowCommand code behind. The problem is when I click button "cmdSelect" in different row,it will display the same information. All my data, the status is '2'. Updated code:

VB

Protected Sub gvCheckList_RowCommand(sender As Object, e As GridViewCommandEventArgs)
     Try
        If e.CommandName = "cmdSelect" Then
            Dim index As Integer = Int32.Parse(e.CommandArgument.ToString())
            Dim row As GridViewRow = gvCheckList.Rows(index)
            Dim MatricNo As String = gvCheckList.DataKeys(index).Value

            sql = "SELECT a.MatricNo, b.Name, b.Faculty, a.Date, a.KaunselingID, b.TelNo, " _
                & "c.Officer " _
                & "FROM KaunselingPL AS a " _
                & "INNER JOIN DbStudent..Personal AS b ON a.MatricNo = b.MatricNo " _
                & "INNER JOIN KaunselingDT AS c ON a.KaunselingID = c.KaunselingID " _
                & "WHERE (a.Status = '2' AND a.MatricNo = '" & MatricNo &"')"

            dbconn = New OleDbConnection(strCon)
            dbconn.Open()

            dbcomm = New OleDbCommand(sql, dbconn)
            dbread = dbcomm.ExecuteReader()
            dbread.Read()
            If (dbread.HasRows()) Then
               Me.lblName.Text = dbread("Name")
               Me.lblMatricNo.Text = dbread("MatricNo")
               Me.lblFaculty.Text = dbread("Faculty")
               Me.lblTelNo.Text = dbread("TelNo")
               Me.lblDate.Text = CDate(dbread("Date")).ToString("dd/MM/yyyy")
               Me.lblOfficer.Text = dbread("Officer")
            End If
            dbread.Close()
            dbcomm = Nothing
            dbconn.Close()
        End If
        Catch ex As Exception
            Response.Write(ex.Message.ToString())
     End Try
    End Sub

ASP.NET

<asp:gridView id="gvCheckList"  CssClass="gridViewTable" autogeneratecolumns="False" allowpaging="True"  Width="100%"  EmptyDataText="No Recordd" runat="server" PageSize="20" OnRowCommand="gvCheckList_RowCommand">
                               
                        <FooterStyle CssClass="gridViewFooter" />
                        <HeaderStyle CssClass="gridViewHeader" />
                        <AlternatingRowStyle CssClass="gridViewAlternating" />

                        <Columns> 
                            <asp:BoundField DataField="ROWNUMBER" HeaderText="No." HeaderStyle-Width="2%"></asp:BoundField>
                            <asp:ButtonField DataTextField="MatricNo" CommandName="cmdSelect" HeaderText="Matric No" meta:resourcekey="ButtonFieldResource1"  HeaderStyle-Width="15%" ></asp:Butt>
                            <asp:ButtonField DataTextField="Name" HeaderText="Name" HeaderStyle-Width="32%"></asp:Butt>
                            <asp:BoundField DataField="Faculty" HeaderText="Fakulti" HeaderStyle-Width="25%"></asp:BoundField>
                            <asp:BoundField DataField="Date" HeaderText="Date" HeaderStyle-Width="25%" DataFormatString="{0:dd/MM/yyyy}"></asp:BoundField> 

                            <asp:BoundField DataField="KaunselingID" HeaderText="Hidden">
                                <ItemStyle CssClass="hidden" VerticalAlign="Top" />
                                <HeaderStyle CssClass="hidden" />
                                <FooterStyle CssClass="hidden" />
                            </asp:BoundField> 
                        </Columns>

                        <FooterStyle CssClass="gridViewFooter" />
                        <HeaderStyle ForeColor="White" CssClass="gridViewHeader" />
                        
                        <PagerSettings FirstPageText="<<" LastPageText=">>" Mode="NextPreviousFirstLast" NextPageText=">" PreviousPageText="<" />
                   </asp:gridView>

Posted 21-Sep-14 15:06pm

U_Hanisa

Updated 24-Sep-14 14:25pm

v6

Add a Solution

Comments

virang_21 21-Sep-14 21:29pm

Your sql statement is not using any value from row. All I can see from where condition is status =2 which will give you same information all the time.

U_Hanisa 21-Sep-14 22:07pm

So what would I do to display all list data in grid view, then when I click button 'Select' on certain row data, it will display that information. Did you get what I mean.

ChauhanAjay 21-Sep-14 23:38pm

You mean to say that if you have a "Select" button on row1 then when you click on the select button it should display the data related to that row "row1". Kindly let me know if I have understood your problem in a correct manner, if not kindly correct me.

U_Hanisa 21-Sep-14 23:46pm

Yes, you right. And when I click select button on row2, it will display data related to row2.

ChauhanAjay 21-Sep-14 23:53pm

Try changing your query from

sql = "SELECT a.MatricNo, b.Name, b.Faculty, a.Date, a.KaunselingID, b.TelNo, " _
& "c.Officer " _
& "FROM KaunselingPL AS a " _
& "INNER JOIN DbStudent..Personal AS b ON a.MatricNo = b.MatricNo " _
& "INNER JOIN KaunselingDT AS c ON a.KaunselingID = c.KaunselingID " _
& "WHERE (a.Status = '2')"

To

sql = "SELECT a.MatricNo, b.Name, b.Faculty, a.Date, a.KaunselingID, b.TelNo, " _
& "c.Officer " _
& "FROM KaunselingPL AS a " _
& "INNER JOIN DbStudent..Personal AS b ON a.MatricNo = b.MatricNo " _
& "INNER JOIN KaunselingDT AS c ON a.KaunselingID = c.KaunselingID " _
& "WHERE (a.Status = '2' and a.MatricNo='"& index.ToString() &"')"

Try the above query and let me know the result.

ChauhanAjay 22-Sep-14 2:20am

Sorry I dont understand what you want to say.

U_Hanisa 22-Sep-14 20:12pm

When I debug,it shows error 'No data exists for the row/column'

ChauhanAjay 22-Sep-14 21:15pm

Basing on which value do you want to display the result. For example I thought that basing on MatricNo you are going to display the values of each row. But when you have run the query you say you don't find any records.

U_Hanisa 22-Sep-14 21:23pm

Yes, you right. It basing MatricNo

ChauhanAjay 22-Sep-14 21:26pm

Can you try using the try catch block in your code so that it displays the error and may be we can get closer to your problem.

U_Hanisa 22-Sep-14 21:33pm

At 'Me.lblName.Text = dbread("Name")', it display error (No data exists for the row/column)

ChauhanAjay 22-Sep-14 21:37pm

What is the name of the column for MatricNo in the gridview.

U_Hanisa 22-Sep-14 21:51pm

Matric No

2 solutions

Solution 1

In the query you will need to change the row.Cells(0).Text to match the column index of the column "Matric No" and then try.

VB

Protected Sub gvCheckList_RowCommand(sender As Object, e As GridViewCommandEventArgs)
	Try
		If e.CommandName = "cmdSelect" Then
			Dim index As Integer = Int32.Parse(e.CommandArgument.ToString())
			Dim row As GridViewRow = gvCheckList.Rows(index)
Dim matricNo As String = row.Cells(1).Text.ToString()
Response.Write("Matric No = "&matricNo)
			sql = "SELECT a.MatricNo, b.Name, b.Faculty, a.Date, a.KaunselingID, b.TelNo, " _
			& "c.Officer " _
			& "FROM KaunselingPL AS a " _
			& "INNER JOIN DbStudent..Personal AS b ON a.MatricNo = b.MatricNo " _
			& "INNER JOIN KaunselingDT AS c ON a.KaunselingID = c.KaunselingID " _
			& "WHERE (a.Status = '2' and a.MatricNo='"& matricNo &"')"
Response.Write("Sql Query = "&sql)		 
			dbconn = New OleDbConnection(strCon)
			dbconn.Open()
		 
			dbcomm = New OleDbCommand(sql, dbconn)
			dbread = dbcomm.ExecuteReader()
			dbread.Read()
			If (dbread.HasRows()) then 
				Me.lblName.Text = dbread("Name")
				Me.lblMatricNo.Text = dbread("MatricNo")
				Me.lblFaculty.Text = dbread("Faculty")
				Me.lblTelNo.Text = dbread("TelNo")
				Me.lblDate.Text = CDate(dbread("Date")).ToString("dd/MM/yyyy")
				Me.lblOfficer.Text = dbread("Officer")
			End If 
			dbread.Close()
			dbcomm = Nothing
			dbconn.Close()
		End If
	Catch ex As Exception
		Response.Write(ex.Message.ToString());
	End Try
End Sub

Posted 22-Sep-14 15:58pm

ChauhanAjay

Updated 24-Sep-14 15:05pm

v4

Comments

U_Hanisa 22-Sep-14 22:08pm

The writing becomes larger

ChauhanAjay 22-Sep-14 22:09pm

I dont understand what you are trying to say

U_Hanisa 22-Sep-14 22:16pm

When I click the select button on Matric No, the writing in gridview becomes larger

ChauhanAjay 22-Sep-14 22:32pm

What about the result are you able to get the correct result.
I am not sure why the writing becomes larger.

U_Hanisa 22-Sep-14 22:35pm

No. When I click select button in gridview, it don't display any data.

ChauhanAjay 22-Sep-14 22:39pm

Can you debug and check the query what is the value against "a.MatricNo" field in the where condition and let me know if that is the correct value or not.

U_Hanisa 22-Sep-14 22:58pm

I try using the try catch block in my code. At 'If (dbread.HasRows()) Then', it false. So it do not read the data. By the way, I don't understand what do you mean.

ChauhanAjay 22-Sep-14 23:04pm

do you know how to debug the code?

U_Hanisa 23-Sep-14 2:00am

Yes. but I don't understand what do you mean by the value against "a.MatricNo" field in the where condition

ChauhanAjay 23-Sep-14 2:47am

this is the where condition
a.MatricNo='"& row.Cells(0).Text &"'" correct.

what is the value of row.Cells(0).Text and if this value exists in the database table from the column MatricNo

U_Hanisa 23-Sep-14 3:03am

oh yes,it's correct, and this value exists in the database. But when I use catch block, it go false when through 'If (dbread.HasRows()) Then'

ChauhanAjay 23-Sep-14 3:10am

Check if the status of that id you have selected from the front end is 2 or not in the database.
For example you have click a select button and the id you are getting is '1' now check if the status for that id '1' is 2 or not.
You will only be able to view the data if the status '2'. May be you are trying to check for id where the status value is not 2 in the database.

U_Hanisa 23-Sep-14 4:10am

My status at data list in database is all '2'

U_Hanisa 22-Sep-14 22:41pm

"Response.Write(ex.Message.ToString());"
What 'Message' do you mean, I don't understand

ChauhanAjay 22-Sep-14 22:42pm

if there is any exception it will write the exception on the browser.

U_Hanisa 23-Sep-14 21:36pm

Excuse me, ChauhanAjay. Last time u teach me how to call data based on drop down list. Now,when I select one of the data from gridview rowCommand, it didn't display at my page. All of my data in gridview, when I select that data I want to update the status from '2' to '3'. Can you help me?

ChauhanAjay 23-Sep-14 22:24pm

Can we have the code of your .aspx page which contains the gridview?

U_Hanisa 23-Sep-14 22:42pm

Please check my updated solution,TQ^^

ChauhanAjay 23-Sep-14 23:09pm

Change this from
<asp:ButtonField DataTextField="MatricNo" CommandName="cmdSelect" HeaderText="Matric No" meta:resourcekey="ButtonFieldResource1" HeaderStyle-Width="15%" >

<asp:ButtonField DataTextField="Name" HeaderText="Name" HeaderStyle-Width="32%">

TO

<asp:ButtonField DataTextField="MatricNo" CommandName="cmdSelect" HeaderText="Matric No" meta:resourcekey="ButtonFieldResource1" HeaderStyle-Width="15%" />

<asp:ButtonField DataTextField="Name" HeaderText="Name" HeaderStyle-Width="32%" />

Please check the updated solution.
Try it out and let me know the result.

U_Hanisa 23-Sep-14 23:21pm

What the difference? By the way, this "Dim NoMatrik As String = gvSenSemak.DataKeys(CurrentRowIndex).Value", (CurrentRowIndex) is not declared.

ChauhanAjay 23-Sep-14 23:26pm

Sorry for the mistake. Copied CurrentRowIndex for some other answer and pasted it here. I have changed it to "index" is the row index of the gridview which we are using in our current code.

U_Hanisa 24-Sep-14 4:25am

Excuse me, what happen?

ChauhanAjay 24-Sep-14 4:35am

Where did this "Dim NoMatrik As String = gvSenSemak.DataKeys(CurrentRowIndex).Value" cum from i did not observer it earlier. It is not a part of the code which you have have pasted.

U_Hanisa 24-Sep-14 20:21pm

Oh ok, sorry for the mistake. I had debug my system, it still didn't display anything.

ChauhanAjay 24-Sep-14 21:03pm

Does it say something like this "Index was out of range error" when you click the button.

ChauhanAjay 24-Sep-14 21:07pm

Check the updated code. I have two response.write statements in the code. When you click the button the response.write statement will print the values of the variables matricNo and sql on the browser can you send me both the values.

U_Hanisa 24-Sep-14 21:35pm

At first respose.writestatement, the value is empty(""). At second response.writestatement, the value is same as sql query. I want to tell you that when it through 'If (dbread.HasRows()) Then', the value is false. So it skip to do the dbread. Do you get what I mean?

ChauhanAjay 24-Sep-14 21:39pm

Can I have the query which populates the gridview?

U_Hanisa 24-Sep-14 21:52pm

SELECT Row_Number() OVER(ORDER BY a.Date) AS ROWNUMBER, a.MatricNo, b.Name, b.Faculty, a.Date, a.KaunselingID, b.TelNo, " _
& "c.Officer " _
& "FROM KaunselingPL AS a " _
& "INNER JOIN DbStudent..Personal AS b ON a.MatricNo = b.MatricNo " _
& "INNER JOIN KaunselingDT AS c ON a.KaunselingID = c.KaunselingID " _
& "WHERE (a.Status = '2')

ChauhanAjay 24-Sep-14 22:01pm

None of your datatextfield or datafield match with the names in your query. Are you sure you are able view data on the gridview and not getting any errors.

U_Hanisa 25-Sep-14 2:53am

Can you please check my new question, TQ. :)
'http://www.codeproject.com/Questions/822923/How-to-update-two-table-in-asp-net'

U_Hanisa 24-Sep-14 23:58pm

I got my solution's answer,thanks for helping^^

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)

U_Hanisa · Accepted Answer · 2014-09-24T18:06:00

VB

If e.CommandName = "cmdSelect" Then
            Dim index As Integer = Convert.ToInt32(e.CommandArgument)
            Dim row As GridViewRow = gvCheckList.Rows(index)

            Me.hidIDKaunseling.Value = Convert.ToString(row.Cells(5).Text)
            Me.lblMatricNo.Text = Convert.ToString(row.Cells(6).Text)

            '---LOAD SENARAI---
            dbconn = New OleDb.OleDbConnection(strCon)
            dbconn.Open()

            sql = "SELECT a.MatricNo, b.Name, b.Faculty, a.Date, a.KaunselingID, b.TelNo, " _
                & "c.Officer " _
                & "FROM KaunselingPL AS a " _
                & "INNER JOIN DbStudent..Personal AS b ON a.MatricNo = b.MatricNo " _
                & "INNER JOIN KaunselingDT AS c ON a.KaunselingID = c.KaunselingID " _
                & "WHERE (a.Status = '2' and a.MatricNo='" & Me.lblMatricNo.Text & "'"

            dbcomm = New OleDbCommand(sql, dbconn)
            dbread = dbcomm.ExecuteReader()
            dbread.Read()

            Me.lblName.Text = dbread("Name")
            Me.lblMatricNo.Text = dbread("MatricNo")
            Me.lblFaculty.Text = dbread("Faculty")
            Me.lblTelNo.Text = dbread("TelNo")
            Me.lblDate.Text = CDate(dbread("Date")).ToString("dd/MM/yyyy")
            Me.lblOfficer.Text = dbread("Officer")

            dbread.Close()
            dbcomm = Nothing
            dbconn.Close()
        End If