Counting Date Fields

mike Delvotti

I'm trying to count the Date cells in my datagridview, then display the total of cells but only if the date is today.

So in brief, if my datagrid contains 2 dates that are today my label will read "2" really I would like it to say "You have 2 appointments today" in one label.

Below is some code I'm using but it's not counting correctly?

if (row.IsNewRow) break;
object obj = row.Cells[2].Value;
DateTime dt;
if (!DateTime.TryParse(obj.ToString(), out dt)&& dt.Date == DateTime.Today) count++;
    {
       label2.Text = count.ToString();
    }

13 Replies

Know the answer? Post it — somebody with the same question will find it here.

Vulpes15y ago
Accepted answer
Oops, having a bad day today :(

It should, of course, be:

DataGridViewRow row = dataGridView1.Rows[i];
0
mike Delvotti15y ago
Vulpes - I find it hard to believe you ever have a bad day, that is spot on thank you very much it works a treat.

On that note I'm going to turn the PC off and rest my eyes but remain very grateful of your assistance.

Thanks to everybody else too.
0
mike Delvotti15y ago
Thanks Vulpes, but I get the below error with that line added:

Cannot implicitly convert type 'System.Windows.Forms.DataGridViewRow' to 'System.Data.DataRow'
0
Vulpes15y ago
Sorry, Mike, I've missed out a line:

int count = 0;

for (int i = 0; i < datagridview1.Rows.Count; i++)
{
DataRow row = dataGridView1.Rows[i];
if (row.IsNewRow) break;
object obj = row.Cells[2].Value;
if (obj == null) continue;
DateTime dt;
if (DateTime.TryParseExact(obj.ToString(), "dd/MM/yyyy HH:mm:ss", null, 0, out dt) && dt.Date == DateTime.Today) count++;
}

label2.Text = count.ToString();
0
mike Delvotti15y ago
Thanks Vulpes, I did use your version but I some how need do declare "row" thats the reason I used the foreach statement.

If i use that directly in my function it won't debug as "row" throws an error?
0
Vulpes15y ago
Just noticed from your penultimate post that you're iterating through the rows twice, once in an outer foreach loop and then again in an inner for loop. You'll therefore be overwriting label2 each time and, if the last row doesn't have any dates, the final value will be "0".

If you're going with my suggested code then all you need is:

int count = 0;

for (int i = 0; i < datagridview1.Rows.Count; i++)
{
if (row.IsNewRow) break;
object obj = row.Cells[2].Value;
if (obj == null) continue;
DateTime dt;
if (DateTime.TryParseExact(obj.ToString(), "dd/MM/yyyy HH:mm:ss", null, 0, out dt) && dt.Date == DateTime.Today) count++;
}

label2.Text = count.ToString();
0
mike Delvotti15y ago
Mahesh, It's using a dataset called "calDataset" to fill the data first from Mysql Database.

Vulpes, my label still displays "0" it's a stange one that it doesn't event count the rows?

This is most weird as when I run the code below on some text it seems to work so can't understand why in won't count through the date columns and count todays date?

I run the below just to see if it finds the word "Test" and it does find it

foreach (DataGridViewRow row in DataGridView1.Rows)
            {
                if (row.IsNewRow) break;
                object obj = row.Cells[4].Value;
                {
                    string content = obj.ToString();
                    if (content == "Test")
                    {
                        label1.Text = "Test";
                    }
                }
            }

Thanks for your help.
0
Vulpes15y ago
It may be a culture problem.

Try replacing:

DateTime.TryParse(obj.ToString(), out dt)

with

DateTime.TryParseExact(obj.ToString(), "dd/MM/yyyy HH:mm:ss", null, 0, out dt)
0
Mahesh Chand15y ago
Even running a SELECT COUNT(*) on Db is better and faster than going through a DataGridView cells :)
0
Mahesh Chand15y ago
Where is this data coming from? I am sure it is coming from a database through a DataSet or DataTable? Or other object? Why don't you search that data source and find number of occurrences match with today's string? Going through each cell, read, and compare data probably is slower.

Something like this:

DataTable.Select('ColumnName='+ TodaysDate)
0
mike Delvotti15y ago
Thanks Prabhu / Vulpes for the replies.

Vulpes yes you are correct the date is in string format like so: dd/MM/yyyy HH:mm:ss I've tried the below but it's still not counting even though I've put 2 test dates for today in cells[2] my label still displays "0"

foreach (DataGridViewRow row in DataGridView1.Rows)
                {
                    int count = 0;
                    for (int i = 0; i < DataGridView1.Rows.Count; i++)
                    {
                        if (row.IsNewRow) break;
                        object obj = row.Cells[2].Value;
                        if (obj == null) continue;
                        DateTime dt;
                        if (DateTime.TryParse(obj.ToString(), out dt) && dt.Date == DateTime.Today) count++;

                    }
                    label2.Text = count.ToString();
                }
0
Vulpes15y ago
If the DGV contains date strings rather than DateTime instances, then I think the way you had it originally is on the right lines except that the first term in your second 'if' statement shouldn't be preceded by '!':

int count = 0;

for (int i = 0; i < datagridview1.Rows.Count; i++)
{
if (row.IsNewRow) break;
object obj = row.Cells[2].Value;
if (obj == null) continue;
DateTime dt;
if (DateTime.TryParse(obj.ToString(), out dt) && dt.Date == DateTime.Today) count++;
}

label2.Text = count.ToString();
0
Prabhu Raja15y ago
Hi mike, try this

        if (this.datagridview1 .Rows.Count > 0) //datagridview1 is your datagridview name
          {
                    int count = 0;
                    for (int i = 0; i < this.datagridview1.Rows.Count - 1; i++)
                    {
                        if (this.datagridview1.Rows[i].Cells[2].Value.Equals(DateTime.Today))
                        {
                            ++count;
                        }
                    }
                    label2.Text = count.ToString();
          }
0