Refactor attempts to get request data to use type-specific methods when the type is known

[imliam]

-$name = $request->input('name');
-$age = (int) $request->get('age');
-$price = (float) $request->data('price');
-$isActive = (bool) $request['is_active'];
+$name = $request->string('name');
+$age = $request->integer('age');
+$price = $request->float('price');
+$isActive = $request->boolean('is_active');

[alexey-m-ukolov]

I like the idea just want to point out that $request->input('name') and $request->string('name') are not the same - former returns (in this example) string|null and latter returns \Illuminate\Support\Str.
I'm not sure if this case should be handled at all - even if we replace with $request->string('name')->toString() we are still losing null. I think only the explicit casts and calls like $request->input('name', '') can be safely replaced (with added ->toString()).

[peterfox]

Might want to consider what happens if null coalese is used:

Suggested change

	$name = (string) $request['name'];
	$age = (int) $request['age'];
	$price = (float) $request['price'];
	$isActive = (bool) $request['is_active'];
	$name = (string) $request['name'];
	$age = (int) $request['age'];
	$price = (float) $request['price'];
	$isActive = (bool) $request['is_active'];
	$name = (string) ($request['name'] ?? null);
	$age = (int) ($request['age'] ?? null);
	$price = (float) ($request['price'] ?? null);
	$isActive = (bool) ($request['is_active'] ?? null);