How to change visibility of each row independently

Question

0.00/5 (No votes)

See more:

Hi again

I scratching my head yet again. Hope you can help.

I have a gridview containing list of equipment. when a person choose a time to book equipment, the system should check the time against the database and toggle the visibility of two controls (checkbox and label). if time clash of overlap is found, then the label should be displayed and checkbox hidden. if not then the other way around is true.

Here is the problem; I have it working as it should, but if time overlap for one equipment is found, then all labels for all equipment is displayed, and checkbox is hidden, if not all checkbox is displayed and label hidden.

I would like to know how I can get it to perform the operation only for rows that have the time overlap or clash, and make other equipment available.

Here is the code behind.

C#

while (reader.Read())
            {
                DataRow dataRow = table.NewRow();

                DateTime startDate = Convert.ToDateTime(reader["StartDate"]);
                DateTime endDate = Convert.ToDateTime(reader["EndDate"]);

                TimeSpan startTime = (TimeSpan)reader["StartTime"];
                TimeSpan endTime = (TimeSpan)reader["EndTime"];
                
                dataRow["SDate"] = startDate;
                dataRow["FDate"] = endDate;
                dataRow["STime"] = startTime;
                dataRow["ETime"] = endTime;
                table.Rows.Add(dataRow);

                
                if (((startDate <= startDateTime && startDateTime <= endDate)
                   || (startDate <= endDateTime && endDateTime <= endDate)
                    || (startDate >= startDateTime && endDateTime >= endDate))
                    && 
                    ((startTime <= starttime && starttime <= endTime)
                || (startTime <= endtime && endtime <= endTime)
                || (startTime >= starttime && endtime >= endTime)))

                {

                            foreach (GridViewRow row in GridView1.Rows)
                            {
               Label LabelUnavailable = (Label)row.FindControl("Label1");
               CheckBox CheckBoxAsset = (CheckBox)row.FindControl("CheckBox1");
                                CheckBoxAsset.Visible = false;
                                LabelUnavailable.Visible = true;
                              }   

                }
                else
                {

                            foreach (GridViewRow row in GridView1.Rows)
                            {
               Label LabelUnavailable = (Label)row.FindControl("Label1");
               CheckBox CheckBoxAsset = (CheckBox)row.FindControl("CheckBox1");
                                CheckBoxAsset.Visible = true;
                                LabelUnavailable.Visible = false;
                            }   
                    
                }

            }

I would really appreciate any suggestion.

Thanks

I changed it as follows, but same result

C#

foreach (GridViewRow gridViewRow in GridView1.Rows)
                {
                    if (((startDate <= startDateTime && startDateTime <= endDate)
                       || (startDate <= endDateTime && endDateTime <= endDate)
                        || (startDate >= startDateTime && endDateTime >= endDate))
                        &&
                        ((startTime <= starttime && starttime <= endTime)
                    || (startTime <= endtime && endtime <= endTime)
                    || (startTime >= starttime && endtime >= endTime)))
                    {



            Label LabelUnavailable = (Label)gridViewRow.FindControl("Label1");
       CheckBox CheckBoxAsset = (CheckBox)gridViewRow.FindControl("CheckBox1");
                        CheckBoxAsset.Visible = false;
                        LabelUnavailable.Visible = true;
                        



                    }

                    else
                    {

                        //foreach (GridViewRow gridViewRow in GridView1.Rows)
                        
             Label LabelUnavailable = (Label)gridViewRow.FindControl("Label1");
       CheckBox CheckBoxAsset = (CheckBox)gridViewRow.FindControl("CheckBox1");
                            CheckBoxAsset.Visible = true;
                            LabelUnavailable.Visible = false;
                        

                    }
                }

            }

Here is GridView and database connection

ASP.NET

