search

LucasRegalar / Expense-Tracker-V2

My first full Stack Application using vanilla PHP
0 stars 0 forks source link

Payload als Parameter #9

Open theDigitalViking opened 4 months ago

theDigitalViking commented 4 months ago

https://github.com/LucasRegalar/Expense-Tracker-V2/blob/2be618b5b2d13514c137967c5f53a27c0846c401/core/functions.php#L37

Den Payload hier als Parameter mitzugeben ist etwas unsauber. Du musst dann checken, ob das array immer so aufgebaut ist, wie es die Funktion erfordert. Dadurch hast du in dem Aufruf im Success Fall auch einen leeren String als error übergeben. Das ist auch ziemlich unsauber.

Besser wäre es die erforderten Parameter zu benennen und direkt defaults mit anzugeben. Dann kann man über die Code Completion sehen, was die Funktion benötigt die Funktion selbst weiß immer, dass sie Daten bekommt, mit denen sie funktioniert.

LucasRegalar commented 4 months ago

So ist besser?

function sendResponse($code = 200, $message = "", $error = "") { http_response_code($code); header("Content-Type: application/json"); echo json_encode([ "message" => $message, "error" => $error, ]); exit(); }