Validate input multiple textboxes

Question

I'm reading data and setting fields in another method with this method, is it possible to validate that each field is not empty or contains other than numbers?

I wanna be able to display one messagebox "Fill in all textboxes" Currently if I add an elsemethod for each I get (in worst case 4 messageboxes)..

    private bool ReadInput()
    {
        double curReading = 0;
        double prevReading = 0;
        double amount = 0;
        double unitNumber = 0;

        if (double.TryParse(tbReading.Text, out curReading))
        {
            CalcData.SetCurrentReading(curReading);
        }

        if (double.TryParse(tbPrevReading.Text, out prevReading))
        {
            CalcData.SetPrevReading(prevReading);
        }

        if (double.TryParse(tbAmount.Text, out amount))
        {
            CalcData.SetAmount(amount);
        }

        if (double.TryParse(tbUnitNumber.Text, out unitNumber))
        {
            CalcData.SetUnitNumber(unitNumber);
        }
        return false;
    }

Take a look at: Validation using Validating event and ErrorProvider - Show Error Summary — Reza Aghaei
– Reza Aghaei, Commented Sep 26, 2016 at 19:01
Also to know more options about windows forms validation, take a look at this post: Validating user input / Give .NET controls status OK or NOK — Reza Aghaei
– Reza Aghaei, Commented Sep 26, 2016 at 19:03

Kevin · Accepted Answer · 2016-09-26 15:53:17Z

2

Something like this maybe:

private bool ReadInput()
{
    double curReading = 0;
    double prevReading = 0;
    double amount = 0;
    double unitNumber = 0;
    var validData = true;

    if (double.TryParse(tbReading.Text, out curReading))
    {
        CalcData.SetCurrentReading(curReading);
    }
    else
    {
        validData = false;
    }

    if (double.TryParse(tbPrevReading.Text, out prevReading))
    {
        CalcData.SetPrevReading(prevReading);
    }
    else
    {
        validData = false;
    }

    if (double.TryParse(tbAmount.Text, out amount))
    {
        CalcData.SetAmount(amount);
    }
    else
    {
        validData = false;
    }

    if (double.TryParse(tbUnitNumber.Text, out unitNumber))
    {
        CalcData.SetUnitNumber(unitNumber);
    }
    else
    {
        validData = false;
    }

    if(!validData)
    {
        //Show your dialog here
    }

    return false;
}

answered Sep 26, 2016 at 15:53

Kevin

1,4729 silver badges9 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

noxious Over a year ago

Cheers, just realized I could fix it that way too :)

Collectives™ on Stack Overflow

Validate input multiple textboxes

1 Answer 1

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related