我有以下 php 代码可以工作,但它太长了,阅读起来很麻烦......
// get the row
if ($taktArticle[0]['t1position3'] > 3 AND $taktArticle[0]['t1position3'] < 7 ) {
$row = "row1";
}
if ($taktArticle[0]['t1position3'] > 7 AND $taktArticle[0]['t1position3'] < 12 ) {
$row = "row2";
}
if ($taktArticle[0]['t1position3'] > 12 AND $taktArticle[0]['t1position3'] < 17 ) {
$row = "row3";
}
if ($taktArticle[0]['t1position3'] > 17 AND $taktArticle[0]['t1position3'] < 22 ) {
$row = "row4";
}
if ($taktArticle[0]['t1position3'] > 22 AND $taktArticle[0]['t1position3'] < 27 ) {
$row = "row5";
}
// get the columns
if ($taktArticle[0]['t1position3'] == 3
or $taktArticle[0]['t1position3'] == 8
or $taktArticle[0]['t1position3'] == 13
or $taktArticle[0]['t1position3'] == 18
or $taktArticle[0]['t1position3'] == 23) {
$col = "col1";
}
if ($taktArticle[0]['t1position3'] == 4
or $taktArticle[0]['t1position3'] == 9
or $taktArticle[0]['t1position3'] == 14
or $taktArticle[0]['t1position3'] == 19
or $taktArticle[0]['t1position3'] == 24) {
$col = "col2";
}
if ($taktArticle[0]['t1position3'] == 5
or $taktArticle[0]['t1position3'] == 10
or $taktArticle[0]['t1position3'] == 15
or $taktArticle[0]['t1position3'] == 20
or $taktArticle[0]['t1position3'] == 25) {
$col = "col3";
}
if ($taktArticle[0]['t1position3'] == 6
or $taktArticle[0]['t1position3'] == 11
or $taktArticle[0]['t1position3'] == 16
or $taktArticle[0]['t1position3'] == 21
or $taktArticle[0]['t1position3'] == 26) {
$col = "col4";
}
if ($taktArticle[0]['t1position3'] == 7
or $taktArticle[0]['t1position3'] == 12
or $taktArticle[0]['t1position3'] == 17
or $taktArticle[0]['t1position3'] == 22
or $taktArticle[0]['t1position3'] == 27) {
$col = "col5";
}
现在。。。我必须从($taktArticle[0]['t1position3']到($taktArticle[0]['t1position11'])重复这个
如您所知,代码将变得庞大...有人知道如何缩短此代码吗?
问候,约翰
您可以创建函数来清理代码。 代码中有明显的模式,查找这些模式并概括这些模式是清理代码的关键要求。 我相信PHP大师可以找到一种更简洁的方法来实现这一目标,但一个基本的例子是这样的:
function get_row($position) {
$row_ranges = array(
array(3, 7),
array(7, 12),
// etc
);
foreach ($row_ranges as $row_index => $range) {
if ($range[0] < $position && $position < $range[1]) {
return sprtintf('row%s', $row_index + 1)
}
}
}
所有行范围都保存在函数内部的集中位置,不再有重复的条件
function get_column($value) {
// looks like you are starting at 3 and have increments of 5
// 3, 8, 13, 18
// you could loop through and calculate these, or hardcode them in
// use `in_array` to clean up the multiple or statements
if (in_array($value, array(3, 8, 13, 18))) {
}
}
克里斯托夫已经对第一部分做出了很好的回答。
至于列,而不是在 IFs 中进行多个比较,请使用 in_array
。
现在。。。我必须从($taktArticle[0]['t1position3']到($taktArticle[0]['t1position11'])重复这个
选择这种次优数据结构的错。
为什么这些数据没有组织为$taktArticle[0]['t1position'][3]
$taktArticle[0]['t1position'][11]
,以便您可以轻松地遍历位置......?
(如果有类似t1position
,所以你也有t2position
、t3position
等——那么这些也应该组织成数组。
我的想法是让数组查找它:
$rowtbl = array(4 => 1, 1, 1, 8 => 2, 2, 2, 2);
$row = 'row'.$rowtbl[$taktArticle[0]['t1position3']];
当然,阵列可能会变大,但您可以从array_merge和范围内构建一些东西。
此外,在我看来,您可以执行以下操作:
$row = ceil(($taktArticle[0]['t1position3']-3)/5);
$col = ($taktArticle[0]['t1position3']-3)%5;
您必须检查 3 和 5 的确切参数(3 是起点,5 是每行的列数。
在列部分,您可以检查数组中是否存在$taktArticle[0]['t1position3']
值,而不是连接多个OR
运算符。
例如,使用 PHP in_array
(http://php.net/manual/en/function.in-array.php):
if(in_array($taktArticle[0]['t1position3'], [3, 8, 13, 18, 23])) {
$col = "col1";
}
尽管这更简洁,但您仍在对此映射中的所有值进行硬编码,因此维护开销将随着您添加新案例而增加。
if(in_array($taktArticle[0]['t1position3'],array(7,12,17,22,27)))
代替此类声明
if ($taktArticle[0]['t1position3'] == 7
OR $taktArticle[0]['t1position3'] == 12
OR $taktArticle[0]['t1position3'] == 17
OR $taktArticle[0]['t1position3'] == 22
OR $taktArticle[0]['t1position3'] == 27)
或者你可以这样做
if((((int)$taktArticle[0]['t1position3'])-7)%5==0||((int)$taktArticle[0]['t1position3'])-7)==0)
对于此声明
if ($taktArticle[0]['t1position3'] == 7
OR $taktArticle[0]['t1position3'] == 12
OR $taktArticle[0]['t1position3'] == 17
OR $taktArticle[0]['t1position3'] == 22
OR $taktArticle[0]['t1position3'] == 27)
另一种方法是在switch中使用布尔运算符来获取行,并使用递归函数来获取col
。//to get Row:
$m = $taktArticle[0]['t1position3'];
switch($m){
case ($m>3 && $m< 7): $row = 'row1'; break;
case ($m>7 && $m< 12): $row = 'row2'; break;
case ($m>12 && $m< 17): $row = 'row3'; break;
case ($m>17 && $m< 22): $row = 'row4'; break;
case ($m>22 && $m< 27): $row = 'row5'; break;
}
//To get Col
function rootNum($num){
return $num-5>0?rootNum($num-5):$num;
}
$n = rootNum($taktArticle[0]['t1position3']);
$col = 'col' . ($n -2);
其中有模式,因此您可以使用除法而不是数组:
$temp = $taktArticle[0]['t1position3']-2
if($temp%5 != 0){
$row = "row".ceil(($taktArticle[0]['t1position3']-2)/5);
}
$col = "col".(($taktArticle[0]['t1position3']-2)%5);