wijzigingen verdwenen about gt_oef1 HOT 7 OPEN

Ik heb mijn lokale branch gefetched en dan gemerged met de Git branch. Mijn wijzigingen van een aantal dagen terug zijn er ook weer (ook in de geschiedenis...).

Krijgen jullie geen mail als iemand een commit heeft uitgevoerd op GIT? of krijg ik die alleen?

from gt_oef1.

Ik heb net enkele commits gedaan maar ik heb dit rechtstreeks op github gedaan. Heb je hier mails van gehad?

from gt_oef1.

neen, geen mail gehad. Op de mails moeten we dus niet vertrouwen. :-S

from gt_oef1.

Het zou mij verbazen dat ik dat was want ik had enkel de throws veranderd en voordat ik ze weer toevoegde had ik een pull gedaan om het safe te spelen.

Ik ben sowieso wel voorzichtig nu want ik merk dat er soms wordt vergeten om te pullen voordat er wordt gepusht. Ook Silvia had dit gedaan met mijn code van de datum klasse een tijd geleden.

Oppassen dus

from gt_oef1.

En toen je de throws veranderde heb je niet opgemerkt dat er een aantal methods veranderd waren? (waar de throws op moeten komen)

Alleszins, De code staat er terug. Ook de compareTo() functie heb ik teruggezet want deze was duidelijk niet getest (25 van de 87 tests slaagden).

Mag ik aub vragen om niet zomaar wijzigingen (op afgewerkte classes) aan te brengen maar dit eerst in een issue te melden zodat we dit kunnen bespreken. Ik heb gisteren een halfuur mogen zoeken waarom mijn Datum classe niet meer werkte...

En als laatste: Test aub je code voordat je deze op Git plaatst, de unittest is er voor gemaakt!

Ps: Iedereen maakt fouten, ook ik. Daar leren we uit. En idd oppassen bij het committen. Ik kijk nu altijd eerst op Git om te zien of lokaal alles up-to-date is. (+ backup....)

from gt_oef1.

Dag allemaal!

Ik heb net een commit & push gedaan, kunnen jullie even checken dat alles nog OK is voor iedereen?

Ik heb een beetje rondgekeken in de klassen die al werden gemaakt in de model package en ik heb de volgende opmerkingen en vragen hierover:
enum Leraar: de constructor heeft geen acces modifier: is dit de bedoeling?
Quiz:
setLeerjaar checkt niet of leerjaar tussen 1 & 6 ligt
setQuizID: werkt dit? Ik zie dat jullie de size +1 toekennen als id ==> wat als je het 1ste element verwijdert en daarna 1 element toevoegt?
er is geen link tussen Quiz en QuizCatalogus
QuizCatalogus
moet ook nog iterable implementeren dacht ik
QuizOpdracht
ik heb een link met opdrachtantwoord toegevoegd
er zijn geen setters, waarom?
Opdracht
er is geen link met OpdrachtCategorie
compareTo: werkt dit?
ook hier wordt de ID toegekend aan de hand van de size() ==> wat als 1ste element met id 1 verwijderd wordt en daarna 1 element toegevoegd wordt?
setOpdrachtCatalogus: wat als er null wordt doorgegeven?

Kunnen we dit maandag eens overlopen?

groeten,

Silvia=

from gt_oef1.

Dit kunnen we maandag overlopen :-) Ik heb momenteel een eigen versie van het Quiz project in MVC stijl. Deze staat op een andere GIT repo. Ik geef jullie toegang tot die repo zodat jullie dit eventueel kunnen bekijken. (Strategy Patterns, Factories, Controllers, JFrame, JPanel, Db data) -> https://github.com/VinzzB/GroepT_Quiz

from gt_oef1.