search

mrTsjolder / SWOP-KULAK

Source code for project Software design
GNU General Public License v2.0
0 stars 1 forks source link

Duration getEndtimefrom #13

Closed isostorm closed 9 years ago

isostorm commented 9 years ago

Duration getEndTimeFrom(begin) houdt momenteel geen rekening met weekends, kan iemand da fixen? Want ik heb het gehad met die tijdzever.

mrTsjolder commented 9 years ago

I'm on it...

mrTsjolder commented 9 years ago

Is dat niet al allemaal in rekening gebracht door de duration zelf? :)

mrTsjolder commented 9 years ago

probleem gevonden (ook wat extra constanten toegevoegd)...

isostorm commented 9 years ago

Nu worden enkel het eerste weekend geskipt, als er meerdere weekends tussen zitten, failt het. Die duration is veel gecompliceerder dan daje zou verwachten. Kzou ook niet kiezen voor een variabele werkweek, dat maakt alles nog complexer. Want kan het zijn dat de between constructor nu ook niet meer werkt?

mrTsjolder commented 9 years ago

het gaat toch maar over 1 dag en de rest van je duration zijn sowieso werkuren... ik denk dat de variabele werkweek niet zo veel complexiteit met zich meebrengt... al uw tests slagen nog, dus als ze goed geschreven zijn, werkt alles nog...

isostorm commented 9 years ago

De testen waren te simpele gevallen, keb ze aangepast, het is supercomplex wi met die weekends, Frederic ging der ne keer na kijken.