Pfthb

I am a newcomer to laravel. I have a controller ProductController like this

 public function buy(Request $request, User $user) {

 StripeStripe::setApiKey("sk_test_xxxxxxxxxxxxxxxxxxxxxxxx");

 $token = $_POST['stripeToken'];

 $charge = StripeCharge::create([
 'amount' => 100,
 'currency' => 'aud',
 'description' => 'Example charge',
 'source' => $token,
 ]);

 if ($charge->status === "succeeded") 
 //-- Processing... --//
 

I would like to ask the more appropriate design style, should I put the part of STTRIE in other places, such as the validator.
If yes, it is to make a rule and a request than verify it in the validator ?

Can someone tell me how to use the rule in the request?

Creating a Stripe charge is not request validation. It's an API call to Stripe. So, it should definitely not stay in the validator.

You can have this logic in a controller for small apps, but for medium/large scale apps with abstraction (e.g. if you want to have the option later to change the payment provider from Stripe to say Braintree), it should be in a service class.

Also, never use $_POST directly. Use $request->input instead. As a thumb rule, if you have 2 ways to do something in code, always use the way that implements higher level libraries (libraries > then framework > then core PHP).

To write the validation using Laravel 5.5+

public function buy(Request $request, User $user)

 // first define your rules
 $rules = [
 'amount' => 'required

For Laravel 5.0 - 5.4:

public function buy(Request $request, User $user)

 // first define your rules
 $rules = [
 'amount' => 'required

The typical responsibilities of a controller in my opinion:

1. Take in a request


2. Return a response

I think it is probably fine to have one conditional or validation check.