Appeler toutes les 3 fonctions en utilisant ou opérateur même après avoir renvoyé vrai en conséquence

Question

J'appelle trois fonctions dans mon code où je veux valider certains de mes champs. Lorsque je tente de travailler avec le code ci-dessous. Il vérifie seulement pour la première valeur jusqu'à ce qu'il obtienne un faux résultat.

Je veux quelque chose comme ça si la fonction fisrt renvoie vrai alors elle devrait aussi appeler la fonction suivante et ainsi de suite. Qu'est-ce qui peut être utilisé à la place de Ou Opérateur pour le faire.

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; 
    } 
} 

S'il vous plaît commentaire, cette méthode est bien en utilisant la variable ref comme l'un des paramètres.

Je vérifie l'événement onSubmit de validation dans winform.

Answer 1

Vous pouvez utiliser le | unique pour OU:

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

Le double-pipe || effectue short-circuit evaluation, la single version|-t évaluation complète.
Idem pour && et &. Voir le MSDN reference.

Réponse à la Modifier:

1. Il n'y a pas besoin de la « ref » devant txtControl, et le retrait qui irait un long chemin pour faire face aux critiques sur votre approche ici. IsFieldEmpty n'apporte aucune modification à txtControl. Vous pouvez renommer CheckFieldEmpty pour l'améliorer un peu plus loin.
2. Il est étrange que vous créiez une instance ErrorProvider à l'intérieur de cette méthode, cela fonctionne-t-il? Il devrait normalement y avoir une instance (permanente) sur le formulaire. Vous voulez probablement que cette méthode soit indépendante du formulaire, il suffit donc d'ajouter un EP en tant que paramètre. Il peut remplacer SetErrorProvider, vous pouvez vérifier le paramètre EP pour null. O, et remplacer EP.Clear(); avec Ep.SetErrortxtControl, "");

Answer 2

rendre explicite ce que vous faites:

bool isFactoryNameEmpty = IsFieldEmpty(ref txtFactoryName, true, "Required"); 
bool isShortNameEmpty = IsFieldEmpty(ref txtShortName, true, "Required"); 
bool isGodownEmpty = IsFieldEmpty(ref cboGodown, true, "Required"); 
if (isFactoryNameEmpty || isShortNameEmpty || isGodownEmpty) 
{ 
    // ... 
} 

(, je suppose que vous avez besoin également d'appeler les trois fonctions parce qu'ils ont des effets secondaires Où?

Answer 3

Ce que vous voyez s'appelle un court-circuit en C#. Si la première expression échoue, il ne sera pas la peine d'essayer les suivantes, car le résultat final a déjà été déterminé.

http://johnnycoder.com/blog/2006/08/02/short-circuit-operators-in-c/

Vous devriez vous | au lieu de || pour obtenir votre résultat.

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

C# opérateurs http://msdn.microsoft.com/en-us/library/6a71f45d.aspx

|| opérateur. http://msdn.microsoft.com/en-us/library/6373h346.aspx

| opérateur. http://msdn.microsoft.com/en-us/library/kxszd0kx.aspx

Answer 4

Pourquoi en auriez-vous besoin? La seule raison à laquelle je peux penser est que votre fonction "IsFieldEmpty" effectue également des calculs ou des modifications sur les données, et cela m'inquiète. Une fonction nommée "IsFieldEmpty" ne devrait vraiment rien faire d'autre.

Dans ce cas, du point de vue facilité d'utilisation/maintenabilité, vous seriez mieux avec:

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

Ou quelque chose de cet acabit.

Answer 5

Les réponses à ce jour supposaient que vous vouliez valider tous les champs, même après l'échec de l'un d'entre eux. Cette hypothèse n'est pas explicite dans votre question initiale. Donc, si cela ne vous dérange pas d'arrêter la validation quand un champ échoue, alors la solution la plus simple est d'utiliser l'opérateur & & au lieu de ||. Cela permettra d'atteindre l'objectif que vous vous êtes fixé: "si la première fonction renvoie vrai, elle devrait également appeler la fonction suivante et ainsi de suite". Cependant, si la première fonction renvoie false, aucune des autres fonctions ne sera appelée, ce qui peut ne pas être ce que vous voulez.