Форум PHP программистов > Проверка Кода

[ Поиск ] - [ Пользователи ] - [ Календарь ]

Bugiman666

28.05.2011 - 18:46

Добрый всем вечер! Шапками не кидать, php ток начал изучать!
Суть вопроса в том что сделат так сказать скрипт или хз как назвать он работае вот только правильно написан код?!
Смысл два файла один отправляет данные другой обрабатывает через post проблема была в том что если неотправльть данные а сразу заходить на второй ьыла ошибка вот я намудрилкод
форма отправки


<form action="rezult.php" method="post" name="form" target="_self">
Введите число:
<br>
<input name="name" type="text" size="3" maxlength="100" /><br >
<input name="sumbit" type="submit" value="start" /><br>

вот файл обработчик


<?php
$a = rand(1,100);
$z = isset($name);
if ($z == 1)
{
$name = $_POST['name'];
echo "Вы ввели число $name<br>";
if ($name == $a) 
{
echo "Вы угадали число";
}
else
{
echo "Вы неугадали число";
}
}

?>

Спустя 3 минуты, 2 секунды (28.05.2011 - 17:49) Bugiman666 написал(а):

Смысл скрипта в том что первый файл отправляет число второй проверяет совпадение с рандомным числом, но если переходить сразу на второй то выдавало ошибки я вроди намудрил и исправил. Но походу получился гкод?

Спустя 14 минут, 8 секунд (28.05.2011 - 18:03) inpost написал(а):

Bugiman666
всё можно реализовать на одной странице. Для того, чтобы убедиться, что человек отправил данные из формы, проверяй, была ли нажата кнопка. Кнопка, она же submit, имеет имя любое: name="submit", а в скрипте проверяй, была ли она отправлена: if(!empty($_POST['submit'])) {

Спустя 14 минут, 48 секунд (28.05.2011 - 18:18) Status-X написал(а):

Сообщение самоудалилось!

Спустя 7 минут, 23 секунды (28.05.2011 - 18:25) Basili4 написал(а):

существует но пустой isset даст true но и empty даст true.
Проверять if(!empty($_POST)){ код проверки}

Спустя 47 секунд (28.05.2011 - 18:26) Bugiman666 написал(а):


<?php 
$a = rand(1,3);
if(!empty($_POST['sumbit']))
{
$name = $_POST['name'];
if($name == $a)
{
echo "Вы угадали число!";
}
else
{
echo "Вы не угадали число!";
}
}

?>

как то так? или код ещё плох?

Спустя 2 минуты, 54 секунды (28.05.2011 - 18:29) Basili4 написал(а):

ну тут то кода 10 строк. Это вполне.

Спустя 1 минута, 57 секунд (28.05.2011 - 18:31) Bugiman666 написал(а):

Цитата (Basili4 @ 28.05.2011 - 15:29)

ну тут то кода 10 строк. Это вполне.

Ну код косячный?или это же можно как-то лучше заделать?

Спустя 5 минут, 8 секунд (28.05.2011 - 18:36) learnPHP написал(а):

проверьте мой код тоже на косячность

<?php
   class db {
     public function __constuct($settind){
       $this->connect()
     }
     public function connect(){
       соединиться_с_майэскюэлем($setting['хост']);
     }
   }
?>

никакой ошибки не выдает, не знаю что делать

Спустя 8 минут, 34 секунды (28.05.2011 - 18:44) Status-X написал(а):

Сообщение самоудалилось!

Спустя 14 минут, 3 секунды (28.05.2011 - 18:58) inpost написал(а):

Завтра будет вопрос, как сделать в форме 2 кнопки, вот сразу пример показываю, так как у меня достаточно часто получается так, что 1 форма имеет 2 и более кнопок с разными событиями.

Спустя 4 минуты, 56 секунд (28.05.2011 - 19:03) Status-X написал(а):

Сообщение самоудалилось!

Спустя 4 минуты, 9 секунд (28.05.2011 - 19:07) inpost написал(а):

А какая разница, проверяешь ли отправку данных, или отправку конкретно кнопки? Строчка в своём виде не меняется, а если лень написать лишние 3-4 символа, то программирование - врядли для него!

Спустя 2 минуты, 48 секунд (28.05.2011 - 19:10) Status-X написал(а):

Сообщение самоудалилось!

Спустя 4 минуты, 2 секунды (28.05.2011 - 19:14) inpost написал(а):

Твоё дело считать так, у меня своё мнение по этому поводу. Уместно перехватывать то событие, которые вызывается, а не ловить ЛЮБОЕ событие, которое теоретически могло было быть, если бы случайным образом не появится другое событие.

Спустя 18 минут, 27 секунд (28.05.2011 - 19:33) Status-X написал(а):

Сообщение самоудалилось!

Спустя 6 минут, 11 секунд (28.05.2011 - 19:39) Bugiman666 написал(а):

ещё вопрос хотел вставить сессию


<?php 
session_start(); 
if (!isset($_SESSION['counter'])) $_SESSION['counter']=0;
echo "Вы обновили эту страницу ".$_SESSION['counter']++." раз. ";
echo "<br><a href=".$_SERVER['PHP_SELF'].">обновить"; 
?>

Мы проверяем, есть ли у нас в сессии переменная counter, если нет, то создаем ее со значением 0, а дальше выводим ее значение и увеличиваем на единицу. Увеличенное значение запишется в сессию, и при следующем вызове скрипта переменная будет иметь значение 1, и так далее.

сессия работает кол-во обновлений растёт но проблема в том что сверху ошибка

Warning: session_start(): Cannot send session cookie - headers already sent by (output started at z:\home\test2.ru\www\test.php:9) in z:\home\test2.ru\www\test.php on line 15

Warning: session_start(): Cannot send session cache limiter - headers already sent (output started at z:\home\test2.ru\www\test.php:9) in z:\home\test2.ru\www\test.php on line 15

тоесть орёт на session_start(); может быть причина работе в денвира?

Спустя 11 минут (28.05.2011 - 19:50) Status-X написал(а):

Сообщение самоудалилось!

Спустя 2 минуты, 55 секунд (28.05.2011 - 19:53) Bugiman666 написал(а):

Снимаю шляпу))) спс. я не знал

Спустя 28 минут, 59 секунд (28.05.2011 - 20:22) ИНСИ написал(а):

Bugiman666 ошибка, потому что ты сперва проверяешь, а потом создаешь переменную:

Цитата

$z = isset($name);
if ($z == 1)
{
$name = $_POST['name'];

Это так, к слову. А по коду, я бы так сделал:

if(isset($_POST['name'])) {
	echo 'Вы ввели число ' . $_POST['name'] . '<br>';
	
	if($_POST['name'] == rand(1,100))
		echo 'Вы угадали число';
	else
		echo 'Вы неугадали число';
}

Спустя 32 минуты, 41 секунда (28.05.2011 - 20:54) Bugiman666 написал(а):

velbox,исправил. Темка close

Спустя 4 часа, 7 минут, 33 секунды (29.05.2011 - 01:02) Invis1ble написал(а):

Да, имхо, проверять лучше не кнопку, а сами данные на существование. Тогда нотайсов никогда не получится.

Быстрый ответ:

Здесь расположена полная версия этой страницы.