如何使这个PHP代码更短


how to make this php code shorter?

我有以下 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,所以你也有t2positiont3position等——那么这些也应该组织成数组。

我的想法是让数组查找它:

$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);