Procedure stockee avec parametre dans le contrôleur

**Korpin** · 25/03/2018, 18h13

Bonsoir,

J'ai un petit soucis que je n'arrive pas à résoudre malgré mes multiples recherches.

En effet, j'ai une procédure stockée qui attend un paramètre (l'id du client) afin d'effectuer un select.

Sur SQL tout se passe bien, quand j'exec j'ai le résultat voulu..

Par contre sous MVC j'ai un 'bug' par rapport (je pense) à mon paramètre...

Voici des exemples de mon code:

Mon contrôleur:

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
using System;
using System.Collections.Generic;
using System.Linq;
using System.Web;
using System.Web.Mvc;
using DAL;
 
namespace EphecAirlinesClient.Controllers
{
    public class OverviewController : Controller
    {
 
        private Ephec_AirlinesEntities contexteEF = new Ephec_AirlinesEntities();
 
        // GET: Overview
        public ActionResult Index(int? id)
        {
 
            return View(contexteEF.Sp_Select_All_Reservation_By_Id_Client(id));
        }
    }
}

Ma procédure stockée (qui fonctionne):

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
ALTER PROCEDURE [dbo].Sp_Select_All_Reservation_By_Id_Client
@IdClient INT
AS
BEGIN
 
	SET NOCOUNT ON;
 
	SELECT	r.RES_Id AS "Réservation n°",
			p.PLA_Date AS "Date du vol", 
			vdepart.VIL_Libelle AS 'Départ', 
			varrivee.VIL_Libelle AS 'Arrivée', 
			r.RES_Prix_Total AS "Prix total"
			FROM Reservation r
			JOIN Client AS c
			ON c.CLI_Id = r.RES_Client_Id
			JOIN Planning p
			ON p.PLA_Id = r.RES_Pla_Id
			JOIN Vol AS v
			ON v.VOL_Id = p.PLA_Vol_Id
			JOIN Ville AS vdepart
			ON vdepart.VIL_Id = v.VOL_Vil_Depart_Id
			JOIN Ville AS varrivee
			ON varrivee.VIL_Id = v.VOL_Vil_Arrivee_Id
			WHERE c.CLI_Id = @IdClient
END

Ma vue:

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
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
@model DAL.Sp_Select_All_Reservation_By_Id_Client_Result
 
@{
    ViewBag.Title = "Visualiser mes vols";
}
 
<h2>Visualiser mes vols</h2>
 
<div>
    <hr />
    <dl class="dl-horizontal">
        <dt>
            @Html.DisplayNameFor(model => model.Réservation_n_)
        </dt>
 
        <dd>
            @Html.DisplayFor(model => model.Réservation_n_)
        </dd>
 
        <dt>
            @Html.DisplayNameFor(model => model.Date_du_vol)
        </dt>
 
        <dd>
            @Html.DisplayFor(model => model.Date_du_vol)
        </dd>
 
        <dt>
            @Html.DisplayNameFor(model => model.Départ)
        </dt>
 
        <dd>
            @Html.DisplayFor(model => model.Départ)
        </dd>
 
        <dt>
            @Html.DisplayNameFor(model => model.Arrivée)
        </dt>
 
        <dd>
            @Html.DisplayFor(model => model.Arrivée)
        </dd>
 
        <dt>
            @Html.DisplayNameFor(model => model.Prix_total)
        </dt>
 
        <dd>
            @Html.DisplayFor(model => model.Prix_total)
        </dd>
 
    </dl>
</div>
<p>
    @Html.ActionLink("Edit", "Edit", new { /* id = Model.PrimaryKey */ }) |
    @Html.ActionLink("Back to List", "Index")
</p>

Mon message d'erreur:

Erreur du serveur dans l'application '/'.
L'élément de modèle passé dans le dictionnaire est de type « System.Data.Entity.Core.Objects.ObjectResult`1[DAL.Sp_Select_All_Reservation_By_Id_Client_Result] », mais ce dictionnaire requiert un élément de modèle de type « DAL.Sp_Select_All_Reservation_By_Id_Client_Result ».
Description : Une exception non gérée s'est produite au moment de l'exécution de la requête Web actuelle. Contrôlez la trace de la pile pour plus d'informations sur l'erreur et son origine dans le code.

