C#

Foothill wrote:

it seems to make the code look a lot more complicated

private btn_Submit_Click(object sender, RoutedEventArgs e)
{
    try
    {
        MainWindow.SetCursor_Wait();
        if (InputsAreGood())
        {
            DataCollection data = GetFormFields():
            await PushDataToDatabaseAsync(data);
            ClosePopOutForm();
        }
        else
        {
            // tell user what fields they missed
        }
    }
    catch (Exception ex)
    {
        // display the error to the user
    }
    finally
    {
        MainWindow.SetCusor_Arrow();
    }
}

private async Task PushToDatabaseAsync(DataCollection data)
{
    // Let any exceptions bubble up the call-stack to be displayed by the calling method.
    // TODO: Make this a real async call to the database if possible.
    await Task.Run(() => PerformDatabaseInsert(data));
    
    // Using "await" means we're back on the UI thread here,
    // so there should be no need to use the Dispatcher.
    if (Dispatcher.CheckAccess())
    {
        UpdateList(data)
    }
    else
    {
        await Dispatcher.BeginInvoke((Action<DataCollection>)UpdateList, data);
    }
}

private void UpdateList(DataCollection data)
{
    lst_MyData.ItemsSource = data.ToArray();
}

Rıza Berkay Ayçelebi wrote:

This program only comrpess text.

Bastard Programmer from Hell
If you can't read my code, try converting it here[^]