Как можно упростить код.

[Predator199]

Скажите пожалуйста, как можно упростить код. Для того чтоб php быстрей обработал?

PHP код:

        $g = file('mod.txt');
        array_splice($g, 1, 72);
        foreach($g as $vs) {
             echo $vs;
        }        
        $j = file('mod.txt');
        foreach($j as $s) {
            $piece = explode("|", $s);
             echo $piece[0];
        }    

        foreach($j as $s) {
            $piece = explode("|", $s);
             echo $piece[1];
        } 


[dem66]

а вот так?

PHP код:

$g = file('mod.txt');
        array_splice($g, 1, 72);
        foreach($g as $vs) {
             echo $vs;
        }        
        $j = file('mod.txt');
        $out = "";
        foreach($j as $s) {
            $piece = explode("|", $s);
             echo $piece[0];
             $out .= $piece[1];

        }    

        echo "\n".$out; 


[Andkorol]

Не нужно дважды считывать один и тот же файл:

PHP код:

$g = $j = file('mod.txt'); 


[dem66]

еще б знать что в том файле..

может быть и file_get_contents() помог бы? без всяких циклов.

[Predator199]

Можно и file_get_contents но с массивом будет проще. Спс.
Этот код можно без цикла? Выводит только 1 строчку.

PHP код:

        array_splice($g, 1, 72); 
        foreach($g as $vs) { 
             echo $vs; 
        } 


[dem66]

хм..
а если вот так?

PHP код:

array_splice($g, 1, 72);
echo implode("\n", $g); 


[Predator199]

Спасибо...

[Predator199]

Этот код можно проще сделать?

PHP код:

    foreach (type_list() as $type => $description){
            $cache .= '<option value="'.$type.'"';
            if($type==$_GET['pm']||$type==$_POST['pm']) $cache .= 'selected="selected"';
            $cache.= '>'.$description.'</option>';
        } 


[masax]

PHP код:

if($type==$_GET['pm']||$type==$_POST['pm']) 


PHP код:

if ($type == $_REQUEST['pm']) 


[masax]

PHP код:

foreach (type_list() as $type => $description)
            $cache .= '<option value="'.$type.'"'.(($type == $_REQUEST['pm']) ? ' selected="selected"' : '').'>'.$description.'</option>'; 


хотя это не проще, а просто меньше))