d在所有ifs之前将$a[0]、'.$a[1]、'.$a[2]、'.$a[3]移动到单独的变量中。现在没有人阅读php手册吗，mysql_fetch和相关函数都不推荐使用，你应该学会使用PDO或mysqli中的一个而不是mysql扩展。Bolek Lole

---

---

d在所有ifs之前将

$a[0]、'.$a[1]、'.$a[2]、'.$a[3]

移动到单独的变量中。现在没有人阅读php手册吗，mysql_fetch和相关函数都不推荐使用，你应该学会使用PDO或mysqli中的一个而不是mysql扩展。Bolek Lolek，请参阅：你可以在

foreach（）中使用开关

construct，并不简单，但在我看来更容易阅读。另外，为什么不在

dzielperson（）

中使用

for（）

循环呢？我正在使用while，因为它写起来更快，因为（）会更好？您试图将太多的逻辑塞进太窄的空间。如果您的数据库表具有更多描述性名称，这将非常有用（对您也是如此）。此外，了解数据库结构也很有用。作为一个快速提示，您可以在所有ifs之前将

$a[0]、'.$a[1]、'.$a[2]、'.$a[3]

移动到单独的变量中。现在没有人阅读php手册了吗，mysql_fetch和相关函数不受欢迎，您应该学会使用PDO或mysqli中的一个，而不是mysql扩展。Bolek Lolek，请参阅：我不同意您的P.P.P.S。，通常，往返数据库的速度很慢，即使在本地主机上也是如此，同时运行多个查询也会使速度更慢。其余的看起来很酷，没有实际测试，但看起来还行，所以你还是得到了我的投票，代码更简单，可读性更强，这是我认为他想问的问题。@Bolek:你的代码的总体思想是：将不同的东西作为一个整体，将它们多次重新排列成具有难以理解的名称的数组。。。试想一下，在半年的时间里，阅读不相关的任务会有多困难，实际上$a[1]是什么，在DB中是什么列？“啊哈，这是来自$data…data…哦…一行数据是$a[0]。”、“$a[1]。”、“$a[2]，”、“$a[3]；当然！这很有道理！”：）想象一下，稍后支持你的代码的人是个疯子，如果他发现很难修改你的代码，他会追捕你。@BolekLolek我没有像Slava那样花那么多时间进行优化，但我还是缩短了很多时间，是的，这很有趣，也让我发笑。@Slava你提出了一个有趣的论点，我想在某些情况下您是对的，这完全取决于您编写查询的能力以及数据库的索引能力，但是根据我的观察，如果您能够编写智能查询，通常比使用多个查询更快。关于你对博莱克的下一句话，到目前为止有人找到你了吗？你有没有从经验中给出关于疯子的建议？@Slava尝试postgresql，但如果你使用如此大的表，你真的必须调整它的配置，它非常快，并且有这个非常酷的扩展，运行

explain analyze QUERY

将运行查询并输出所选计划的每个步骤的估计值以及实时信息，您可以调整估计系数，使其始终选择最佳（最有效）计划。你也可以使用这些信息来很好地调整你的查询，在优化方面帮了我很大的忙。我不同意你的P.P.P.S.，通常往返数据库的速度很慢，即使是在本地主机上，同时运行多个查询也会使它更慢。其余的看起来很酷，没有实际测试，但看起来还行，所以你还是得到了我的投票，代码更简单，可读性更强，这是我认为他想问的问题。@Bolek:你的代码的总体思想是：将不同的东西作为一个整体，将它们多次重新排列成具有难以理解的名称的数组。。。试想一下，在半年的时间里，阅读不相关的任务会有多困难，实际上$a[1]是什么，在DB中是什么列？“啊哈，这是来自$data…data…哦…一行数据是$a[0]。”、“$a[1]。”、“$a[2]，”、“$a[3]；当然！这很有道理！”：）想象一下，稍后支持你的代码的人是个疯子，如果他发现很难修改你的代码，他会追捕你。@BolekLolek我没有像Slava那样花那么多时间进行优化，但我还是缩短了很多时间，是的，这很有趣，也让我发笑。@Slava你提出了一个有趣的论点，我想在某些情况下您是对的，这完全取决于您编写查询的能力以及数据库的索引能力，但是根据我的观察，如果您能够编写智能查询，通常比使用多个查询更快。关于你对博莱克的下一句话，到目前为止有人找到你了吗？你有没有从经验中给出关于疯子的建议？@Slava尝试postgresql，但如果你使用如此大的表，你真的必须调整它的配置，它非常快，并且有这个非常酷的扩展，运行

