Warning: Cannot use a scalar value as an array in /home/admin/public_html/forum/include/fm.class.php on line 757

Warning: Invalid argument supplied for foreach() in /home/admin/public_html/forum/include/fm.class.php on line 770

Warning: Invalid argument supplied for foreach() in /home/admin/public_html/forum/topic.php on line 737
Форумы портала PHP.SU :: Калькулятор

 PHP.SU

 Программирование на PHP, MySQL и другие веб-технологии
PHP.SU Портал     На главную страницу форума Главная     Помощь Помощь     Поиск Поиск     Поиск Яндекс Поиск Яндекс     Вакансии  Пользователи Пользователи


 Страниц (1): [1]   

> Описание: Не работает калькулятор
Поиск в теме | Версия для печати
ruslanlizogub
Отправлено: 17 Июня, 2013 - 20:54:29
Post Id


Новичок


Покинул форум
Сообщений всего: 24
Дата рег-ции: Июнь 2013  


Помог: 0 раз(а)

[+]
Всем привет.
Вот составил простенький калькулятор, но не могу понять где допускаю ошибку.Не отображается вторая форма с результатом.Просьба тыкнуть носом в ошибку.
PHP:
скопировать код в буфер обмена
  1. <?PHP
  2. // Функция вывода формы ввода.
  3. function show_form()
  4. {
  5.         echo '<html>';
  6.         echo '<head>';
  7.         echo '<title>Сумматор</title>';
  8.         echo '</head>';
  9.         echo '<body>';
  10.         echo '<form action="sum.php" method="post">';
  11.         echo '<input type="text" name="a" />';
  12.         echo '<select name="operation" size=1>';
  13.         echo '<option value="plus">+</option>';
  14.         echo '<option value="minus">-</option>';
  15.         echo '<option value="division">/</option>';
  16.         echo '<option value="multiplication">*</option>';
  17.         echo '</select>';
  18.         echo '<input type="text" name="b" />';
  19.         echo '<input type="submit" value="=" />';
  20.         echo '</form>';
  21.         echo '</body>';
  22.         echo '</html>';
  23. }
  24. // Функция вывода результата.
  25. function show_result($a, $b, $operation)
  26. {
  27.         $operation = 'operation';
  28.         switch ($operation) {
  29.                 case 'plus':
  30.                         $result = $a + $b;
  31.                         break;
  32.                 case 'minus':
  33.                         $result = $a - $b;
  34.                         break;
  35.                 case 'division':
  36.                         if ($b == 0) {
  37.                                 echo 'Делить на ноль не надо';
  38.                         }
  39.                         else {
  40.                         $result = $a / $b;
  41.                         }
  42.                         break;
  43.                 case 'multiplication':
  44.                         $result = $a * $b;
  45.                         break;
  46.         }
  47.         return;
  48.         echo '<html>';
  49.         echo '<head>';
  50.         echo '<title>Сумматор</title>';
  51.         echo '</head>';
  52.         echo '<body>';
  53.         echo '<form action="sum.php" method="post">';
  54.         echo '<input type="text" name="a" value = " ' . $a . ' " />';
  55.         echo '<select name="operation" size=1>';
  56.         echo '<option value="plus">+</option>';
  57.         echo '<option value="$minus">-</option>';
  58.         echo '<option value="division">/</option>';
  59.         echo '<option value="multiplication">*</option>';
  60.         echo '</select>';
  61.         echo '<input type="text" name="b" value = " ' . $b . ' " />';
  62.         echo '<input type="submit" value="=" />';
  63.         echo "<b>$result</b>";
  64.         echo '</p>';
  65.         echo '<p>';
  66.         echo '<a href="sum.php">Хочу суммировать еще</a>';
  67.         echo '</p>';
  68.         echo '</body>';
  69.         echo '</html>';
  70. }
  71.  
  72. // Точка входа.
  73.  
  74. // Показываем результат операции или форму ввода.
  75. if (isset($_POST['a']) and isset($_POST['b']) and isset($_POST['operation']))
  76. {
  77.         show_result($_POST['a'], $_POST['b'], $_POST['operation']);
  78. }
  79. else
  80. {
  81.         show_form();
  82. }
  83. ?>
  84.  
 
 Top
IllusionMH
Отправлено: 17 Июня, 2013 - 21:02:09
Post Id



Активный участник


Покинул форум
Сообщений всего: 4254
Дата рег-ции: Февр. 2011  
Откуда: .kh.ua


Помог: 242 раз(а)
ruslanlizogub, после return; жизни нет
 
 Top
ruslanlizogub
Отправлено: 17 Июня, 2013 - 21:14:00
Post Id


Новичок


Покинул форум
Сообщений всего: 24
Дата рег-ции: Июнь 2013  


Помог: 0 раз(а)

[+]
IllusionMH пишет:
ruslanlizogub, после return; жизни нет