<asp:GridView ID="GridView1" runat="server" AutoGenerateColumns="False" BackColor="White" 
                                    Width="700px" BorderColor="#999999" BorderStyle="Solid" BorderWidth="0px" 
                                    CellPadding="16" DataSourceID="SqlDataSource2"  ForeColor ="#333333" GridLines="None"
                                    OnSelectedIndexChanged="GridView1_SelectedIndexChanged" ShowFooter="True" 
                                    OnRowCreated="RowCreated" HorizontalAlign="Center" Visible="False" DataKeyNames="AssetID">
                        <AlternatingRowStyle BackColor="#f1f1f1" />
                        <Columns>
                           <asp:TemplateField>                             
                                
                                <FooterTemplate>
                                    
                                    <asp:Button ID="Confirm" CssClass="buttonConfirm" runat="server" Text="Confirm" Visible="True" />
                                   <asp:Button ID="Cancel" CssClass="buttonCancel" runat="server" Text="Cancel"  Visible="True" OnClick="Cancel_Click" />          
                                    
                        
                                </FooterTemplate>
                                
                                <ItemTemplate>
                                    <asp:CheckBox ID="CheckBox1" runat="server"   />
                                    <asp:Label ID="Label1" runat="server" ForeColor="Red" Text="Unavailable" ></asp:Label>
                                </ItemTemplate>
                            </asp:TemplateField>
                            <asp:BoundField DataField="Asset" HeaderText="Asset" SortExpression="Asset" />
                            <asp:BoundField DataField="Descriptions" HeaderText="Descriptions" SortExpression="Descriptions" />
                            <asp:BoundField DataField="ObtainMethod" HeaderText="ObtainMethod" SortExpression="ObtainMethod" />
                            <asp:TemplateField>
                                <ItemTemplate>
                                    <asp:Image style="" AlternateText="Image Not Found"  ID ="Image1" Width="60px" runat="server" ImageUrl='<%# Bind("Images") %>' />
                                </ItemTemplate>
                            </asp:TemplateField>
                        </Columns>

                        
                        <EmptyDataRowStyle ForeColor="red" HorizontalAlign="Center"/>
                        <FooterStyle BackColor="#ffffff" Width="100%" Height="100%"/> 
                        <HeaderStyle BackColor="white"  ForeColor="White" />
                        <PagerStyle BackColor="#999999" ForeColor="Black" HorizontalAlign="Center" />
                        <SelectedRowStyle BackColor="#000099" ForeColor="White" />
                        <SortedAscendingCellStyle BackColor="#F1F1F1" />
                        <SortedAscendingHeaderStyle BackColor="#808080" />
                        <SortedDescendingCellStyle BackColor="#CAC9C9" />
                        <SortedDescendingHeaderStyle BackColor="#383838" />
                    
                        <EmptyDataTemplate>
                            <br>
                            </br>
                            <div style=" background-color:#fffbec; height:20px; width:70%;  border-removed 1px solid red; border-removed 1px solid red;border-radius:8px; padding:2px;">
                                    Please choose your Campus to display available assets


                            </div>
                            <br>
                            </br>
                            <br>
                            </br>
                       </EmptyDataTemplate>
                        
                        
                    </asp:GridView>

                               
                    
                    

                    <asp:SqlDataSource ID="SqlDataSource2" runat="server" ConnectionString="<%$ ConnectionStrings:AssetBookingSystemConnectionString %>" SelectCommand="SELECT * FROM [tblAssets]">
                    </asp:SqlDataSource>

Here is the complete code behind

