Discussion :

[Louison]

Bonjour,

Je fait un projet .NET ASP C#, et j'essaie d'afficher un tableau contenant des enregistrements venant d'une BD. Sur chaque ligne de ce tableau j'ai un bouton (ou une image c'est égal) permettant de supprimer dans la BD l'enregistrement sélectionné.

J'ai donc créé un tableau simple en ASP (je veux pas de GridView), sur lequel je parcours ma méthode de retour de données, jusque la aucun problème.

Mon problème est "comment récupérer l'ID de l'utilisateur à supprimer pour le passer à une méthode C#". J'ai donc créé un bouton dynamiquement, lui ai passé en paramètre l'iD pour pouvoir le récupérer en C#, mais j'ai une jolie erreur lors de l'affichage du site. Voici mon code ASP :

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
 
<%@ Page Language="C#" MasterPageFile="~/MasterPage.master" AutoEventWireup="true" CodeFile="remTechnicien.aspx.cs" Inherits="remTechnicien" Title="Page sans titre" %>
 
<asp:Content ID="Content1" ContentPlaceHolderID="head" Runat="Server">
</asp:Content>
<asp:Content ID="Content2" ContentPlaceHolderID="ContentPlaceHolder1" Runat="Server">
    <form id="form1" runat="server">
    <br /><h2>Supprimer un technicien</h2><br />
 
    <table id="Liste" class="normal">
        <thead><tr><td></td><td><b>Nom d'utilisateur</b></td><td><b>Nom</b></td><td><b>Prénom</b></td></tr></thead>
    <% int i = 0; foreach (Technicien technicien in showList()) {%>
        <tr>
            <td>
            <%
                Button Bt = new Button();
                Bt.Text = "oui";
                Bt.ID = Convert.ToString(i);
                Bt.CommandName = Convert.ToString(i);
                Bt.Click += new EventHandler(Button1_Click);
                this.Controls.Add(Bt);
                i++; 
            %>
            
            
            </td>
            <td><%= technicien.userName%></td>
            <td><%= technicien.name%></td>
            <td><%= technicien.prename%></td>
        </tr>
    <%  } %>
    </table>
    </form>
    </asp:Content>

Et mon erreur :

[InvalidOperationException: La collection a été modifiée ; l'opération d'énumération peut ne pas s'exécuter.]
System.Web.UI.ControlCollectionEnumerator.MoveNext() +8648494
System.Web.UI.Control.RenderChildrenInternal(HtmlTextWriter writer, ICollection children) +143........

J'ai également essayé de créer un bouton directement en ASP, mais je n'arrive pas passer de code à l'option "CommandName", il m'affiche le code et non son interprétation :

Code :

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

<asp:Button ID="Button1" runat="server" CommandArgument="test" CommandName="<%= technicien.userName %>" Text="Button" />

J'ai essayé beaucoup de choses, je n'arrive à rien, mon but est vraiment de récupérer l'iD pour le passer à C#. Je suis preneur de toute solution, je ne sais plus trop comment faire.

Merci pour votre aide

[oyigit]

Peut-on voir la fonction showList() ?

[Louison]

Oui biensur, voici le code entier C# :

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
 
public partial class remTechnicien : System.Web.UI.Page
{
    protected void Page_Load(object sender, EventArgs e) {
 
    }
 
    public List<Technicien> showList() {
        DataProvider dataProvider = DataProvider.getInstance();
        List<Technicien> techniciens = dataProvider.getTechniciens();
        return techniciens;
    }
 
    public void remTechniciens(int idTechnicien) {
      //Je lancerai ici le SQL pour supprimer le technicien d'après l'id en paramètre
 
    }
 
    protected void Button1_Click(object sender, EventArgs e) {
        Button btn = (Button)sender;
        string id = btn.CommandName;
        System.Windows.Forms.MessageBox.Show(id);
    }
}

la méthode Button1_Click() est appelée lors du clic sur le bouton présent à chaque ligne (créé dynamiquement), et la récupération de l'argument "CommandeName" est fonctionnel, cette méthode est donc ok, c'est vraiment au niveau ASP que ça croche...

Merci pour votre aide, car la je bloque complet. Et je le redis, si vous avez une solution plus élégante je suis preneur (sauf GridView car on devrait bcp modifier notre applic)

[oyigit]

Ok. ça ne semble pas venir de là.

On dirait plutôt que l'erreur vient de :

Code :

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

this.Controls.Add(Bt);

dans ton code.

Lorsqu'une page contient des tags <%= %> il ne faut pas modifier directement sont arborescence de contrôles. Essaie de rajouter un placeholder autour de ton tag.

[Louison]

Ok j'ai rajouté le placeHolder, en effet c'était bien le add qui posait problème.. Par contre maintenant à chaque fois que j'ai une nouvelle ligne, un bouton supplémentaire est ajouté :

ligne 1 : 1 bouton
ligne 2 : 2 boutons
ligne 3 : 3 boutons....

Voici le code modifié :

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
 
    <% foreach (Technicien technicien in showList()) {
           Button Bt; %>
        <tr>
            <td>
            <asp:PlaceHolder ID="plControl" runat="server" />
            <%
                Bt = new Button();
                Bt.Text = "oui";
                Bt.ID = "test";
                Bt.CommandName = "oui";
                Bt.Click += new EventHandler(Button1_Click);
                this.plControl.Controls.Add(Bt);
            %>
            </td>
            <td><%= technicien.userName%></td>
            <td><%= technicien.name%></td>
            <td><%= technicien.prename%></td>
        </tr>
    <%  } %>

Merci d'avance

[Louison]

J'ai essayé de créer un placeHolder en c# directement, puisqu'en faite j'ai plusieurs boutons car c'est toujours le même placholder qui contient de plus en plus d'éléments (1 de plus à chaque tour de boucle). Voici donc mon code :

Code :

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

1
2
3
4
5
6
7
8
                PlaceHolder placeh = new PlaceHolder();
                placeh.ID = "test";
                Button Bt = new Button();
                Bt.Text = Convert.ToString(i);
                Bt.ID = Convert.ToString(i);
                Bt.CommandName = Convert.ToString(i);
                Bt.Click += new EventHandler(Button1_Click);
                placeh.Controls.Add(Bt);

Malheureusement les boutons ne s'affichent plus....
Je ne sais vraiment plus quoi faire... Avez-vous d'autres idées pour lancer une méthode C# avec paramètre depuis ASP ?