Версия для печати темы
Нажмите сюда для просмотра этой темы в оригинальном формате
Форум программистов > PHP: Общие вопросы > Прошу проверить правильность написания кода.


Автор: Nightwalker 16.7.2006, 16:11
Всем привет!Проверьте пожалуйтса правильно ли написан код,и нету ли в нем чего-нибудь лишнего и ненужного?

Код

<html>
<head>
<?
Error_Reporting(E_ALL & ~E_NOTICE);
include "102938/config.php";
if (count($_GET)==0) $_GET['id_page']=1;
$_GET['id_page'] = (int) $_GET['id_page'];
$_GET['id_article'] = (int) $_GET['id_article'];
$_GET['news'] = (int) $_GET['news'];
if (!empty($_GET['news']))
 {
 $query="SELECT * FROM ".$pref."news WHERE id=".$_GET['news'];
 $res = mysql_query($query);
  if ($res && mysql_num_rows($res)>0)
    {
    $page = mysql_fetch_array($res);
    $title = $page['html_title'];
    $description = $page['html_description'];
    $keywords = $page['html_keywords'];
    }
 }
elseif (!empty($_GET['id_article']))
 {
 $query="SELECT * FROM ".$pref."articles WHERE id_article=".$_GET['id_article'];
 $res = mysql_query($query);
  if ($res && mysql_num_rows($res)>0)
    {
    $page = mysql_fetch_array($res);
    $title = $page['html_title'];
    $description = $page['html_description'];
    $keywords = $page['html_keywords'];
    }
 }
elseif (!empty($_GET['id_page']))
 {
 $query="SELECT * FROM ".$pref."artpage WHERE id_artpage=".$_GET['id_page'];
 $res = mysql_query($query);
  if ($res && mysql_num_rows($res)>0)
    {
    $page = mysql_fetch_array($res);
    $title = $page['html_title'];
    $description = $page['html_description'];
    $keywords = $page['html_keywords'];
    }
 }
?>
<base href="<?= $base ?>" />
<title><?= $title;?></title>
  <META HTTP-EQUIV="Content-Type" CONTENT="text/html; charset=windows-1251">
  <META NAME="keywords" CONTENT="<?= $keywords; ?>">
  <META NAME="description" CONTENT="<?= $description; ?>">
  <link rel="StyleSheet" type="text/css" href="main.css">
  <link rel="SHORTCUT ICON" href="cska.ico">
</head>
<link rel="SHORTCUT ICON" href="cska.ico">
<body leftmargin="0" topmargin="0" marginwidth="0" marginheight="0">
<table width="100%" border="0" cellspacing="0" cellpadding="0" height="100%">
  <tr>
    <td height="183">
      <table  border="0" cellpadding="0" cellspacing="0" width="100%">
        <tr>
          <td width="555"><img src="img/header.jpg" border="0" alt="Не официальный сайт ПФК ЦСКА Москва"></td>
          <td  bgcolor="#BBBBBB" width="100%"></td>
          <td width="225" valign="top" background="img/right.gif">
            <table  border="0" cellpadding="0" cellspacing="0" width="225" id="txtgame">
              <tr>
                <td  height="2"></td>
              </tr>
              <tr>
                <td  background="img/rightzag.gif" height="21">
                  <div id="zgr">информация</div>
                </td>
              </tr>
              <tr>
                <td>&nbsp; </td>
              </tr>
            </table>
          </td>
        </tr>
      </table>
    </td>
  </tr>
  <tr>
    <td>
      <table border="0" cellpadding="0" cellspacing="0" width="100%" height="100%" bgcolor="#BBBBBB">
        <tr>
          <td valign="TOP">
            <!--left-->
            <table  border="0" cellpadding="0" cellspacing="0" width="150" background="img/left.gif">
              <tr>
                <td height="15"><img src="img/top_menushka.gif"></td>
              </tr>
              <? include "menu.php"?>
              <tr>
                <td  height="21"><img src="img/downs_menushka.gif" width="150"></td>
              </tr>
            </table>
            <!--end_left-->
          </td>
          <td valign="TOP" width="100%">
            <!--center-->
            <table  border="0" cellpadding="0" cellspacing="0" width="100%">
              <tr>
                <td  height="49">
                  <table  border="0" cellpadding="0" cellspacing="0" width="100%">
                    <tr>
                      <td width="106"><img src="img/zagolovoktxtleft.gif"></td>
                      <td   width="184" background="img/fontxtz.gif" valign="bottom" align="center">
                        <div id="zg"> магазин</div>
                      </td>
                      <td width="81"><img src="img/shoose.jpg"></td>
                      <td background="img/zagolovoktxtback.gif" >&nbsp;</td>
                      <td width="30"><img src="img/zagolovoktxtright.gif"></td>
                    </tr>
                  </table>
                </td>
              </tr>
              <tr>
                <td valign="top" width="100%">
                  <table  border="0" cellpadding="0" cellspacing="0" width="98%" align="center">
                    <tr>
                      <td valign="top">
                        <div id="center"> <? include "get.php" ?></div>
                      </td>
                    </tr>
                  </table>
                </td>
              </tr>
            </table>
            <!--end_center-->
          </td>
          <td background="img/right.gif" valign="TOP">
            <!--right-->
            <table  border="0" cellpadding="0" cellspacing="0" width="225">
              <tr>
                <td  background="img/rightzag.gif" height="21">
                  <div id="zgr">реклама</div>
                </td>
              </tr>
              <tr>
                <td>1</td>
              </tr>
            </table>
            <!--end_right-->
          </td>
        </tr>
         <? include "downs.php"?>
      </table>
    </td>
  </tr>
</table>
</body>
</html>


 

Автор: st1ng3r 16.7.2006, 16:53
Так, признавайся откуда код скатал? smile Потому что сразу видно что не твой.
И зачем спрашивать других, когда ты сам можешь сохранить этот код в файле с расширением .php и закинуть на сервак да протестить!
Что сложного? Если что-то конкретно не запахает, говори... 

Автор: Nightwalker 16.7.2006, 18:48
не пахает присоеденённый файл get 

Автор: Guedda 16.7.2006, 19:39
Цитата(Nightwalker @  16.7.2006,  19:48 Найти цитируемый пост)
присоеденённый файл get  

Что есть присоединённый файл? 

Автор: Nightwalker 16.7.2006, 20:28
Да 

Автор: Shark 16.7.2006, 20:53
Nightwalker, покажи мне пальце... где именно файл(!) get ты присоединяешь... на скока мне извесно это далеко не файл 

Автор: Nightwalker 16.7.2006, 21:31
Скажите как присоеденить файл где написано id="center"?
 

Автор: Shark 16.7.2006, 21:37
Nightwalker, ты про эту строчку <div id="center"> <? include "get.php" ?></div> ????
если да, то она должна выглядеть так 
Код

<?
include "get.php"; // точку с запятой надо ставить
?>
  

и как тебе уже говорили, научись писать код сам.....
и тока тогда ты будешь находить ошибки настолько простые 

Powered by Invision Power Board (http://www.invisionboard.com)
© Invision Power Services (http://www.invisionpower.com)