Finding in a file

[robertaccettura]

I have the following:

function lookupaddress($maildatabasefile, $ticket){
	$filename = "$maildatabasefile";
	$fp = fopen($filename, "r");
	$maildatabase = fread($fp, filesize($filename));
	fclose($fp);
	$maildbrow = explode(";", $maildatabase);
	while($i <= sizeof($maildbrow)) {		
		$dbrowcontents = explode(":", $maildbrow[$i]);
		if ($dbrowcontents[0] == "$ticket"){
			$emailaddress = $dbrowcontents;
			break;
		} else {
			$emailaddress = "NO ADDRESS";
		}
		$i++;
	}	
return $emailaddress;
}

the file it reads contains

1:[e-mail];
2:[e-mail];
3:[e-mail];

it should return the email address (right of the 🙂 of the line that contains the ticket (number left of the 🙂

Any idea why it doesn't?

Edit: Made a few changes, though same problem.

[Code_guy]

you wrote:
if ($dbrowcontents[0] == "$ticket"){

instead of:
if ($dbrowcontents[$i] == "$ticket"){

also you don't have a semicolon at the end of each email

😉

[robertaccettura]

sorry, when I forgot to put the ; in the test file...

doesn't matter since it still doesn't work 🙁

As for changing "0" to $i... that would break the script...

It breaks it up first as lines:

1:[e-mail]

2:[e-mail]

then breaks the line itself

1

[e-mail]

$dbrowcontents[0] should always be 0 since it is the first object in the array, in this case always the ticket number to the left of the :

[Code_guy]

sorry,
I didn't look at the code closely enough...

you need to write:
$emailaddress = $dbrowcontents[1];

instead of:
$emailaddress = $dbrowcontents;

[robertaccettura]

Thanks

Yep, that's definatly a mistake.

Still doesn't find a match though

Turns up as "no address" regardless....

[philipolson]

function get_ticket($filename, $ticket) {
    $lines = file($filename);
    foreach ($lines as $line) {
        $data = explode(':', $line);
        if ($data[0] == $ticket) {
            return $data[1];
        }
    }
    return false;
}

if ($email = get_ticket('somefile.txt', 3)) {
    print $email;
} else {
    print "Invalid ticket number";
}

Btw, remove the ; from your database.

[Code_guy]

oh,

initialize the $i to 0 before your while loop

[robertaccettura]

Originally posted by philipolson

function get_ticket($filename, $ticket) {
    $lines = file($filename);
    foreach ($lines as $line) {
        $data = explode(':', $line);
        if ($data[0] == $ticket) {
            return $data[1];
        }
    }
    return false;
}

if ($email = get_ticket('somefile.txt', 3)) {
    print $email;
} else {
    print "Invalid ticket number";
}

Btw, remove the ; from your database. [/B]

That would work if I were starting from scratch....

This is adding to an additional system.

If I removed the ; here, I would have to do a ton of reworking....

What I don't get is why my code is isn't finding a match.

[robertaccettura]

<action event="bump">This thread</action>