Laravel lumen : Post API method

Question

Question is quite simple, I am using dingo package, Currently below is my code for an API call.

public function publishApplicant(Request $request,$id){
    $changeStatus = \App\User::where('id', $id)->update(array('admin_published' => 1));

    if($changeStatus){
        return response()->json([
            'message' => 'Applicant published.',
            'code' => 200,
            'status' => 1
        ], 200);
    }

    return response()->json([
        'message' => 'Applicant published.',
        'code' => 500,
        'status' => 0
    ], 200);
}

What can be improved in the above code?

phaberest · Accepted Answer · 2017-09-10 08:17:25Z

Not a big improvement, but since $changeStatus is responsible for a different 'code' and 'status' and the rest is identical, we may do something like the following:

public function publishApplicant(Request $request,$id)
{
    $changeStatus = \App\User::where('id', $id)->update([
        'admin_published' => 1
    ]);

    return response()->json([
        'code'    => $changeStatus ? 200 : 500,
        'message' => 'Applicant published.',
        'status'  => $changeStatus ? 1 : 0
    ], 200);
}

I've also applied some PSR-2 coding style adaptments, nothing too fancy but it makes it a bit more readable.

Anyway you may want to change the message in case of failure.

stan · Accepted Answer · 2017-09-09 18:43:17Z

1

You could store the response in a variable and only call return once. That seems cleaner. Also, put constants like messages and HTTP codes into constant variables.

answered Sep 9, 2017 at 18:43

stan

2091 silver badge8 bronze badges

Add a comment |

Stack Exchange Network

Laravel lumen : Post API method

2 Answers 2

You must log in to answer this question.

Hot Network Questions

Laravel lumen : Post API method

2 Answers 2

You must log in to answer this question.

Related

Hot Network Questions