Mangler feedback på kode

Hej R.S.,

Jeg har lige hurtigt skimmet noget af det igennem, og de ting jeg selv bider mærke i er at du benytter camelCasing til dine namespaces. Microsoft selv anbefaler at man bruger PascalCasing for disse, og jeg synes personligt det giver god mening, når man har et langt namespace.

https://msdn.microsoft.com/en-us/library/ms229026(v=vs.110).aspx

Omkring dine metoder så vil jeg igen nævne at jeg kan set du bruger camelCasing og her er Microsofts også at det skal være PascalCasing.

https://msdn.microsoft.com/en-us/library/4df752aw.aspx

Jeg ved godt jeg er lidt OCD på det punkt, men ser jeg metoder med camelCasing i c#, hvor jeg arbejder så bliver de renamed

For at tage fat i noget andet, så kan jeg se at du f.eks. i din abstrakte klasse ClassItemsMethod, har en metode som hedder create(), men den smider en exception hvis property'en Name ikke er udfyldt.

Jeg tænker her er det smartere at du laver en constructor, der tager imod et name, og hvis name er null (eller måske tomt?) så smider du en fejl her. Der er næsten ikke noget mere bøvlet at finde ud af hvorfor en metode smider en fejl, fordi den forventer noget på ens object er udfyldt. Så for at gøre det nemmere vil jeg sørge for at sende det med ind ved oprettelsen.

Der er lidt at tage et kig på, og har du spørgsmål, så spørg da endelig.

Hej Martin.

Vidste godt nok ikke at det er standart at namespace skal starte med stort.

ang. method så er jeg nok mere vant til Java og PHP hvor det skal starte med småt men skal nok tage den med i mine overvejelser.

ClassItemsMethod kan du have helt ret i. ved ikke hvorfor jeg ikke tænkte over det. Ville også ændre Aguments i den da den uanset hvad skal sættes så det har jeg tænkt skal klares i constructor

Tak for din feedback det gav mig noget at tænke over