Благодарю, return; я убирал, не появляется результат.
А логика построения верная, или нужно что-то изменить?
 
 Top
MADONZY
Отправлено: 18 Июня, 2013 - 03:50:02
Post Id


Гость


Покинул форум
Сообщений всего: 104
Дата рег-ции: Июнь 2013  


Помог: 0 раз(а)
ruslanlizogub пишет:
А логика построения верная, или нужно что-то изменить?

Нет, логики в твоем коде ноль...
Твои ошибки:
1) html код в функциях используется очень редко! И то для того что-бы просто указать тип и структуру элемента (тэга). А весь сайт в функциях не пишут Улыбка
2) ты слишком часто вызываешь конструкцию echo! В php строки будут восприниматься до тех пор, пока найдется точка с запятой.
3) задавать аргумент который нигде не нужен (я про $operation = 'operation') не нужно. Так как это ничего тебе не даст. Сделай так: $operation = $_POST['operation'];
4) надеюсь обработчик 'sum.php' это этот-же файл? Ато иначе понту нет дальше что-то объяснять Улыбка
5) конструкция return ""; обрывает код и все что далее пропускается, так как функция поняла что ей нужно отдать. У тебя около return ничего не стоит...что она тебе должна отдать? Поставь return $result; И удали все что ниже него.

А ещё лучше, удали весь код и напиши все по-человечести с полученными данными через $_POST...if($_SERVER['REQUEST_METHOD'] == 'POST') ... вот-так Улыбка
(Добавление)
И ещё, если будешь выводить вот так:
Цитата:
echo '<a href="sum.php">Хочу суммировать еще</a>';

то все отосланные файлы могут повторяться, так-что каждый раз после отправки формы добавляй: header("Location: sum.php"); таким образом ты будешь возвращаться в GET)
 
 Top
AlexAnder
Отправлено: 18 Июня, 2013 - 06:45:55
Post Id



Частый посетитель


Покинул форум
Сообщений всего: 915
Дата рег-ции: Авг. 2012  
Откуда: Россия


Помог: 34 раз(а)

LURK =\ пишет:
Спойлер (Отобразить)

(Отредактировано автором: 18 Июня, 2013 - 06:46:52)



-----
Оказывается, недостаточно читать справочники, чтобы правильно писать коды. sadex ©

Форумы стали местом обучения программированию, а не решения трудных вопросов. KingStar ©
 
 Top
ruslanlizogub
Отправлено: 19 Июня, 2013 - 00:48:21
Post Id


Новичок


Покинул форум
Сообщений всего: 24
Дата рег-ции: Июнь 2013  


Помог: 0 раз(а)

[+]
MADONZY пишет:
ruslanlizogub пишет:
А логика построения верная, или нужно что-то изменить?

Нет, логики в твоем коде ноль...
Твои ошибки:
1) html код в функциях используется очень редко! И то для того что-бы просто указать тип и структуру элемента (тэга). А весь сайт в функциях не пишут Улыбка
2) ты слишком часто вызываешь конструкцию echo! В php строки будут восприниматься до тех пор, пока найдется точка с запятой.
3) задавать аргумент который нигде не нужен (я про $operation = 'operation') не нужно. Так как это ничего тебе не даст. Сделай так: $operation = $_POST['operation'];
4) надеюсь обработчик 'sum.php' это этот-же файл? Ато иначе понту нет дальше что-то объяснять Улыбка
5) конструкция return ""; обрывает код и все что далее пропускается, так как функция поняла что ей нужно отдать. У тебя около return ничего не стоит...что она тебе должна отдать? Поставь return $result; И удали все что ниже него.

А ещё лучше, удали весь код и напиши все по-человечести с полученными данными через $_POST...if($_SERVER['REQUEST_METHOD'] == 'POST') ... вот-так Улыбка
(Добавление)
И ещё, если будешь выводить вот так:
Цитата:
echo '<a href="sum.php">Хочу суммировать еще</a>';

то все отосланные файлы могут повторяться, так-что каждый раз после отправки формы добавляй: header("Location: sum.php"); таким образом ты будешь возвращаться в GET)

Заработал скрипт, помог 3 совет, все остальные рекомендации учту, огромное спасибоУлыбка)
 
 Top
Поиск в теме | Версия для печати
Страниц (1): [1]
Сейчас эту тему просматривают: 0 (гостей: 0, зарегистрированных: 0)
« Вопросы новичков »


Все гости форума могут просматривать этот раздел.
Только зарегистрированные пользователи могут создавать новые темы в этом разделе.
Только зарегистрированные пользователи могут отвечать на сообщения в этом разделе.
  



Powered by PHP  Powered By MySQL  Powered by Nginx  Valid CSS  RSS

 
Powered by ExBB FM 1.0 RC1. InvisionExBB