PHP - Need Some Suggestions On Exception Handling And Redirect To An Error Page

I have set an SQL field as unique so that duplicates can't be entered, however rather then having the page error out and show the Error: Duplicate entry 'entry' for key 'field' I would like to be able to have a little div appear and say something like "this entry already exists" I already have it set to show that the item has been added but I have no idea were to start to make this work.

Any ideas?

Thanks,
Jim

Hello,
Am writing a script that involves user input.
Take an example: a user fills in a wrong username or password at the page login.php, my login processor (processor.php) detects it, how is the error "WRONG USERNAME OR PASSWORD" supposed to be transferred back to login.php.
So far I have been using a session variable to transfer the error but am sure there is a better way to do this without displaying the error on processor.php itself.

Thanx in advance

Hi,

I'm a beginner in PHP OOP and I'm with some doubts about the correct way of handling errors in PHP.

Look at this function for example:


public function deleteFileFromDisk($fileNameToBeDeleted) {

    $handle = unlink($fileNameToBeDeleted);

    if (!$handle) {
        $result = "(this->deleteFileFromDisk) - Error, " . $fileNameToBeDeleted . " not deleted.";
    } else {
        $result = "(this->deleteFileFromDisk) - Success, " . $fileNameToBeDeleted . " deleted.";
    }
    return $result;
}


Is this the correct way of doing it, or I can do better than this?

Let me add some details of what I'm achieving...

I'm running class methods, and I need to control errors in the process. If any call to the object throw an error I need to catch, stop all the process and send an e-mail with the error.

Here are the object interactions:


$testar_classe = new geoIpImportCSV('geolitecity', 'http://geolite.maxmind.com/download/geoip/database/GeoLiteCity_CSV/');
$testar_classe->downloadAndSaveFile('./', $testar_classe->obtainDownloadFileName());
$testar_classe->uncompressZipFile($testar_classe->obtainDownloadFileName(), '.');
$testar_classe->deleteLine(1, 'GeoLiteCity-Location.csv');               
$testar_classe->deleteLine(1, 'GeoLiteCity-Blocks.csv');
$testar_classe->deleteDataFromTable('tabela1');
$testar_classe->deleteDataFromTable('tabela2');
$testar_classe->insertLinesToDb('GeoLiteCity-Location.csv', 'tabela1');
$testar_classe->insertLinesToDb('GeoLiteCity-Blocks.csv', 'tabela2');
$testar_classe->deleteFileFromDisk($testar_classe->obtainDownloadFileName());
$testar_classe->deleteFileFromDisk('GeoLiteCity-Blocks.csv');
$testar_classe->deleteFileFromDisk('GeoLiteCity-Location.csv'); 


Which is the best way of handle this? Create a new method to take care of the exceptions? There are any examples on how to do this?

Best Regards.

Hi
I'm completely new to error handling in PHP and wanted to ask whether I'm doing it right and, if not, what the right way would look like

class DBConnection {

public function execute($sql) {

$query = @pg_query($this->dbconn, $this->prepare($sql));

try {

if (!$query) {

throw new DBException();

}

} catch (DBException $e) {

echo "Query execution failed";

exit;

}

}
}

I have a prepared statement that returns an Article from my database.  It then binds the results-set to variables.

Most of the fields in the query are "required", so I *assume* that I am guaranteed to always get values back for those fields...

Is that presumptuous?

Here is a snippet of my code...

// Execute query.
mysqli_stmt_execute($stmt);

// Store results.
mysqli_stmt_store_result($stmt);

// Check # of Records Returned.
if (mysqli_stmt_num_rows($stmt)==1){
// Article was Found.
$articleExists = TRUE;

// Bind result-set to variables.
mysqli_stmt_bind_result($stmt, $articleID, $title, $description, $keywords, 
$heading, $subHeading, $publishedOn, $author,
$body, $referenceListing, $endnoteListing);

// Fetch record.
mysqli_stmt_fetch($stmt);

// Close prepared statement.
mysqli_stmt_close($stmt); // ????



Is it sufficient to have code like this...

Code: [Select]
<title><?php echo $title; ?></title>
...or do I need more error-handling??

Hope that makes sense?!

Thanks,


Debbie

My script displays results until I hit an error.  I continually get fatal timeout errors that I am unable to handle or ignore.  Yes I googled and found the custom error handler AND the simpler try/catch.  Still cant figure it out...
It has been 1 hour of playing with this without success so I have to ask  .  Please help.  thanks : )

