erreur C# svp !

L'index était hors limites. Il ne doit pas être négatif et doit être inférieur à la taille de la collection.
Nom du paramètre : index

voici le bout de code qui fait ça :

Code:

---

string productId = Convert.ToString(GridView1.DataKeys[row.RowIndex].Value);

---

Le message d'erreur est assez clair, row.rowindex est null.
tu as cliqué sur une ligne qui existe pas ou quelque chose comme ca.


tu dois mettre ca au début de ta méthode pour éviter cette erreur

Code:

---

1 2	if(row.index == -1) return;

---

:)

que faut -il faire pour quil soit pas null ?

voici ce que jai fais et ca marche pas !

Code:

---

1 2 3 4 5 6 7

if (row.RowIndex == -1) { return; } else { string productId = Convert.ToString(GridView1.DataKeys[row.RowIndex].Value);

---

tu récupère comment l'objet row ?

je le recupere comme ceci :

Code:

---

1 2 3 4 5

for (int i = 0; i < GridView1.Rows.Count; i++) { GridViewRow row = GridView1.Rows[i]; } string productId = Convert.ToString(GridView1.DataKeys [row.RowIndex].Value);

---

Citation:

---

Envoyé par Djclo

je le recupere comme ceci :

Code:

---

1 2 3 4 5

for (int i = 0; i < GridView1.Rows.Count; i++) { GridViewRow row = GridView1.Rows[i]; } string productId = Convert.ToString(GridView1.DataKeys [row.RowIndex].Value);

---

---

C'est vraiment le code que tu utilises ???
Ta boucle ne sert à rien là !

Cela n'explique quand même pas pourquoi l'index n'est pas bon...

Essaie ça:

Code:

---

1 2	string productId = Convert.ToString(GridView1.DataKey[row.RowIndex].Values(0));

---

Si ta DataKeys est la première,!!

ca amarche pas !

Euh ! Question con !

tu as bien définis une (ou des) clé(s) dans ta grille ?

Ce bout de code est définit dans quelle méthode ???

ah si je lai defini en tant datakeynames !

A priori ce la ne vient pas de l'index de ta row.

Je ne vois qu'un problème au niveau des clés

Vérifies ce que tu as mis dans DataKeyNames

ce bout de code est dans cette methode

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17

protected void addToCart_Command(object sender, CommandEventArgs e) { for (int i = 0; i < GridView1.Rows.Count; i++) { GridViewRow row = GridView1.Rows[i]; bool isChecked = ((CheckBox)row.FindControl("_chkSelect")).Checked; if (isChecked) { if (row.RowIndex == -1) { if (e.CommandName == "add") { string productId = Convert.ToString(GridView1.DataKeys[row.RowIndex].Value); // ajouter le produit au panier ShoppingCartAccess.AddItem(productId); } }

---

le DataKeyNames est bien present :

<asp:GridView AllowPaging="true"
DataKeyNames="ProductID"
OnPageIndexChanging="ChangePage_PageIndexChanging"
PageSize="3"
Width="100%"
ID="GridView1" runat="server"
AllowSorting="True"
AutoGenerateColumns="False"
CellPadding="4"
ForeColor="#333333"
GridLines="None">
<Columns>
<asp:TemplateField HeaderText="Sélection" HeaderStyle-Font-Size="Small">
<ItemTemplate>
<asp:CheckBox ID="_chkSelect" runat="server" />
</ItemTemplate>
</asp:TemplateField>

Citation:

---

Envoyé par Djclo

Code:

---

1 2 3 4

// Normalement, ce test est inutile, RowIndex ne peut pas être égale à -1 if (row.RowIndex == -1)

---

---

Citation:

---

Envoyé par Djclo

le DataKeyNames est bien present :

<asp:GridView AllowPaging="true"
DataKeyNames="ProductID"
OnPageIndexChanging="ChangePage_PageIndexChanging"
PageSize="3"
Width="100%"
ID="GridView1" runat="server"
AllowSorting="True"
AutoGenerateColumns="False"
CellPadding="4"
ForeColor="#333333"
GridLines="None">
<Columns>
<asp:TemplateField HeaderText="Sélection" HeaderStyle-Font-Size="Small">
<ItemTemplate>
<asp:CheckBox ID="_chkSelect" runat="server" />
</ItemTemplate>
</asp:TemplateField>

---

Et dans ta source de donnée, tu as bien une colonne qui s'appelle ProductID ?

Essaye de définir la propriété datakeynames après avoir défini la source de données.

jai une colonne ProductID...

Je ssaye de chercher de mon coté

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17

protected void addToCart_Command(object sender, CommandEventArgs e) { for (int i = 0; i < GridView1.Rows.Count - 1; i++) { GridViewRow row = GridView1.Rows[i]; bool isChecked = ((CheckBox)row.FindControl("_chkSelect")).Checked; if (isChecked) { if (row.RowIndex == -1) { if (e.CommandName == "add") { string productId = Convert.ToString(GridView1.DataKeys[row.RowIndex].Value); // ajouter le produit au panier ShoppingCartAccess.AddItem(productId); } }

---

Heu si tu écris

string productId = Convert.ToString(GridView1.DataKeys[row.RowIndex].Value);

cela signifie que tu as autant d'éléments dans GridView1.DataKeys que de lignes.

Perso j'écrirai plutôt
string productId = Convert.ToString(GridView1.DataKeys[0].Value);
si je n'ai qu'une clé, ce qui est aussi ton cas.

Citation:

---

Envoyé par Spoonnny

Heu je me trompe peut-être mais écrire

string productId = Convert.ToString(GridView1.DataKeys[row.RowIndex].Value);

signifie que tu as autant d'éléments dans GridView1.DataKeys que de lignes.

Perso j'écrirai plutôt
string productId = Convert.ToString(GridView1.DataKeys[0].Value);
si je n'ai qu'une clé, ce qu'y est aussi ton cas.

---

Non, il y a normalement autant de DataKeys que de lignes.
S'il y a plusieurs clés c'est ceci qu'il faut faire

Code:

---

1 2 3

// je récupère la deuxième clé GridView1.DataKeys[row.RowIndex].Values(1)

---

Mettre un breakpoint à l'appel de la méthode et passé en pas en pas, ajouter des espions et voir ce que cela donne, parce que là, sans plus d'indication, c'est pas gagné

Citation:

---

Envoyé par tom974

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17

protected void addToCart_Command(object sender, CommandEventArgs e) { for (int i = 0; i < GridView1.Rows.Count - 1; i++) { GridViewRow row = GridView1.Rows[i]; bool isChecked = ((CheckBox)row.FindControl("_chkSelect")).Checked; if (isChecked) { if (row.RowIndex == -1) { if (e.CommandName == "add") { string productId = Convert.ToString(GridView1.DataKeys[row.RowIndex].Value); // ajouter le produit au panier ShoppingCartAccess.AddItem(productId); } }

---

---

Non ce que tu as mis n'est pas correct !

si tu mets ceci :

Code:

---

1 2	i < GridView1.Rows.Count -1

---

Tu ne prends pas en compte la dernière ligne !
(Ne pas confondre avec les boucles en VB !)

Enfin ça marcheeeeeeeeeeeeeeeeee !!!!!!!!!!!!!!!!!

merci a tous les bons developpeurs C#

encore merci et bon week end

il fallais faire :

Il fallait faire quoi ??
Il manque un morceau :D

si ta qu une clé, faire :

Code:

---

string productId = Convert.ToString(GridView1.DataKeys[0].Value);

---

et si ten a plusieurs :

Code:

---

string productId = Convert.ToString(GridView1.DataKeys[row.rowindex].Value);

---

Euh là tu récupères toujours la clé de la première ligne... ou alors on a pas le même composant...

Je te recommande de lire la documentation :

http://msdn.microsoft.com/fr-fr/libr...ys(VS.80).aspx