ElsLommelen
commented
3 years ago Vermits het aanspreken van de databank volledig buiten dit package gelaten is, lijkt het me properder om ook deze functies erbuiten te laten. Nadeel is vooral dat die functies toch wat extra ballast met zich meebrengen in het package:
- een extra dependency, nl. RODBC, dus een extra package dat bij gebruikers geïnstalleerd wordt als ze het package willen gebruiken (kan wel onder suggests, zodat het niet bij elk gebruik van dhcurve geladen wordt)
- om unittests voor deze functies te maken, is er een testdatabank in Access nodig (intern in het package opgeslagen, of eentje dat voor de test codematig aangemaakt wordt als dat al kan)
Bijkomend gaat het ook lastig zijn om een unittest te ontwikkelen die ook op andere OS werkt (omdat Access enkel op Windows draait). Ik zou uiteraard ook de tests kunnen aanpassen zodat ze enkel in Windows uitgevoerd worden, maar eigenlijk test ik bij voorkeur toch op alle OS om de rest van het package ook voor niet-Windows-gebruikers bruikbaar te houden.
Enfin, toch wel wat moeilijkheden te overwinnen om die functies toe te voegen. Wat ik me afvraag: zou het evt. interessant kunnen zijn om een apart package te maken met alle functies voor het aanspreken van de databank, waarin behalve die hulpfuncties ook functies zitten om de gegevens uit de databank op te vragen? Of is dat meer werk dan dat het voordelen biedt t.o.v. de functies in je eigen repo te hebben? (Reken dat het meeste werk van een package kruipt in het schrijven van de functies en de documentatie erbij, of toch als het voor een beperkte gebruikersgroep is en je niet teveel tijd wil steken in unittests en het automatisch testen van code. Voor dhcurve zit dat er dus wel allemaal in, inbodb is een voorbeeld waar dit (nog) niet het geval is.) Een groot voordeel van de functies in package-vorm lijkt mij dat ze gemakkelijk te gebruiken zijn na installatie van het package. Nadeel is een extra repo, dus mss een beetje overdreven als het maar over enkele functies gaat, en een klein beetje code in een andere repo?
leymanan
ik heb zelf enkele hulpfuncties geschreven om status van enerzijds metingen, en anderzijds van dhcurves aan te passen. (zie github leymanan/dhcurves_ANB: develop-branch)
Misschien kunnen die meegenomen worden in het package? Moet niet persé, ik roep ze nu zelf aan via source(file = here::here("src/hulpfuncties.R")), dus ik kan dat ook zo blijven doen.