Discussion :

[DenisBuzz]

Bonjour
Je voudrai trouver un "idclient" non présent dans ma table.
J'ai donc créer une fonction IdClient() retournant un string:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
 
protected string IdClient()
    {
        int idclient = 1;
        bool trouve = false;
 
        while( trouve == false){
            string sql = "Select idclient from clients where idclient="+idclient;
            using (SqlConnection conn = new SqlConnection(GetConnectionString()))
            using (SqlCommand cmd = new SqlCommand(sql, conn))
            {
                conn.Open();
 
 
               int id = (int)cmd.ExecuteScalar();
 
                conn.Close();
                if (id == idclient)
                {
                    idclient++;
                }
                else
                {
                    trouve = true;
                }
 
            }           
        }
        string idcli = idclient.ToString();
        return idcli;
    }

cependant j'obtient l'erreur suivante:

System.NullReferenceException: La référence d'objet n'est pas définie à une instance d'un objet.

sur la ligne:

Code :

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

int id = (int)cmd.ExecuteScalar();

Je ne comprend pas pourquoi.

Merci de m'aider

[PatStan17]

Tu dois construire ta requête SQL comme ça :

Code :

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

string sql = "Select idclient from clients where idclient="+idclient + "; SELECT @@IDENTITY";

P.JEAN

[DenisBuzz]

J'ai modifié ma requête comme tu m'as dit mais toujours la même erreur au même endroit .

[PatStan17]

escuses j'avais mal lu ta question ... je croyais que tu voulais récuperer l'identifiant de la ligne que tu ajoutes avec ta requête .....




P.JEAN

[kheironn]

Cmd doit être null, avec le using ça m'étonne un peu...
par contre tu as fait des Conn.open()/Close() qui sont inutils car fait par le using. C'est peut-être de là que viens ton erreur.

[DenisBuzz]

Si j'enlève les conn.Open et les conn.Close ça me met une autre erreur:

ExecuteScalar nécessite une Connection ouverte et disponible. La connexion est actuellement fermée.

Donc on est obligé de le mettre. Et dans ma table j'ai bien un idclient qui est égal à 1.

[kheironn]

il me semblait que ça ouvrait et fermait les connexions...
je n'utilise jamais cette synthaxe. avec le fait main, je sais ce que je fais et quand je le fais!

[Robotech]

Logique tu demandes un produit scalaire
tu plantes simplement des lors que ta requête ne renvoi rien
il faut que tu utilises un COUNT

Code :

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

string sql = "Select COUNT(idclient) from clients where idclient='"+idclient + "'";

[Robotech]

Par contre essaye de nettoyer ton code car hum c'est crading(sorry)

[kheironn]

Logique tu demandes un produit scalaire
tu plantes simplement des lors que ta requête ne renvoi rien
il faut que tu utilises un COUNT
string sql = "Select COUNT(idclient) from clients where idclient='"+idclient + "'";

Ah oui, tiens ! Pourquoi Scalar ? Je pense que tu as plutôt besoin d'un cmd.ExecuteReader().

[Robotech]

Heu excuse moi kheironn, mais si executeScalar te renvoi 0 tu as l'id qu'il te faut et tu n'as plus qu'a utiliser la variable idclient ,je t'avoue ne pas être super concentré mais un count et executescalar me paraissent tout a fait cohérent ici.

[kheironn]

DenisBuzz,
Je viens de relire ton code et javoue ne pas comprendre quelque chose : le besoin final.
D'ailleur ta méthode est mal nommée. En général on donne un nom contenant un verbe, par exemple FindFirstFreeCustomerId(), ce que semble faire cette dernière.
Comment utilises-tu cette méthode ? Pourquoi recherche un id libre ? Une valeur auto-incrémentée ne conviendrait-elle pas ? SqlServer gère très bien ces choses là. J'ai l'impression que ta bas est suceptible d'avoir des trous et que c'est la raison de ce choix de fonctionnalité.

[Robotech]

+1 pour kheironn,ma réponse ne signifie pas non plus que je comprend ce genre de besoin