Question

Problem: I am trying to update a List. If a certain item’s ID already exists in the List, I want to add onto that item’s quantity. If not, then I want to add another item to the list.

            cart = (List<OrderItem>)Session["cart"];

            for(int counter = cart.Count-1; counter >= 0; counter--)
            {
                if (cart[counter].productId == item.productId)
                {
                    cart[counter].productQuantity += item.productQuantity;
                }
                else if (counter == 0)
                {
                    cart.Add(item);
                }
            }

cart[counter] and item represent an instance(s) of a custom object of mine. Currently when I finally find a matching ID, everything APPEARS as though it should work, but I get a StackOverflowException thrown in my custom object class.

    public int productQuantity
    {
        get
        {
            return _productQuantity;
        }
        set
        {
            productQuantity = value;
        }
    }

It gets thrown right at the open-bracket of the “set”. Could somebody please tell me what the heck is wrong because I’ve been going at this for the past 2+ hours to no avail. Thank you in advance.

Answer 1

the problem is in your setter of the productQuantity

it should read:

set
    {
        _productQuantity= value;
    }

edit (naming convention):

public class Vertex3d
{
    //fields are all declared private, which is a good practice in general 
    private int _x; 

    //The properties are declared public, but could also be private, protected, or protected internal, as desired.
    public int X
    { 
        get { return _x; } 
        set { _x = value; } 
    }
}