PHP code execution sequence

Question

I am trying to execute some lines of php code, but it seems that tey are not being execute in the required order. Here is a code snippet:-

if( !empty($_POST['val']) )
               {
                    $val = Get_Val($sid, $_POST['val'], $lnk);
                    if($val)
                    { 
                        echo "<br />Here Value : " . $val;
                    }
                    else
                    {
                        echo "Invalid Value.";
                    }
              }

When I echo the value before returning in function Get_Val() it shows a positive number for some set of valid arguments, which means that the If-condition is true, but when I execute the code the Else part is being executed. Though the output appear in order, they are not consistent. I hope I have made the problem clear. Any amount of help is appreciated. Thanks! Here is Get_Val() function:-

function Get_Val( $sid, $a, $link)
{
    //check is name is already present in table 
    $query = "SELECT val FROM store WHERE name = \"" . $a . "\"";  //val is auto incremented in sql
    $result = mysql_query( $query ,$link ) or die( mysql_error());
    if($result)
    {   
        $count = mysql_num_rows($result);
        if( $count == 0 ) //insert name and the return val
        {
            $query_x = "INSERT INTO store(name) VALUES('" . $a . "')";
            $result = mysql_query( $query_x ,$link ) or die( mysql_error());
            if($result)//If new name inserted then return the 'val'
            {
            Get_Val($sid, $a,$link);
            }
        }
        else
        {
            $row = mysql_fetch_assoc( $result );
            echo "Val in Get_Val : " . $row['val'];
            return $row['val'];
        }
    }
    else
    {
        echo "Unexpected Error Occured...!!!";
        exit(0);
    }
}

Also try if(isset($val)). Are you sure Get_Val() is returning a value? — ngen
– ngen, Commented Apr 8, 2012 at 17:24
make sure you sanitize $_POST['val'], or else, sql injection is positive — Vlad Balmos
– Vlad Balmos, Commented Apr 8, 2012 at 17:43

Roland Rusch · Accepted Answer · 2016-01-29 13:49:45Z

0

Get_Val does not return a value if $count == 0. Add a return statement before the recursive call. Like this:

...
if( $count == 0 ) //insert name and the return val
{
    $query_x = "INSERT INTO store(name) VALUES('" . $a . "')";
    $result = mysql_query( $query_x ,$link ) or die( mysql_error());
    if($result)//If new name inserted then return the 'val'
    {
       return Get_Val($sid, $a,$link);
    }
}
...

answered Jan 29, 2016 at 13:49

Roland Rusch

1146 bronze badges

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

Collectives™ on Stack Overflow

PHP code execution sequence

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related