markussagen/assignment_2

Assignment 2 for the course IOOPM HT 2018

👋 MLOps Engineer / AI Research Engineer / Software Engineer

Working on

Lot's of cool things;
Either for my own curiousity or to make tools and applications that will enable others to work smarter.

Contact

• 📫 How to reach me: Contact by @email

assignment_2's Issues

Assignment 2

Struktur

• Är koden välstrukturerad samt stilistiskt och formateringsmässigt konsekvent?
• Finns det “död kod” eller metoder/rutiner som inte anropas eller behövs?
• Finns det block av (mer eller mindre) upprepad kod som kunde extraheras och kapslas in i en metod?
• Används symboler istället för ”magiska nummer” eller strängkonstanter?
• [] Finns det klasser/metoder som är väldigt omfattande och/eller komplicerade och borde brytas upp i fler mindre klasser/metoder/rutiner?

Variabler

• Har alla variabler definierats med vettiga namn som är meningsfulla, tydliga och konsekventa?
• Initieras alla variabler med vettiga värden?
• Finns det redundanta variabler eller variabler som inte används?

Loopar och villkorssatser

• Är alla loopar, villkorssatser och logiska uttryck kompletta, korrekta och korrekt nästlade?
• Testas de vanligaste fallen först i if ... elseif ... -kedjor?
• Har alla case-satser en default-klausul?
• Är villkoren för loopars terminering uppenbara och alltid uppnåeliga?
• Initieras indexeringsvariabler korrekt innan de loopar där de används?
• Finns det uttryck och satser inuti loopar som kunde brytas ut och placeras utanför loopen?

Defensiv programmering

• Testas indexeringsvariabler och pekare korrekt mot gränsvärdena för arrayer, listor, etc.?
• Kontrolleras att indata (från filer, terminalen, etc.) är valida och komplett?
• Städas minne (alt. referenser) till allt allokerat data bort korrekt?
• [] Kontrolleras att filer som programmet öppnar existerar innan programmet gör åtkomst till dem?

case 0 & 3 onödiga?

https://github.com/Mk3Johnson/Assignment_2/blob/d855f9ab783f36ac3a18d71a9f6c12ba35900e53/user_interface.c#L227-L238

Bara jobbigt att läsa

https://github.com/Mk3Johnson/Assignment_2/blob/d855f9ab783f36ac3a18d71a9f6c12ba35900e53/business_logic.c#L150

Adding to carts

Jobbigt att inte kunna se vilka varor som finns, måste själv komma ihåg. Kan komma in dit före varor finns i lager.

Quit

Kunde bara lägga till så att det skriver, vad det är man ska skriva för att quita?

Varför float?

https://github.com/Mk3Johnson/Assignment_2/blob/d855f9ab783f36ac3a18d71a9f6c12ba35900e53/user_interface.c#L55

char_match

https://github.com/Mk3Johnson/Assignment_2/blob/d855f9ab783f36ac3a18d71a9f6c12ba35900e53/utils.c#L129

Vad menas med detta?

Borde inte hända

https://github.com/Mk3Johnson/Assignment_2/blob/d855f9ab783f36ac3a18d71a9f6c12ba35900e53/business_logic.c#L378

Replenish

Kan inte se vilka varor som är tillgängliga att replenisha. Måste gå upp i programmet för att se. Borde listas.

Fel meddelande vid pris

Får "Cart index must be a number larger than 0!" om man sätter ett pris på en vara under 0<=

Detta händer aldrig?

https://github.com/Mk3Johnson/Assignment_2/blob/d855f9ab783f36ac3a18d71a9f6c12ba35900e53/business_logic.c#L346

Jobbig att läsa med alla if satser

https://github.com/Mk3Johnson/Assignment_2/blob/d855f9ab783f36ac3a18d71a9f6c12ba35900e53/utils.c#L137-L162