Haan schreef op woensdag 15 september 2010 @ 15:12:
Hebben jullie een paar concrete voorbeelden van wat er echt fout is in die tutorial / boek, voordat ik meteen verkeerde dingen aan leer?

(ik ben me overigens bewust van het feit dat de tutorial MVC 1 is, en ik zelf MVC 2 gebruik), komt de sessie MVC 2 van Scott Hanselman op DevDays2010 een half jaar later toch nog van pas

in code snippet 1.3 gaat het al fout:
C#:
1
2
3
4
5
6
| public void Delete(Dinner dinner) {
foreach(var rsvp in dinner.RSVPs) {
entities.RSVPs.DeleteObject(dinner.RSVPs);
}
entities.Dinners.DeleteObject(dinner);
} |
moet natuurlijk entities.RSVPs.DeleteObject(rsvp); zijn..
code snippet 1.55:
C#:
1
2
3
4
5
6
7
8
9
10
11
12
13
14
| [HttpPost]
public ActionResult Create(Dinner dinner) {
if (ModelState.IsValid) {
dinner.HostedBy = "SomeUser";
dinnerRepository.Add(dinner);
dinnerRepository.Save();
return RedirectToAction("Details", new { id=dinner.DinnerID });
}
return View(new DinnerFormViewModel(dinnerToCreate));
} |
waar komt die dinnerToCreate ineens vandaan?
of gewoon in de tekst staat:
"For example, consider the following code snippet from the
e file in our project (which we will be looking
at shortly)."
de 'e file'??? uit de context kan je natuurlijk wel afleiden dat het over de Site.Master file gaat.
en dan zijn ze gewoon stukken code vergeten te printen... zoals de NearestDinners method.. die moet je dus zelf maar verzinnen :-)
er staat ook ineens:
"While still a 1.0 product, ASP.NET MVC sits on top of, and was developed
with, everything that makes System.Web a powerful and stable part of the .NET Framework."
Het book gaat over MVC 2.. niet 1.0!!
Dat zijn degene die ik nog herinner, maar er zijn er nog een 10-tal meer. Het is dus duidelijk dat ze gewoon H1 van het eerste boek hebben gekopieerd en een paar aanpassingen hebben gedaan zodat het met de Entity Framework werkt.. maar een heeeeleeeboel fouten hebben gemaakt/laten staan..
Verder wel een goed boek... en mooie reference..
maar ik zou eerder MVC 2 in Action of Pro ASP.NET MVC 2 Framework aanraden