Discussion :

[onizuka159]

Bonjour, je code en C#, asp.net et visual studio 2005.

J'effecture un executescalar pour pouvoir récupérer une valeur dans ma base de donnée. Il me récupere bel et bien la valeur, mais lorsqu'il n'y a pas de valeur il m'indique une erreur "la référence dobjet nest pas défni à une instance d'un objet".

voici mon code

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
requete2 = "Select L_Duree from temps_passe where N_tache="+numtache+" and D_temp_passe='"+date.ToShortDateString()+"' and N_collaborateur="+session.Num_Utilisateur;
        connection objconex3 = new connection();
        objconex3.command = new SqlCommand(requete2, objconex3.laconnection);
        decimal valeur_existante = 0;
        objconex3.seconnecter();
        valeur_existante=(decimal)objconex3.command.ExecuteScalar();            
        objconex3.déconnecter();

connection est une classe que j'ai créé. J'aimerais que lorsque la base de donnée contien uen valeur, qu'elle soit récupéré dans valeur_existante (ca, ca fonctione trésbien) mais lorsqu'il n'y en na pas, que valeur_existante garde sa valeur 0. Ne plus avoir l'exception, l'erreur donc.

[rattlehead]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

Select isnull(L_Duree,0) from temps_passe

[SaumonAgile]

Essaie avec :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

valeur_existante= Convert.ToDecimal(objconex3.command.ExecuteScalar());

[onizuka159]

j'ai utiliser le convert, merci beaucoup, ca fonctionne

[guitoux1]

Il faut que tu test le retour de ExecuteScalar() avant de la caster en décimal :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
 
requete2 = "Select L_Duree from temps_passe where N_tache="+numtache+" and D_temp_passe='"+date.ToShortDateString()+"' and N_collaborateur="+session.Num_Utilisateur;
        connection objconex3 = new connection();
        objconex3.command = new SqlCommand(requete2, objconex3.laconnection);
        decimal valeur_existante;
        objconex3.seconnecter();
        object objRetour = objconex3.command.ExecuteScalar();
        valeur_existante= objRetour == null ? 0 : (decimal)objRetour;        
        objconex3.déconnecter();

[SaumonAgile]

Il faut que tu test le retour de ExecuteScalar() avant de la caster en décimal :

ça fonctionne, mais c'est bien compliqué non ?

[guitoux1]

ça fonctionne, mais c'est bien compliqué non ?

Ben je sais pas, je dirais que c'est normal. Vérifier qu'une variable n'est pas null avant de la caster.

Mais oui effectivement, le Convert.ToDecimal() est plus "simple" (en fait, il est juste plus court à écrire), mais si tu regarde ce que fait Convert.ToDecimal(object value) tu veras :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
 
public static decimal ToDecimal(object value)
{
    if (value != null)
    {
        return ((IConvertible) value).ToDecimal(null);
    }
    return 0M;
}

Donc c'est exactement la même chose. C'est juste qu'en postant ma réponse, j'avais pas vu que t'avais déjà répondu

(T'es trop rapide )

[SaumonAgile]

Mais oui effectivement, le Convert.ToDecimal() est plus "simple"

C'était juste pour être contrariant

(T'es trop rapide )

C'est ce que dit ma femme aussi

[guitoux1]

C'est ce que dit ma femme aussi

lol
c'est pour ça que tu t'es mis à l'informatique

[SaumonAgile]

c'est pour ça que tu t'es mis à l'informatique

Non c'est parce que n'aime bien les nordinateurs