search

pxlit-projects / 3tiwfullstack-labs-AngelChezziPXL

3tiwfullstack-labs-3tiwfullstack-labs-3TIWFullStack2425 created by GitHub Classroom
0 stars 0 forks source link

Foutafhandeling indien geen data gevonden #2

Open AngelChezziPXL opened 3 days ago

AngelChezziPXL commented 3 days ago

@tomschuyten Dag meneer Schuyten

Is de code in de link ok voor het afhandelen van foutmeldingen? In de video wordt hier niet veel over gezegd. Als ik de code strikt volg vanuit de video krijg ik een 200 code met een null return value.

https://github.com/pxlit-projects/3tiwfullstack-labs-AngelChezziPXL/blob/8be5be6eef9519f0f29d871fc66032363cb70416/backend-java/SpringCloudPXL/department-service/src/main/java/be/pxl/services/department/services/DepartmentService.java#L28

tomschuyten commented 23 hours ago

Ok voor mij!

AngelChezziPXL commented 22 hours ago

@tomschuyten

https://github.com/pxlit-projects/3tiwfullstack-labs-AngelChezziPXL/blob/f3ed8aab21bef39c9e8083e184bfe2c9f9fe0420/backend-java/SpringCloudPXL/employee-service/src/main/java/be/pxl/services/employee/services/EmployeeService.java#L47C5-L65C6

Ik heb gemerkt dat als ik de findAllBy...Id(ongeldigId) aanroep er toch een 200 code retourneert met een lege array. Zelfs met orElsThrow(...). Is het ok hoe ik het heb opgelost of wordt dit normaal niet gedaan? Ik heb ook een vraag betreffende de best practices voor httpstatus codes. Worden de 'if controles' beter in de service of in de controller uitgevoerd?