ansaurus

Question

PHP & MySQL problem.

Answer 1

+1 A:

You should check if the values are empty(). If they are, omit them from your database insert query or mark them as "null".

Without seeing the database table design or any code that has to deal with the database, it is hard to say how we can better help you.

UPDATE

With the new information from the INSERT query, here is how you would apply the empty (untested pending syntax errors):

$tags = array_filter($tags, function ($v) { return !empty($v);});

for ($x = 0; $x < count($tags); $x++){
    $query1 = "INSERT INTO tags (tag) VALUES ('" . mysqli_real_escape_string($mysqli, strtolower($tags[$x])) . "')";
}

Should remove the empty values from the array, pending I did not make a simple mistake.

EDIT

Here is one option to do it, but yea. Since you are using the loop already, you can just add an if inside the loop. With the array_filter it should remove any empty values from the array.

Moved the function definition. The above should work.

Brad F Jacobs 2010-08-09 17:40:29

how would i add that to my code?

myTIME 2010-08-09 17:41:58

I dont want them to get entered into the database at all.

myTIME 2010-08-09 17:43:13

Depending on your DB Schema, this may not be possible. If you post that up, I (we) can tell you if that would be possible for your situation. Given normal database designs (if it is normal) the answer would be they have to be entered into the DB. But again, without seeing the structure this is impossible to tell.

Brad F Jacobs 2010-08-09 17:46:08

I get the following now `Warning: array_filter() [function.array-filter]: The second argument, 'empty', should be a valid callback in `

myTIME 2010-08-09 18:00:11

Answer 2

A:

Why don't you check if the string is empty before inserting it into the database?

for ($x = 0; $x < count($tags); $x++){
    if ($tags[$x] != '') {
         $query1 = "INSERT INTO tags (tag) VALUES ('" . mysqli_real_escape_string($mysqli, strtolower($tags[$x])) . "')";
    }
}

A. M. 2010-08-09 17:59:52

Answer 3

A:

I think you just need to beef up what you split on.

I'd probably do something more like this

$input = ",,,,,,,, ,, ,,,,a,d, <html> , ruby-on-rails , ad, <html>";

$tags = preg_split( "/(?:\s*,\s*)+/", trim( strip_tags( $input ), ', ' ) );
$tags = array_map( 'mysqli_real_escape_string', str_replace( ' ', '-', $tags ) );

Then, when it comes to your query, take advantage of MySQL's multiple insert syntax

$query1 = "INSERT INTO tags (tag) VALUES ('" . implode( "'),('", $tags ) . "');";

But you may want to look into duplicate insert handling.

Peter Bailey 2010-08-09 18:14:43

ansaurus

tags:

views:

answers:

PHP & MySQL problem.

related questions