質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.32%

  • PHP

    21364questions

    PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

PHPのformからうまく値を反映できません。

解決済

回答 2

投稿

  • 評価
  • クリップ 0
  • VIEW 175

GeekTK

score 4

多分PHPのformからうまく値を反映できていないので、phpファイルの$denom = 2.0 * $this->a;の部分がdevision by zero というエラーが出ます。全然何をすれば良いか分からずもがいています。回答お願いします。

<!DOCTYPE html>
<html lang="en">
  <head>
    <title> Answer </title>
    <meta charset="utf-8"/>
    <style>
    </style>
  </head>

<body>

<?php
class Equation{
  private $a;
  private $b;
  private $c;

  public function __construct($a, $b, $c){
    $this->a = (int)$a;
    $this->b = (int)$b;
    $this->c = (int)$c;
  }

  public function determination(){
      $root_part = sqrt($this->b * $this->b - 4.0 * $this->a * $this->c);
      if ($root_part>0){
        return 2;
      }elseif($root_part==0){
        return 1;
      }else{
        return 0;
      }
    }

      public function posans(){
        $root_part = sqrt($this->b * $this->b - 4.0 * $this->a * $this->c);
        $denom = 2.0 * $this->a;
        $pos = (-$this->b + $root_part) / $denom;
        return $pos;
    }
      public function negans(){
        $root_part = sqrt($this->b * $this->b - 4.0 * $this->a * $this->c);
        $denom = 2.0 * $this->a;
        $neg = (-$this->b - $root_part) / $denom;
        return $neg;
    }
  }



function Equation(){
  $a = $_GET["a"];
  $b = $_GET["b"];
  $c = $_GET["c"];

  $denom1 = 2.0 * $a;
  if (($b * $b - 4.0 * $a * $c)>=0){
    $root_part1 = round(sqrt($b * $b) - 4.0 * $a * $c,2);
  }else{
    $root_part1 = round(sqrt(abs($b * $b)) - 4.0 * $a * $c,2);
  }

  $fomula = new Equation(a, b, c);
  $ans1 = $fomula->posans();
  $ans2 = $fomula->negans();

  if ($fomula->determination() === 2){
    printf($ans1 , $ans2);
  }elseif($fomula->determination() === 1){
    printf($ans1);
  }elseif($fomula->determination() === 0){
    printf(-$b / $denom1, "+", $root_part1 / $denom1, "i");
    printf(-$b / $denom1, "-", $root_part1 / $denom1, "i");
    printf("Imaginary parts are rounded off to two decial place");
  }
}

Equation();

?>
</body>
</html>
<!DOCTYPE html>
<html lang="en">
  <head>
    <title> Quadratic Equation Solver </title>
    <meta charset="utf-8"/>
  </head>
<form action="quadratic.php" method="get">
  <body>
    <h1> Quadratic Equation Solver </h1>

    <p>
      This program calculates quadratic equation. First I need some data.
    </p>

    <h2>Throw Parameters</h2>

    <div class="in_block">

      <p>
        Quadratic Equation<br />
        ax<sup>2</sup>+bx+c=0
      </p>


      <p><input type="text" name="a" /></p>
      <p><input type="text" name="b" /></p>
      <p><input type="text" name="c" /></p>
      <p><input type="submit" value="calculates" /></p>

</div>
</form>


  </body>
</html>
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • mts10806

    2018/05/14 06:57

    Equation()の冒頭でvar_dump($_GET);した結果を教えてください。想定通りの情報がきていますか?また、Equationクラスをnewするときの引数が変数ではなく定数になっています。クラスと関数を別々に作った理由がよく分かりませんが、同名にするのはあまり良くありません。クラス内にまとめるか、別名にした方が良いでしょう。

    キャンセル

回答 2

checkベストアンサー

+1

かなかなー?

      :
  $fomula = new Equation($a, $b, $c);    // ←ここ、$が抜けてる
  $ans1 = $fomula->posans();
  $ans2 = $fomula->negans();
      :

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

0

このコードの問題はそもそも変数をキャストして扱っていることにあります。
下記、コンストラクタ部分。

  public function __construct($a, $b, $c){
    $this->a = (int)$a;
    $this->b = (int)$b;
    $this->c = (int)$c;
  }

なぜキャストが問題かというと「どんな情報も数値として扱わせてしまう」からです。

例として、下記は全て出力結果は「0」となります。

$a = "a";
echo (int)$a;
$b = "あ";
echo (int)$b;
$c = null;
echo (int)$c;
$d = array();
echo (int)$d;


※$d つまり配列の場合は何か値が設定されていれば「1」となります

luckerさんの回答と私のコメントでも指摘しているように、
引数で変数を渡していないことは問題として1つあるのですが、
本来は存在しないはずの定数a,b,cがそのまま素通りされているのは
キャストしているからというのもあります。

値が定義されていない変数の値はnullなので上記の検証コード上では「0」となります。
0のまま扱われるので今回の現象が起きています。

あと定数を定義せずに利用しているのでNoticeレベルのエラーが出ているはずです。
はじめの内(または開発途中段階)はエラー表示はOnにしておきましょう。

入力値が0だったり、数字以外の文字列が入ってきた場合も全て「0」として扱われます。

つまり、キャストして変換するのではなく、入力値によってその後の処理を行うかどうかの入力値検証をされた方が、コードとしては理想かなと思います。
今回の場合ですと、数値ではないもの・数値であっても0であれば入力エラーとする 形ですね。

ご検討ください。

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.32%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

同じタグがついた質問を見る

  • PHP

    21364questions

    PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。