[英]Can this PHP code structure be improved?
我有存储有关网络数据包的数据的类:
var $from_ip;
var $to_ip;
var $from_port;
var $to_port;
var $tcp_length;
var $tcp_stream_text;
var $tcp_stream;
var $tcp_sequence_dec;
现在,我正在创建函数来打印它:
function Print_data()
{
echo "<table>";
echo "<tr><td>";
echo "From IP:";
echo "</td><td>";
echo $this->from_ip;
echo "</td></tr>";
echo "<tr><td>";
echo "To IP:";
echo "</td><td>";
echo $this->to_ip;
echo "</td></tr>";
echo "</table>";
}
有什么方法可以使其最佳化吗? 您可能会注意到,会有很多相同的行。 (例如,可能是编写一个函数来显示它)。 但是vars呢?
您的变量不需要var
,但是如果它们在类中,我建议定义它们的访问级别( private
, public
, protected
等):
private $from_ip;
private $to_ip;
private $from_port;
private $to_port;
private $tcp_length;
private $tcp_stream_text;
private $tcp_stream;
private $tcp_sequence_dec;
这就是我(个人)编写Print_data()
函数的方式:
function Print_data() {
?>
<table>
<tr>
<td>From IP:</td>
<td><?php echo $this->from_ip ?></td>
</tr>
<tr>
<td>To IP:</td>
<td><?php echo $this->to_ip ?></td>
</tr>
</table>
<?php
}
尝试使用数组! 使用关联数组将键设置为您想要的任何值,我刚刚引用了变量名。 这些值就是您现在拥有的值。
$data = array(
"To IP" => $to_ip,
"From IP" => $from_port,
"To Port" => $to_port,
"TCP Length" => $tcp_length,
"TCP Stream Text" => $tcp_stream_text,
"TCP Stream" => $tcp_stream,
"TCP Sequence Dec" => $tcp_sequence_dec
);
然后迭代! 使用foreach循环获取键和值,以及繁荣。 清洁得多
function Print_data() {
echo "<table>";
foreach ($this->data as $key => $value) {
echo "<tr><td>$key</td><td>$value</td></tr>";
}
echo "</table>";
}
由于PHP关键字“ var”有什么作用? 说,不推荐使用var。
这是我建议的解决方案,
<?php
class TCP {
protected $from_ip;
protected $to_ip;
protected $from_port;
protected $to_port;
protected $tcp_length;
protected $tcp_stream_text;
protected $tcp_stream;
protected $tcp_sequence_dec;
private $template = "<table>
<tr>
<td>From IP:</td>
<td>%s</td>
</tr>
<tr>
<td>To IP:</td>
<td>%s</td>
</tr>
</table>";
static public function render_template() {
return sprintf(self::$template, self::$from_ip, self::$to_ip);
}
}
print TCP::render_template();
这应该适合您的班级
var $from_ip, $to_ip, $from_port, $to_port, $tcp_length, $tcp_stream_text, $tcp_stream, $tcp_sequence_dec;
function Print_data()
{
echo <<<EOF
<table>
<tr><td>
From IP:
</td><td>
$this->from_ip
</td></tr>
<tr><td>
To IP:
</td><td>
$this->to_ip
</td></tr>
</table>
EOF;
}
也许不是最快的,但是我使用类似的东西将对象转换为assoc。 数组:
public function printObject($object)
{
$reflectionClass = new ReflectionClass(get_class($object));
foreach ($reflectionClass->getProperties() as $property)
{
$property->setAccessible(true);
echo "Prop: {$property->getName()}, Value: {$property->getValue($object)} \n";
$property->setAccessible(false);
}
}
声明:本站的技术帖子网页,遵循CC BY-SA 4.0协议,如果您需要转载,请注明本站网址或者原文地址。任何问题请咨询:yoyou2525@163.com.