Adding an If/else error/success message to PHP MYSQL function

Question

Im wondering how i could add a success or fail message to return at the end of this function

function InsertIntoDB($field1, $field2, $field3, $field4, $field5, $field6, $field7, $field8, $field9){
$sql = "INSERT INTO `caches` ( `url` ,  `username` ,  `password` ,  `cachename` ,  `lat` ,  `long` ,  `message` ,  `notes` ,  `tags`  ) VALUES(  '{$field1}' ,  '{$field2}' ,  '{$field3}' ,  '{$field4}' ,  '{$field5}' ,  '{$field6}' ,  '{$field7}' ,  '{$field8}' ,  '{$field9}'  ) "; 
mysql_query($sql) or die(mysql_error());
}

Id like it to return "OK" on success, and "FAIL" on failure.

Why not return true/false as success/failure? Returning strings means you are going to have == 'OK' littered throughout your code. — DisgruntledGoat
– DisgruntledGoat, Commented Sep 7, 2009 at 0:23

karim79 · Accepted Answer · 2009-09-07 00:15:27Z

You shouldn't hard-code messages like "OK" and "FAIL" into a function as its possible return values, that's a bit scary in terms of localization and the flexibility of your application. You should return a boolean and deal with messages outside of that function.

It makes little sense to test the returned strings if you're calling that function from elsewhere in your application where you only care about the semantics of what has happened as opposed to the returned message. What if you decide to change 'FAIL!' to 'Unsuccessful' for example? You will end up having to change all the other code that relies upon the return of your function. Why not just do something like this:

function InsertIntoDB($field1, $field2, $field3, $field4, $field5, $field6, $field7, $field8, $field9){
     $sql = "INSERT INTO `caches` ( `url` ,  `username` ,  `password` ,  `cachename` ,  `lat` ,  `long` ,  `message` ,  `notes` ,  `tags`  ) VALUES(  '{$field1}' ,  '{$field2}' ,  '{$field3}' ,  '{$field4}' ,  '{$field5}' ,  '{$field6}' ,  '{$field7}' ,  '{$field8}' ,  '{$field9}'  ) "; 
     return mysql_query($sql);
}

if(InsertIntoDB(...)) {
    echo 'Success!';
} else {
    echo 'Fail!';
}

Then somewhere else in your application, you won't need to:

if(InsertIntoDB(...) == 'SUCCESS!') {
    ...
}

but instead:

if(InsertIntoDB(...)) {
    ...
}

or:

if(!InsertIntoDB(...)) {
    ...
}

carl · Accepted Answer · 2009-09-06 23:53:51Z

0

What do you define to be success?

If you define mysql_query() returning true to be a success, then the following will work:

function InsertIntoDB($field1, $field2, $field3, $field4, $field5, $field6, $field7, $field8, $field9){
     $sql = "INSERT INTO `caches` ( `url` ,  `username` ,  `password` ,  `cachename` ,  `lat` ,  `long` ,  `message` ,  `notes` ,  `tags`  ) VALUES(  '{$field1}' ,  '{$field2}' ,  '{$field3}' ,  '{$field4}' ,  '{$field5}' ,  '{$field6}' ,  '{$field7}' ,  '{$field8}' ,  '{$field9}'  ) "; 
     return mysql_query($sql) ? "OK" : "FAIL";
}

answered Sep 6, 2009 at 23:53

carl

50.7k17 gold badges76 silver badges83 bronze badges

1 Comment

nickf Over a year ago

read the other answers - though this is exactly what the question was asking for, returning a string isn't a great practice.

Amber · Accepted Answer · 2009-09-06 23:51:46Z

0

function InsertIntoDB($field1, $field2, $field3, $field4, $field5, $field6, $field7, $field8, $field9){
$sql = "INSERT INTO `caches` ( `url` ,  `username` ,  `password` ,  `cachename` ,  `lat` ,  `long` ,  `message` ,  `notes` ,  `tags`  ) VALUES(  '{$field1}' ,  '{$field2}' ,  '{$field3}' ,  '{$field4}' ,  '{$field5}' ,  '{$field6}' ,  '{$field7}' ,  '{$field8}' ,  '{$field9}'  ) "; 
$result = mysql_query($sql);
if($result) return "OK";
else return "FAIL";
}

answered Sep 6, 2009 at 23:51

Amber

531k89 gold badges643 silver badges558 bronze badges

Comments

mauris · Accepted Answer · 2009-09-07 01:53:58Z

0

Why not use boolean instead of "OK" and "FAIL"?

function InsertIntoDB($field1, $field2, $field3, $field4, $field5, $field6, $field7, $field8, $field9){
     $sql = "INSERT INTO `caches` ( `url` ,  `username` ,  `password` ,  `cachename` ,  `lat` ,  `long` ,  `message` ,  `notes` ,  `tags`  ) VALUES(  '{$field1}' ,  '{$field2}' ,  '{$field3}' ,  '{$field4}' ,  '{$field5}' ,  '{$field6}' ,  '{$field7}' ,  '{$field8}' ,  '{$field9}'  ) "; 
     return mysql_query($sql);
}

if(InsertIntoDB(....)){
   // succeeded
}else{
   // failed
}

answered Sep 7, 2009 at 1:53

mauris

43.6k16 gold badges102 silver badges132 bronze badges

Comments

nickf · Accepted Answer · 2009-09-07 01:59:43Z

0

As everyone else has suggested, to indicate success or failure, you really should use a boolean. If you really want that extra bit of semantic sugar for readability, then you could define some constants:

define('OK', true);
define('FAIL', false);

function insertIntoDB() {
    $sql = "...";
    return mysql_query($sql) ? OK : FAIL;
}

if (insertIntoDB() == OK) {
    // win!
}

But I'll just reiterate, that's only if you particularly feel you need this extra readability. This method would probably make future maintainers of your code say "WhyTF did he do that?"

answered Sep 7, 2009 at 1:59

nickf

548k199 gold badges660 silver badges727 bronze badges

Collectives™ on Stack Overflow

Adding an If/else error/success message to PHP MYSQL function

5 Answers 5

Comments

1 Comment

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

5 Answers 5

Comments

1 Comment

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related