|
| |
|
|
| |
для: Batis
(25.12.2008 в 15:10)
| | | Дело в том, что писать код нужно так, что бы было понятно не только Вам, но и всем остальным. Стиль - вещь интимная, есть несколько стандартов стилей, и Вы вправе выбрать любой или использовать свой собственный. Но несколько общих правил, особенно на этом форуме, желательно соблюдать. Основоные:
1.Фигурные скобки нужно распологать ниже выражения. Вот так:
<?
//if ($result == 'true') {echo "<p>Ваш урок успешно обновлен!</p>";}
//else {echo "<p>Ваш урок не обновлен!</p>";}
if ($result == 'true')
{
echo "<p>Ваш урок успешно обновлен!</p>";
}
else
{
echo "<p>Ваш урок не обновлен!</p>";
}
|
а в данном случае лучше обойтись совсем без них:
<?
if ($result == 'true')
echo "<p>Ваш урок успешно обновлен!</p>";
else
echo "<p>Ваш урок не обновлен!</p>";
|
2. Выражение, находящееся внутри другого нужно смещать вправо табулятором. Вот так:
<?
if (isset($title) && isset($meta_d) && isset($meta_k) && isset($date)
&& isset($description) && isset($text) && isset($author))
{
$result = mysql_query ("UPDATE lessons SET title='$title', meta_d='$meta_d',
meta_k='$meta_k', date='$date', description='$description', text='$text',
author='$author' WHERE id='$id'");
if ($result == 'true')
echo "<p>Ваш урок успешно обновлен!</p>";
else
echo "<p>Ваш урок не обновлен!</p>";
}
|
а закрывающую фигурную скобку ставить точно под открывающей.
3. Не писать длинных строк, использовать переносы.
Вот и всё впринципе. | |
| |
|
|
| |
|
|
| |
для: Trianon
(25.12.2008 в 14:33)
| | | Если бы мне оказывали такие медвежьи услуги почаще, я бы давно вырос из чайников ))). Вопрос в том, как автор распорядится полученными знаниями. | |
| |
|
|
| |
|
|
| |
для: Batis
(25.12.2008 в 15:10)
| | | например для меня криво вот это
я обычно использую and
или например
а надо скорее $result == true
а еще проще if($result)
и (для перфекционистов-шизофреников :) вместо
UPDATE lessons SET title='$title'
|
я пишу
UPDATE `lessons` SET `title`='$title'
...
|
| |
| |
|
|
| |
|
|
| |
для: Николай2357
(25.12.2008 в 07:54)
| | | Спасибо огромное!!! Вы действительно обладаете знаниями :)))
Ещё вопрос что означает кривой стиль? имеются ли какие нибудь стили кода? вернее стили написания кода?
я правильно понял? | |
| |
|
|
| |
|
|
| |
для: Николай2357
(25.12.2008 в 14:29)
| | | нет, если Вам хочеться решить конкретно эту проблему автора - ради бога..
Медвежью услугу Вы ему в этом окажете, совершенно однозначно. | |
| |
|
|
| |
|
|
| |
для: Trianon
(25.12.2008 в 13:15)
| | | Всё я правильно рассматриваю. Переменная $description у автора не определена, по причине ошибки в именах элементов. А по сему блок с запросом на UPDATE вообще не включается. А если бы он включался, то просто в поле description прописался бы null... Разве не так? | |
| |
|
|
| |
|
|
| |
для: Николай2357
(25.12.2008 в 09:08)
| | | Вы пытаетесь рассматривать не тот скрипт , на который жалуется автор. | |
| |
|
|
| |
|
|
| |
для: Trianon
(25.12.2008 в 08:58)
| | | Там до запроса дело вообще не доходит. Код написан в таком кривом стиле, что не сразу и поймёшь... А разве помешал бы null при апдейте? По моему просто записалась бы пустота.... | |
| |
|
|
| |
|
|
| |
для: Batis
(25.12.2008 в 03:20)
| | | потому что ниже ( в операторе UPDATE) у Вас остались неопределенные переменные. | |
| |
|
|
| |
|
|
| |
для: Batis
(25.12.2008 в 01:04)
| | | Вот тут у Вас дырка. Имена элементов одинаковые, а значит одного из них нет.
<label>Введите краткое описание уроков с тэгами абзацев урока:
<br> <textarea name="text" id="description" cols="40" rows ="5" /> "$myrow[description]" </textarea>
</label>
</p>
<p>
<label>Введите полный текст урока с тэгами:
<br> <textarea name="text" id="text" cols="40" rows ="20" /> "$myrow[text]" </textarea>
</label>
|
| |
| |
|
|
|
