imploding and using database information in arrays

Question

I have a database table that has the IPs of users who visit the website. I'm trying to create an IP ban system for a page. I want the IPs that that are listed as blocked from my database table to be listed in an array of IPs that cannot access this particular page. I keep getting errors and I can't figure out how to get it to work properly. This is what i'm using so far:

PHP:

$getBlockedUsers = mysqli_query($con,"SELECT ip FROM blog_comments WHERE blocked='yes'");
$separateIps = implode(", ", mysqli_fetch_array($getBlockedUsers));

if (in_array ($_SERVER['REMOTE_ADDR'], $separateIps)) {
   header("location: http://www.google.com");
   exit();
}

I'm connected to the database correctly

Why not just search for the row with ip = $_SERVER[REMOTE_ADDR]? — Barmar
– Barmar, Commented Jul 2, 2013 at 6:43

Chelsea Urquhart · Accepted Answer · 2013-07-02 06:46:58Z

1

$getBlockedUsers = mysqli_query($con,"SELECT true FROM blog_comments WHERE blocked='yes' AND ip='" . $_SERVER['REMOTE_ADDR'] . "' LIMIT 1");

if ($getBlockedUsers->num_rows > 0) {
   header("location: http://www.google.com");
   exit();
}

answered Jul 2, 2013 at 6:46

Chelsea Urquhart

1,4281 gold badge12 silver badges18 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Nikhil Mohan · Accepted Answer · 2013-07-02 06:47:48Z

0

Remove the implode function,

$separateIps = mysqli_fetch_all($getBlockedUsers, MYSQL_NUM);

in_array expects parameter 2 to be an array. When using implode you will get a string instead of an array.

edited Jul 2, 2013 at 6:47

answered Jul 2, 2013 at 6:42

Nikhil Mohan

8916 silver badges13 bronze badges

4 Comments

Barmar Over a year ago

That's just one row of the table, not all the IPs.

Leri Over a year ago

@Barmar I am pretty sure he meant mysqli_fetch_all.

Josh Over a year ago

I believe Barmar is right. I'll need to test this. It has removed the errors. But I'm not sure if this will work for all IPs.

Barmar Over a year ago

@PLB That won't work either, because it returns a multidimensional array.

Collectives™ on Stack Overflow

imploding and using database information in arrays

2 Answers 2

Comments

4 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

4 Comments

Your Answer

Sign up or log in

Post as a guest

Related