Question

I am calling three functions in my code where i want to validate some of my fields.
When I tries to work with the code given below. It checks only for first value until it gets false result.

I want some thing like that if fisrt function returns true then it should also call next function and so on. What can be used instead of Or Operator to do this.

    if (IsFieldEmpty(ref txtFactoryName, true, "Required") ||
        IsFieldEmpty(ref txtShortName, true, "Required") ||
        IsFieldEmpty(ref cboGodown, true, "Required"))
    { }

EDIT

public bool IsFieldEmpty(ref TextBox txtControl, Boolean SetErrorProvider,string msgToShowOnError)
{
    ErrorProvider EP = new ErrorProvider();
    if (txtControl.Text == string.Empty)
    {
        EP.SetError(txtControl, msgToShowOnError);
        return true;
    }
    else
    {
        EP.Clear();
        return false;
    }
}

Please comment, Is this method fine using ref variable as one of the parameter.

I am checking for validation onSubmit event in winform.

Answer 1

You can use the single | for OR:

 if (IsFieldEmpty(ref txtFactoryName, true, "Required") |
    IsFieldEmpty(ref txtShortName, true, "Required") |
    IsFieldEmpty(ref cboGodown, true, "Required"))
{ }

The double-pipe || is performing short-circuit evaluation, the single version | does full evaluation.
The same for && and &.

See the MSDN reference.

Response to the Edit:

1. There is no need for the ‘ref’ in front of txtControl, and removing that would go a long way in addressing the criticism on your approach here. IsFieldEmpty makes no changes to txtControl. You could rename to CheckFieldEmpty to improve it a little further.
2. It is strange that you create an ErrorProvider instance inside this method, does that work at all? There should normally be one (permanent) instance on the Form. You probably want this method to be independent of the Form, so just add an EP as a parameter. It can replace the SetErrorProvider, you can check the EP parameter for null. O, and replace EP.Clear(); with Ep.SetErrortxtControl, "");