Представленип о гкоде

Карочь хочу узнать ваше мнение, сколько займет мой код по шкале "гкода" от 1 до 10 да и кто дал оченку, скидываем код, который относиться лишь к 1/10
по этой же шкале
[code]
<?php
$head = 'Диалоги';
require_once ('../inc/func.php');
require_once ('../inc/head.php');

if(!$user['id']) {
header('Location: /');
exit();
}

$vsego = mysql_result(mysql_query("select count(`id`) from `dialogs` where `us1`='".$user['id']."'",0);
if($vsego == 0) {
echo'<div class="block"><b>Ошибка</b><hr />Диалоги отсутсвуют</div>';
require_once ('../inc/end.php');
exit();
}



$in_page = 10;
$a = mysql_query("select `id`,`us2`,`last_id`,`last_time`,`new` from `dialogs` where `us1`='".$user['id']."' order by `id` desc limit $start, $in_page";
while($b = mysql_fetch_assoc($a))
{


$l_msg = mysql_fetch_assoc(mysql_query("select `mess` from `dialogs_msg` where `id`='".$b['last_id']."'");
($b['new'] == 1 ? $color = 'red' : $color = '#fff');


echo '
<table><tr style="border-bottom:1px solid #DAE1E8;background:#fff;">
<td>
'.us($b['us2']).' ['.time($b['last_time']).'] <br />
<a href="/msg/'.$b['us2'].'"><font style="font-size:10px; color:'.$color.';">'.substr($l_msg['mess'], 0, 50).'...</font></a>
</td><td style="padding:9px" align="center" width="5%">
<a href="/msg/'.$b['us2'].'"> Диалог»</a>
</td></tr>
</table>
';

}

if($vsego > $in_page) echo page('/?', $in_page, $start, $vsego);

require_once ('../inc/end.php');
?>
[/code]
[code]<?
$id = guard($_GET['id']);
if($id == $user['id']) {
header('Location: /msg');
exit();
}
echo'<div class="title"><b>Диалог с '.nk($id).'</b></div>';
echo'
<div class="menu" style="font-size:10px; text-align:center;">
<form action="/msg/'.$id.'/add" method="post">
Сообщение:<br/>
<textarea name="mess"></textarea><br>
<input type="submit" value="Отправить"/>
</form>
</div>
';

// dialog(id) - мой диалог с собеседником
$dialog = mysql_fetch_assoc(mysql_query("select `id` from `dialogs` where `us1`='".$user['id']."' and `us2` = '".$id."'");
// dialog2(id) - диалог собеседника со мной
$dialog2 = mysql_fetch_assoc(mysql_query("select `id` from `dialogs` where `us1`='".$id."' and `us2` = '".$user['id']."'");

$total = mysql_result(mysql_query("select count(`id`) from `dialogs_msg` where `dialog_id`='".$dialog['id']."'",0);
if(!$dialog['id'] || $total == 0) {
echo'
<div class="msg"><center>Сообщения отсутсвуют</center></div>';
require_once ('../inc/end.php');
exit();
}

//Чтобы в цикле по 10 раз не вызывать, заранее берем ник, онлайн и лвл
$us1 = us($user['id']);
$us2 = us($id);

$in_page = 10;
$msg_q = mysql_query("select `us`,`mess`,`time`,`new` from `dialogs_msg` where `dialog_id`='".$dialog['id']."' order by `id` desc limit $start, $in_page";
while($msg = mysql_fetch_assoc($msg_q))
{

($msg['new'] == 1 ? $color = 'red' : $color = '#000');
($msg['us'] == $user['id']? $align='left':$align='right');

echo '
<div style="text-align:'.$align.';">
<span class="msg">
<font color="'.$color.'">'.$msg['mess'].'</font><br />
<div style="font-size:10px; text-align:'.$align.';"><br />'.($msg['us'] == $user['id']? $us1:$us2).'['.times($msg['time']).']</div>
</span>
</div>
';

if($msg['new'] == 1 && $msg['us'] != $user['id']){

//Отмечаю у себя как прочитано
mysql_query("UPDATE `dialogs_msg` SET `new` = '0' WHERE `dialog_id`='".$dialog['id']."' AND `us` = '".$id."'";
mysql_query("UPDATE `dialogs` SET `new` = '0' WHERE `id`='".$dialog['id']."'";
//Отмечаю у собеседника о том что я прочел
mysql_query("UPDATE `dialogs_msg` SET `new` = '0' WHERE `dialog_id`='".$dialog2['id']."' AND `us` = '".$id."'";
mysql_query("UPDATE `dialogs` SET `new` = '0' WHERE `id`='".$dialog2['id']."'";
}

}

if($total > $in_page) echo page('/msg/'.$id.'&', $in_page, $start, $total);
echo'<div class="main"><a href="/msg"> К списку диалогов</a></div>';
?>
[/code]

Изм. 2 раз. / Посл. изм.

#FWLone 9.56

(22 фев 2015, 17:43)

Переменные более осмысленные создавай. А не: $b.
Желательно еще не давать транслитные имена, типа: $vsego. Считается как ГК. Лучше назвать $total или $count

спасибо, учту

Я конечно начинающий, но по мне то на на троечку !

обновил пост

не юзай лишние echo

echo'<div class="title"><b>Диалог с '.nk($id).'</b></div>';

echo'

<div class="menu" style="font-size:10px; text-align:center;">

<form action="/msg/'.$id.'/add" method="post">

Сообщение:<br/>

<textarea name="mess"></textarea><br>

<input type="submit" value="Отправить"/>

</form>

</div>

';

нужно писать так

echo '<div class="title"><b>Диалог с '.nk($id).'</b></div>

<div class="menu" style="font-size:10px; text-align:center;">

<form action="/msg/'.$id.'/add" method="post">

Сообщение:<br/>

<textarea name="mess"></textarea><br>

<input type="submit" value="Отправить"/>

</form>

</div>';

AOC, спасибо) я с установкой формы не определился, снизу смотриться красивей, но тут в примере я её в начало вставил) а так она до навигации стояла, учту

Изм. 2 раз. / Посл. изм.

#FWLone 9.56

(22 фев 2015, 19:57)

11 из 10