Search array from a txt doc

catnip_uk

Hi

I am looking to search an array as part of a error validation page from an external txt file (fee/promo.cfg).

Basically, I want the page to display a text error if the user has not entered a correct promo code in the 'txtPromoCode' box ie: if the code exists in the txt document then no error is generated. (The txt document also serves as a flat file database to an admin can update pormo codes)

Please look at the code below in what I have entered as bold is what I am having problems with, the rest of the validations workfine.

Kudose

What about:

$codeEntered = 'whatever'; //your $_POST or something
$path = "/path/to/file/location.cfg";
$handle = fopen($path, "r");
$file = fread($handle, filesize($path));
$codes = explode("\n", $file);
if(in_array($codeEntered, $codes))
 echo 'Found.';
else
 echo 'You can\'t rip ME off!';

Installer

This will remove any carriage-returns as well as newlines in the file, and do a case-insensitive search:

$fp = fopen('fees/promo.cfg', 'r');
$bol_form_error = true;
while (!feof($fp)) {
    $promo_line = strtolower(trim(fgets($fp)))
    if ($strtolower($promo_value) == $promo_line);
        $bol_form_error = false;
        break;
    }
fclose($fp);
if ($bol_form_error) { 
    $str_form_error = '<p class="form-error">Error: Please enter a valid promotional code.</p>';
}

catnip_uk

Hi...Many thanks for the help but....

I am getting the following error on the first 'if' statement

Parse error: parse error, unexpected T_IF in /var/www/vhosts/***************/httpdocs/test/index3.php on line 74

Which begins if ($strtolower etc)

Installer wrote:

This will remove any carriage-returns as well as newlines in the file, and do a case-insensitive search:

$fp = fopen('fees/promo.cfg', 'r');
$bol_form_error = true;
while (!feof($fp)) {
    $promo_line = strtolower(trim(fgets($fp)))
    if ($strtolower($promo_value) == $promo_line);
        $bol_form_error = false;
        break;
    }
fclose($fp);
if ($bol_form_error) { 
    $str_form_error = '<p class="form-error">Error: Please enter a valid promotional code.</p>';
}

etully

In this line:

// Check to see if the client has entered a valid promotional code
if(!bol_form_error == 0 && strlen($_POST['txtPromoCode']) > 1)

Did you mean to leave the $ sign out? Shouldn't that be $bol_form_error

laserlight

Looks like a typo error.

$fp = fopen('fees/promo.cfg', 'r');
$bol_form_error = true;
while (!feof($fp)) {
	$promo_line = strtolower(trim(fgets($fp)));
	if (strtolower($promo_value) == $promo_line) {
		$bol_form_error = false;
		break;
	}
}
fclose($fp);
if ($bol_form_error) {
	$str_form_error = '<p class="form-error">Error: Please enter a valid promotional code.</p>';
}

EDIT: missed a terminating semi-colon

catnip_uk

Nearly there I think, but now getting an error as below, maybe it's something to do with the txt doc it's trying to read from? (promo.cfg txt doc attached)

Notice: Undefined variable: promo_value in /var/www/vhosts/**************/httpdocs/test/index3.php on line 70

Many thanks in advance

Kudose

Is the variable set?

If you replaced your entire bolded code with laserlights code, the initialization was omitted.

$promo_value = $_POST['txtPromoCode'];

catnip_uk

Is this what you mean?....but still does not parse :o(

if (strtolower($_POST['txtPromoCode']) == 0) {

Installer

I'm sorry the code I posted caused you so much trouble, but, for pete's sake, it was only a couple of typos. Both of them are pretty obvious and easily corrected by someone willing to spend 30 seconds looking for them (with the error messages as big clues), and especially by someone reading it to figure out how it's supposed to work and not merely copy-pasting with crossed fingers.

As for "$promo_value", that's the name of the variable you used in your original code ( $promo_value = $_POST['txtPromoCode']; ). I didn't really see any need to repost all your code.

Replace your bold code with this:

if(!($bol_form_error == 0) && (strlen($_POST['txtPromoCode']) > 1)) {
    $promo_value = $_POST['txtPromoCode']; 
    $fp = fopen('fees/promo.cfg', 'r'); 
    $bol_form_error = true; 
    while (!feof($fp)) { 
        $promo_line = strtolower(trim(fgets($fp))); 
        if ($strtolower($promo_value) == $promo_line); 
            $bol_form_error = false; 
            break; 
        } 
    fclose($fp); 
    if ($bol_form_error) { 
        $str_form_error = '<p class="form-error">Error: Please enter a valid promotional code.</p>'; 
    } 
}

If it doesn't work, check for typos, then look for a problem elsewhere with your posted values.

Kudose

Don't mean to pick on you Installer -- just saving you a headache.

if ($strtolower($promo_value) == $promo_line);

Should be

if (strtolower($promo_value) == $promo_line){

Installer

Thanks, Kudose. 🙂

catnip_uk

Hey!...thanks guys I really appreciate all your help, and 'Installer' sorry about being picky but the developer that works for me is on holiday and I only know basic php and need to get this page working so apologies....

Anyway.....

The new amended code does parse to the next page but does not throw up a validation error if a user posts an in-correct promotional code from the txt doc.

I have looked through the rest of the code and cannot see anything glaringly obvious?

Again...many many thanks for all your help so far.

catnip_uk

Ignore