Как правильно организовать ООП php

Question

[grabbee]

Делаю класс доступа к GET POST данным, там просто типа

<?
namespace core;

 class Requests {   
  
public static function GET($VarName, $Default = NULL)
  {
   return isset($_GET[$VarName]) ? $_GET[$VarName] : $Default;
  }

отдельным классом делаю стандартную обработку данных типа

<?          
namespace contr;

use core\Requests;
 
 class RequestPrepare extends Requests { 
   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);
   }

Но проверяются всегда все входящие данные, числа через одну, строки через вторую. Сейчас так

$request = new contr\RequestPrepare ; 
 echo $request->correctInt( $request->GET('str') );

Это правильно? Или не очень...

Comments

[grabbee]

@Aliance, прошу прощения что не привёл листинг всего класса, помимо GET данных имеются POST и COOKIES - которые там же присутствуют в классе Requests. То есть в вашем случае switch ($type) нужно будет повторять как минимум в POST и COOKIES...

[Илья Лесных]

@grabbee что мешает написать так:

const STRING = 1;
const INT = 2;
// ...
public function get($key, $default = null, $type = self::STRING) {
    return $thi->_get($_GET, $key, $default, $type);
}

public function post($key, $default = null, $type = self::STRING) {
    return $thi->_get($_POST, $key, $default, $type);
}

public function cookie($key, $default = null, $type = self::STRING) {
    return $thi->_get($_COOKIE, $key, $default, $type);
}

private function _get(array $list, $key, $default = null, $type = self::STRING) {
    if (! isset($list[$key])) {
        return $default;
    }

    switch ($type) {
        case self::STRING:
            $value = addslashes(trim(strip_tags($list[$key])));
            break;
        case self::INT:
            $value = (int) $list[$key];
            break;
        default:
            throw new Exception('Unknown type');
            break;
    }

    return $value;
}

[grabbee]

@Aliance, я пока только разбираюсь в ООП проектировании. Допустим слэшить данные где-то и не нужно будет. То есть предыдущий класс подошел бы, а теперь нет. Получается правильно делать расширение класса и перезагрузку всех методов по вашей схеме?

[Илья Лесных]

@grabbee если вы работаете один, то правильно так - как вам удобно.
можно ввести еще один параметр - флаг необходимости слеширования. можно второй метод, без слеширования, можно новый тип для свича.

Answer 1

[Илья Лесных]

1. mysql_real_escape_string не будет работать без ресурса подключения.
2. забудьте про CAPSLOCK и почитайте, что такое camelCase, ваш код будет приятнее читать.
3. зачем статичные методы?
4. почитайте про паттерн (шаблон) проектирования - Singleton (одиночка), обычно объект Request реализуется именно так.
5. зачем нужно наследование в 2 класса?
6. вместо различных методов correct*, я бы переписал функцию получения следующим образом:

const STRING = 1;
const INT = 2;
// ...
function get($value, $default = null, $type = self::STRING) {
    if (! $value) {
        return $default;
    }

    switch ($type) {
        case self::STRING:
            $value = addslashes(trim(strip_tags($value)));
            break;
        case self::INT:
            $value = (int) $value;
            break;
        default:
            throw new Exception('Unknown type');
            break;
    }

    return $value;
}

Comments

[andreyqin]

1. Будет

[Илья Лесных]

@andreyqin

link_identifier
Соединение MySQL. Если идентификатор соединения не был указан, используется последнее соединение, открытое mysql_connect(). Если такое соединение не было найдено, функция попытается создать таковое, как если бы mysql_connect() была вызвана без параметров. Если соединение не было найдено и не смогло быть создано, генерируется ошибка уровня E_WARNING.

(с) www.php.net/manual/ru/function.mysql-real-escape-s...

[andreyqin]

Указан в скобках как необязательный параметр. Посмотрите примеры.

[Илья Лесных]

@andreyqin со зрением все окей? если он не указан - берется последний. Использование этой функции в таком роде может быть и без подключения, поэтому это может не работать, а если и будет - то сама по себе такая зависимость кривая. Это не говоря уже о том, что этот адаптер и вовсе deprecated

[andreyqin]

Мы сейчас говорим про то, будет или не будет функция работать без дескриптора соединения.

[grabbee]

Не ссорьтесь :) я только два дня в ООП - это кусок всунут из процедурного кода. Плюс ко всему, моё приложение не возможно без подключения к БД , а подключение будет в самом начале - то есть оно всегда будет, а если его не будет, то не функция выдаст ошибку, а само приложение преднамеренно ляжет.

Answer 2

[Павел Соловьёв]

непонятно зачем вам это, но есть пару замечаний если метод объявлен как статический то и вызывайте его как статический

public static function GET

Requests::GET('str');

return ($data); - зачем скобки?