PHP - Don't load the page if exception is caught - php

Question

I'm new to OO PHP...I'm trying to create a PHP class called MyClass and some methods that should:
validate that parameters are numeric
validate that parameters are defined
if any of the above fail I need to send an exception with explanation
When I call the MyFunction method like this:
$quotation = new Quotations\MyClass();
echo $quotationResult = $quotation -> MyFunction('A', 'B', 2, 'a');
to test if it will throw an exception but for some reason page doesn't load saying - "localhost is currently unable to handle this request". But when I call it like this:
$quotation = new Quotations\MyClass();
echo $quotationResult = $quotation -> MyFunction('A', 'B', 2, 3);
it works fine (I can see blank page).
Why it doesn't allow me to print an exception in the browser?
My script:
<?php
namespace Quotations;
class MyClass {
var $is_number;
var $is_defined;
private $number;
private $defined;
private function isNumeric($w, $c){
if(is_numeric($w) && is_numeric($c)){
$number = true;
}
else{
$number = false;
}
return $number;
}
private function isDefined($t, $f, $w, $c){
if(isset($t, $f, $w, $c)){
$defined = true;
}
else{
$defined = false;
}
return $defined;
}
function MyFunction($to, $from, $weight, $cube) {
try{
if(!$this -> isNumeric($weight, $cube)){
throw new InvalidArgumentException('Arguments are not numeric');
}
if(!$this -> isDefined($to, $from, $weight, $cube)){
throw new BadMethodCallException('Arguments are missing');
}
}catch(InvalidArgumentException $e){
echo 'Caught exception: ', $e->getMessage(), "\n";
}catch(BadMethodCallException $e){
echo 'Caught exception: ', $e->getMessage(), "\n";
}
}
}
?>

Answer 1

Use backslashes before the execption names to look for them in the global namespace. Otherwise, it looks for them in the Quotations namespace.
function MyFunction($to, $from, $weight, $cube) {
try {
if(!$this -> isNumeric($weight, $cube)){
throw new \InvalidArgumentException('Arguments are not numeric');
}
if(!$this -> isDefined($to, $from, $weight, $cube)){
throw new \BadMethodCallException('Arguments are missing');
}
} catch(\InvalidArgumentException $e) {
echo 'Caught exception: ', $e->getMessage(), "\n";
} catch(\BadMethodCallException $e) {
echo 'Caught exception: ', $e->getMessage(), "\n";
}
}

Answer 2

First step is to enable error reporting -
error_reporting(E_ALL);
ini_set("display_errors", 1);.
From looking at your code you seem to be missing a closing bracket your code should look like this.
namespace Quotations;
class MyClass {
var $is_number;
var $is_defined;
private $number;
private $defined;
private function isNumeric($w, $c) {
if(is_numeric($w) && is_numeric($c)){
$number = true;
}
else {
$number = false;
}
return $number;
}
private function isDefined($t, $f, $w, $c){
if( isset( $t, $f, $w, $c ) ){
$defined = true;
}
else {
$defined = false;
}
return $defined;
}
function MyFunction($to, $from, $weight, $cube) {
try {
if(!$this -> isNumeric($weight, $cube)){
throw new InvalidArgumentException('Arguments are not numeric');
}
if(!$this -> isDefined($to, $from, $weight, $cube)){
throw new BadMethodCallException('Arguments are missing');
}
} catch(InvalidArgumentException $e) {
echo 'Caught exception: ', $e->getMessage(), "\n";
} catch(BadMethodCallException $e) {
echo 'Caught exception: ', $e->getMessage(), "\n";
}
}
}
Edit:
I saw you said in the comments you have added the } and still no luck so I looked through your code and found the following problem
So this is my last solutions I can think of xD
function MyFunction( $to, $from, $weight, $cube ) {
try
{
try
{
if( !$this->isNumeric( $weight, $cube ) ) {
throw new \InvalidArgumentException( 'Arguments are not numeric' );
}
} catch ( \InvalidArgumentException $e ) {
echo 'Error: ' . $e->getMessage();
}
try
{
if( !$this->isDefined( $to, $from, $weight, $cube ) ){
throw new \BadMethodCallException( 'Arguments are missing' );
}
} catch ( \BadMethodCallException $e ) {
echo 'Error: ' . $e->getMessage();
}
} catch ( Exception $e ) {
echo 'Error: ' . $e->getMessage();
}
}
Hope this helps