Solved

MonthCalendar and DataGridView - Very Slow Loading.

Posted on 2009-04-06
7
2,017 Views
Last Modified: 2013-12-17
I have a DataGridView with several columns.  If a user selects a cell in the date column a monthCalendar control appears at that location.  

My issue is that it is extremely slow (it appears fast, but attempting to select a date seems to take a long time).  I have used the exact same code (posted below) for a textBox click event and it works perfectly fast.  Does anybody know what may be wrong?

if (dataGridView.CurrentRow.Cells["transactionDate"].Selected)
                    {
                        monthCalendar.Visible = true;
                        monthCalendar.Top = 40;
                        monthCalendar.Left = 50;
                        monthCalendar.BringToFront();
                        monthCalendar.Capture = true;
                        while (monthCalendar.Visible == true)
                        {
                            Application.DoEvents();
                        }
                        dataGridView.CurrentRow.Cells["transactionDate"].Value = _accountDate;
                        return;
                    }
 
 private void monthCalendar_MouseUp(object sender, MouseEventArgs e)
        {
            switch (monthCalendar.HitTest(e.Location).HitArea)
            {
                case MonthCalendar.HitArea.Date:
                    monthCalendar.Visible = false;
                    break;
                case MonthCalendar.HitArea.NextMonthDate:
                    break;
                case MonthCalendar.HitArea.PrevMonthDate:
                    break;
            }
            _accountDate = monthCalendar.SelectionStart.ToShortDateString();
        }

Open in new window

0
Comment
Question by:angus_young_acdc
[X]
Welcome to Experts Exchange

Add your voice to the tech community where 5M+ people just like you are talking about what matters.

  • Help others & share knowledge
  • Earn cash & points
  • Learn & ask questions
  • 4
  • 3
7 Comments
 
LVL 15

Expert Comment

by:oobayly
ID: 24075530
I'd suggest using a custom DataGridViewColumn that hosts a DateTimePicker control, once you've created it, it can be used all over the place with very little code.

This MSDN article explains how to do it. Once you understand how it works, it's simple to create a whole host (no pun intended) of custom columns. So far I've got a DateTimePicker column, a NumericUpDown column and a column that allows editing of binary data by displaying it as a hex string.

http://msdn.microsoft.com/en-us/library/7tas5c80.aspxhttp://msdn.microsoft.com/en-us/library/7tas5c80.aspx
0
 
LVL 15

Author Comment

by:angus_young_acdc
ID: 24075677
In the example provided it is adding the CalendarColumn to the grid as a new column.  How can I assign the custom class to a column that already exists?  Or how can I actually use this?  I currently have lots of code for the other columns and seems like a re-write would be required.
0
 
LVL 15

Expert Comment

by:oobayly
ID: 24076102
Well, presuming that your columns are autogenerated when binding to a datatable, you can just add the following code:
It uses the same header text and dataproperty as the original column that contains a date, inserts itself in the same position, and then hides the original.

As for your code, I'm guessing it's to do with the while loop. Why not remove the loop, and set the column's date after you hide the Calendar control:
dataGridView.CurrentRow.Cells["transactionDate"].Value = monthCalendar.SelectionStart;

Why are you converting the date selected by the user to a string, is the Date column not DateTime?
You're only testing for one hittest type, so why use switch?
// Using Calendar column
      DataGridViewColumn autoGenDateCol = dataGridView1.Columns["Date Column"];
      CalendarColumn  calendarCol = new CalendarColumn ();
      calendarCol.HeaderText = autoGenDateCol.HeaderText;
      calendarCol.DataPropertyName = autoGenDateCol.DataPropertyName;
      calendarCol.Name = "Date Column2";
      dataGridView1.Columns.Insert(autoGenDateCol.Index, calendarCol);
      autoGenDateCol.Visible = false;
 
