[ASP.net MVC / C#] Delete action werkt niet

[jvaneijk]

Hoi allemaal,

Ik heb een heel simpel MVC projectje met een database met het Entity framework
Nu wil het zo zijn dat als ik een delete action maak mijn delete niet werkt. de get action van de delete geeft netjes een Object aan de hand van een id

C#:

public ActionResult Delete(int id)
        {
            var productgroupToDelete = (from p in _entities.Productgroup
                                   where p.productgroupID == id
                                   select p).FirstOrDefault();

            return View(productgroupToDelete);
        }

maar de post action doet echter helemaal niets,
en geeft een NullReferenceException

HTML:

<h2>
        Delete</h2>
    <p>
        Are you sure you want to delete the productgroup
        Puhh?</p>
    <form action="/Productgroup/Delete/6" method="post">
    <p>
        <input type="submit" value="Delete" /></p>
    </form>

Hierboven het relevante html source als de get verstuurd is. Hier staat netjes een action met een id als ik het zo bekijk.

C#:

[AcceptVerbs(HttpVerbs.Post)]
        public ActionResult Delete(Productgroup productgroupToDelete)
        {
            try
            {
                var originalProductgroup = (from p in _entities.Productgroup
                                       where p.productgroupID == productgroupToDelete.productgroupID
                                       select p).FirstOrDefault();

                _entities.DeleteObject(originalProductgroup);
                _entities.SaveChanges();
                return RedirectToAction("Index");
            }
            catch
            {
                return View();
            }
        }

Als ik daarna in mijn post action kijk van de delete is productgroupToDelete echter helemaal leeg.
Ik weet alleen niet waarom deze niet gevuld wordt en hoop dat iemand mij hiermee kan helpen.

Ik ben nog erg nieuw met MVC en entity framework dus zal wel een stomme fout zijn die ik over het hoofd zie.

hieronder ook nog even mijn Delete view

HTML:

<%@ Page Title="" Language="C#" MasterPageFile="~/Views/Shared/Site.Master" Inherits="System.Web.Mvc.ViewPage<FRSTestPortal.Models.Productgroup>" %>

<asp:Content ID="Content1" ContentPlaceHolderID="TitleContent" runat="server">
    Delete
</asp:Content>
<asp:Content ID="Content2" ContentPlaceHolderID="MainContent" runat="server">
    <h2>
        Delete</h2>
    <p>
        Are you sure you want to delete the productgroup
        <%= Model.productgroupName %>?</p>
    <% using (Html.BeginForm(new { id = Model.productgroupID }))
       { %>
    <p>
        <input type="submit" value="Delete" /></p>
    <% }%>
    <div>
        <%=Html.ActionLink("Back to List","Index") %>
    </div>
</asp:Content>

Ik heb voor zover ik weet alles goed gedaan maar kom er echt niet meer uit..

[Niemand_Anders]

Op de eerste Delete moet je filteren op de HttpVerbs.Get en bij de tweede delete moet je ook 'int id' als argument gebruiken (6 in het voorbeeld), echter dan als post. Welke overloaded Delete wordt gebruikt wordt dus dan bepaald aan de hand van de http verb.

Nu kan MVC niet 6 omzetten naar Productgroup en geeft deze dus 'null' door.

[Rockafello]

Overigens lijkt me dit niet verstandig voor een delete:

code:

var originalProductgroup = (from p in _entities.Productgroup
                                       where p.productgroupID == productgroupToDelete.productgroupID
                                       select p).FirstOrDefault();

die FirstOrDefault zou ik hier weghalen. En eerst kijken of je maar 1 record

code:

var originalProductgroup = (from p in _entities.Productgroup
                                       where p.productgroupID == productgroupToDelete.productgroupID
                                       select p); 

if (originalProductgroup .Count() > 0)
                return originalProductgroup .First();
            else
                return null;

Voor hetzelfde geld pakt ie de verkeerde, maar dit weet ik niet zeker

[Niemand_Anders]

Rockafello schreef op donderdag 23 juli 2009 @ 13:58:
Overigens lijkt me dit niet verstandig voor een delete:

code:

var originalProductgroup = (from p in _entities.Productgroup
                                       where p.productgroupID == productgroupToDelete.productgroupID
                                       select p).FirstOrDefault();

die FirstOrDefault zou ik hier weghalen. En eerst kijken of je maar 1 record

code:

var originalProductgroup = (from p in _entities.Productgroup
                                       where p.productgroupID == productgroupToDelete.productgroupID
                                       select p); 

if (originalProductgroup .Count() > 0)
                return originalProductgroup .First();
            else
                return null;

Voor hetzelfde geld pakt ie de verkeerde, maar dit weet ik niet zeker

Maar de FirstOrDefault() doet precies wat jij nu handmatig wilt doen. Als er geen record is wordt een null value terug gegeven. Zou je op de query (zonder count() controle) direct First gebruiken en de record niet bestaan in de database, dan krijg je een exceptie. Ofwel als er geen First() is, dan geeft Linq Default(ENTITY) terug. Default wordt vooral gebruikt in code met generics.

Default(int) geeft 0
Default(ProductGroup) geeft null

In beide gevallen moet de aanroepende code nog steeds controleren of je een entity terug wordt gegeven of een null value.