How to add item to listbox and then send email to target recipient

Question

0.00/5 (No votes)

See more:

Good day guys, I'm a semi-intermediate here in c#. Actually, the said topic name it won't work with my codes implemented, i need some help on how to improve my timer tick procedure code as shown what I've tried below.

What I have tried:

here is my code as follows.

private void timer1_Tick(object sender, EventArgs e)
        {
            statmessage();

            int i = listBox1.SelectedItems.Count + 1;

          if (i < 0)
            {
                var exp = (EItem)listBox1.Items[i];
                var timeVisible = DateTime.Now - exp.Added;
                if (timeVisible.TotalSeconds > 5)
                    if (counter == 100)
                    {
                   
                    SendMessage();
                    counter = counter - 1;
                    ++i;
                    }
                    else if (counter <= 0)
                    {

                    statmessage();
                    }
                
            }

        }

Your feedback is highly appreciated and will help to improve the said concern, Thank you very much

Posted 30-Jan-19 14:51pm

Member 13998948

Updated 31-Jan-19 2:11am

Add a Solution

Comments

Richard MacCutchan 31-Jan-19 4:34am

The value of i can never be less than 0.

Richard Deeming 4-Feb-19 14:12pm

That sounds like a challenge! Time to build a machine capable of coping with int.MaxValue items in a single listbox. :)

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)

ZurdoDev · Answer 1 · 2019-01-31T02:12:00

Solution 1

As Richard mentioned in the comments, it should be pretty obvious, right? All of your code is inside of an if statement that will only run when i is less than 0. However, I is set to the count of the listbox items. How can I ever be less than 0?

Simple debugging would have found this much faster than the time it took you to post the question. :)

Posted 31-Jan-19 2:12am

ZurdoDev

Comments

Richard Deeming 4-Feb-19 14:11pm

"... I is set to the count of the listbox items ..."

It's actually set to one more than the count. So technically, it could be less than zero if there were int.MaxValue items in the listbox - assuming the OS hadn't given up by that point. :)

ZurdoDev 4-Feb-19 15:15pm

That might be a little beyond the complexity of this question. :)