Code: [Select]
try
{
if(($result=shell_exec("$commandLine")) == false)
{
echo "------>error";
throw new Exception;
}
else
{
echo "---->good";
}
}
catch (Exception $e)
{
echo "@@@__>exception caught";
}

Hello everyone,

quick description of 'error handling' so that everyone understands the solution i am seeking. let php and the database handle the error is not an appropriate answer here.

find out what i mean: using xampp, go to a login screen of your website, then turn off the database before submitting the login data. the connection will be attempted then fail. the browser will display its own error page which is white bg with a message. this is ridiculous. i made a custom handler that allows me to ignore this failure and show my own website with an error message.

now how can i detect an error with a select or update statement? i am not a db designer, so i really don't know how to do it. what say i fetch filed['testmyerror'] and it doesn't exist or there is a problem. how do i write php code that is used to ignore this problem and display my own message. naturally i will log the error and/or log and send email to myself.

anyone able to help? just a tip in the right direction?

Thank you very much, i appreciate this forum and its members always.

Hopefully there is a simple solution to this.

I have a form that when submitted sends a message to an email address from the form.  A customer fills it out, so there is no single one email address.  However, half of the time, the retards that use this form dont know their own email address and dont bother to put the .net at the end of it.  or they misspell it.  The other half of the time they dont even bother to put the @ and the domain.  They still think this is AOL for some god unknown reason.  And I cant account for all the ways that they can not figure out how to fuck up their email address, so an email-validator doesnt work because they can even beat that and still manage to break it.  For example, one guy named Mike continues to put the first half of his email in as Miek or Mik or just spelled completely wrong, and the 550 error that comes up when he puts that garbage in, which is most of the time (I think the guy is blind or something) is because his email address doesnt exist on the mail server because he cant spell it right, when on the off chance he actually does get the @domain.com in the end.  I really hate my retard customers.

So my simple little form blows up on them.

But what I am trying to do is to have something like this

if(@mail(args))
  {
  //success, I'll call them back shortly
  }
else
  {
  //youre a retard, please put in your email address correctly
  }

However, the error handling isnt working.  mail() is supposed to return FALSE if it encounters an error, an error that I'd rather put in an error log and not display to the customer, however, the error suppressor doesnt work.

Any Suggestions?

Hi.

I'm trying to display all the missing fields (errors) when the user hits SUBMIT.

My logic:

IF there is NO EMAIL THAN add $errortrack array = $errormsg[EMAIL]

IF there is NO OLDPASSWORD THAN add $errortrack array = $errormsg[OLDPASS]

and so on.

At the end, Print ALL the Error messages on the form using <?php foreach ( $errortrack as $key => $value) { 
                echo "<dt>$key:</dt>"; } ?>

The only thing it prints out is "0:" ever though there should be other errors.

What am I missing? 

Site Link http://www.fusionfashionhair.com/newpassform.php
My PHP Code:

<?php
session_start();
?>
<?php
$submit = $_POST['submit'];
// Form Data
$email = $_POST['email'];
$password_old = $_POST['password_old'];
$password_new = $_POST['password_new'];
$password_new_con = $_POST['password_new_con'];
$errorcount = 0;

$errormsg['Email'] = "Email Entered is Invalid";
$errormsg['OldPass'] = "Old Password Entered is Incorrect";
$errormsg['NewPass'] = "New Password Entered is Incorrect";
$errormsg['NewPassCon'] = "New Confirmed Password Entered is Incorrect";
$errormsg['SecCode'] = "Security Code is Incorrect";
$errormsg['NoErr'] = "No Errors, Continue";

$errortrack = array ($errormsg['NoErr']);

