...
De code voldoet daarom ook de kwaliteitseisen in het PvA. De code voldoet aan de kwaliteitseisen uit het PvA. Het maakt gebruik van SOLID via de Single Responsibility Principle. Van GRASP wordt Protected Variations gebruik gemaakt, omdat deze klasse een Interface implementeert. Hierdoor zal het altijd dezelfde inputs en outputs hebben, terwijl de logica anders kan wordenclass heeft een taak en dat is dat de data van de tabbladen worden opgehaald. Het voldoet ook aan de onze Definition of Done. Er is eerst gecontroleerd of het is uitgewerkt in het SRS en SDD en of dit overeenkomtdeze documenten overeenkomen. De code hoort horen bij de aanmaken job de usecase . Het staat niet in de sequence diagram, maar wordt benoemd in een Design desicion. Nadat de code is geprogrammeerd zijn er tests voor geschreven, is het nagekeken door groepsgenoten en is het via Jenkins en SonarQube op codesmells gecontroleerd.We hebben nog een tijdje gehad dat we niet met Jenkins controleerden, omdat we het nog niet werkend hadden. Toen het eenmaal werkte, hebben we meteen de codesmells en bugs opgelostview data. Er zijn nog geen testen voor geschreven omdat we de nuttalk nog niet hebben gehad, zodra wij deze nuttalk hebben gehad zal deze class de nodige unit testen krijgen. Daarnaast hebben we het nog niet met Jenkins gekoppeld omdat wij als groep het nog niet werkend hebben gekregen. Komende sprint gaan we dat wel doen zodat alle code smells en bugs eruit worden gehaald.
Ik geef de code een goede voldoende.