C#

 protected void CheckAvailability_Click(object sender, EventArgs e)
        {

            DateTime startDateTime = DateTime.ParseExact(StartDate.Text, "yyyy-MM-dd", CultureInfo.InvariantCulture);
            TimeSpan starttime = TimeSpan.ParseExact(StartTime.SelectedValue, "h\\:mm", CultureInfo.InvariantCulture);


            DateTime endDateTime = DateTime.ParseExact(FinishDate.Text, "yyyy-MM-dd", CultureInfo.InvariantCulture);
            TimeSpan endtime = TimeSpan.ParseExact(FinishTime.SelectedValue, "h\\:mm", CultureInfo.InvariantCulture);

            //Checking asset is available for booking:


            if ((endDateTime > startDateTime) || (endDateTime == startDateTime))

                GridView1.Visible = true;

            else
                GridView1.Visible = false;

            if ((startDateTime == DateTime.Now.Date) || (endDateTime == DateTime.Now.Date))

                SameDayBooking.Visible = true;
            else
                SameDayBooking.Visible = false;


            if (((FinishTime.SelectedIndex < StartTime.SelectedIndex) || (FinishTime.SelectedIndex == StartTime.SelectedIndex))
                 && ((endDateTime < startDateTime) || (endDateTime == startDateTime)))
            {
                timeError.Visible = true;
                GridView1.Visible = false;


            }
            else
            {
                timeError.Visible = false;
                GridView1.Visible = true;
            }


            
                string cs = System.Configuration.ConfigurationManager.ConnectionStrings["AssetBookingSystemConnectionString"].ConnectionString;



                SqlConnection con = new SqlConnection(cs);
                SqlCommand cmd = new SqlCommand();
                SqlDataReader reader;
                cmd.CommandText = "SELECT StartDate, EndDate, StartTime, EndTime FROM tblBooking";
                cmd.CommandType = CommandType.Text;
                cmd.Connection = con;

                con.Open();
                reader = cmd.ExecuteReader();


                DataTable table = new DataTable();
                table.Columns.Add("SDate");
                table.Columns.Add("FDate");
                table.Columns.Add("STime");
                table.Columns.Add("ETime");



                while (reader.Read())
                {
                    DataRow dataRow = table.NewRow();

                    DateTime startDate = Convert.ToDateTime(reader["StartDate"]);
                    DateTime endDate = Convert.ToDateTime(reader["EndDate"]);

                    TimeSpan startTime = (TimeSpan)reader["StartTime"];
                    TimeSpan endTime = (TimeSpan)reader["EndTime"];




                    dataRow["SDate"] = startDate;
                    dataRow["FDate"] = endDate;
                    dataRow["STime"] = startTime;
                    dataRow["ETime"] = endTime;
                    table.Rows.Add(dataRow);

                    foreach (GridViewRow gridViewRow in GridView1.Rows)
                    {
                        if (((startDate <= startDateTime && startDateTime <= endDate)
                           || (startDate <= endDateTime && endDateTime <= endDate)
                            || (startDate >= startDateTime && endDateTime >= endDate))
                            &&
                            ((startTime <= starttime && starttime <= endTime)
                        || (startTime <= endtime && endtime <= endTime)
                        || (startTime >= starttime && endtime >= endTime)))
                        {

                            Label LabelUnavailable = (Label)gridViewRow.FindControl("Label1");
                            CheckBox CheckBoxAsset = (CheckBox)gridViewRow.FindControl("CheckBox1");
                            CheckBoxAsset.Visible = false;
                            LabelUnavailable.Visible = true;


                        }

                        else
                        {



                            Label LabelUnavailable = (Label)gridViewRow.FindControl("Label1");
                            CheckBox CheckBoxAsset = (CheckBox)gridViewRow.FindControl("CheckBox1");
                            CheckBoxAsset.Visible = true;
                            LabelUnavailable.Visible = false;



                        }
                    }

                }
                reader.Close();

                con.Close();


            
        }

    }
}

Posted 8-Jun-15 1:07am

Awoldeselassie

Updated 8-Jun-15 9:08am

v4

Add a Solution

Comments

[no name] 8-Jun-15 8:30am

It looks your label and checkbox control visibility is set as per the last record in reader object, as you are iterating through all gridview rows every time you process 1 reader entry. Please correct if wrong.

Have you tried to set this visibility in RowDataBound event for gridview control?

Awoldeselassie 8-Jun-15 8:58am

Your first sentence is correct. And yes, I have tried. But the RowDataEvent is outside the button event handler, therefore when I refer to the checkbox or label in the IF statement, it keeps saying 'it doesn't exists in this context' that would have been much easier, but I don't know how to reference, for e.g. 'CheckBoxAsset.Visible = false;' and 'LabelUnavailable.Visible = true;' in the IF statement as shown above. Sorry if I sound stupid

