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

Question

[Lorem Ipsum]

$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";

Comments

[Денис Юрьев]

это троллинг какой то?

[Lorem Ipsum]

Денис Юрьев, нет, почему?

Answer 1

[FanatPHP]

Как у тебя только в глазах не рябит от всех этих знаков препинания
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>

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

Comments

[Lorem Ipsum]

1. Не HTML писать внутри РНР, а наоборот - РНР внктри HTML.

Но мне нужно заносить всё в одну переменную...
Я потом в массив пихаю, преобразую в JSON и кидаю в ответ на ajax-запрос.

[DevMan]

Lorem Ipsum, и как этому мешает написанное?

[FanatPHP]

ob_get_contents спасёт отца русской демократии

[Lorem Ipsum]

У Вас два пункта №3.

А вот зачем мне экранировать вывод (htmlspecialchars)? Это в пункте №3 который первый.

[FanatPHP]

HTML - это определенным образом структурированная информация
если мы в нее добавляем данные неизвестного формата, то их надо соответствующим образом оформлять
чтобы случайно не испортить исходное форматирование.
в данном случае ты выводишь данные в HTML. Значит надо форматировать все символы, которые могут оказаться служебными в HTML.

[Lorem Ipsum]

FanatPHP, А, ну да.