0

Добрый день! У меня такой вопрос. Как мне можно оптимизировать ниже представленный (надеюсь на нестрогий суд, так как занимаюсь не долго)код.

Делала сайт на Codeigniter, что требовалась. Вывести данные из 2 таблиц(да таблицы не приведены к 3нф т.к. база была сделана до меня и мне не дали время на нормальную проработку базы, а хотят только наращивать функционал). И по требованию было сделано 4 поля обязательных, которые всегда выводятся и 26 необязательны, которые были сделаны чекбоксами(это ужасно) и вот я получаю такой код на выводе.

<tr>
                    <th> Станция назначение </th>
                    <th> Станция операции </th>
                    <th> Номер вагона </th>
                    <th> Дата </th>
                    <th> Время </th>

                 <? if($op == "yes"):?>
                <th> ОП </th>
                <? endif;?>
                 <? if($dpr == "yes"):?>
                <th> ДПР </th>
                <? endif;?>
                 <? if($inpoezd == "yes"):?>
                <th> Индекс поезда </th>
                <? endif;?>
                 <? if($gruz == "yes"):?>

и т.д.

<? foreach ($disla as $item):?>
                    <?

                        $x = $item->naznachenie;
                        $q = $item->operation;
                        for($i=0; $i< count($item->naznachenie); $i++)
                        {
                            if ($x != $y)
                            {
                                echo "<tr><td>".$x."</td><td></td></tr>";
                                $t = true;
                                $y = $x;
                            }
                            else
                            {
                                $t = false;
                            }
                        }
                    for ($i=0; $i< count($item->operation); $i++)
                    {
                        if ($q != $w or $t == true)
                        {
                            echo "<tr><td></td><td>".$q."</td></tr>";
                            $w = $q;
                        }
                    }
                    ?>
                <tr>
                    <td></td>
                    <td></td>

                    <td><a href="#" style='text-decoration: none;' id="<?=$item->number_vagona?>" onclick="TellId(this)"><?=$item->number_vagona?></a></td>
                    <?

                    $date_out = date("d.m.Y", strtotime($item->date));

                    if(strtotime($date11) == strtotime($date_out) or strtotime($date1) == strtotime($date_out) and ($item->time) > $time)
                    {
                        echo "<td style='color: #3caa3c;'><strong>".$date_out."</strong></td>"; 
                    }
                    elseif(strtotime($date_out) <= strtotime($date5) and strtotime($date_out) > strtotime($date10))
                    {
                        echo "<td style='color: orange;'><strong>".$date_out."</strong></td>"; 
                    }
                     elseif(strtotime($date_out) <= strtotime($date10))
                    {
                        echo "<td style='color: red;'><strong>".$date_out."</strong></td>"; 
                    }

                    else
                        echo "<td><strong>".$date_out."<strong></td>";

                    ?>
                    <td><?=$item->time ?></td>
                    <? if($op == "yes"):?>
                        <td><?=$item->op ?></td>  
                    <? endif;?>

                    <? if($comment == "yes"):?>
                        <td><input onchange="uodate(<?=$item->number_vagona ?>,'comment',this)" type="text"  value="<?=$item->comment ?>" id="noborder"/></td>          
                    <? endif;?>

при выводе происходит сравнение первых двух столбцов, вывод даты по цвету.

я его конечно сократила очень сильно, сами представьте 40 столбцов на выбор юзера. вроде бы все работает, но скорость работы просто ужасная. Понимаю, что во многом я делала не правильно. Просьба напишите замечания, где что можно сделать лучше? Заранее спасибо большое!

3
  • Дам маленький совет! Нужно html встраивать в php а не php в html если не используйется шаблонизатор! ----------------------- Это же ужасно <td><?=$item->time ?></td> <? if($op == "yes"):?> <td><?=$item->op ?></td> <? endif;?> не кажется так?
    – Palmervan
    19 июл 2012 в 10:16
  • да, правы, спасибо.
    – kira
    19 июл 2012 в 10:20
  • просто, когда все начиналось не рассчитывалось на такой объем было 6 столбца, которые спокойно выводились.
    – kira
    19 июл 2012 в 10:22

1 ответ 1

0

в Codeigniter не работал, но нас колько знаю это фреймворк в котором реализован паттерн MVC. Если вы не знаетет его - то вперед изучать, иначе вы не сможете эффективно использовать этот да и другие фрэймворки Т.е. вы хотите сказать что это кусок кода относящийся исключительно к представлению?

Если у вас так много необязательных полей то я бы вам посоветовал посмотреть в сторону использовании модели EAV (Entity-Attribute-Value) можно почитать тут

2
  • ну я его изучала, и все у меня сделано по модели MVC, как только было это возможно(в моем представлении), данный кусок кода представляет вывод таблицы. И я задаю конкретный вопрос, как мне оптимизировать мой вывод учитывая количество столбцов, и что пользователь их может регулировать чекбоксами.
    – kira
    19 июл 2012 в 10:14
  • спасибо, за ссылку буду изучать.
    – kira
    19 июл 2012 в 11:04

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.