global mysqli variable?

**~~Twinkie~~** · 04-12-2009 01:36 PM

Trying some simple testing to form the backbone of a subscription system. Why am I getting this error when I make a subscription?

PHP Code:


<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
<title>Mail Subscription</title>
</head>
<body>

<?php
function DBCon() {
    global $con;
    $con = mysqli_connect("localhost", "user", "nopass","mtwinkie_testdb");
    if (mysqli_connect_errno()) {
        printf("Connect failed: %s\n", mysqli_connect_error());
        exit();
    }
}

function Validate($email) {
    if(!filter_var($email, FILTER_VALIDATE_EMAIL))
        die("<h1 align=\"center\">Invalid Email.</h1>");
    else if (strlen(email) > 150)
        die("<h1 align=\"center\">Email too long.</h1>");
    $query = "SELECT id FROM Newsletter WHERE email = '" . email ."';";
    $res = mysqli_query($con,$query) or
        die(mysqli_error($con));
    if (mysqli_num_rows($res) < 1)    
        die("<h1 align=\"center\">Your already subscribed.</h1>");
    return true;
}

if (!isset($_REQUEST["do"])) {
?>

<h1>Mail Subscription</h1>
<form action="#" method="post">
<b>Your Email Address:</b><br />
<input type="text" name="email" /><br /><br />
<b>Action:</b><br />
<input type="radio" name="do" value="sub" /> Subscribe
<input type="radio" name="do" value="unsub" /> Unsubscribe
<p><input type="submit" value="Submit Form" /></p>

<?php
} else if ($_REQUEST["do"]=="sub") {
    DBCon();
    if (Validate($_REQUEST["email"])) {
        $query = "INSERT INTO Newsletter (ID, Email) VALUES (\"" . $email . "\");";
        $res = mysqli_query($con,$query) or
            die(mysqli_error($con));
        mysql_close($con);
        die("<h1 align=\"center\">Thank you for signing up!</h1>");
    }
} else if ($_REQUEST["do"]=="unsub") {
    die("<h1 align=\"center\">Not available yet =(</h1>");
} else die("<h1 align=\"center\">Logical Script Error.</h1>");
?>

</body>
</html>

As far as I can tell the global declaration should solve this problem?

**misson** · 04-12-2009 03:12 PM

2 problems:

Originally Posted by Twinkie

Code:

function Validate($email) {
    if(!filter_var($email, FILTER_VALIDATE_EMAIL))
        die("<h1 align=\"center\">Invalid Email.</h1>");
    else if (strlen(email) > 150)
        die("<h1 align=\"center\">Email too long.</h1>");
    $query = "SELECT id FROM Newsletter WHERE email = '" . email ."';";
    $res = mysqli_query($con,$query) or
        die(mysqli_error($con));
    if (mysqli_num_rows($res) < 1)    
        die("<h1 align=\"center\">Your already subscribed.</h1>");
    return true;
}

You left the '$' off of 'email' in 2 lines here. Also, grammar maven says that that should be "You're" ("you are"), not "Your" (possessive).

Originally Posted by Twinkie

PHP Code:


        $query = "INSERT INTO Newsletter (ID, Email) VALUES (\"" . $email . "\");";

This is the source of the error. You say you're going to provide 2 values, ID and Email, but only provide 1. If the ID column is auto_increment, leave ID out of the insert statement.

**~~Twinkie~~** · 04-12-2009 03:33 PM

Thanks misson, I already the first two errors after I posted, but that's not what the problem was. I got a warning that I did not have a valid mysqli variable supplied in the email validation function. I later fixed it my moving the mysqli portion of the function where the DBCon() was called, but why can't the script posted above read the $con variable in the Validate() function is $con is a global variable?

**misson** · 04-12-2009 03:44 PM

Validate() doesn't declare $con as global. I recommend passing $con as a parameter rather than using a global; it's cleaner.

**~~Twinkie~~** · 04-12-2009 04:02 PM

You're probably right, but I declare it as a global variable in the DBCon() function. Shouldn't that make it accessible in the Validate() function?

**misson** · 04-12-2009 04:28 PM

Originally Posted by Twinkie

You're probably right, but I declare it as a global variable in the DBCon() function. Shouldn't that make it accessible in the Validate() function?

Nope. Each function has its own scope. The "global" declaration imports a global variable into a function's scope (actually, it creates a local variable that holds a reference to the global variable). It doesn't make a variable a superglobal.

In any case, pass the connection around as a parameter and you don't need to worry about globals.

**~~Twinkie~~** · 04-12-2009 05:25 PM

Thanks misson, I can't rep you because I did it before

Thread: global mysqli variable?

LinkBack

Thread Tools

Display

global mysqli variable?

Re: global mysqli variable?

Re: global mysqli variable?

Re: global mysqli variable?

Re: global mysqli variable?

Re: global mysqli variable?

Re: global mysqli variable?

Similar Threads

Problema con variable

MySQLi Not working

mysqli not available

SmartFTP Global Queue

perl script for guestbook, need help

Posting Permissions