Variable Context Question

Question

Dans l'extrait de code suivant, si j'omets la ligne de code entourée par /////, j'obtiens une erreur qui se lit comme suit: "Utilisation de la variable locale non affectée CurrentDate". Il me semble un peu idiot de donner juste une valeur arbitraire à CurrentDate, y a-t-il un meilleur moyen de contourner cela?

DateTime CurrentDate; 

/////////////////////////// 
CurrentDate = DateTime.Now; 
/////////////////////////// 

if(1==1) 
{ 
CurrentDate = DateTime.Now.AddDays(1); 
} 

if(CurrentDate == DateTime.Now) 
{ 
... 
} 

Answer 1

Ne pas faire if (1 == 1)? Sérieusement si le compilateur vous donne cette erreur c'est soit parce que votre code est faux, soit parce que c'est trop complexe et pourrait être mieux exprimé d'une autre manière où vous n'avez pas besoin d'accéder à des variables non assignées.

Pouvez-vous trouver un exemple réel où vous obtenez cette erreur où il n'y a pas de solution évidente en faisant un simple refactoring? Cela rendrait votre question plus responsable.

Cela dit que si vous ne courez dans une de ces situations, il y a quelques autres approches que vous pouvez utiliser:

DateTime CurrentDate = DateTime.MaxValue; 
DateTime CurrentDate = default(DateTime); 
DateTime? CurrentDate = null; 

J'aime la dernière option, car elle exprime ce que vous voulez dire - que vous ne connaître la valeur. Cela rend le code un peu plus détaillé, car vous disposez d'un niveau de redirection supplémentaire chaque fois que vous souhaitez accéder à une valeur. Vous pouvez utiliser le temps passé à taper .Value pour déterminer si vous avez correctement géré la situation où elle pourrait être nulle.

Également: Avez-vous considéré que la valeur de DateTime.Now pouvait changer entre le premier et le deuxième appel? Cette déclaration finale if semble ne pas faire ce que vous vouliez.

Answer 2

Vous pouvez le faire:

DateTime CurrentDate; 


if(1==1) 
{ 
CurrentDate = DateTime.Now.AddDays(1); 
} 
else 
{ 
/////////////////////////// 
CurrentDate = DateTime.Now; 
/////////////////////////// 
} 

if(CurrentDate == DateTime.Now) 
{ 
... 
} 

Cela permettra d'éliminer l'erreur du compilateur.

NOTE: dans ce VS2008 sera compilation:

if(1==1) 
{ 
CurrentDate = DateTime.Now.AddDays(1); 
} 
//else 
//{ 
///////////////////////////// 
//CurrentDate = DateTime.Now; 
///////////////////////////// 
//} 

if(CurrentDate == DateTime.Now) 
{ 
// 
} 

Answer 3

pourquoi ne pas simplement affecté directement

DateTime CurrentDate = DateTime.Now; 

Answer 4

L'exemple est un peu alambiquée, mais, non, il n'y a pas un meilleur moyen de contourner cela.

Vous devez attribuer une valeur à une position variable dans un bloc conditionnel, même si vous êtes sûr que le bloc exécutera toujours (le 1 = 1 dans votre exemple)

Cependant, je recommanderais contre DateTime Maintenant que la valeur d'initialisation, parce que par un certain mirracle, le bloc ne s'exécute pas, la situation devrait être facilement détectable, et DateTime.Now est bien plus réel que DateTime.MinValue

Answer 5

J'ai toujours mis mes objets à zéro qu'il lève une exception de référence nulle si une valeur n'est pas créée. Cependant, comme Hans a souligné que null ne fonctionne pas dans cette situation, Microsoft a décidé de rendre DateTime nullable dans leur sagesse infinie. En tant que tel, j'utilise normalement DateTime.MinValue, car il me donne une valeur spécifique à vérifier et autre que le temps de voyager sera toujours dans le passé, contrairement DateTime.MaxValue qui vient ici pour certains d'entre vous archaïque 32bit peeps.

Exemple

DateTime CurrentDate; 

/////////////////////////// 
CurrentDate = DateTime.MinValue; 
/////////////////////////// 

if(1==1) 
{ 
CurrentDate = DateTime.Now.AddDays(1); 
} 

if(CurrentDate == DateTime.Now) 
{ 
... 
}