2011-09-14 4 views
0

Просто ищите лучшее решение. Я извлекаю все записи из mysql. Затем я отправлю эти данные по электронной почте. Его работа прекрасна. Есть ли лучшее решение? благодаряИзвлечение записей из mysql

$query = "SELECT * FROM order_details WHERE order_id = '".$data['order_id']."'"; 
while($row = mysql_fetch_array($result,MYSQL_ASSOC)) { 
    $message_admin .= "<table style='text-align:center;' border='1' cellspacing='0' cellpadding='7'> 
     <tr> 
      <td>Source</td> 
      <td>".$data['source']."</td> 
     </tr> 
     <tr> 
      <td>Email</td> 
      <td>".$data['email']."</td> 
     </tr> 
     <tr> 
      <td>Message</td> 
      <td>".$data['message']."</td> 
     </tr> 
     <tr> 
      <td></td> 
      <td></td> 
     </tr> 
    </table>"; 
    } 
+0

Лучше в каком смысле? И вы хотите опубликовать весь код, включая SQL-запрос. – TJHeuvel

+0

Почему, что случилось с этим? –

+0

Зависит от того, что вы подразумеваете под «лучше» – symcbean

ответ

0

Вы выиграли бы от: http://us.php.net/manual/en/language.types.string.php#language.types.string.syntax.heredoc

Вместо сложной манипуляции строки вы делаете сейчас.

Вы устанавливаете $ row, но используете $ data?

+0

Когда я печатаю эту переменную. 'echo $ message_admin'. Он отобразил данные. Я передаю эту переменную по почте. И я получаю пустое сообщение. Что не так с этим кодом? Спасибо, что указали мне ошибку '$ row'. –

+0

Почтовая функция '$ headers = 'MIME-Version: 1.0'. "\ Г \ п"; \t \t \t \t \t \t $ headers. = 'Content-type: text/html; charset = iso-8859-1 '. "\ Г \ п"; \t \t \t \t \t \t почта ($ to_client, $ subject_client, $ message_admin, $ заголовки); ' –

+0

Я не уверен, что это неправильно. Сначала попробуйте отправить простое тестовое сообщение с php, а затем введите код, который вы написали. – Ariel

Смежные вопросы