Amirhosein Al Amirhosein Al - 6 months ago 8
PHP Question

On form submission the browser loads the php file that had the code for handling the form

I'm writing my first php code... I'm trying to submit a form.

I have two php files. index.php(which contains the form) and process.php(which contains the method that handles the form).

But on form submission, the browser heads to process.php, so nothing is displayed.
I'm trying to echo the result in index.php .

And keep in mind this is my very first php code... :-)

This is index.php

<!DOCTYPE html>
<html>
<?php
include 'process.php';
$newletter1 = new newsletter();
?>
<head>
<meta charset="utf-8">
<title></title>
</head>
<body>
<form action="process.php" method="post">
<input type="text" name="email" placeholder="Your Email Address..."><br><br>
<input type="submit">
</form>
<h4><?php $newletter1 -> abc(); ?></h4>
</body>
</html>


And this is process.php

class newsletter
{

public function abc()
{
if (isset($_POST["email"])) {
$input = $_POST["email"];
if (empty($input)) {
echo "Please provide an email address!";
}else{
echo "Thanks for subscribing " . $input;
}
}else{
echo "ELSE is running...";
}
}

}

Answer

Your script process.php is just a class definition.

A class does nothing unless it is instantiated and a method called.

As you are including it and instantiating it in your index.php I would suggest changing your <form> tag so it runs itself, leaving href="" will do that.

<?php
  // run this only if we are being set info by the user
  // so not when the form is first loaded.
  $to_show_later = '';
  if ($_SERVER['REQUEST_METHOD'] == 'POST' ) {
      include 'process.php';
      $newletter1 = new newsletter();
      $to_show_later = $newsletter1->abc();
  }
?>
<!DOCTYPE html>
<html>
<head>
<meta charset="utf-8">
<title></title>
</head>
<body>
    <form action="" method="post">
      <input type="text" name="email" placeholder="Your Email Address..."><br><br>
      <input type="submit">
    </form>
    <h4><?php echo $to_show_later; ?></h4>
</body>
</html>

It's also bad practice to echo directly out of a class method, so change this so it return the data.

class newsletter
{

  public function abc()
  {
    $reply = '';
    if (isset($_POST["email"])) {
      $input = $_POST["email"];
      if (empty($input)) {
        $reply = "Please provide an email address!";
      }else{
        $reply = "Thanks for subscribing " . $input;
      }
    }else{
      $reply = "ELSE is running...";
    }
    return $reply;
  }

}
Comments