// Your modified code
if (dataGridView.CurrentRow.Cells["transactionDate"].Selected)
                    {
                        monthCalendar.Visible = true;
                        monthCalendar.Top = 40;
                        monthCalendar.Left = 50;
                        monthCalendar.BringToFront();
                        monthCalendar.Capture = true;
                        dataGridView.Enabled = false;
                        return;
                    }
 
 private void monthCalendar_MouseUp(object sender, MouseEventArgs e)
        {
            if (monthCalendar.HitTest(e.Location).HitArea == MonthCalendar.HitArea.Date){
              dataGridView.CurrentRow.Cells["transactionDate"].Value = monthCalendar.SelectionStart;
              monthCalendar.Visible = false;
              dataGridView.Enabled = true;
            }
        }

Open in new window

0
Learn by Doing. Anytime. Anywhere.

Do you like to learn by doing?
Our labs and exercises give you the chance to do just that: Learn by performing actions on real environments.

Hands-on, scenario-based labs give you experience on real environments provided by us so you don't have to worry about breaking anything.

 
LVL 15

Author Comment

by:angus_young_acdc
ID: 24077966
Hi oobayly

Unfortunately not, they have been created just via Edit Columns as there is no real binding, it's taking a value from XML to populate the table.  

Good idea about hiding after populating the date, shall try that out now.  

I'm converting the date to a string as that's just the way it is read/written to the XML, so just seemed easier (albeit uglier) rather than just converting it later.

As for the switch... I'm not sure, should I just change it to an if statement for the HitArea.Date?
0
 
LVL 15

Accepted Solution

by:
oobayly earned 500 total points
ID: 24078064
Well, when you add the CalendarColumn to your project, it should appear in your list of available column types when editing the columns for a DataGridView, so editing them should be pretty easy, and you could dump the DataGridViewTextBoxColumn that is currently displaying the date.

Fair enough about using the date as a string if the datasource is xml, though I tend to do the conversions when loading and saving my tables, so any controls have the datatype that they expect. I suppose it's personal preference.

I'd use if instead of switch as your only checking for a single case. If you were handling any other HitArea then I'd definitely use a switch statement. Again, it's personal preference as I find it easier to read.
0
 
LVL 15

Author Comment

by:angus_young_acdc
ID: 24085191
Hi again,

I have tried that code but unfortunately I can't seem to get it working correctly.  I'm either getting a null reference exception or an invalid cast exception.

They all happen here:

 public override void InitializeEditingControl(int rowIndex, object
            initialFormattedValue, DataGridViewCellStyle dataGridViewCellStyle)
        {
            // Set the value of the editing control to the current cell value.
            base.InitializeEditingControl(rowIndex, initialFormattedValue,
                dataGridViewCellStyle);
            CalendarEditingControl ctl =
                DataGridView.EditingControl as CalendarEditingControl;
            ctl.Value = (DateTime)this.Value;  // Exception thrown
        }
0
 
LVL 15

Author Closing Comment

by:angus_young_acdc
ID: 31566937
Got it working by removing code and doing a bit of a hack.  Also it is a lot faster (most likely due to your suggestion about the loop).  Thanks a lot1
0

Featured Post

DevOps Toolchain Recommendations

Read this Gartner Research Note and discover how your IT organization can automate and optimize DevOps processes using a toolchain architecture.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Introduction Hi all and welcome to my first article on Experts Exchange. A while ago, someone asked me if i could do some tutorials on object oriented programming. I decided to do them on C#. Now you may ask me, why's that? Well, one of the re…
A long time ago (May 2011), I have written an article showing you how to create a DLL using Visual Studio 2005 to be hosted in SQL Server 2005. That was valid at that time and it is still valid if you are still using these versions. You can still re…
If you're a developer or IT admin, you’re probably tasked with managing multiple websites, servers, applications, and levels of security on a daily basis. While this can be extremely time consuming, it can also be frustrating when systems aren't wor…
If you’ve ever visited a web page and noticed a cool font that you really liked the look of, but couldn’t figure out which font it was so that you could use it for your own work, then this video is for you! In this Micro Tutorial, you'll learn yo…

734 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question