hildalange / TDA553-lab1

0 stars 0 forks source link

CRITICAL: Misunderstanding the LSP #28

Closed I-Hansson closed 1 year ago

I-Hansson commented 1 year ago

https://github.com/hildalange/TDA553-lab1/blob/71d5489e3870f5300ba2956a33a825ef995dff14/src/java/lab2reflection.txt#L10

Ni bryter mot LSP då ni anser att kontraktet för gas och brake enbart ska göra så ni skall gasa eller bromsa. Men i Scania och carTransporter så måste även ett speciellt villkor uppfyllas (Platformen får inte vara nere/uppe) . Vad skulle ni kunna ändra för att följa LSP? (Hint: Ni behöver inte ändra någon kod)

I-Hansson commented 1 year ago

Bra! Ni följer LSP genom att säga " kontraktet för gas/brake är att öka/bromsa hastigheten på fordonet om fordonet är i ett stadie där den kan röra sig, annars gör vi ingenting".