explain analyze QUERY

将运行查询并输出所选计划的每个步骤的估计值以及实时信息，您可以调整估计系数，使其始终选择最佳（最有效）计划。您还可以使用这些信息很好地调整查询，这在优化方面对我有很大帮助。

$sql='SELECT DISTINCT id,title,img,description,adder,added,
GROUP_CONCAT(DISTINCT cid,"-",caty ) AS caty,
GROUP_CONCAT(DISTINCT oid,"-",obs,"-",face,"-",rola,"-",typ) AS obs
FROM film

LEFT JOIN f_o ON f_o.f_id = film.id
LEFT JOIN obs ON f_o.o_id = obs.oid

WHERE film.id ='.$fid;

$wynik=mysql_fetch_assoc(mysql_query($sql));
if(isset($wynik['id'])){
echo '<pre>';
print_r($wynik);
echo '</pre>';
////
$array  = explode(',', $wynik['obs']);

$r=array();//director - 0
$s=array();//Screenwriter - 1
$ak=array();//actors - 2
$akn=array();//actors 2 plan - 3
$np=array();//From Idea By - 4
$p=array();//producers - 5
$m=array();//music - 6

foreach ($array as $item)
{
    $a = explode('-', $item);
    if( $a[4] == 0 )
    {
        $r[] = $a[0].','.$a[1].','.$a[2].','.$a[3];
    }
    elseif($a[4] == 1 )
    {
        $s[] = $a[0].','.$a[1].','.$a[2].','.$a[3];
    }
    elseif($a[4] == 2 )
    {
        $ak[] = $a[0].','.$a[1].','.$a[2].','.$a[3];
    }
    elseif($a[4] == 3 )
    {
        $akn[] = $a[0].','.$a[1].','.$a[2].','.$a[3];
    }
    elseif($a[4] == 4 )
    {
        $np[] = $a[0].','.$a[1].','.$a[2].','.$a[3];
    }
    elseif($a[4] == 5 )
    {
        $p[] = $a[0].','.$a[1].','.$a[2].','.$a[3];
    }
    elseif($a[4] == 6 )
    {
        $m[] = $a[0].','.$a[1].','.$a[2].','.$a[3];
    }
}

function dzielperson($data){    
    $i = 0;
    $ile=count($data);
    while ($i < $ile) {
        $a  = explode(",", $data[$i]);
        $caty='<a href="/person/'.dolink($a[1]).'-'.$a[0].'" class="link1">'.$a[1].'</a>'.($i==($ile-1) ? '':', ');
        $i++;
    }
    return $caty;
}
echo '<br>Title: '.$wynik[title];
echo '<br>Desription: '.$wynik[description];
echo '<br>directors: '.dzielperson($r);
echo '<br>screenwriters: '.dzielperson($s);
echo '<br>actors: '.dzielperson($ak);
echo '<br>actors 2 plan: '.dzielperson($akn);
echo '<br>From Idea By '.dzielperson($np);
echo '<br>Producers: '.dzielperson($p);
echo '<br>Music: '.dzielperson($m);

}

Array
(
    [id] => 1
    [title] => Pirates
    [img] => /images/1page_img1.jpg
    [description] => 
    [adder] => baambaam
    [added] => 1349387322
    [obs] => 1-aktor1-foto.jpg-shrek-3,2-aktor2-foto2.jpg-batman-0,3-aktor3-f1.png-Pirat-1,4-aktorzyna4-f2.png-Pirat 3-1
)

