Code Review

Nume: Ghita Raul-Andrei

Grupa: 213
Aplicatia a fost creata de: The Laboratory Boss

Lista functionalitatilor care lipsesc,
sunt incomplete sau nu functioneaza
corespunzator.
- Update student
- Remove student
- Search student
Lista functiilor care nu sunt acoperite
de teste.

- Functiile de set
- Functiile din StudentFileRepository.h
- Functiile din SecretaryController.h
- Din Lista.h:
Functia setElement
Lista functiilor nespecificate.

- In domain:
student.h: functiile de get si set
studentValidator.cpp: functia StudentValidator
testSimpluStudent.h: lipsa totala de specificatii
testStudent.h: lipsa totala de specificatii
- In repository:
StudentInMemoryRepository.cpp: lipsa totala de specificatii
- In controller:
SecretaryController.cpp: lipsa totala de specificatii ( cu exceptia
destructorului)
- In util:
Conversions.h: lipsa totala de specificatii
Lista.h: lipsa totala de specificatii
- In ui:
Console.cpp: lipsa totala de specificatii
Corectitudinea separarii interfetei
modulelor de implementarea lor.
- In StudentInMemoryRepository.h, la functia size:
Implementarea ar trebui sa apara in
StudentInMemoryRepository.cpp
- In SecretaryController.h la functiile getNumberOfStudents si
getAllAtudents:
Implementarea ar trebui sa apara in SecretaryController.cpp
Corectitudinea organizarii aplicatiei.

- In domain apare un fisier header studentRepository.h care nu prea isi
are locul acolo
- In fisierul testStudentInMemoryRepository.h apare acceasi functie care
este implementata in testCaseStudentInMemory.h
Functiile in care nu inteleg ce se
intampla (rolul, implementarea).
- In momentul adaugarii unui student, daca numele contine un singur
caracter, utilizatorul este avertizat de faptul ca Numele studentului
nu poate sa fie vid. Daca are un caracter, nu este vid .
Problemele gasite in aplicatie si
volumul de munca necesar pentru
remediere (in ore).
- Lipsa validarii in momentul alegerii tipului de repository folosit
Pentru o valoare intreaga diferita de 0/1 programul moare in
momentul in care se incearca orice modificare/ afisare
Pentru o valoare diferita de tipul intreg programul intra in bucla
infinita
- Lipsa validarii ID-ului in momentul adaugarii unui student
ID- ul ar trebui sa fie un numar intreg, dar introducerea unui
caracter de orice alt tip nu returneaza nici o exceptie
Sunt permise si numere negative (ID- ul unei persoane ar trebui sa
fie un numar pozitiv)
- Lipsa validatorului pentru campul Grupa Studentului
Introducerea unui caracter omoara programul (bucla infinita)
- Primul caracter din numele studentului se pierde la momentul
adaugarii
- Lipsa validarii corespunzatoare a optiunii introduse dupa afisarea
meniului
In momentul introducerii unor caractere programul se inchide fara
nici un avertisment/ mesaj