Как правильно расставить «»?

Question

[Николай Громов]

Здравствуйте. Подскажите как правильно расставить "" и ' ', сначала все было нормально, потом пришлось вставить еще один цикл и тут все началось. И еще вопрос, по моему if и selected неправильно записаны, selected должно отрабатывать при выполнении if?

<?php foreach($products as $product){
            echo    "<tr>
                        <td>{$this->escapeHtml($product->getId())}</td>
                        <td>{$this->escapeHtml($product->getName())}</td>
                        <td>{$this->escapeHtml($product->getPrice())}</td>";
            echo        '<td><select class="form-control">'
                        .foreach ($form as $element){ 
                            <option value='$element->get('id')' 
                                if ($product->getCategoryId() == $element->getId())' selected="selected" >
                                    {$this->escapeHtml($element->getParentId())}
                            </option>'
                        }
                        '</td>';
            echo        '<td>{$this->escapeHtml($product->getDescription())}</td>
                        <td>{$this->escapeHtml($product->getImage())}</td>';
            echo        '<td><a class="btn btn-success" href="'.$this->url('admin/products', array('action' => 'edit', 'id' => $product->getId())).'"><span class="glyphicon glyphicon-pencil" ></span> Редактировать</a>&nbsp;&nbsp;
                             <a class="btn btn-danger" href="'.$this->url('admin/products', array('action' => 'delete', 'id' => $product->getId())).'" onclick="if(confirm(\'Удалить запись?\'))  {document.location = this.href;} return false;"><span class="glyphicon glyphicon-remove"></span> Удалить</a>
                        </td>
                    </tr>';
        }
        ?>

Answer 1

[Shaks]

<table>
    <?php foreach($a as $b):?>
        <tr>
            <td><?php echo $b?></td>
        </tr>
    <?php endforeach;?>
</table>

Этим примером я хочу сказать, что не нужно в пхп файле генерить хтмл, нужно в хтмл шаблоне делать вставки пхп. Иначе дрочится с кавычками будете постоянно. Да и подход ваш еще в духе пхп3, а щас какбы 2017й год на дворе

Comments

[Николай Громов]

а что можете сказать по поводу if и selected?

[Shaks]

вы действительно думаете ктото будет разбираться в этом гавне?

[Николай Громов]

Shaks: вот исправил вроде можете посмотреть if и selected

<?php foreach($products as $product):?>
            <tr>
                <td><?php echo $this->escapeHtml($product->getId())?></td>
                <td><?php echo $this->escapeHtml($product->getName())?></td>
                <td><?php echo $this->escapeHtml($product->getPrice())?></td>
                <td>
                    <select class="form-control">
                        <?php foreach ($form as $element):?> 
                            <option value="$element->get('id')" <?php if ($product->getCategoryId() == $element->getId()) { echo 'selected="selected"'}?> >
                                <?php echo $this->escapeHtml($element->getParentId())?>
                            </option>
                        <?php endforeach;?>
                    </select>
                </td>
                <td><?php echo $this->escapeHtml($product->getDescription())?></td>
                <td><?php echo $this->escapeHtml($product->getImage())?></td>
                <td>
                    <a class="btn btn-success" href="<?php $this->url('admin/products', array('action' => 'edit', 'id' => $product->getId()))?>"><span class="glyphicon glyphicon-pencil" ></span> Редактировать</a>&nbsp;&nbsp;
                    <a class="btn btn-danger" href="<?php $this->url('admin/products', array('action' => 'delete', 'id' => $product->getId()))?>" onclick="if(confirm(\'Удалить запись?\'))  {document.location = this.href;} return false;"><span class="glyphicon glyphicon-remove"></span> Удалить</a>
                </td>
            </tr>
        <?php endforeach;?>

[Shaks]

Николай Громов
используйте тернарный оператор, вот так:

<option value="<?php echo $element->get('id')?>" <?php echo $product->getCategoryId() == $element->getId() ? "selected=\"selected\"" : ''?> >
                        <?php echo $this->escapeHtml($element->getParentId())?>
                    </option>

[Николай Громов]

Shaks: Спасибо!