ImpAnonym
@ImpAnonym
Lorem ipsum dolor sit amet

Как оптимизировать код, сделать профессиональнее?

$tableHTML = '<div class="scaleInTop">' . "\n";
$tableHTML .= '<div class="borderedBlock">' . "\n";
$tableHTML .= '<table class="table table-bordered align-middle">' . "\n";
$tableHTML .= '        <tr class="tableGreen text-center">' . "\n";
$tableHTML .= '            <td colspan="3">Карточка торговой точки</td>' . "\n";
$tableHTML .= '            <td>Дата создания: ' . date('d.m.Y', strtotime($callCompanyData['result']['DATE_CREATE'])) . '</td>' . "\n";
$tableHTML .= '            <td>Дата изменения: ' . date('d.m.Y H:i:s', strtotime($callCompanyData['result']['DATE_CREATE'])) . '</td>' . "\n";
$tableHTML .= '        </tr>' . "\n";
$tableHTML .= '        <tr class="tableGrey">' . "\n";
$tableHTML .= '            <td>Наименование Компании</td>' . "\n";
$tableHTML .= '            <td colspan="4">' . (!empty($callCompanyData['result']['TITLE']) ? $callCompanyData['result']['TITLE'] : ' ') . '</td>' . "\n";
$tableHTML .= '        </tr>' . "\n";
$tableHTML .= '        <tr class="tableGrey">' . "\n";
$tableHTML .= '            <td>Торговая точка</td>' . "\n";
$tableHTML .= '            <td colspan="4">' . (!empty($callCompanyData['result']['UF_CRM_1587463057321']) ? $callCompanyData['result']['UF_CRM_1587463057321'] : ' ') . '</td>' . "\n";
$tableHTML .= '        </tr>' . "\n";
$tableHTML .= '        <tr class="tableGrey">' . "\n";
$tableHTML .= '            <td>Телефон</td>' . "\n";
$tableHTML .= '            <td colspan="2">' . (!empty($collectedPhones) ? $collectedPhones : ' ') . '</td>' . "\n";
$tableHTML .= '            <td>E-Mail</td>' . "\n";
$tableHTML .= '            <td colspan="2">' . (!empty($collectedEmails) === true ? $collectedEmails : ' ') . '</td>' . "\n";
$tableHTML .= '        </tr>' . "\n";
$tableHTML .= '        <tr class="tableGrey">' . "\n";
$tableHTML .= '            <td>Фактический адрес</td>' . "\n";
$tableHTML .= '            <td colspan="4">' . (!empty($callCompanyData['result']['UF_CRM_1587463057321']) ? $callCompanyData['result']['UF_CRM_1587463057321'] : ' ') . '</td>' . "\n";
$tableHTML .= '        </tr>' . "\n";
$tableHTML .= '</table>' . "\n";
$tableHTML .= '</div>' . "\n";
  • Вопрос задан
  • 96 просмотров
Решения вопроса 1
FanatPHP
@FanatPHP
Чебуратор тега PHP
Как у тебя только в глазах не рябит от всех этих знаков препинания
1. Не HTML писать внутри РНР, а наоборот - РНР внктри HTML.
2. Использовать оператор сведения с null
3. Экранировать вывод по умолчанию
т.е
<tr class="tableGrey">
    <td>Фактический адрес</td>
    <td colspan="4"><?= htmlspecialchars($callCompanyData['result']['UF_CRM_1587463057321']) ?? ' ') ?></td>
</tr>

3. Научиться пользоваться шаблонизатором Twig, т.е.
<tr class="tableGrey">
    <td>Фактический адрес</td>
    <td colspan="4">{{ callCompanyData.result.UF_CRM_1587463057321 ?? ' ' }}</td>
</tr>


Чтобы записать этот вывод в переменную надо пользоваться функциями буферизации вывода
Ответ написан
Пригласить эксперта
Ваш ответ на вопрос

Войдите, чтобы написать ответ

Войти через центр авторизации
Похожие вопросы