kbevers
commented
3 years ago Jeg synes bestemt det giver mening.
En udfordring med at flytte kode fra fire.cli-laget til et dybere API-lag er blandt andet, at der i den nuværende regnearksfunktionalitet bliver skrevet til brugerens terminal med fire.cli.print. CLI-modulets funktionalitet er det yderste lag af applikationen og bør derfor ikke være en afhængighed for dybere-liggende moduler.
Kan vi løse den her med indføring af en logger? Så de forskellige funktioner i stedet for at bruge fire.cli.print() smider deres information i en log, som CLI'en efterfølgende kan vælge at skrive på skærmen til brugeren.
xidus
Regnearksfunktionaliteten i fire.cli.niv har bør ligge i et separat modul med dette tema. Det er forsøgt startet i #475 med et fire.io.regneark-modul, og her er tilføjet en række nye funktioner i forbindelse med dette PR.
En udfordring med at flytte kode fra fire.cli-laget til et dybere API-lag er blandt andet, at der i den nuværende regnearksfunktionalitet bliver skrevet til brugerens terminal med fire.cli.print. CLI-modulets funktionalitet er det yderste lag af applikationen og bør derfor ikke være en afhængighed for dybere-liggende moduler.
En ekstra udfordring er derfor, at brugerens nuværende oplevelse med de eksisterende tilbagemeldinger fra programmet i terminalen skal afkobles fra funktionaliteten, som ikke bør vide noget om, hvordan den bliver anvendt; samtidig skal brugeren ikke miste væsentlige tilbagemeldinger fra programmet.
Med accept af #475 vil der desuden være kode med overlappende funktionalitet, så det vil være nødvendigt at få skrevet de to modulers funktionalitet sammen eller skifte det ene ud med det andet.
Ændringerne forudsætter, at vi har en samlet plan for tilsvarende refaktoriseringer af hele kodebasen.