wijzigingen verdwenen

[VinzzB]

Iemand heeft mijn wijzigingen van een aantal dagen gewist? Per ongeluk?
Ik had oa de setDatum() method veranderd.. Ik had deze aangepast zodat bij een foute datum terug de oude datum wordt ingevuld... + Mijn Test is ook terug veranderd. + enkele wijzigingen zijn weg. Mijn nieuwe SetDatumInternal is weg....pfff

Ik ben er zeker van dat ik het geupped had na de wijziging van Isaak (verwijderen throws). Mijn lokaal project geeft ook aan dat ik geen wijzigingen meer heb (tenzij ik nu pull of fetch, dan worden mijn wijzigingen lokaal gewist, wat ik nu niet ga doen...)

Zelfs de geschiedenis is weg in Git. Iemand had niet de laatste versie van git gehaald...

Ik denk dat jij het was Isaak? Jij hebt nog wijzigingen aangebracht in de datum classe.

Ik zal nu alles wel corrigeren / migreren, maar aub, kijk altijd even op git of je wel over de laatste versie beschikt voor je begint te committen!

[VinzzB]

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?

[Silvia24]

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

[VinzzB]

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

[ghost]

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

[VinzzB]

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....)

[Silvia24]

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=

[VinzzB]

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