Fetching a result from selection query in php function

Question

I have a php function of selection from mySql database:

function Master_file($name, $latin ){

  $HOST_DB ="localhost";

  $NAME_DB="nom";
  $USER_DB ="utilisaeur";
  $PWD_DB="K3Pud1";
  $connect = mysql_connect($HOST_DB,$USER_DB,$PWD_DB);
  $db=mysql_select_db($NAME_DB);


  $qry = "SELECT tax_id FROM  master where name =".$name." and latin =".$latin;
  echo $qry;
  $result = mysql_query($qry);

  while ($Res_user = mysql_fetch_assoc($result) ) {

    return $Res_user['tax_id'];
  }
}

an error is shown Warning: mysql_fetch_assoc(): supplied argument is not a valid MySQL result resource in /home/admin/public_html/hitlist/include/fg_membersite.php on line 446 and the line is while ($Res_user = mysql_fetch_assoc($result)

So what is the problem ? How can i fix it?

Your query failed. I assume you're missing quotes around the variables. — Fabrício Matté
– Fabrício Matté, Commented May 26, 2013 at 21:26
Is the database connection valid? It could be your not actually connected to the database yet, as you don't have any error catching, you may not know about it — Phil Cross
– Phil Cross, Commented May 26, 2013 at 21:27
@FabrícioMatté i change the query to $qry = "SELECT tax_id` FROM master where name =".$name." and latin =".$latin;` but the same result — Lamloumi Afif
– Lamloumi Afif, Commented May 26, 2013 at 21:31
$result = mysql_query($qry) or die(mysql_error()); may be informative for the next times. — Fabrício Matté
– Fabrício Matté, Commented May 26, 2013 at 21:37
Why are you fetching in a while loop if you're going to return during the first iteration? — Barmar
– Barmar, Commented May 26, 2013 at 21:43

Phil Cross · Accepted Answer · 2013-05-27 13:09:55Z

1

Try this

function Master_file($name, $latin ){

    $dsn = 'mysql:host=localhost;dbname=nom';
    $username = 'utilisaeur';
    $password = 'K3Pud1';

    try {
        $db = new PDO($dsn, $username, $password);
        $db->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
    } catch (PDOException $e) {
        echo $e->getMessage();
        exit;
    }


    $result = $db->prepare("SELECT tax_id FROM  master where name =:name");
    $result->bindValue(':name', $name);
    $result->execute();

    foreach($result->fetchAll(PDO::FETCH_ASSOC) as $row){
        echo $Res_user['tax_id'] . '<br />';
    }
}

EDIT The function above has just been updated to use PDO, display any errors, and output the tax_id value to the browser

edited May 27, 2013 at 13:09

answered May 26, 2013 at 21:40

Phil Cross

9,33212 gold badges54 silver badges86 bronze badges

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

2 Comments

Lamloumi Afif Over a year ago

nice it works but the same error is repeat only 30 times but before your answer it is 400 rows contains Warning: mysql_fetch_assoc(): supplied argument is not a valid MySQL result resource in. i added a message under the while condition and it is shown one time. so how can i avoid this error completely?

Phil Cross Over a year ago

@lamloumi I have updated the function. I have made several changes, which include using PDO instead of mysql functions. PDO is a lot better to use, check out: php.net/pdo also, I have made a couple other changes as well, so if you want to give it a go and see if it works.

The Alpha · Accepted Answer · 2013-05-26 21:56:19Z

0

You may try this, since your returning here return $Res_user['tax_id']; so I think you need a single row instead

function Master_file($name, $latin ){

    $HOST_DB ="localhost";
    $NAME_DB="nom";
    $USER_DB ="utilisaeur";
    $PWD_DB="K3Pud1";

    $connect = mysql_connect($HOST_DB,$USER_DB,$PWD_DB);
    if (!$connect) {
        die("Could not connect: " . mysql_error());
    }

    $db=mysql_select_db($NAME_DB, $connect);
    if (!$db) {
        die ("Can't use " . $NAME_DB . " : " . mysql_error());
    }

    $qry = "SELECT tax_id FROM  master where name ='" . $name . "' and latin = '" . $latin . "'";
    $result = mysql_query($qry);
    if( $result ){
        $row = mysql_fetch_assoc($result);
        return $row['tax_id'];
    }
}

edited May 26, 2013 at 21:56

answered May 26, 2013 at 21:50

The Alpha

147k30 gold badges294 silver badges313 bronze badges

Collectives™ on Stack Overflow

Fetching a result from selection query in php function

2 Answers 2

2 Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related