Как правильно организовать ООП php Делаю класс доступа к GET POST данным, там просто типа correctInt( $request->GET('str') ); Это правильно? Или не очень...
Нет, не совсем правильно. Ваш класс RequestPrepare наследуется от класса Requests, который, в свою очередь, не имеет никакой логики и просто предоставляет доступ к данным из $_GET. Таким образом, класс RequestPrepare не должен наследоваться от Requests, так как это нарушает принцип единственной ответственности и не соответствует принципам SOLID.
Вместо этого, вы можете просто использовать класс Requests внутри класса RequestPrepare, чтобы получить данные из $_GET и обработать их правильно. Например:
namespace contr; use core\Requests; class RequestPrepare { public static function correctData($data) { $data = strip_tags( (string)$data ); $data = mysql_real_escape_string ( trim( $data ) ); return $data; } public static function correctInt($data) { $data = abs( (int)$data ); return $data; } public static function process() { $request = new Requests(); $str = $request->GET('str'); echo self::correctInt($str); } } RequestPrepare::process();
Таким образом, вы избегаете наследования классов и направляете логику обработки данных к классу RequestPrepare, что делает код более чистым и подходящим для объектно-ориентированного программирования.
Нет, не совсем правильно. Ваш класс RequestPrepare наследуется от класса Requests, который, в свою очередь, не имеет никакой логики и просто предоставляет доступ к данным из $_GET. Таким образом, класс RequestPrepare не должен наследоваться от Requests, так как это нарушает принцип единственной ответственности и не соответствует принципам SOLID.
Вместо этого, вы можете просто использовать класс Requests внутри класса RequestPrepare, чтобы получить данные из $_GET и обработать их правильно. Например:
namespace contr;use core\Requests;
class RequestPrepare {
public static function correctData($data)
{
$data = strip_tags( (string)$data );
$data = mysql_real_escape_string ( trim( $data ) );
return $data;
}
public static function correctInt($data)
{
$data = abs( (int)$data );
return $data;
}
public static function process()
{
$request = new Requests();
$str = $request->GET('str');
echo self::correctInt($str);
}
}
RequestPrepare::process();
Таким образом, вы избегаете наследования классов и направляете логику обработки данных к классу RequestPrepare, что делает код более чистым и подходящим для объектно-ориентированного программирования.