film:id,title img,description,adder,added
obs:oid,obs,face,rola,typ
f_o:f_id,o_id

$result = array(
    array(),
    array(),
    array(),
    array(),
    array(),
    array(),
    array()
);

foreach ($array as $item)
{
    $a = explode('-', $item);
    $result[ $a[4] ][] = $a[0].','.$a[1].','.$a[2].','.$a[3];
}

function dzielperson($data){
    $i = 0;
    $ile=count($data);
    while ($i < $ile) {
        $a  = explode(",", $data[$i]);
        $caty='<a href="/person/'.dolink($a[1]).'-'.$a[0].'" class="link1">'.$a[1].'</a>'.($i==($ile-1) ? '':', ');
        $i++;
    }
    return $caty;
}
echo '<br>Title: '.$wynik[title];
echo '<br>Desription: '.$wynik[description];
echo '<br>directors: '.dzielperson($result[0]);
echo '<br>screenwriters: '.dzielperson($result[1]);
echo '<br>actors: '.dzielperson($result[2]);
echo '<br>actors 2 plan: '.dzielperson($result[3]);
echo '<br>From Idea By '.dzielperson($result[4]);
echo '<br>Producers: '.dzielperson($result[5]);
echo '<br>Music: '.dzielperson($result[6]);

$result = json_decode('[[],[],[],[],[],[],[]]');

$names = array_flip(array('r', 's', 'ak', 'akn', 'np', 'p', 'm'));
echo '<br>directors: '.dzielperson($result[$names['r']]);
echo '<br>screenwriters: '.dzielperson($result[$names['s']]);
echo '<br>actors: '.dzielperson($result[$names['ak']]);
echo '<br>actors 2 plan: '.dzielperson($result[$names['akn']]);
echo '<br>From Idea By '.dzielperson($result[$names['np']]);
echo '<br>Producers: '.dzielperson($result[$names['p']]);
echo '<br>Music: '.dzielperson($result[$names['m']]);

$sql='SELECT DISTINCT id,title,img,description,adder,added
FROM film
WHERE film.id ='.$fid;
$wynik=mysql_fetch_assoc(mysql_query($sql));
if(isset($wynik['id'])){

// you should use constants instead of just if(type==_some_meaningless_number_):
$personTypeMap = array(
    'r'    //director - 0
    ,'s'   //Screenwriter - 1
    ,'ak'  //actors - 2
    ,'akn' //actors 2 plan - 3
    ,'np'  //From Idea By - 4
    ,'p'   //producers - 5
    ,'m'   //music - 6
);
// so above should be something like:
// define('PERSON_TYPE_DIRECTOR', 0);
// define...
// then you wouldn't need that array-map above as well as would be easier to understand who is what

// init all people subarrays:
$people = array_fill_keys($personTypeMap, array());

$sql='SELECT oid, obs, typ   #add other fields if you actually use them
FROM f_o
INNER JOIN obs ON f_o.o_id = obs.oid
WHERE f_o.f_id ='.$fid;
$peopleResult = mysql_query($sql);
while ($person=mysql_fetch_object($peopleResult)) {
    $people[$personTypeMap[$person->typ]][] = dzielperson($person);
}

function dzielperson($person){    
   return '<a href="/person/'.dolink($person->obs)."-{$person->oid}\" class=\"link1\">{$person->obs}</a>";
}

// join people in all categories through comma:
foreach ($people as &$category) {
    $category = implode(', ', $category);
}

echo '<br>Title: '.$wynik['title'];
echo '<br>Desription: '.$wynik['description'];
echo '<br>directors: '.$people['r'];
echo '<br>screenwriters: '.$people['s'];
echo '<br>actors: '.$people['ak'];
echo '<br>actors 2 plan: '.$people['akn'];
echo '<br>From Idea By '.$people['np'];
echo '<br>Producers: '.$people['p'];
echo '<br>Music: '.$people['m'];