Как подправить код?

Question

[viktorross]

здравствуйте, есть такой код

class users {

	var $id;
	var $id_pp;
	var $error;
	var $login_error;
	var $clean;
	var $array;
	var $tmp;
	var $last;

	public function __construct($id=0)
	{

		global $db;
		if($id) {
			$this->id=$id;
			$this->array=array();
			$this->array=$db->fetchAssoc('select * from '.TABLE_USERS.' where id='.$id);

			foreach ($this->array as $key=>$value) $this->array[$key] = cleanStr($value);

		}
		$this->error = array();
	}

	function getId() {
		return $this->id;
	}

static function getEmail($id='') {

		global $db;
		if(!$id) return;
		$email=$db->fetchRow('select email from '.TABLE_USERS.' where id="'.$id.'"');
		return $email;
	}
}

вот так я вызываю эту функцию

$email=users::getEmail($id);
как мне получить getEmail не по id а например $jfd ?

Comments

[FanatPHP]

Что такое $jfd и зачем оно нужно?

Answer 1

[index0h]

Поменяйте имя функции на getEmailById и напишите отдельно функцию, получения по чему-то другому.

Ваш код ужасен. Очень рекомендую к прочтению: https://github.com/index0h/php-conventions
Что касается работы с БД, ознакомьтесь с паттерном Repository, в частности с его реализацией Doctrine и не плодите энтропию за зря.

Comments

[FanatPHP]

паттерном Repository, в частности с его реализацией Doctrine

Скажите, эта гениальная мысль вам самому в голову пришла или вы прочитали ее где-то?

[index0h]

FanatPHP, к чему сарказм?))

[FanatPHP]

Ну как бы доктрина это дбал и орм, а репозиторий немного совсем про другое.

Автору нужен не репозиторий, а орм.

И он у него есть. Кривой, но есть.

В общем, доктрина это не репозиторий, и репозиторий нужен автору как рыбке зонтик

[index0h]

FanatPHP

Ну как бы доктрина это дбал и орм, а репозиторий немного совсем про другое.

Ага, еще это обработчик аннотаций, ивент диспатчер и куча всего другого. Но обычно, когда говорят о doctrine - имеется ввиду orm, которая как раз реализует паттерн репозиторий.

[FanatPHP]

index0h, да ты я смотрю знаток. Столько умных слов знаешь. Ещё бы понимал их смысл -цены бы тебе не было

[index0h]

FanatPHP, тот же комментарий))