ForEach problem...

Kold

I have an array using the following format (taken from print_r);

Array
(
[0] => 6
[User_ID] => 6
[1] => TEXT
[User_Text] => TEXT
)

The problem i have is that i an using a ForEach loop to go through all of them, however, obviously everything is counted twice.

I would like the numbered array elements OR the named ones to be used, not both. Any ideas?

Thanks!

LordShryku

foreach($array as $key => $value) {
   if(is_numeric($key)) {
      echo $value;
   }
}

TheDefender

I guess my question would be why are you doubling the data in the array to begin with?!?

Post the code you use to generate your array, and maybe we can help you clean that up, so you can avoid this scenario...

LordShryku

I do the same in some parts of my database extraction layer. It's not too uncommon...

TheDefender

I know it isn't too uncommon, but even when working within the db extraction layer, I still try to avoid duplicating data if I don't need it. (But that's just me personally)

I would be interested in seeing how you use it though LordShryku. I always enjoy the lessons you throw at me, and have used a lot of what you and I have discussed in the past, in my coding. (Don't think your lessons ever fall on deaf ears with me, even when I disagree with you...) 🙂

LordShryku

Well, I'm at home now, and don't have my extraction script handy, but I use MySQL, PostgreSQL, MSSQL, and Oracle all in conjunction on one of my sites. I ended up building that functionality in there to allow flexability, expecially for Oracle. My coding style is all lowercase, except for globals. Oracle has a force uppercase on it's fields, so I had to remember when flipping back and forth between databases. This helped ease that a bit by allowing me to specify numbers when I wanted to with Oracle, and still use row names with others. Another one was when I wanted to loop results from a large selection, but still only pick one out. Helps me from having to count the number of fields when I specifically know which field is to be returned.

My return comes back in an array consisting of rows as the first element and fields as the second element, so

Array (
   [rows] = Array (
              [fields] = field result
            )
)

I can see drawmack or Weedpacket bashing this already, but oh well, it's more of personal preference for me. I do have variables I can pass to my functions to specify fields names or numeric keys to be returned, but default that to both.

TheDefender

As always LS, I appreciate the lesson plan. I personally don't see where or how I can incorporate any of that into my own scripts (especially, since I code exclusively on MySQL), but it is very interesting food for thought. I can see where you are going with the thought process though, and appreciate you taking the time to explain your thoughts to me. 🙂

OK, I am off for now, you have a good night out there.

Kold

Thanks for the replies, interesting stuff 🆒 🙂

Here is my code;

$SQLList = mysql_query("SELECT $NeededCols FROM tblUsers WHERE ID='$TID'", $dbhndl);

if ($Returned = mysql_fetch_array($SQLList)) {

do {

// Turn numbers to names (User_Num different to User_ID)
If($Returned['User_Num']){
	$Returned['User_Num'] = NameFromID($Returned['User_Num']);
}

print_r($Returned); //Generates the quote in my 1st post...
exit; // Dont run the rest of the script whilst debugging...

I think thats the code doing-the-double :p

Thanks
K.

xblue

when working with mysql without any database abstraction layer, simply change mysql_fetch_array() to mysql_fetch_assoc() to loop through the fields with foreach, or mysql_fetch_row() with a for loop should work out fine as well.