initialiser une classe de type []

Bonjour,

a l'interieur d'une classe : OrderDetails j'ai :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

public class OrderDetails { private int _orderdetailid; private int _orderid; private int _productid; private Product[] _products; private int _quantity; public OrderDetails() { _products = new Product; }

---

as product

la question : comment initialiser la classe product car il y'a product[] et pas seulement product ?

Bonjour,
Il s'agit bien d'un tableau de produits et donc il faut prévoir la taille de ton tableau :
Par exemple :

Code:

---

products = new Product [10];

---

merci, je t'explique exactement le probleme :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

for ( int i = 0 ; i < paypalinformation.Order.OrderDetails.Products.Length ; i++ ) { items = new PaymentDetailsItemType(); items.Amount = new BasicAmountType(); items.Amount.Value = paypalinformation.Order.OrderDetails.Products[i].Price.ToString(); items.Amount.currencyID = CurrencyCodeType.EUR; items.Quantity = paypalinformation.Order.OrderDetails.Products[i].Quantity.ToString(); items.Name = paypalinformation.Order.OrderDetails.Products[i].Name; items.Number = paypalinformation.Order.OrderDetails.Products[i].ProductID.ToString(); itemArray.SetValue( items , i ); }

---

cette boucle for est pour toutes les elements dans shoppingcart, l'erreur affiché est :

Citation:

---

Object reference not set to an instance of an object.

---

C'est normal, dans ton constructeur tu n'a pas declaré _products comme tableau. Regardes ce que j'ai écrit dans mon message ci-dessus.

Merci

toujours la meme erreur :

Citation:

---

Object reference not set to an instance of an object

---

je ne comprens pas trop ton problème essaye de mettre un point d'arrêt sur ton code et le débugguer.

son problème c'est qu'il a surement rien à Products[i] et c'est pour ça que ça crash
tu peux faire ça pour éviter de lire un élément qui n'existe pas

Code:

---

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

for ( int i = 0 ; i < paypalinformation.Order.OrderDetails.Products.Length ; i++ ) { if(Products[i]!=null) { items = new PaymentDetailsItemType(); items.Amount = new BasicAmountType(); items.Amount.Value = paypalinformation.Order.OrderDetails.Products[i].Price.ToString(); items.Amount.currencyID = CurrencyCodeType.EUR; items.Quantity = paypalinformation.Order.OrderDetails.Products[i].Quantity.ToString(); items.Name = paypalinformation.Order.OrderDetails.Products[i].Name; items.Number = paypalinformation.Order.OrderDetails.Products[i].ProductID.ToString(); itemArray.SetValue( items , i ); } }

---

Oui mais déjà au départ le tableau n'était pas instancié !

je crois que là il l'a instancié mais il a quand même une erreur

Bonjour,
Pourquoi ne pas passer plutôt par une List<Product> plutôt qu'un tableau ?

Citation:

---

Envoyé par fabi19022

Bonjour,
Pourquoi ne pas passer plutôt par une List<Product> plutôt qu'un tableau ?

---

Ca ne change pas grand chose, si la liste n'est pas instanciée, il y aura une NullReferenceException... Mais, c'est effectivement plus facile à manipuler.

Citation:

---

Envoyé par mimosa803

Code:

---

products = new Product [0];

---

---

Dans ton cas, je supposerai que si tu n'as aucun produit, tu dois instancier un tableau de produits de longueure 0... Si tu crées un tableau Product[10], ton algo va essayer d'accéder à des objets non instanciés...
Sinon +1 pour l'utilisation d'une liste... Sinon tu dois créer un nouveau tableau à chaque fois que tu ajoutes un produit, avec les anciens produits + le nouveau. :aie:

Code:

---

1 2	List<Product> _Products=new List<Product>();

---