php代碼優(yōu)化建議

代碼如下：$list數(shù)據(jù)量是上千條；$userJobnumberArray大概有幾萬條數(shù)據(jù)。但是這種循環(huán)執(zhí)行起來非常慢，請問有什么優(yōu)化的方法？

$list = array(
    '1' => array('1','6','124'),
    '2' => array('2','5','156'),
    '4' => array('4','734','221')
);

$userJobnumberArray = array(
    array(
        'deptid' => '1',
        'jobnumber' => 'X1414'
    ),
    array(
        'deptid' => '734',
        'jobnumber' => '34j6'
    ),
);

$userGroup = array();
foreach ($list as $k => $v) {
    foreach ($userJobnumberArray as $l => $j) {
        if (in_array($j['deptid'], $v)) {
            $userGroup[$k][] = $j['jobnumber'];
            unset($userJobnumberArray[$l]);
        }
    }
}

回答

編輯回答

刮刮樂

$userJobnumberArray 這個為什么不變成map的形式[deptid => jobnumber]?

2017年11月12日 02:11

編輯回答

我甘愿

先拿$list，去重整理成字符串，作為取$userJobnumberArray 的in 條件。幾千個沒什么壓力吧。。。

2017年8月13日 10:40

編輯回答

你的瞳

其實，你只需要先把$userJobnumberArray數(shù)據(jù)的先用一次循環(huán)把他改成$userJobnumberArray=[deptid=>jobnumber] 這個格式，然后在執(zhí)行你的外循環(huán)，內(nèi)循環(huán)就不要了，直接判斷 isset($userJobnumberArray[deptid]),如果存在就直接取就好了.

下面這個如何？？？能優(yōu)化地方，也就是in_array() 這個時間復(fù)雜度為n的操作變成了數(shù)組索引調(diào)用這個復(fù)雜度為1的調(diào)用。

<?php

$list = array(
    '1' => array('1','6','124'),
    '2' => array('2','5','156'),
    '4' => array('4','734','221')
);

$userJobnumberArray = array(
    array(
        'deptid' => '1',
        'jobnumber' => 'X1414'
    ),
    array(
        'deptid' => '734',
        'jobnumber' => '34j6'
    ),
);

$ujba = [];

foreach($userJobnumberArray as $v){
    $ujba[$v['deptid']]=$v['jobnumber'];
}
unset($userJobnumberArray);
$userGruop = [];
foreach ($list as $k => $v) {
    foreach($v as $deptid){
        if(isset($ujba[$deptid]))
        $userGruop[$k][] = $ujba[$deptid];
    }
}

print_r($userGruop);

2017年1月21日 06:37

編輯回答