我应该如何使我的方法按时运行

Question

This is just a simple problem that I wanted to solve for the sake of improving my skills. I want my method selectingtime to run and I also want the condition I put in the RunEvent to receive the value of string selecttime from selectingtime. If you did reply on this, I thank you so much.

public partial class Form1 : Form
{
    int hourt;  
    int minutet;
    string hourformatt;
    string selectedtime;
    Timer tm = new Timer(); 

    public Form1()
    {
        InitializeComponent();           
    }

    private void Form1_Load(object sender, EventArgs e)
    {
        tm.Interval = 1000; 
        tm.Tick += new EventHandler(RunEvent); 
        tm.Start();  
    }

    private void RunEvent(object sender, System.EventArgs e)
    {
        label1.Text = DateTime.Now.ToLongTimeString(); 
        DateTime dateT = DateTime.Now; // created datetime

           if (dateT.ToString("hh:mm tt") == selectedtime) // condition where dateT.ToString is equal to selectedtime 
           {
               MessageBox.Show("Please work"); // expected output whenever dateT.ToString is equal to selected time.
           }                
    }

    private void selectingtime()
    {
        hourt = Convert.ToInt32(textBox1.Text); //textbox for hour
        minutet = Convert.ToInt32(textBox2.Text); // textbox for minute
        hourformatt = textBox3.Text; // textbox for AM or PM
        selectedtime = hourt + ":" + minutet + " " + hourformatt; // example 05:31 PM
    }

    private void button1_Click(object sender, EventArgs e)  
    {
        selectingtime();         
    }

Answer 1

private void RunEvent(object sender, System.EventArgs e)
{
    selectingtime(); //This?
    label1.Text = DateTime.Now.ToLongTimeString(); 

    DateTime dateT = DateTime.Now; // created datetime

   if (dateT.ToString("hh:mm tt") == selectedtime) // condition where dateT.ToString is equal to selectedtime 
   {
       MessageBox.Show("Please work"); // expected output whenever dateT.ToString is equal to selected time.
   }
}

Though, I suppose you may get an error on the INT32 converting when the texbox is null, empty, or has non-numerical characters. In that case, you may want to implement a try parse. Also, if you are stuck on the idea of "Getting" the select time from the method, then you need to have it return a string -- though this may meet your request, I don't think it is necessarily an "improvement".

PS You do not need to need to create a tick event with a new event handler instance, passed with the event method.

    private void Form1_Load(object sender, EventArgs e)
    {
        tm.Interval = 1000;
        tm.Tick += RunEvent;
        tm.Start();
    }

    private void RunEvent(object sender, System.EventArgs e)
    {
        var selectedTime = selectingtime();
        label1.Text = DateTime.Now.ToLongTimeString();

        DateTime dateT = DateTime.Now; // created datetime

        if (dateT.ToString("hh:mm tt") == selectedTime) // condition where dateT.ToString is equal to selectedtime 
        {
            textBox1.Clear();
            textBox2.Clear();
            MessageBox.Show("Please work"); // expected output whenever dateT.ToString is equal to selected time.
        }
    }

    private string selectingtime()
    {
        DateTime time;
        string timeFormat = string.Format("{0}:{1} {2}", textBox1.Text, textBox2.Text, textBox3.Text);
        DateTime.TryParse(timeFormat, out time);
        return time.ToString("hh:mm tt");
    }