search

BruteSolutions / training

Repo used for learning git and the workflow.
2 stars 0 forks source link

Implementera Truck. #6

Open davnils opened 12 years ago

davnils commented 12 years ago

Implementera klassen Truck som ärver från Vehicle och Autonomous med följande medlemmar:

Truck(std::string brand, int year) : public

Förslagsvis baseras cost på ett värde som beräknas baserat på märket. Sätt även ett värde på radius. Tänk på att implementera någon version av de ärvda metoderna.

Följ instruktioner i wikin och lägg klassdeklarationer/definitioner i separata filer (.h / .cpp) med filnamn som följer klassnamn.

davnils commented 12 years ago

@Jimipe är granskare.

kazie commented 12 years ago

Min generella tanke: http://i2.kym-cdn.com/photos/images/original/000/234/739/fa5.jpg

davnils commented 12 years ago

Jag ser att du har commitat båda filerna till develop. Tanken är att alla komponenter ska implementeras i egna grenar. Merge till develop sker först efter kodgranskning här i den här tråden (alltså skickas resultatet med en pull request).

(Förtydligande: arbeta bara i branch 006-truck)

Koden är borttagen från develop så det vore bra om du kollar instruktionerna och försöker igen.

Rent kodmässigt är det viktigt att placera implementationen i Truck.cpp (den var tom tidigare). Tänk även på att använda rätt metodnamn på de som ärvs från abstrakta klasser.

Bara skriv om det är något som är oklart.

kazie commented 12 years ago

Jag var glad att jag ens fick koden att skicka nånstans. :> Jag har varken fattat exakt hur jag ska skriva (brist på specifikation, och IDE) eller exakt var man ska committa saker.

Ville bara ha nått gjort i alla fall...

2012/2/1 David Nilsson < reply@reply.github.com

Jag ser att du har commitat båda filerna till develop. Tanken är att alla komponenter ska implementeras i egna grenar. Merge till develop sker först efter kodgranskning här i den här tråden (alltså skickas resultatet med en pull request).

Koden är borttagen från develop så det vore bra om du kollar instruktionerna och försöker igen.

Rent kodmässigt är det viktigt att placera implementationen i Truck.cpp (den var tom tidigare). Tänk även på att använda rätt metodnamn på de som ärvs från abstrakta klasser.

Bara skriv om det är något som är oklart.

Reply to this email directly or view it on GitHub: https://github.com/BruteSolutions/training/issues/6#issuecomment-3756039

davnils commented 12 years ago

Syftet med den här uppgiften är bara att alla ska prova-på och lära sig arbetsflödet, så det är skitbra att du testar.

Nu har du pushat till "001-test" vilket blir lite fel eftersom varje komponent ska ligga på en egen branch (om du inte testar?). Truck bör ligga på "006-truck". Tänk även på att det i det här fallet bör finnas innehåll i både .h/.cpp. Det är bra att separera class-deklarationen från själva metoderna (signaturer räcker i headerfilen).