Conforme escrito no comentário acima, você deve dividir e conquistar para facilitar sua vida (especialmente ao escrever o código enquanto brinca com ele nessa grande função). Isso funciona tão fácil quanto:
function file_put($number, $data)
{
$path = sprintf("C:/temp/wamp/www/file%d.txt", $number);
file_put_contents($path, $data);
}
por exemplo, está apenas substituindo as muitas linhas duplicadas em que você só precisa de um arquivo (numerado) em que coloca alguma string.
Mas você também pode fazer isso com coisas mais complexas, como a operação do banco de dados. Você provavelmente deseja remover o tratamento de erros da sua vista, bem como tomar cuidado para se conectar ao banco de dados quando necessário e uma maneira mais flexível de buscar os dados. Isso pode ser feito movendo o (suavemente obsoleto)
mysql_* funções em uma ou duas classes próprias, para que fique fora de sua vista. Isso tornará seu uso muito mais fácil (o que mostro primeiro):// Create your database object to use it later on:
$config = array(
'server' => 'localhost',
'name' => 'root',
'password' => '',
'db' => 'test',
);
$db = new MySql($config);
Eu chamei a classe de banco de dados
MySql pois representa a conexão mysql e funciona com a antiga extensão mysql. Você só precisa passar esse objeto de banco de dados para a função em sua pergunta. Combinado com o file_put função, ficaria assim:function checkin(MySql $DB, $TechID, $ClientID, $SiteID)
{
$query = sprintf("SELECT `Type` FROM `Log` WHERE `TechID` = '%d' ORDER BY LogTime DESC LIMIT 1", $TechID);
file_put(5, $query);
$result1 = $DB->query("SELECT COUNT(*) FROM Log");
$result2 = $DB->query($query);
foreach ($result1 as $row1) {
list($count) = $row1;
$data = "ClientID:$ClientID TechID:$TechID SiteID:$SiteID Count:$count"
file_put(3, $data);
foreach ($result2 as $row2) {
file_put(4, $data);
}
}
}
Ainda o
checkin A função está perto de ser grande (12 linhas de código já), mas é muito menor que sua primeira versão porque delega o trabalho de escrever os arquivos e acessar o banco de dados. Espero que esta demonstração seja útil. O que segue é o exemplo de código completo:/**
* MySql Exception
*/
class MySqlException extends RuntimeException
{
}
/**
* MySql Database Class
*/
class MySql
{
private $server;
private $name;
private $password;
private $db;
private $connection;
public function __construct(array $config)
{
$this->server = $config['server'];
$this->name = $config['name'];
$this->password = $config['password'];
$this->db = $config['db'];
}
private function connect($server, $name, $password)
{
$this->connection = mysql_connect($server, $name, $password);
if (!$this->connection) {
$this->error("Unable to connect to '%s' as user '%s'", $server, $name);
}
}
private function select($db)
{
if (!mysql_select_db($db, $this->connection)) {
$this->error("Unable to select database '%s'", $db);
}
}
private function close()
{
$this->connection && mysql_close($this->connection);
}
private function connectSelect()
{
$this->connect($this->server, $this->name, $this->password);
$this->select($this->db);
}
/**
* @param $query
* @return MySqlResult
*/
public function query($query)
{
$this->connection || $this->connectSelect();
$result = mysql_query($query, $this->connection);
if (!$result) {
$this->error("Unable to execute query '%s'", $query);
}
return new MySqlResult($result);
}
/**
* @param string $format
* @param ...
* @throws MySqlException
*/
private function error($format)
{
$args = func_get_args();
array_shift($args);
$format .= ': %s';
$args[] = $this->connection ? mysql_error($this->connection) : mysql_error();
throw new MySqlException(vsprintf($format, $args));
}
public function __destruct()
{
$this->close();
}
}
/**
* MySql Result Set - Array Based
*/
class MySqlResult implements Iterator, Countable
{
private $result;
private $index = 0;
private $current;
public function __construct($result)
{
$this->result = $result;
}
public function fetch($result_type = MYSQL_BOTH)
{
$this->current = mysql_fetch_array($this->result, $result_type);
return $this->current;
}
/**
* Return the current element
* @link http://php.net/manual/en/iterator.current.php
* @return array
*/
public function current()
{
return $this->current;
}
public function next()
{
$this->current && $this->fetch();
}
/**
* Return the key of the current element
* @link http://php.net/manual/en/iterator.key.php
* @return mixed scalar on success, or null on failure.
*/
public function key()
{
return $this->current ? $this->index : null;
}
/**
* Checks if current position is valid
* @link http://php.net/manual/en/iterator.valid.php
* @return boolean The return value will be casted to boolean and then evaluated.
* Returns true on success or false on failure.
*/
public function valid()
{
return (bool)$this->current;
}
/**
* Rewind the Iterator to the first element
* @link http://php.net/manual/en/iterator.rewind.php
* @return void Any returned value is ignored.
*/
public function rewind()
{
$this->fetch();
}
/**
* Count of rows.
*
* @link http://php.net/manual/en/countable.count.php
* @return int The count of rows as an integer.
*/
public function count()
{
return mysql_num_rows($this->result);
}
}
// Create your database object to use it later on:
$config = array(
'server' => 'localhost',
'name' => 'root',
'password' => '',
'db' => 'test',
);
$db = new MySql($config);
function file_put($number, $data)
{
$path = sprintf("C:/temp/wamp/www/file%d.txt", $number);
file_put_contents($path, $data);
}
function checkin(MySql $DB, $TechID, $ClientID, $SiteID)
{
$query = sprintf("SELECT `Type` FROM `Log` WHERE `TechID` = '%d' ORDER BY LogTime DESC LIMIT 1", $TechID);
file_put(5, $query);
$result1 = $DB->query("SELECT COUNT(*) FROM Log");
$result2 = $DB->query($query);
foreach ($result1 as $row1) {
list($count) = $row1;
$data = "ClientID:$ClientID TechID:$TechID SiteID:$SiteID Count:$count";
file_put(3, $data);
foreach ($result2 as $row2) {
file_put(4, $data);
}
}
}
checkin($db, 1, 2, 3, 4);
