Discussion :

[ClaudeBg]

Bonjour,

Voici mon problème :

J'ai une table "Client" et une table "Adresse"

Il y a une relation entre les deux tables (Client.Adresse, Client.AdresseReference) bien visible graphiquement.

Je recherche un Client dans la base -> monClient
Je recherche une Adresse dans la base -> monAdresse

Je n'arrive pas en fait à créer la relation entre le client et son adresse.

Si j'écris :

monClient.Adresse = monAdresse

Ca fonctionne tant que je ne ferme pas mon programme (en fait, apparemment la liaison se fait avec l'adresse en mémoire, pas avec l'adresse dans la table).

Si je réouvre mon programme, je ne sais plus joindre les propriétés de l'adresse du client (monClient.Adresse.rue par exemple) parce que monClient.Adresse vaut "null".

Je n'arrive pas à trouver la bonne syntaxe, j'ai essayé en utilisant monClient.AdresseReference également, ou la méthode attach, mais pas moyen de trouver.

La réponse est sûrement toute bête, mais j'ai du louper quelque chose

Merci d'avance

Claude

[Arnard]

Appels à SaveChanges() correctement faits ?

[ClaudeBg]

Bonjour,

Oui, voici une partie du code concerné (à cet endroit on a un Client "newClient" correctement initialisé excepté qu'on ne lui a pas affecté d'adresse, et une adresse "newAdresse" initialisée excepté sa clé primaire "Id" ):
Trouver est une méthode d'extension renvoyant l'adresse éventuellement trouvée dans la table.
Ce code s'exécute sans recevoir les messages d'erreur prévus, l'adresse se retrouve bien dans la table adresse en évitant les doublons, et le client dans la table client.

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
32
33
34
35
36
37
38
39
40
41
 
                                            // GERER L'ADRESSE
                                            // ---------------
            Adresse ad = newAdresse.Trouver();                  // chercher si l'adresse existe déjà
            if (ad != null)                                     // si oui
            {
                newClient.Adresse = ad; 
            }
            else                                                // s'il s'agit d'une nouvelle adresse
            {
                try
                {
                    newAdresse.SetNewId();                      // affecter une clé primaire libre
                    Bdd.AddToAdresse(newAdresse);               // ajouter l'adresse
                    Bdd.SaveChanges();                          // sauver modifications
                    newClient.Adresse =                         // affecter l'adresse enregistrée au client
                         (from a in Bdd.Adresse
                          where a.Id == newAdresse.Id
                          select a).FirstOrDefault();
                }
                catch (Exception ex1)                           // erreur 
                {
                    MessageBox.Show("Une erreur est survenue lors de l'écriture de l'adresse du client:\n\n"
                        + ex1.Message, "Erreur", MessageBoxButtons.OK, MessageBoxIcon.Error);
                    return false;
                }
 
            }
                                            // GERER LE CLIENT
                                            // ---------------
            try
            {
                Bdd.AddToClient(newClient);                 // ajouter le client
                Bdd.SaveChanges();                          // sauver modifications
            }
            catch (Exception ex2) 
            {
                MessageBox.Show("Une erreur est survenue durant l'écriture du client:\n\n"
                    + ex2.Message, "Erreur", MessageBoxButtons.OK, MessageBoxIcon.Error);
                return false;
            }

Lors de l'affichage, si je tente d'afficher le client et que je mets un point d'arrêt dans ma méthode d'affichage (figure "Avant"), on voit bien que tout est correctement initialisé.

Ensuite, je quitte le programme, je demande le réaffichage du même client en passant pas les mêmes méthodes, et le même point d'arrêt (figure "Apres") montre que le champs "adresse" du client est maintenant null. Je comprends mal pourquoi ces différences de comportement.

merci d'avance

Claude

[Arnard]

Un problème de contexte vraisemblablement.
Peut-on voir le code de la fonction Trouver ? Le reste me semble juste

[ClaudeBg]

Bonjour,

