Discussion :

[Leelith]

Bonjour!

Lorsque j'essaye d'insérer une ligne dans ma base de donnée, je passe par la méthode suivante:

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
 
 public class Method
    {
        private static ISMS.Models.isms_databaseEntities db = new isms_databaseEntities();
 
        public void AddMethod(string _methodName)
        {
            tbl_method newMethod = new tbl_method();
            newMethod.met_name = _methodName;
            newMethod.tbl_client.cli_id = (int)1;
            db.AddTotbl_method(newMethod);
            db.SaveChanges();
        }
 
   // du code
}

Le seul problème, c'est qu'à l'éxecution, ça plante sur la ligne

Code :

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

1
2
 
newMethod.tbl_client.cli_id = (int)1;

lorsque j'essaye d'insérer la clé étrangère pour indiqué quel utilisateur (ici tbl_client) possède cette nouvelle ligne de la table tbl_method. Est-il nécessaire d'instancier également la table tbl_client, sachant qu'on insère dans la clé étrangère ce trouvant dans la table tbl_client?

Ceci, notamment à cause du mapping que fait EF. Je pense qu'il ne créer pas un champ avec une clé étrangère, mais plutôt va référencer sur le champ de la table contenant notre clé étrangère (tbl_client) de notre table principale (tbl_method), ce qui entraine donc une erreur de type

Object reference not set to an instance of an object

[Louis-Guillaume Morand]

bon alors déjà, le

Code :

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

(int)1

t'oublie le framework va automatiquement le convertir en int.

ensuite

Code :

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

newMethod.tbl_client.cli_id

tbl_client c'est effectivement le loading d'EF mais comme on fait du lazy loading c'est vide.

je pense que tu dois avoir une autre propriété sur ton objet tbl_method. par exemple
tbl_method.id_client = 1; ou un truc du genre. Il faudrait connaitre la structure de ta table.




pour finir, il va falloir revoir tes conventions de nommage si un jour tu bosses en équipe. Coté sql, appeler une table tbl_method n'est pas gênant (meme si tu verras plus souvent T_METHOD ou t_method), par contre, côté modèle C#, c'est moche. Ton objet représente une méthode mais pas une table de méthode quand tu fais tbl_method m = new tbl_method.
donc comme EF te le permet, prend l'habitude dans le designer EF, de renommer les classes coté C#, car ces noms n'ont pas besoin d'être identiques à la DB. Met plutôt Method, Client, etc. Au moins les noms sont parlants.
le nommage reste ton choix, mais il y a des conventions implicites qu'une majorité des développeurs utilisent, et le mieux est de rapidement prendre l'habitude d'utiliser ces conventions. Ca sera plus simple quand tu reprendras des projets d'autres personnes, ou l'inverse

[Leelith]

Merci pour ta réponse.

Effectivement, le nommage c'est un point que je dois améliorer Je vais corriger ça, et je te poste dès que possible le schéma des tables pour que ça soit plus parlant que ça

[Leelith]

Voilà donc une petite capture des 2 tables concernées :

[Leelith]

Je suis actuellement en train de lire un bouquin sur LINQ pour voir si j'arrive à résoudre mon problème.

Si quelqu'un à des propositions/solution, je suis preneur (toujours bloqué )

[Leelith]

J'ai trouvé une possible solution ici.

Seul soucis, je ne suis pas bien sûr de comprendre la signification des possibles problèmes que cette solution peut apporter:

This solution is essentially just a workaround that hides some of the limitations of independent associations, and like all workarounds it has limitations. The key then is to make an informed tradeoff between the benefits and the drawbacks. So here are some of the key drawbacks:

1. Properties in the partial class are not recognized by the Entity Framework, so you can't use them in your LINQ queries.
2. By referencing the fully qualified EntitySet name in our setter, we have coupled your Entity Class directly to the EntitySet. Now for most people this is not a problem, but it won't work if you try to re-use your entity classes between contexts or if you use MEST.
3. There are probably more... I'll add them as I think of them!

Si quelqu'un voit de quoi ça parle, merci d'avance

Bonne soirée!

L.

[Leelith]

J'avance toujours dans ma découverte. Je pense qu'il faut en fait jouer avec les entityreference et les EntityKey.

Seul soucis, lorsque je veux créer une nouvelle entrée dans ma base de donnée en passant par EF, j'écris ceci:

Code :

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

1
2
3
4
5
6
7
8
9
10
 