Détails de l'exception: System.InvalidOperationException: L'élément de modèle passé dans le dictionnaire est de type « System.Data.Entity.Core.Objects.ObjectResult`1[DAL.Sp_Select_All_Reservation_By_Id_Client_Result] », mais ce dictionnaire requiert un élément de modèle de type « DAL.Sp_Select_All_Reservation_By_Id_Client_Result ».

Merci d'avance...

EDIT:

J'ajoute le fait que quand j'essaye avec SqlParameter j'ai le problème suivant:

Erreur CS1503 Argument 1*: conversion impossible de 'System.Data.SqlClient.SqlParameter' en 'int?' EphecAirlinesClient C:\Users\j0n\source\Projet\EphecAirlinesClient\EphecAirlinesClient\Controllers\OverviewController.cs 26 Actif

En gros je sais le faire en ADO.Net sauf que le prof souhaite qu'on fasse une partie d'un projet avec Entity Framework..

**DotNetMatt** · 25/03/2018, 20h39

Envoyé par Korpin

Code :

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

1
2
3
4
public ActionResult Index(int? id)
{
    return View(contexteEF.Sp_Select_All_Reservation_By_Id_Client(id));
}

Ma vue:

Code :

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

@model DAL.Sp_Select_All_Reservation_By_Id_Client_Result

Le probleme n'a rien a voir avec la procedure stockee ou meme avec Entity Framework.

Dans ta vue, tu dis que ton Model doit etre un seul objet de type DAL.Sp_Select_All_Reservation_By_Id_Client_Result.
Or dans ton controller tu injectes une liste d'objets.

2 solutions selon ce que tu souhaites faire :

1 - Tu laisses la vue telle quelle, et tu injectes un objet unique dans le controller, par exemple :

Code :

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

1
2
3
4
public ActionResult Index(int? id)
{
    return View(contexteEF.Sp_Select_All_Reservation_By_Id_Client(id)).FirstOrDefault();
}

2 - Tu laisses le controller tel quell et tu modifies la vue pour qu'elle accepte une liste d'objets :

Code :

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

@model List<DAL.Sp_Select_All_Reservation_By_Id_Client_Result>

**MoscoBlade** · 26/03/2018, 13h39

Bonjour,

Pour ce problème

Envoyé par Korpin

Erreur CS1503 Argument 1*: conversion impossible de 'System.Data.SqlClient.SqlParameter' en 'int?' EphecAirlinesClient C:\Users\j0n\source\Projet\EphecAirlinesClient\EphecAirlinesClient\Controllers\OverviewController.cs 26 Actif

Remplace dans Index de ton controller id par id.Value.
int? est différent de int

Mosco

**Korpin** · 26/03/2018, 22h46

Bonsoir

Merci pour vous réponses, j'aurai l'occasion d'essayer cela demain!

**Korpin** · 27/03/2018, 19h59

Hello,

Merci pour vos réponses!

Quand j'ajoute .FirstOrDefault() dans mon contrôleur, ma vue n'accepte plus mon @foreach que voici:
Apparemment, cela ne fonctionne pas sur des variables de type stored proc

Auriez-vous une solution?

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
42
43
44
45
46
47
48
49
50
51
52
53
54
55
@model DAL.Sp_Select_All_Reservation_By_Id_Client_Result

@{
    ViewBag.Title = "Index";
}

<h2>Index</h2>

<p>
    @Html.ActionLink("Create New", "Create")
</p>
<table class="table">
    <tr>
        <th>
            @Html.DisplayNameFor(model => model.Réservation_n_)
        </th>
        <th>
            @Html.DisplayNameFor(model => model.Date_du_vol)
        </th>
        <th>
            @Html.DisplayNameFor(model => model.Départ)
        </th>
        <th>
            @Html.DisplayNameFor(model => model.Arrivée)
        </th>
        <th>
            @Html.DisplayNameFor(model => model.Prix_total)
        </th>
        <th></th>
    </tr>

@foreach (var item in Model) {
    <tr>
        <td>
            @Html.DisplayFor(modelItem => item.Réservation_n_)
        </td>
        <td>
            @Html.DisplayFor(modelItem => item.Date_du_vol)
        </td>
        <td>
            @Html.DisplayFor(modelItem => item.Départ)
        </td>
        <td>
            @Html.DisplayFor(modelItem => item.Arrivée)
        </td>
        <td>
            @Html.DisplayFor(modelItem => item.Prix_total)
        </td>
        <td>
            @Html.ActionLink("Mettre à jour", "Update", new { /* id=item.PrimaryKey */ }) |
            @Html.ActionLink("Supprimer", "Delete", new { /* id=item.PrimaryKey */ })
        </td>
    </tr>
}
</table>

Seconde question, dans ma procédure stockée, j'ai des AS afin de renommer le nom de mes colonnes.
Hors, quand je lance mon appli, cela ne prend pas en compte les AS.
Y'a-t-il un moyen de modifier cela à part écrire manuellement le nom de mes colonnes dans mon tableau (cela fonctionne mais j'ai du mal à croire qu'il n'y a pas une solution miracle lol)?

Grand merci d'avance ;-)

**DotNetMatt** · 27/03/2018, 21h10

Envoyé par Korpin

Quand j'ajoute .FirstOrDefault() dans mon contrôleur, ma vue n'accepte plus mon @foreach que voici:
Apparemment, cela ne fonctionne pas sur des variables de type stored proc

Ca ne veut rien dire une "variable de type stored proc"

Ta variable est de type Sp_Select_All_Reservation_By_Id_Client_Result.

Si tu veux faire un foreach tu dois avoir une collection d'objets. Donc tu dois modifier ton model comme ceci :

Code :

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

@model List<DAL.Sp_Select_All_Reservation_By_Id_Client_Result>

et supprimer le FirstOrDefault() dans ton controller, afin de transmettre une collection a ta vue.

Envoyé par Korpin

Seconde question, dans ma procédure stockée, j'ai des AS afin de renommer le nom de mes colonnes.
Hors, quand je lance mon appli, cela ne prend pas en compte les AS.
Y'a-t-il un moyen de modifier cela à part écrire manuellement le nom de mes colonnes dans mon tableau (cela fonctionne mais j'ai du mal à croire qu'il n'y a pas une solution miracle lol)?

En principe tu peux renommer les colonnes dans le model EF. Si tu as mis a jour les noms de colonne apres avoir genere ton model il faut le mettre a jour.

**Korpin** · 28/03/2018, 15h24

Bonjour!

Top, je vais donc faire avec une List<>.

Petite question opti, j'ai tendance à déclarer une Liste directement dans le contrôleur mais ne vaudrait-il mieux pas que j'utilise un modèle reprenant les champs ainsi que la liste, que je ferais passer à mon contrôleur?

Merci beaucoup pour toutes tes réponses en tout cas ;-)

Pour les noms des colonnes, enfait j'utilise un user de ma DB qui n'a accès qu'aux stored.. pas aux tables. les AS sont dans ma stored, je me suis dit que cela venait peut-être de là mais je vais checker dans le modèle EF ce soir.
Au pire je les modifierai manuellement dans mon code, ce n'est pas bien grave à mon niveau lol.

**DotNetMatt** · 28/03/2018, 17h06

Envoyé par Korpin

Petite question opti, j'ai tendance à déclarer une Liste directement dans le contrôleur mais ne vaudrait-il mieux pas que j'utilise un modèle reprenant les champs ainsi que la liste, que je ferais passer à mon contrôleur?

Oui tu peux ajuster ton model comme bon te semble. Le principal c'est que pour ton foreach, tu dois disposer d'une liste quelque part : soit le model directement, soit une propriete du model. A toi de voir ce qui est le plus pratique et adapte a ton cas.

**Korpin** · 28/03/2018, 17h11

Parfait, un grand merci à toi!

Je devrais pouvoir me débrouiller avec tous ces éléments ;-)

Je clôture en résolu

EDIT:

Voici comment j'ai procédé:

Contrôleur:

Code :

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

1
2
3
4
5
6
        public ActionResult Index(int id)
        {
            var MaListe = contexteEF.Sp_Select_All_Reservation_By_Id_Client(id).ToList();
 
            return View(MaListe);
        }

Vue:

Code :

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

@model IEnumerable<DAL.Sp_Select_All_Reservation_By_Id_Client_Result>

(avec un foreach)

**footsteps** · 10/05/2018, 19h57

Bonne chose que ce soit résolu.
Regarde maintenant l'architecture.
Pas de logique, d'appel à la DB dans le controller!
Bonne continuation.

**Korpin** · 11/05/2018, 10h20

@footsteps

C'est une question que je me suis déjà plusieurs fois posé.. même si j'utilise Entity Framework, suis-je obligé de créer mes propres modèles? C'est une convention? Car les modèles sont déjà générés par EF..
Je comprends la logique quand je souhaite utiliser plusieurs modèles dans une même vue, mais pour le reste?

Peux-tu m'éclairer?

Thx!

**footsteps** · 11/05/2018, 19h27

Entityframework crée des entités. (images des tables de ta BDD)
Tu devrais créer tes Models pour le business et des Viewmodels pour ta vue.
C'est une gymnastique, mais indispensable.
Si tu veux, tu mappes d'une couche à l'autre.
Bonne continuation.

Procedure stockee avec parametre dans le contrôleur [Débutant]

ASP.NET MVC

Discussions similaires

Partager

Partager