Aucun problème, voici :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
        public static Adresse Trouver(this Adresse adresse)
        {
            return 
                (from a in Bdd.Adresse
                 where (a.Boite.ToLower() == adresse.Boite.ToLower()
                 && a.CodeP.ToLower() == adresse.CodeP.ToLower()
                 && a.Localite.ToLower() == adresse.Localite.ToLower()
                 && a.Numero.ToLower() == adresse.Numero.ToLower()
                 && a.Pays.ToLower() == adresse.Pays.ToLower()
                 && a.Rue.ToLower() == a.Rue.ToLower())
                 select a).FirstOrDefault();
        }

C'est tout bête (tous les champs sont non nullables)
Note que si j'ajoute un client dont l'adresse n'existe pas, cette méthode renvoie null et donc le résultat n'est pas utilisé.

J'ai vérifié, que la méthode retourne null ou non, dans les deux cas j'ai strictement le même phénomène.

Bref, que j'affecte à mon client l'adresse retournée par cette méthode ou la nouvelle adresse créée puis relue de la base, c'est pareil.

Vu que je commence à me demander si le problème vient de l'enregistrement ou de la relecture, voici la méthode qui me renvoie le client qui a le numéro le plus élevé. La méthode d'affichage dont j'ai donné une partie du code teste évidemment si le client est null :

Code :

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

1
2
3
4
5
6
7
        private void AfficheClientDern()
        {
            AfficheClient(
                (from c in Bdd.Client
                orderby c.NumClient descending
                select c).FirstOrDefault());
        }

Je peux donner le code de toute la form s'il faut, il n'y a pas de secret défense, c'est destiné de toutes façons ensuite à être "open-source".
C'est juste que j'essaye de cerner au max le problème pour éviter de faire perdre du temps inutile à ceux qui veulent bien m'aider.


Bon, je viens en faite juste de m'apercevoir que le problème venait du select et non de l'enregistrement.

Si je remplace le bout de code ci-dessus par ceci :

Code :

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

1
2
3
4
5
6
7
8
9
        private void AfficheClientDern()
        {
            var test =
                (from c in Bdd.Client
                 orderby c.NumClient
                 select new {c.Nom,c.Adresse.Localite }).FirstOrDefault();
 
            MessageBox.Show(test.Localite);
        }

C'est à dire en demandant le renvoi explicite d'un champs de l'adresse dans une nouvelle structure, ça fonctionne et le messagebox me donne bien le nom de la localité du client.

Bref, le client a correctement été enregistré avec son adresse, mais je ne récupère pas l'adresse lors du select fait simplement sur le client.

Pourtant, logiquement, la propriété de navigation "Adresse" devrait être un simple pointeur, je ne comprends pas pourquoi la valeur de ce pointeur n'est pas rapatriée.

Et si je suis contraint d'inventer un nouveau type de client pour renvoyer un client avec une adresse, ça va devenir très lourd à gérer, surtout si on a une table qui pointe sur une autre table elle-même pointant sur une troisième table.

Y a-t-il une autre méthode?
Et intuitivement j'ai le sentiment qu'il faudrait utiliser le champs "AdresseReference" qui a été ajouté automatiquement par linq to entities, et qui ressemble furieusement dans sa dénomination à un pointeur. Mais j'ai essayé plusieurs manipulations avec ce champs, je n'arrive à rien de concluant.

Merci

Claude

[ClaudeBg]

Bonjour,

J'ai fini par trouver, après m'être arraché les cheveux.
Ca venait bien du select, il fallait inclure la table pointée.

Code :

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

1
2
3
4
5
6
7
        private void AfficheClientDern()
        {
            AfficheClient(
                (from c in Bdd.Client.Include("Adresse")
                 orderby c.NumClient descending
                 select c).FirstOrDefault());
        }

Je trouve ça curieux de devoir préciser la table pointée lorsqu'on rapatrie un simple pointeur, mais puisque ça fonctionne...

Seul problème, le nom de la table passé en "bête" string reporte la vérification à l'exécution et plus à la compilation, ce qui est dommage. Mais bon, peut-être que ce sera résolu à la nouvelle version?

Merci de t'être intéressé à mon problème, et vraiment désolé pour le dérangement

Merci
Claude