JSM er igang med denne

Planlægning af projekt.

Opsætning af Github Repo
Opsætning af Github Project
Opsætning af database
Udarbejdelse af problemformulering
Figma og User Stories
Oprettelse af Projekt Rapport dokument på Google docs

Jeg kan ikke umiddelbart vurdere hvem gruppemedlemmerne ud fra disse Github brugere (pånær en):
JSM80
amorags
oliverager
shadpre
Venligst skriv i readme filen fulde navne på gruppemedlemmerne.

Det ser alt i alt fornuftigt ud. Der er ikke nogen kritiske mangler, og den ene test I har som fejler i pipelinen er stadig under bagatelgrænsen for hvad der må være af fejl.

De anbefalinger jeg har til refactoring er følgende:

• Anvend gerne .NET's dependency injection system i stedet for at manuelt instantiere indenfor class scope. Dvs at tilføje BoxDAO til jeres builder i Program.cs:

builder.Services.AddScoped<BoxDAO>();

og anvend en constructor i jeres class som kræver denne, eksempelvis:

 public class BoxController : ControllerBase
    {
        private BoxDAO _boxDao;
        public BoxController(BoxDAO boxDao)
        {
            _boxDao = boxDao;
        }
}

På nuværende tidspunkt ser jeres tests ud til at afhænge af hinanden og den eksakte rækkefølge. Det vil være en god praksis at lave uafhængige tests, så hver test sørger for at rydde og forberede data forinden i en arrange fase. På den måde sørger I for at have retvisende tests som ikke tilfældigvis fejler.

Jeres null checks som ser sådan her ud vil altid være falsk:
if (newBox == null)
Null checks bør tjekke reference og ikke ved hjælp af ==, sådan her:
if (ReferenceEquals(newBox,null))

Opgaven er bestået.