public void AddQuestion(string _questionName)
        {
 
            tbl_question newQuestion = new tbl_question();
            newQuestion.que_name = _questionName;
            newQuestion.que_ponderation = _ponderation;
            newQuestion.tbl_domainReference.EntityKey = new System.Data.EntityKey("Models.databaseEntities.tbl_domain", "dom_id", UneValeur);
            db.AddTotbl_question(newQuestion);
            db.SaveChanges();

Cependant j'ai une exception qui pointe sur mon Models.databaseEntities.tbl_domain... quelqu'un aurait-il une idée?

merci d'avance et bonne soirée

[Louis-Guillaume Morand]

pour commencer, tu ne peux pas travailler avec de tels objets

tbl_question, c'est le nom de ta table, c'est un choix que tu as fait, d'habitude on met juste T_QUESTION ou QUESTION mais ton choix n'est pas génant
PAR CONTRE, ton objet ne doit pas s'appeler tbl_question, c'est moche, et pas logique pour un cerveau humain. le designer edmx te permet d'avoir une classe qui s'appelle juste question et lié à une table tbl_question. donc passe sur ton designer, et renomme tes entités, ca sera plus propre.


ensuite, on ne sait pas ce qu'il y a pas dans tbl_domain ou ce que tu veux faire.
t'as essayé un truc comme ca

Code :

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

1
2
3
4
5
6
7
8
9
10
 
tbl_domain dom = from m in tbl_domainSet
                        where id = tavalue select m;
 
   tbl_question newQuestion = new tbl_question();
            newQuestion.que_name = _questionName;
            newQuestion.que_ponderation = _ponderation;
            newQuestion.tbl_domain = dom;
            db.AddTotbl_question(newQuestion);
            db.SaveChanges();

en gros toujours travailler avec des instances d'objets complets, pas juste une référence à laquelle tu ne dois pas toucher directement logiquement (je crois)

[Skyounet]

Code :

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

newQuestion.tbl_domainReference = xxx

Avec xxx un objet qui existe déjà dans ta BDD (donc il faut d'abord le récupérer), ou xxx un new TaClasse();

En gros

Code :

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

1
2
3
var domain = context.tbl_domainReference.FirstOrDefault(d => d.Id == 1);
 
newQuestion.tbl_domainReference = domain;

Ou

Code :

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

1
2
3
newQuestion.tbl_domainReference = new tbl_domainReference;
newQuestion.tbl_domainReference.xxx = xxx;
...

[Leelith]

pour commencer, tu ne peux pas travailler avec de tels objets

tbl_question, c'est le nom de ta table, c'est un choix que tu as fait, d'habitude on met juste T_QUESTION ou QUESTION mais ton choix n'est pas génant
PAR CONTRE, ton objet ne doit pas s'appeler tbl_question, c'est moche, et pas logique pour un cerveau humain. le designer edmx te permet d'avoir une classe qui s'appelle juste question et lié à une table tbl_question. donc passe sur ton designer, et renomme tes entités, ca sera plus propre.


ensuite, on ne sait pas ce qu'il y a pas dans tbl_domain ou ce que tu veux faire.
t'as essayé un truc comme ca

Code :

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

1
2
3
4
5
6
7
8
9
10
 
tbl_domain dom = from m in tbl_domainSet
                        where id = tavalue select m;
 
   tbl_question newQuestion = new tbl_question();
            newQuestion.que_name = _questionName;
            newQuestion.que_ponderation = _ponderation;
            newQuestion.tbl_domain = dom;
            db.AddTotbl_question(newQuestion);
            db.SaveChanges();

en gros toujours travailler avec des instances d'objets complets, pas juste une référence à laquelle tu ne dois pas toucher directement logiquement (je crois)

Ok, j'ai changé les noms. C'est chiant, il a fallut aller dans toutes les classes pour tout changer, mais effectivement maintenant c'est plus lisible

Sinon, si j'ai donc bien compris il faut donc récupérer un object dans une variable, et ensuite on peut alors faire pointer ma requête vers cette object?

Code :

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

newQuestion.tbl_domainReference = xxx

Avec xxx un objet qui existe déjà dans ta BDD (donc il faut d'abord le récupérer), ou xxx un new TaClasse();

En gros

Code :

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

1
2
3
var domain = context.tbl_domainReference.FirstOrDefault(d => d.Id == 1);
 
newQuestion.tbl_domainReference = domain;

Ou

Code :

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

1
2
3
newQuestion.tbl_domainReference = new tbl_domainReference;
newQuestion.tbl_domainReference.xxx = xxx;
...

Est-ce que récupérer un object en faisant:

Code :

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

1
2
3
4
 
var sample = from xxx in db.myTable
                  where xxx.id == 1
                  select new (xxx.champ1, xxx.champ2)

Ou ce n'est pas identique à une select normal (sans new, cependant je ne suis pas sûr de la syntaxe...) ?

Merci en tout cas pour ces précisions

[Leelith]

Dailleurs, maintenant que j'ai renommé mes classes dans mon modèle, j'ai une erreur à la compilation parce que Entity Framework n'a pas mis à jours les fichiers de mappage.
Est-ce que je dois changer manuellement tous mes anciens nom ou est-ce qu'il y a un moyen de faire ça automatiquement?

[Louis-Guillaume Morand]

Dailleurs, maintenant que j'ai renommé mes classes dans mon modèle, j'ai une erreur à la compilation parce que Entity Framework n'a pas mis à jours les fichiers de mappage.
Est-ce que je dois changer manuellement tous mes anciens nom ou est-ce qu'il y a un moyen de faire ça automatiquement?

bizarre, il aura dû si t'etais passé par le designer. en tout cas, normalement tu peux tout faire via le designer, meme le correctif du mappage, faut jamais editer à la main quoi que ce soit

[Leelith]

Je suis effectivement passé par le designer. J'ai sélectionné mes classes (comme on voit sur l'extrait de mes classes, et j'ai simplement renommé. Mais ça n'a pas mis à jours... à la compilation, il va me chercher des erreurs dans les fichiers de mapping :/

[Louis-Guillaume Morand]

Je suis effectivement passé par le designer. J'ai sélectionné mes classes (comme on voit sur l'extrait de mes classes, et j'ai simplement renommé. Mais ça n'a pas mis à jours... à la compilation, il va me chercher des erreurs dans les fichiers de mapping :/

tu peux pas refaire les mappings via le designer? c'est dans la partie du bas quand tu selectionnes une classe

[Leelith]

Si je sélectionne une classe, j'ai en bas du diagramme le "Mapping Details - MaClasse". C'est bien là?

Parce que sinon, j'ai pas grand chose là dedans :

• Maps to tbl_matable
• et la possibilité de rajouter une table ou vue

[Leelith]

Je crois avoir trouvé: en faite, dans mes vues (c'est un projet MVC), la partie inherits n'est pas changée... du coup, faut la changer, sinon ça marche pas. Ouai