for statement help

bimmer528

I have been working, trying to fix this for what it seems like all day. Actually, it has been. 🙁

This is the problem I am having


for ($i=0; $id[1][$i]; $i++) {
for ($i=0; $data[1][$i]; $i++) {

echo "<br>" .$id[1][$i]. "" .  $data[1][$i] . "<br>";

}}

id is always in this format 1,2,3,4...incremented. My problem is grouping data with id. For example, id 1 could have 2 data entries while id 2 only has 1.

The script is just outputting this way

1 mydata
2 mydata
3 mydata
mydata
mydata
mydata

This isn't being pulled from a db. It's reading a file.

Help?

jmrouleau

Ok...a for loop should be in the following format:

$i=10
for($x=0; $x<$i; $x++){

echo $array[$x];

}

This will print the values in $array at indexes 0-9.

The thing about a for loop is that you need to know how many times it is going to loop....Either by a counter in the code or a static number. You don't need a seperate for loop for each array.

jmrouleau

I stand corrected....I guess the second expression can be any conditional. However, the nested for loop will only revert back to the first when the conditional is false...so it will go through the whole array.

You will probably need to use an if statement or two instead.

bimmer528

what about starting a foreach id group data to id.... but linking data to know which id?

jmrouleau

What exactly are you trying to do??

Is there a reason for the multidimensional array??

Try attaching the full script also.

bimmer528

I'm Sorry I was not more clear or failed to provide enough information.

Please excuse my messy code. I'm learning 🙂


preg_match_all( "/\<Player\>(.*?)\<\/Player\>/s", 
  $contents , $data );

  foreach( $data[1] as $value )
  {
  preg_match_all( "/\<Name\>(.*?)\<\/Name\>/", 
  $value, $name );
  preg_match_all( "/\<Score\>(.*?)\<\/Score\>/", 
  $value, $score );
  preg_match_all( "/\<Kills\>(.*?)\<\/Kills\>/", 
  $value, $kills );
  preg_match_all( "/\<Deaths\>(.*?)\<\/Deaths\>/", 
  $value, $deaths );
  preg_match_all( "/\<TOP\>(.*?)\<\/TOP\>/", 
  $value, $top);
preg_match_all( "/\<Ping\>(.*?)\<\/Ping\>/", 
  $value, $ping);
preg_match_all( "/\<IP\>(.*?)\<\/IP\>/", 
  $value, $ip);
  preg_match_all( "/\<Round\>(.*?)\<\/Round\>/s", 
  $value, $rnd);
  preg_match_all( "/\<Nr\>(.*?)\<\/Nr\>/", 
  $value, $nr );
preg_match_all( "/\<V\>(.*?)\<\/V\>/", 
  $value, $v );
preg_match_all( "/\<W\>(.*?)\<\/W\>/", 
  $value, $w );
preg_match_all( "/\<Dmg\>(.*?)\<\/Dmg\>/", 
  $value, $dmg );
preg_match_all( "/\<Kill\>(.*?)\<\/Kill\>/", 
  $value, $rndkill );
preg_match_all( "/\<L\>(.*?)\<\/L\>/", 
  $value, $l );
preg_match_all( "/\<B\>(.*?)\<\/B\>/", 
  $value, $b );
preg_match_all( "/\<H\>(.*?)\<\/H\>/", 
  $value, $h);




echo " Player=> " .$name[1][0].
"<br> Score=> ". $score[1][0] .
"<br>  Kills=> ". $kills[1][0].
"<br> deaths => " .$deaths[1][0].
" <br> TOP=> " .$top[1][0]. 
"<br>  ping=> " .$ping[1][0]. 
"<br>  IP=> <Hidden> 

<br>";


$name=$name[1][0];
$score=$score[1][0];
$kills= $kills[1][0];
$deaths=$deaths[1][0];
$top=$top[1][0]; 
$ping=$ping[1][0]; 
$ip=$ip[1][0]; 



for($i=0; $nr[1][$i]; $i++)
for($i=0; $v[1][$i]; $i++)

{ {


echo 
"   round=>" . $nr[1][$i].
 "   Vic=>" . $v[1][$i]. 


"    Weapon=>  ". $w[1][$i] .
"   dmg =>  ". $dmg[1][$i] .
"  got kill =>  ". $rndkill[1][$i] .
"   Leg=>  ". $l[1][$i] .
"   Body=>  ". $b[1][$i] .
"  Head =>  ". $h[1][$i] .
"<br><br>";
}}

This script reads from a xml file Located Here

This is the output on my page The page

Sorry for the messy output. I just wrote it fast to see the results and they are just temporary.

You can see when each players round's are done, the vic's keep going. Those vic's actually belong grouped with round and are misplaced.

From the xml file

<round>
records a new round. There may be more then one vic round.
</round>

jmrouleau

for($z=0; $nr[1][$z]; $z++) 
for($i=0; $v[1][$i]; $i++) 

{ { 


echo 
"round=>" . $nr[1][$z]. 
"<br>Vic=>" . $v[1][$i]. 


"<br>Weapon=>  ". $w[1][$i] . 
"<br>dmg =>  ". $dmg[1][$i] . 
"<br>got kill =>  ". $rndkill[1][$i] . 
"<br>Leg=>  ". $l[1][$i] . 
"<br>Body=>  ". $b[1][$i] . 
"<br>Head =>  ". $h[1][$i] . 
"<br><br>"; 
}}

Ok...for each round is there more than one weapon, dmg, etc??

If so, I would suggest changing $i in the first loop to $z or something because it is probably causing the loop to act weird. The z will carry over into the second loop if you need it there.

Otherwise, you only need one for loop. If you want to add a conditional put everything inside the for in an if statement.

jmrouleau

By the way, the XML link doesnt work.

bimmer528

jmrouleau- I can't thank you enough. This is as close as it's come.

What it is doing though is outputting the victims per round twice in that round. Click on the url. The order of i and z are correct.

Any ideas?

Edit-

Wait I just realized its not duplicating by the round but the following round's data is being entered into the previous along with the correct data for that round.


- <Player>
  <ID>29</ID> 
  <Team>1</Team> 
  <Name>[Da]DivineOne</Name> 
  <Score>68</Score> 
  <Kills>9</Kills> 
  <Deaths>2</Deaths> 
  <TOP>0</TOP> 
  <Ping>56</Ping> 
  <PL>0</PL> 
  <Time>8</Time> 
  <IP>69.193.236.85</IP> 
- <Damage>
- <Round>
  <Nr>4</Nr> 
- <Hit>
  <V>{RD}-Killa[Ae]</V> 
  <W>Desert Eagle</W> 
  <Dmg>100</Dmg> 
  <H>1</H> 
  <B>2</B> 
  <Kill>1</Kill> 
  </Hit>
- <Hit>
  <V>[-oS-]Raver</V> 
  <W>Desert Eagle</W> 
  <Dmg>54</Dmg> 
  <H>1</H> 
  </Hit>
  </Round>
- <Round>
  <Nr>5</Nr> 
- <Hit>
  <V>{RD}-Killa[Ae]</V> 
  <W>MP5 SD</W> 
  <Dmg>100</Dmg> 
  <B>7</B> 
  <L>1</L> 
  <Kill>1</Kill> 
  </Hit>
- <Hit>