gauteaven / radiatorby

Starterkode ING301 Prosjekt Del A

Se oppgavebeskrivelsen i ing301public!

Skriv din egen dokumentasjon av SmartHus-applikasjonen her...

tests/demo_house.py og smarthouse/domain.py

radiatorby's Issues

Feedback Prosjekt Del A

Hei! Etter å ha vurdert innleveringen din, vil jeg dele noen tilbakemeldinger på arbeidet ditt med SmartHus-applikasjonen, samt noen ekstra filer du la til i innsendingen.

Hva du gjorde bra:

1. Struktur på SmartHus: Du har gjort en solid jobb med å definere og implementere klassene Room, Floor, Device, Actuator, Sensor og SmartHouse. Det ser ut som om du har fanget opp nøkkelkonseptene i oppgaven og relatert disse til objektorientert programmering på en svært passende måte. Dine klasser og metoder følger godt oppgavekravene og virker å håndtere registrering og manipulering av etasjer, rom og enheter på en strukturert måte.

2. Implementasjon av funksjonalitet: Din implementasjon av operasjoner som å legge til rom, etasjer og enheter fremstår godt gjennomarbeidet. Bruken av dictionaries i Room-klassen for å lagre enhetene er en smart løsning, og din generelle måte å knytte forskjellige komponenter sammen på (som rom til etasje og enheter til rom) er gjort på en forståelig måte.

3. Tester: Fra testresultatet ser det ut som om alle testene passerte, noe som indikerer at grunnleggende funksjonalitet i koden din fungerer som forventet. Dette er veldig bra!

Ting å forbedre og merke seg:

1. Klassediagram: Det ser ut som du mangler et klassediagram. Jeg vil anbefale deg å skape et klassediagram som viser forholdet mellom de ulike klassene. Dette vil ikke bare hjelpe deg å få en bedre forståelse av systemets struktur, men også gjøre det enklere for andre å forstå koden din.

2. Metodenavn: I Sensor-klassen har du en statisk metode kalt anus som returnerer en måling. Selv om det teknisk sett fungerer, bør navngivningen av metoder være mer beskrivende og profesjonell for å lette forståelsen og vedlikeholdet av koden.

3. Duplikat kode: Jeg la merke til at du har opprettet en separat Lyspære-klasse i en egen fil. Den ser ut til å gjenskape logikk som allerede er dekket av Actuator-klassen din. Det er viktig å bruke den eksisterende koden du har og unngå duplisering hvor mulig.

4. Feilhåndtering: Det ser ikke ut til å være noen form for feilhåndtering eller validering av input. I en reell applikasjon er det viktig å sikre mot ugyldige verdier som kan føre til uønskede tilstander eller feil.

5. Ubrukte Filer: drit2.py og drit3.py ser ut til å være testfiler som ikke er relatert til hovedoppgaven. Husk å rydde opp i kodebasen før innlevering for å holde fokus på oppgaven.

Tips til videre arbeid:

• Utforsk hvordan du kan gjøre kodebasen din mer modulær og gjenbrukbar, spesielt med tanke på håndtering av enhetene.

Dette er en solid innsats, og jeg ser potensial for ytterligere forbedringer og utvidelser av prosjektet ditt. Fortsett det gode arbeidet!"

Etter dere ha lest gjennom kan dere bare markere dette "Issue" som "resolved" 😉