[no name] 8-Jun-15 9:04am

Do you bind this table object as datasource to your gridview control?

Awoldeselassie 8-Jun-15 9:12am

The datasource for the gridview is a table callen 'tblAssets' as it display the assets.
Datasource for the reader to check if time overlap is occurring, is a table called tblBooking. It looks like its working, but it displays one or the other. If it find time overlap or clash, all assets are unavailable. otherwise all are available.

[no name] 8-Jun-15 9:21am

Dont you hace any provision to hold this overlap flag value in your datasource which you bind to grid control. If yes then it should work in RowDataBound event to set visibility, otherwise I think you need to maintain this flag somewhere in datasource.

But you need to move that logic to set visibility from your code as above, this will set all rows based on 1 reader entry.

Tadit Dash (ତଡିତ୍ କୁମାର ଦାଶ) 8-Jun-15 9:26am

Have you debugged and seen what is happening exactly? Which condition it is going into?

Sinisa Hajnal 8-Jun-15 9:44am

Put the flag in the query (that is, in your returned result) and access it directly. It is the cleanest "row by row" access.

Awoldeselassie 8-Jun-15 9:53am

Yes I have debugged, it is working and not throwing any exceptions, but not as its required. And no, as you can see from the code, I don't hold the overlap value, I used an if statement to toggle the visibility of the checkbox and label.

Sinisa Hajnal, I don't know what you mean by put the flag in the query. elaborate please?

Tadit Dash (ତଡିତ୍ କୁମାର ଦାଶ) 8-Jun-15 10:37am

Adding a new comment all the time won't notify us. If you want to notify us, then you need to reply to that comment using the "Reply" button.

Now coming to the problem. According to code, it is looping through all the rows and making the Label and CheckBoxes Visible false or true. So, what is the issue exactly?

Awoldeselassie 8-Jun-15 11:10am

Ok, thanks for the tip, My bad.

The problem is, even though it is looping through the rows, it only display one or the other (visible label for all rows, or visible checkbox for all rows). So if I have 10 row, but only one of the equipment is booked for the specified time, all checkbox is set to false and all label is set to true. What it should do is check each row, and set the visibility to true or false for each row independently depending on the specified time.
Thaks

Tadit Dash (ତଡିତ୍ କୁମାର ଦାଶ) 8-Jun-15 13:01pm

I can see that you have changed the code. But as you are saying that it is still not working, that means the condition is failing, I believe. Have you tried debugging again?

Awoldeselassie 8-Jun-15 15:40pm

Yes I have, but debugging is not doing anything as the code is working properly. Its just not doing what I want it to do.

[no name] 8-Jun-15 11:48am

You can see all rows with either control visible because, you have only current overlap value say "True" from current item from reader and you are iterating with all rows in grid control. So in this case above code will set label visibility to true for all rows in grid control.

What you should do is, maintain the overlap value for each individual row and when you bind data to gridview control you should set label / checkbox visibility based on "current row data". In above code you set all rows with 1 reader object so its now working as you expects.

Can you add code how do you bind data to gridcontrol? I hope you are using ItemTemplate here as you need lable/checkbox inside gridview.

Awoldeselassie 8-Jun-15 12:00pm

I have added the code for GridView and databind. It is one reader, but shouldn't it loop through all rows as long as I use foreach?

Awoldeselassie 8-Jun-15 12:02pm

And yes, as you can see, I am using ItemTemplate for the checkbox and label.

[no name] 8-Jun-15 14:09pm

ok, so you used SqlDataSource to bind data to gridview control which I believe do not have any detail which you can use to decide overlap flag? how do you bind data to templated fields. Is this loop for reader you put in RowDataBound event? If not how do you assign values to them?

Awoldeselassie 8-Jun-15 14:55pm

what you said earlier makes a lot of sense, its only using the current read to loop through the rows. data is bind using ItemTemplate as is shows in the code. It works but its just looping through

