I need help with this chunk of code

[Anon]

This is what the file(/etc/postfix/virtual) looks like:

Left side is address
Right side is user

[e-mail] usr0001
[e-mail] usr003
[e-mail] jogn
[e-mail] usr0001
[e-mail] jogn
This is what the code looks like:
<code>
//vars
$users[]="";
static $add=0;
static $countA=0;
static $countB=0;

				//Place file in array
				$filearray=file("/etc/postfix/virtual");

				while (list ($line_num, $line) = each($filearray)) 
				{
					//Is the line an address?
					if(ereg("@", $line) && !ereg("#", $line))
					{
						//Split the address and user
						//0 : addy   //1 : user
						$line=explode(" ", $line);

						//Add user
						if($line[1]!="")
						{
							$current=chop($line[1]);
							$add=1;

							++$countA;
							$countB=0;

							while(list($index, $value)=each($users))
							{
								++$countB;
								$value=chop($value);
								//print("<option>$countA:$countB-$current:$value");

								if(strcmp($current, $value)==0)
									$add=0;
							}

							if($add)
							{
								$users[]=$current;
								print("<option>$current");
							}
						}
					}
				}
			?>

</code>

This code is supposed to read in the lines and parse out the user. It's supposed to eliminate any redundancy at the same time.
I've spent many days trying to debug this and came to the conclusion that theres something in php that I'm missing. I could do this in a sec in C++, oh well.

[Anon]

Try this - although I can't be sure it 100%does what you're after. It uses a slippet I picked up on php.net's annotated manual.

$filearray=file("/etc/postfix/virtual");

while (list ($line_num, $line) = each($filearray)) {
if(ereg("@", $line) && !ereg("#", $line)) {
$line = explode(" ", $line);
$users[] = $line[1];
}
}
// now flip the array and get key values
$hash = array_flip($users);
$users = array_keys($hash);

Note: this will only work in PHP4.

[Anon]

Indeed, you are missing the reset() command to reset the array pointer before you start the while(list()=each()) loop.

reset($user);

and all your troubles are over. Manuals continue to be your friend.

[Anon]

Oh my god, thank you Vincent. I read through the manual like 6 times, but I'm so used to c/c++ that I didn't even notice that command. Thank you so much!!!!

[Anon]

Why do I have to use the static on the add? It doesn't work otherwise. Reading through the literature, it would seem that $add would persist all of the way through the chunk of code. But it seems to instantiate a new $add when it hits that second while loop.

[Anon]

That is qute strange because STATIC is only used to make a variable persist when a function exits.

I tried your code with and without the STATIC, and it works fine.

How do you implement this script? is it as a function or an object? DO you have any global variable named "$add"?