Problème avec LinkButton / CommandArgument

Version imprimable

Bonjour,

Dans mon code behind, j'ai une méthode que j'appel dans le Page_Load.
Dans cette méthode jai un morceau de code qui doit me générer des LinkButton en fonction d'un nombre de page.
Au clic (onclick), je souhaite appeler une autre méthode dans laquelle je veux récupérer l'argument (CommandArgument).

Voici le morceau de code:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 for (i = 1; i <= this._nbPage; i++) { LinkButton LB = new LinkButton(); LB.ID = "HL" + i; LB.Text = i.ToString(); LB.Style.Add("padding-right", "3px"); //LB.Command += new CommandEventHandler(ChangeIndex_Click); LB.OnClientClick = "ChangeIndex_Click"; LB.CommandName = "go"; LB.CommandArgument = (i - 1).ToString(); this.paginator.Controls.Add(LB); }
Voici l'autre méthode que je veux appeler:
Code:

1 2 3 4 5 6 7 protected void ChangeIndex_Click(object sender, EventArgs e) { LinkButton senderLink = sender as LinkButton; int Index = int.Parse(senderLink.CommandArgument); this._pageIndex = Index; }
Les LinkButton sont bien générés, mais la méthode ChangeIndex_Click n'est jamais appelée et j'ai l'impression que les arguments ne sont pas affectés.
Je précise que je n'est aucun message d'erreur.

Pouvez-vous m'aider SVP.:mouarf:

Merci par avance.:ccool:

02/02/2010, 16h25
laedit

Utilise la propriété OnClick plutôt que OnClientClick.

OnClientClick contient le nom de la méthode javascript qui sera exécutée tandis que OnClick contient le nom de la méthode du code-behind qui sera exécutée.
02/02/2010, 16h53
nasimpat7

J'ai essayé de mettre OnClick à la place, mais il semble que via mon code behind je n'est pas accès à cette méthode.

Code:

LB.OnClick = ...;

J'obtient d'ailleurs cette erreur :

Citation:

CS0122: 'System.Web.UI.WebControls.LinkButton.OnClick(System.EventArgs)' est inaccessible en raison de son niveau de protection
02/02/2010, 17h23
laedit

Effectivement, tu n'y as accès qu'à partir du côté client (page asp). Mais tu devrais y arriver en passant par l'événement Click du LinkButton.
02/02/2010, 17h46
nasimpat7

En effet, j'ai bien accès à l'événement.
Je fais comme suit:

Code:

LB.Click += new EventHandler(this.ChangeIndex_Click);

Je passe bien par ma méthode et je récupère bien mon argument.
Mais j'ai un autre problème dont je vous parlerai un peu plus tard par manque de temps.

A+

Bonjour,

Comme vous l'aurez deviné, ce code à pour but de faire de la pagination.
Plus précisément sur un contrôle asp:repeater.

Grâce à vous, je parviens maintenant à passer mon index de page et à le récupérer.

Mais je n'arrive pas à l'affecter à mon CurrentPageIndex de mon PagedDataSource, je ne comprend pourquoi.:?

Voici le fameux code:

Code:

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
    protected void doPagination()
    {
        //DataSet myItems = new DataSet();
        DataView myItems = (DataView)SqlDataSource1.Select(DataSourceSelectArguments.Empty);
 
        // Populate the repeater control with the Items DataSet
        PagedDataSource objPds = new PagedDataSource();
        objPds.DataSource = myItems;
 
        // Indicate that the data should be paged
        objPds.AllowPaging = true;
 
        // Set the number of items you wish to display per page
        objPds.PageSize = 7;
 
        this._nbPage = objPds.PageCount;
        objPds.CurrentPageIndex = this._pageIndex;
 
        Repeater1.DataSource = objPds;
        Repeater1.DataBind();
 
        int i;
 
        for (i = 1; i <= this._nbPage; i++)
        {
            LinkButton LB = new LinkButton();
            LB.ID = "HL" + i;
 
            LB.Text = i.ToString();
            LB.Style.Add("padding-right", "3px");
            LB.Click += new EventHandler(this.ChangeIndex_Click);
 
            LB.CommandName = "go";
            LB.CommandArgument = (i - 1).ToString();
 
            this.paginator.Controls.Add(LB);
 
        }
    }

ma méthode qui récupère l'index :

Code:

1
2
3
4
5
6
7
    protected void ChangeIndex_Click(object sender, EventArgs e)
    {
        LinkButton senderLink = sender as LinkButton;
        int Index = int.Parse(senderLink.CommandArgument);
        this._pageIndex = Index;
 
    }

Et ma propriété pageIndex:

Code:

1
2
3
4
5
6
7
    private int pageIndex;
 
    public int _pageIndex
    {
        get { return this.pageIndex; }
        set { this.pageIndex = value; }
    }

Quand je rentre manuellement l'index, celui-ci est bien pris en compte.
Ex:

Code:

objPds.CurrentPageIndex = 1;

La 2ème page est bien affichée.

Je précise que la méthode doPagination() est appelée dans le Page_Load.

Merci par avance pour votre aide.:ccool:

08/02/2010, 09h56
laedit

Mets un point d'arrêt dans ChangeIndex_Click afin de voir si cela récupère bien la valeur attendue.

Et je ne vois pas l'intérêt de passer par CommandArgument pour récupérer un entier. Il ne serait pas plus pratique de passer par le Tag du LinkButton ?
08/02/2010, 10h38
nasimpat7

Bonjour,

J'ai dit précédemment que j'arrivais bien à récupérer la valeur voulue, et d'ailleurs grâce à toi, là n'est plus le problème.:ccool:

Le problème que j'ai actuellement, est que l'index n'est pas pris en compte bien que la propriété soit bien modifiée par le setter.
08/02/2010, 10h44
laedit

Et si tu mets un point d'arrêt dans DoPagination afin de vérifier que l'index que tu lui passe est bien celui que tu as récupéré ?

Ensuite, au niveau du nommage, la convention veut que qu'une propriété publique commence par une majuscule : _pageIndex => PageIndex ;)

Finalement,
en séparant le code suivant de la méthode doPagination() et en le plaçant directement dans dans le Page_Load, miracle ça fonctionne.:lol:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
           int i;
 
            for (i = 1; i <= this._nbPage; i++)
            {
                LinkButton LB = new LinkButton();
                LB.ID = "HL" + i;
 
                LB.Text = i.ToString();
                LB.Style.Add("padding-right", "3px");
                //LB.Command += new CommandEventHandler(ChangeIndex_Click);
                LB.Click += new EventHandler(this.ChangeIndex_Click);
                //LB.OnClientClick = "ChangeIndex_Click";
 
                LB.CommandName = "go";
                LB.CommandArgument = (i - 1).ToString();
 
                this.paginator.Controls.Add(LB);
 
 
            }

Voilà à quoi ressemble mon Page_Load:

Code:

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
protected void Page_Load(object sender, EventArgs e)
    {
            ...
            doPagination();
 
            int i;
 
            for (i = 1; i <= this._nbPage; i++)
            {
                LinkButton LB = new LinkButton();
                LB.ID = "HL" + i;
 
                LB.Text = i.ToString();
                LB.Style.Add("padding-right", "3px");
                //LB.Command += new CommandEventHandler(ChangeIndex_Click);
                LB.Click += new EventHandler(this.ChangeIndex_Click);
                //LB.OnClientClick = "ChangeIndex_Click";
 
                LB.CommandName = "go";
                LB.CommandArgument = (i - 1).ToString();
 
                this.paginator.Controls.Add(LB);
 
 
            }
    }

Pour standardiser tout ça, je vais faire un petit contrôle utilisateur.

Merci encore pour votre aide.:ccool: