ASP.NET MVC appikation - Shoppingcart.

https://www.youtube.com/watch?v=ZBd0MnKb7u0

Som jeg skrev i webudvikler-gruppen på facebook, så kunne du anvende en foreach (som jeg kan se du har forsøgt at replikere, men ændret på, så den ikke fungerer :-)).



skal i stedet være



Det er muligt, at du skal kalde .ToList() på cart.Where, altså



men prøv lige uden først.

Grunden til at du gemmer de produkter der skal slettes i en ny liste er, at du ikke kan gennemløbe en liste samtidig med at du sletter elementer i den samme liste. Derfor kopierer du referencerne til de objekter der skal slettes over i en ny liste. Den nye liste kan så gennemløbes, mens du sletter objekterne i den oprindelige liste.

Din Delete burde kunne klares med flg.:

Hvad med:

cart = cart.Where(x => x.Product.productId != id).ToList();

?

Hehe! Narturligvis, arne!

Fedt tak for feedback, det lidt ligesom en nål i en høstak. Så det var dejligt med andre øjne på det. Var ved at køre død i det.

Jeg prøver lige at se om jeg kan få det til at virke, men tak for jeres feedback. :0)

Nu ser det godt nok ud til den fjerne alle items fra Listen. Det er ikke helt det den skal.


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

            var lstSlet = cart.Where(x => x.Product.productId == id).ToList();
            lstSlet.ForEach(x => cart.Remove(x));

            Session["cart"] = cart;

            return View("Cart");

Jeg har nu også forsøgt med et for loop, hmm

  public ActionResult Delete(int id)
        {
            int index = isExisting(id);
            List<Item> cart = (List<Item>)Session["cart"];

            for (int i = cart.Count - 1; i >= 0; i--)
            {
                cart.RemoveAt(i);
            }
              // cart.RemoveAt(index);

            //var lstSlet = cart.Where(x => x.Product.productId == id);
            //cart.ForEach(x => cart.Remove(x));
            Session["cart"] = cart;

            return View("Cart");


        }

I din egen version sletter du jo netop alle elementer, så den gør det du beder den om ;-)

Umiddelbart virker det som om, at alle produkter i din liste har det samme id og at det så er derfor alt bliver slettet. Du skal starte med, at sikre dig, at dit datagrundlag er som det skal være.

Du bør nok anvende Arnes udgave, da den er mere direkte til opgaven, men princippet i både Arnes og min løsning, er det du har brug for.

Ja det fandt jeg ud af,  men det er nu mere antal (quantity) jeg vil have slette, hvor mange stk af hver vare.

OK, men det er jo en anden opgave end at slette den enkelte vare fra kurven (med mindre der ikke er flere tilbage på varelinjen).

Jeg tror der er behov for, at du beskriver helt præcis, hvad det er du vil opnå og hvilken datamodel du arbejder med. Det er umuligt at hjælpe effektivt, hvis man selv skal sidde og gætte sig frem til en masse parametre.

Hvis du med Delete blot vil fratrække én enhed fra varelinje med det oplyste id, kunne det være noget i stil med dette:

Jeg benenytter mig af EntityFramework, hvor til jeg har oprettet følgende klasser til hvor jeg har en proptity "quantity" som skal tage højde for antal stk man lægger i.

Første klasser her er item, som ligger i mit controllerlag.

*--name spaces--*

namespace Outlet.Controllers
{
    public class Item
    {
        private Product product = new Product();
        private int quantity;

        public Item()
        {

        }

        public Item(Product product, int quantity)
        {
            this.Product = product;
            this.Quantity = quantity;
        }

        public Product Product
        {
            get{ return product; }
            set{ product = value; }
        }

        public int Quantity
        {
            get{ return quantity; }
            set{ quantity = value; }
        }
    }
}

Efterfølgende har jeg også ShoppingCartController hvor delete metoden  ligger i.

*--name spaces--*


namespace Outlet.Controllers
{
    public class ShoppingCartController : Controller
    {
        public Connection db = new Connection();

        // GET: Cart
        public ActionResult Index()
        {
            return View();
        }

        private int isExisting(int id)
        {
            List<Item> cart = (List<Item>)Session["cart"];

            if(cart.Count > 0)
            {
                for (int i = 0; i < cart.Count; i++)
                    if (cart[i].Product.productId == id)
                        return i;
             
            }

            if (cart.Count < 0)
            {
                for (int i = 0; i < cart.Count; i--)
                    if (cart[i].Product.productId == id)
                        return i;
            }
         

            return -1;
        }

        public ActionResult Delete(int id)
        {
            int index = isExisting(id);
            List<Item> cart = (List<Item>)Session["cart"];

           

            for (int i = cart.Count - 1; i >= 0; i--)
            {
           
               
             
                cart.RemoveAt(i);
            }
              // cart.RemoveAt(index);

            //var lstSlet = cart.Where(x => x.Product.productId == id);
            //cart.ForEach(x => cart.Remove(x));
            Session["cart"] = cart;

            return View("Cart");


        }
       
        public ActionResult OrderNow(int id)
        {
            if (Session["cart"] == null)
            {
                List<Item> cart = new List<Item>();
                cart.Add(new Item(db.Products.Find(id), 1));
                Session["cart"] = cart;
            }

            else
            {
                List<Item> cart = (List<Item>)Session["cart"];
                int index = isExisting(id);
                if (index == -1)
                    cart.Add(new Item(db.Products.Find(id), 1));
                else
                    cart[index].Quantity++;
                Session["cart"] = cart;

            }
            return View("Cart");
        }
    }
}

Det jeg vil have er at antal stk (quantity) minimeres når man klikker delete, frem for sådan det er nu, hvor den bare sletter alt i listen.

Prøv lige mit sidste post, dog med denne rettelse (produktid fra produktet):

Fedt, jeg prøver det lige af senere. Er lige ved at refactor noget kode :0)

Det virker nu, tak for hjælpen ! :0)

public ActionResult Delete(int id)
        {
            int index = isExisting(id);
            List<Item> cart = (List<Item>)Session["cart"];

            Item item = cart.FirstOrDefault(l => l.Product.productId == id);

            for (int i = cart.Count - 1; i >= 0; i--)
            {
               

                if (item != null)
                {
                    if (item.Quantity > 1)
                        item.Quantity--;
                    else
                        cart.RemoveAt(i);
                }
             
            }

            Session["cart"] = cart;

            return View("Cart");


        }

Tak, jeg fik det til at virke :0)

og du tog selv point fordi du ikke mener at du har faet hjaelp????

Nej jeg fatter ikke helt, hvordan de point fungere, jeg ville bare lukke tråden :0)

Velbekomme.

Jeg forstår nu ikke helt alt den kode du har i din Delete. Det virker overflødig at tjekke på om den eksisterer (isExisting), samt din løkke omkring sletningen/nedskrivningen af varelinjen.

Dette burde være længe rigelig:



Den sidste tildeling til Session burde heller ikke være nødvendig, da du arbejder med en reference til et objekt der allerede ligger i session. Du kunne i det mindste prøve at se om det fungerer uden, inden du sætter den ind.