if ($_POST[submit]){
    if ($errorstop = "go") {
    $errorstop="go";
    while ($errorstop<>"stop") {
        // check for existance
      if ($email) {
            echo "True - Continue 1";
            echo "<p>----------</p>";
        } else {
            $errortrack = array ($errormsg['Email']);
            $errorcount++;
            $errorstop="stop";
        }
            
        // check for existance
      if ($password_old) {
            echo "True - Continue 2";
            echo "<p>----------</p>";
        } else {
            $errortrack = array ($errormsg['OldPass']);
            $errorcount++;
            $errorstop="stop";
        }        

        // check for existance
      if ($password_new) {
            echo "True - Continue 3";
            echo "<p>----------</p>";
        } else {
            $errortrack = array ($errormsg['NewPass']);
            $errorcount++;
            $errorstop="stop";
        }        

        // check for existance
      if ($password_new_con) {
            echo "True - Continue 4";
            echo "<p>----------</p>";
        } else {
            $errortrack = array ($errormsg['NewPassCon']);
            $errorcount++;
            $errorstop="stop";
        }            
        $errortrack = array ($errormsg="EVERYTHING IS OK");
        $errorstop="stop";
    }//End While Loop

    } else {
        while($errorcount>=0) {
            // Test display all error messages
            echo "<p>----------</p>";
            echo "<p>Error Count = '$errorcount'</p>";


        }
        die ("PLEASE FILL IN ALL FIELDS");
        
    }
}
?>


My Form Code with the PRINT ERROR:
Code: [Select]
    <form action='newpassform.php' method='post' id="regform">
      <fieldset>
        <legend>Change Password</legend>
        <p><?php foreach ( $errortrack as $key => $value) { 
                echo "<dt>$key:</dt>"; } ?></p>
        <p>
          <label for='email'>Email:</label>
          <input name='email' type='text' maxlength="25" value='<?php echo $email; ?>'/>
        </p>
        <p>
          <label for='password_old'>Old Password:</label>
          <input name='password_old' type='password' maxlength="32" />
        </p>
        <p>
          <label for='password_new'>New Password:</label>
          <input name='password_new' type='password' maxlength="32"/>
        </p>
        <p>
          <label for='password_new_con'>Confirm Password:</label>
          <input name='password_new_con' type='password' maxlength="32"/>
        </p>
        <p><span class="required">*</span> Note, username and password are case sensitive</p>
        <p>Forgot your password? <a href="forgot_password.php">Click Here</a></p>
        <p>Login <a href="login.php">Here</a></p>
       <h2>Security Check</h2>
      <p>Enter letters below exactly how they are displayed.  Letter are case sensitive. </p>
      <br />
      <img src="captcha.class.php?usefile=1" />
      <!--OR-->
      <!--<img src="image.php" />-->
     
      <input id='user_code' name='user_code' type='text' size='10' >
      <p>&nbsp;</p>
        <input class="reset" type='reset' value='Cancel' name='reset'>
                <input class="submit" type='submit' value='Continue' name='submit'>
      </fieldset>
    </form>
    <!--End of Form-->

Has anyone seen a decent database class for nicely wrapping up all the PDO functions? Ideally I'm after an example to handle database errors properly, insert/update/select queries, prepared statements and transactions. I've had a crack at writing my own and it's a bit messy with error handling everywhere. Maybe I need to put that in a separate class or something.

E.g. here's how I'd do a database update:
$db = new Database();    // In page controller
$params['type'] = $type;
$params['details'] = $details;
$query = 'insert into site_logs (type, details) values (:type, :details)';
$result = $db->preparedUpdate($query, $params);   // returns false if error or # of rows if successful

Just some examples would be great and I'll re-factor my one to make it better. Thanks!

Hey guys i was wondering if someone could help or guide me. I have been messing around with php for quite some months now to the stage where i can code pretty much what i need to. but recently i started using paypals IPN and during testing i realised some problems arise for example when testing and people buying things say the payment failed due to a line not executing in your script for what ever reason, the person has lost their money yet you have gained it and they are lost. Take this code below for example:


