MYSQLi PHP function fetch array [closed]

Question

Closed. This question is not reproducible or was caused by typos. It is not currently accepting answers.

This question was caused by a typo or a problem that can no longer be reproduced. While similar questions may be on-topic here, this one was resolved in a way less likely to help future readers.

Closed last year.

Improve this question

I am trying to retrieve a zone by running a PHP function based off of a place that has already been submitted.

Using FORM method GET, after submission, the variable that I am retrieving is:

 $place = mysqli_real_escape_string($_GET['place]);

The variable immediately after is zone:

 $zone = getZone($pol);  // here is the PHP function call

Above both of these variables is the function getZone, which looks like this:

 function getZone($place)
 {
   $searchZone = "SELECT ZONE FROM zones WHERE PLACE = '".$place."'";
   $result = mysqli_query($dbc, $searchZone);
   $row = mysqli_fetch_array($result);
   return $row['ZONE'];  
 }

I can run the query in the database, and it returns the ZONE.

Now, the mysqli_fetch_array, which normally works for me, is failing to produce the result from the query.

Does anyone see what is wrong?

What kind of feedback do you expect ? an array or a simple row ? — ceadreak
– ceadreak, Commented Jan 29, 2015 at 20:58

user2801966 · Accepted Answer · 2015-01-29 21:11:31Z

2

This might help

  //Assuming $dbc as connection variable
  function getZone($dbc,$place)
   {
     $searchZone = "SELECT ZONE FROM zones WHERE PLACE = '".$place."'";
     $result = mysqli_query($dbc, $searchZone);
     $row = mysqli_fetch_array($result);
     return $row['ZONE'];  
   }
   include 'path/to/connectionfile';//Only if you haven't  already  done  that 
   $zone = getZone($dbc,$pol);

answered Jan 29, 2015 at 21:11

user2801966

4883 silver badges11 bronze badges

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

1 Comment

halfer Over a year ago

Bear in mind that string concatenation in SQL queries is usually dangerous; it can open up a SQL injection vulnerability.

Marc B · Accepted Answer · 2015-01-29 20:58:59Z

1

You've forgotten about PHP's variable scope rules:

$result = mysqli_query($dbc, $searchZone);
                       ^^^^---- undefined

Since $dbc is undefined in the function, you're using a local null handle, which is invalid. If you'd had ANY kind of error handling in your code, you'd have been told about the problem.

Try

global $dbc;
$result = mysqli_query(...) or die(mysqli_error($dbc));

instead. Never assume success. Always assume failure, check for that failure, and treat success as a pleasant surprise.

answered Jan 29, 2015 at 20:58

Marc B

362k44 gold badges433 silver badges508 bronze badges

2 Comments

John Beasley Over a year ago

I went ahead and included the database file so that I can use my $dbc connection. Still no results.

Slava Rozhnev Over a year ago

This is bad practice to use die(mysqli_error($dbc))

halfer · Accepted Answer · 2023-10-19 10:02:02Z

0

I figured it out, thanks to the assistance of Marc B. I took into account that I was not providing my connection string, so I added it to the file. Problem is, I needed to add it to the actual function, like so:

 function getZone($place)
 {
   include ("../include/database.php");
   // then the rest of the code

After I included the database connection, I am now able to retrieve the zone.

edited Oct 19, 2023 at 10:02

halfer

20.2k20 gold badges110 silver badges207 bronze badges

answered Jan 29, 2015 at 21:07

John Beasley

3,09911 gold badges55 silver badges104 bronze badges

Collectives™ on Stack Overflow

MYSQLi PHP function fetch array [closed]

3 Answers 3

1 Comment

2 Comments

Comments

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

1 Comment

2 Comments

Comments

Related