жду критики

[crautcher]

и стоит ли разбить метод на более мелкии составные части , дабы улучшить конструктивность?

<?php
function Hear($msg)		#Обрабатывает сообщение и отправляет юзeру Ответ + вопрос
{

	$this -> SetMinds('Со мной что то не тaк !'); //задаём ответ который не должен появится , если появится - действительно чтото не так 
	if (!$this -> SeakOrNot()); 					//проверка готовности к работе
		if (!$this-> AdminCommands($msg))			//Не является ли текст командой
			$this->GenerateAnswer($msg);//Выделяет вопрос из полученого сообщения или задаёт фолс

	if (!$this->foundQ)
		$this -> Dunno();  //Вывод сообщений когда незнаем что хотят

	$answer =$this -> GetA( $this -> GetQID($this -> foundQ ) ); //получаем ответ на заданный вопрос

	if (!empty($answer))
		$this-> SetMinds ( $answer ); 			//шлем ответ юзеру
	else
		{
		$this -> LastQuestion = $this->asked;
		$this -> Dunno(); 
		}

	if ($this->LastQuestion)					//если был задан вопрос в прошлом сообщении
			{
			$id = $this -> GetQID($this->LastQuestion); 	//пытаемся получить айдишник последнего вопроса
			if (!empty($id))
				{
				$answer = GetAnswer($msg);		//выделяем ответ на вопрос из сообщения
				if ($answer != '')
					{
					if (!($this -> sql -> query("select text from answers where text='".$answer."' AND questionId = '".$id."' ")))
						$this->AddA($answer,$id); //добавляем ответ если такого нет
					}
				}
			}

	$this->LastQuestion= $this -> GenerateQuestion();    //спрашиваем юзера
	$this -> SendMind( $this->LastQuestion );

}

?>

[Gold Dragon]

crautcher, ты оригинал выдрал метод из класса и предлагаешь нам что?

[crautcher]

crautcher, ты оригинал выдрал метод из класса и предлагаешь нам что?

еслиб я все класы которые используются сюда выложил , никтоб даже и не подумал смотреть всю эту кучу кода ...

[Gold Dragon]

а ты думаешь сейчас понятно что все эти методы и свойства делают? Да и просто не понятно что ты хочешь услышать..

[crautcher]

а ты думаешь сейчас понятно что все эти методы и свойства делают? Да и просто не понятно что ты хочешь услышать..

ну я коменты писал для этого . Что услышать ? Ну незнаю , вы же тут все люди с опытом , указать на недостатки. ладно в топку тему

[Gold Dragon]

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

Edited September 30, 2011 by Gold Dragon