public function insertIntoDatabase($firstName,$lastName,$emailAddress,$merchant,$code,$database,$gameNumber)
{
$year = date("Y");
// Check what database we need to submit too::
$sql = "INSERT INTO playerGame (playerFirstName,
playerLastName,
playerEmailAddress,
playerAccountType,
playerCode,
playerGameDate,
playerGameNumber) VALUES (?,?,?,?,?,?,?)";

$stmt = $this->conn->prepare($sql);
$stmt->bind_param('ssssssi',$firstName,$lastName,$emailAddress,$merchant,$code,$year,$gameNumber);
$stmt->execute();
$stmt->close();

return true;
}


this is called on success the paypal IPN script. But what happens if this fails to execute? I suppose i could wrap it in a variable on the other side and test it

if($task)
{
    echo "code executed";
}

but what happens if this fails ? of am i just being paranoid ? hope someone can help dont want to be ripping people off =] Thanks!

Hello,

If I have an index file with:

date_default_timezone_set('GMT');
error_reporting(E_ALL | E_STRICT); 

abstract class A {
    abstract public static function YesIReallyMeanAbstractStatic();
}

class B extends A {
    public static function YesIReallyMeanAbstractStatic() {
    }
}


This will not error.

However if I have:


date_default_timezone_set('GMT');
error_reporting(E_ALL | E_STRICT); 

require('class_a_in_a_different_file.php'); // Or use __autoload() to do the same.

class B extends A {
    public static function YesIReallyMeanAbstractStatic() {
    }
}


This will throw the error:
"Static function A::YesIReallyMeanAbstractStatic() should not be abstract"

I know abstract statics are debatable...
but it makes no sense to me for it to be allowed in the first example but disallowed in other.

So I was wondering if this is a bug? Or is there something else going on here?

I assume the parser is evaluating the included file and then throwing the error which doesn't happen if both classes are in the same file as they can then be evaluated altogether.

I'm sure it's not much, but I'm not understanding something with custom error handlers:

I've created a custom error handler, which I initially set when my page loads :

set_error_handler(array ( new ErrorHandler(), 'handleError' ));

 

It seems to catch all internal PHP errors, such as if I: var_dump($non_existing_var); right after the set_error_handler....

Now, I have an object that throws an exception after:

set_error_handler(array ( new ErrorHandler(), 'handleError' )); 
$locale = new \CorbeauPerdu\i18n\Locale(...); // this should throw an exception 
...

I thought that with an error handler set, I could 'skip' the try/catch for it, but doing so, PHP spits out in its internal log:

PHP Fatal error: Uncaught CorbeauPerdu\i18n\LocaleException: ....

 

My error handler doesn't catch it at all before, thus my page breaks!!

If I want it to go through the error handler, I have to init my Locale with a try/catch and use trigger_error() like so:

set_error_handler(array ( new ErrorHandler(), 'handleError' ));

try { 
  $locale = new \CorbeauPerdu\i18n\Locale(...); // this should throw an exception 
}
catch(Exception $e) { 
  trigger_error($e->getMessage(), E_USER_ERROR); 
} 
...

Is this normal ? I thought one of the goals of the error_handler was to catch anything that wasn't dealt with?

Thanks for your answers!

Hello.  I have a web page that has several operations on it, and I'm not sure the best way to design it.

This is a subscription page were a user can choose different renewal plans.

Before a certain date, the user can "renew" for a certain lower price (operation #1), or he/she can "upgrade" for a certain lower price (operation #2).

After a certain date, the user can choose from one of three plans (operation #3, #4, or #5).

The way I have things design for the "after" part is just creating a form for each one, so that when a user chooses one of them, then I can take the appropriate actions.

My question is this...

1.) Is it okay to have five forms on the one web page to more clearly delineate each request/operation?

2.) Or should I create one form, and then assign a hidden form value to each button so I know how to handle the selection?

 

 

 

 

 

Edited July 25, 2020 by SaranacLake

This topic has been moved to PHP Applications.

http://www.phpfreaks.com/forums/index.php?topic=351056.0