[no name] 8-Jun-15 15:02pm

so do you have something like for templated fields -

protected void GridView1_RowDataBound(object sender, GridViewRowEventArgs e)
{
while(reader.Read)
{
//here your if loop to check flag and set visibility of control
}
}

[no name] 8-Jun-15 15:02pm

this comments is as per above code you provided?

Awoldeselassie 8-Jun-15 15:12pm

ok thanks, I have posted the entire code behind for the event handler. data is checked based on the user input. so I had to use a button and event handler. so you can see the 'while(reader.Read)'. I dont know if I got the foreach in the wrong place, or even if I should be looping through the table rows in the code behind, driving me nuts, lol

Awoldeselassie 12-Jun-15 7:30am

After a week, I finally figured out what the problem is: the IF statement inside the foreach loop is checking the table every time it loops rather than the row. So if there is time overlap or not, its always going to be the same result for all rows. I just need a way of getting the IF statement to check each row rather than the table.

[no name] 12-Jun-15 7:39am

Thats correct. I tried to explain same in my previous comments..

Awoldeselassie 12-Jun-15 8:20am

yes you did, had to find out the hard way... every time the foreach loop iterate, it has to check using the IF, but I don't know how to get it to do that for the individual row, any suggestions?
At least I know the problem, half the battle is won..lol

[no name] 12-Jun-15 9:17am

You are binding data for grid control with SqlDataSource, so on CheckAvailability just build an page level bool list to maintain ovelap flag value for each booking. Then on RowDataBound event you can just iterate through this list to see if current booking has any overlap and then you can decide visibility of control ....

I have tried to explain this in my solution I post here...may not work perfectly but just a suggestion... :)

Awoldeselassie 12-Jun-15 10:06am

Ok, thanks for all your suggestions and for trying hard to help. Your solution was helpful.

[no name] 12-Jun-15 10:09am

I know lot of improvements are needed in my solution if it works for your basic requirement you can improve it later on

Awoldeselassie 12-Jun-15 10:25am

I hope it works, you suggestion is the best I found. I just didn't expect to change most of the code as I expected the foreach to check for each row, and not the table every time it loops.
thanks again man.

1 solution

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)

User 4180254 · Accepted Answer · 2015-06-12T03:17:00

You can move code in seperate method to populate data from tblooking.

C#

private DataTable GetBookingDetails(object sender, EventArgs e)
{
 // your code to get from tblBooking here, then
    while (reader.Read())
    {
       //condition to check overlap flag then populate list<bool> which have value for each row for tblBooking now..maintain this on page level e.g. IsOverlapForBooking
    }
}</bool>

C#

protected void CheckAvailability_Click(object sender, EventArgs e)
{
    // comment your code to read tblBooking and reader implementation here
    GridView1.DataSourceID = null; // because we want to bind here on click event, this will error our as you have SqlDataSource attached to grid control.
    GridView1.DataSource = (DataView)SqlDataSource1.Select(DataSourceSelectArguments.Empty);
    GridView1.DataBind();
}

So here now you have DataSource availale to bind with GridControl with SqlDataSource, then List<bool> overlap flag for each booking you have in tblBooking table which you build in GetBookingDetails and page level variable, so when you click CheckAvailability button now use RowDataBound to iterate through overlap flag list and then based on value you can set visibility of lable and checkbox control..

C#

protected void GridView1_RowDataBound(object sender, GridViewRowEventArgs e)
{
    if (e.Row.RowType == DataControlRowType.DataRow)
    {
        Label label = e.Row.FindControl("label") as Label;
        CheckBox checkBox = e.Row.FindControl("checkBox") as CheckBox;
         
        //this you populated on GetBookingDetails for each row
        if(IsOverlapForBooking[e.Row.RowIndex])
        {
           // overlap = true then do something
        }
        else
        {
           // overlap = false then do something
        }
    }
}

this may not be a perfect solution but you asked for suggestion and i tried. Hope you can find some better solution than this... lol :)