comparing two strings, what am I doing wrong?

[DavidSeck.com]

Hi,

I am working with the Facebook API right now, an I have kind of a
problem, but I don't know what I am doing wrong.

So I have a few arrays, f.ex.:

User albums:

array(2) {
[0]=>
array(10) {
["aid"]=>
string(19) "2322329421568036697"
["cover_pid"]=>
string(19) "2322329421568483956"
["owner"]=>
string(9) "540709454"
["name"]=>
string(5) "test2"
["created"]=>
string(10) "1212610115"
["modified"]=>
string(10) "1212610148"
["description"]=>
string(42) "Description 2 balaldlsaüdkopas jdkaj daj"
["location"]=>
string(9) "Location2"
["link"]=>
string(56) "http://www.facebook.com/album.php?
aid=20313&id=540709454"
["size"]=>
string(1) "6"
}
[1]=>
array(10) {
["aid"]=>
string(19) "2322329421568036696"
["cover_pid"]=>
string(19) "2322329421568483947"
["owner"]=>
string(9) "540709454"
["name"]=>
string(5) "test1"
["created"]=>
string(10) "1212609951"
["modified"]=>
string(10) "1212610028"
["description"]=>
string(23) "Description 1 blablabla"
["location"]=>
string(9) "Location1"
["link"]=>
string(56) "http://www.facebook.com/album.php?
aid=20312&id=540709454"
["size"]=>
string(1) "9"
}
}

User photos:

array(15) {
[0]=>
array(8) {
["pid"]=>
string(19) "2322329421568483956"
["aid"]=>
string(19) "2322329421568036697"
["src_small"]=>
string(95) "http://photos-e.ak.facebook.com/photos-ak-sf2p/
v258/189/57/540709454/t540709454_467572_7021.jpg"
["src_big"]=>
string(95) "http://photos-e.ak.facebook.com/photos-ak-sf2p/
v258/189/57/540709454/n540709454_467572_7021.jpg"
["src"]=>
string(95) "http://photos-e.ak.facebook.com/photos-ak-sf2p/
v258/189/57/540709454/s540709454_467572_7021.jpg"
["link"]=>
string(57) "http://www.facebook.com/photo.php?
pid=467572&id=540709454"
["caption"]=>
string(9) "caption21"
["created"]=>
string(10) "1212610147"
}
[1]=>
array(8) {
["pid"]=>
string(19) "2322329421568483957"
["aid"]=>
string(19) "2322329421568036697"
["src_small"]=>
string(95) "http://photos-f.ak.facebook.com/photos-ak-sf2p/
v258/189/57/540709454/t540709454_467573_7347.jpg"
["src_big"]=>
string(95) "http://photos-f.ak.facebook.com/photos-ak-sf2p/
v258/189/57/540709454/n540709454_467573_7347.jpg"
["src"]=>
string(95) "http://photos-f.ak.facebook.com/photos-ak-sf2p/
v258/189/57/540709454/s540709454_467573_7347.jpg"
["link"]=>
string(57) "http://www.facebook.com/photo.php?
pid=467573&id=540709454"
["caption"]=>
string(9) "caption22"
["created"]=>
string(10) "1212610147"
}
[2]=>
array(8) {
["pid"]=>
string(19) "2322329421568483958"
["aid"]=>
string(19) "2322329421568036697"
["src_small"]=>
string(95) "http://photos-g.ak.facebook.com/photos-ak-sf2p/
v258/189/57/540709454/t540709454_467574_7613.jpg"
["src_big"]=>
string(95) "http://photos-g.ak.facebook.com/photos-ak-sf2p/
v258/189/57/540709454/n540709454_467574_7613.jpg"
["src"]=>
string(95) "http://photos-g.ak.facebook.com/photos-ak-sf2p/
v258/189/57/540709454/s540709454_467574_7613.jpg"
["link"]=>
string(57) "http://www.facebook.com/photo.php?
pid=467574&id=540709454"
["caption"]=>
string(9) "caption23"
["created"]=>
string(10) "1212610147"
......
[9]=>
array(8) {
["pid"]=>
string(19) "2322329421568483950"
["aid"]=>
string(19) "2322329421568036696"
["src_small"]=>
string(95) "http://photos-g.ak.facebook.com/photos-ak-sf2p/
v258/189/57/540709454/t540709454_467566_6827.jpg"
["src_big"]=>
string(95) "http://photos-g.ak.facebook.com/photos-ak-sf2p/
v258/189/57/540709454/n540709454_467566_6827.jpg"
["src"]=>
string(95) "http://photos-g.ak.facebook.com/photos-ak-sf2p/
v258/189/57/540709454/s540709454_467566_6827.jpg"
["link"]=>
string(57) "http://www.facebook.com/photo.php?
pid=467566&id=540709454"
["caption"]=>
string(8) "caption4"
["created"]=>
string(10) "1212610025"
}
[10]=>
array(8) {
["pid"]=>
string(19) "2322329421568483951"
["aid"]=>
string(19) "2322329421568036696"
["src_small"]=>
string(95) "http://photos-h.ak.facebook.com/photos-ak-sf2p/
v258/189/57/540709454/t540709454_467567_7106.jpg"
["src_big"]=>
string(95) "http://photos-h.ak.facebook.com/photos-ak-sf2p/
v258/189/57/540709454/n540709454_467567_7106.jpg"
["src"]=>
string(95) "http://photos-h.ak.facebook.com/photos-ak-sf2p/
v258/189/57/540709454/s540709454_467567_7106.jpg"
["link"]=>
string(57) "http://www.facebook.com/photo.php?
pid=467567&id=540709454"
["caption"]=>
string(8) "caption5"
["created"]=>
string(10) "1212610025"

There is this aid, it is the album id, and I wanted to only show
pictures of a special aid, so I wrote this:

for($i=0;$i<count($albums);$i++){
echo "<br><b>".$albums[$i]['name']."</b><br>";
$album_aid = $albums[$i]['aid'];
for($j=0;$j<count($photos);$j++){
$photo_aid = $photos[$j]['aid'];
if($photo_aid == $album_aid){
echo $album_aid." ".$photo_aid;
echo "<a href=\"".$photos[$j]['link']."\"><img src=\"".$photos[$j]
['src_small']."\" alt=\"Photo\" border=\"0\" /></a>";
}
}
echo "<br>";
}

but i keeps showing me all of the pictures, even if the aids are
different. I looked in the manual, but I do not know what I am doing
wrong. Can someone please help me?

[=?UTF-8?B?SXbDoW4gU8OhbmNoZXogT3J0ZWdh?=]

Da***********@gmail.com wrote:

but i keeps showing me all of the pictures, even if the aids are
different. I looked in the manual, but I do not know what I am doing
wrong. Can someone please help me?

You're doing arrays the wrong (newbie) way, and implementing a horrible
O(n^2) search algorithm.
First, always use foreach() to iterate through an array.

Second, if you're doing searches, don't iterate until you find the photo -
build the photos array in such way that the array key is the photo ID. e.g:

$photos[1234567] = array('link'=>'foobar' , ... );
Try it out.

--
----------------------------------
Iván Sánchez Ortega -ivansanchez-algarroba-escomposlinux-punto-org-

Un ordenador no es un televisor ni un microondas, es una herramienta
compleja.

[axlq]

In article <g3**********@hercules.cohp1>,
Iván Sánchez Ortega <[e-mail]:

>O(n^2) search algorithm.

First, always use foreach() to iterate through an array.

foreach() is also O(n^2), for a 2-D array.

-A

[Tim Roberts]

Da***********@gmail.com wrote:

>
There is this aid, it is the album id, and I wanted to only show
pictures of a special aid, so I wrote this:

for($i=0;$i<count($albums);$i++){
echo "<br><b>".$albums[$i]['name']."</b><br>";
$album_aid = $albums[$i]['aid'];
for($j=0;$j<count($photos);$j++){
$photo_aid = $photos[$j]['aid'];
if($photo_aid == $album_aid){
echo $album_aid." ".$photo_aid;
echo "<a href=\"".$photos[$j]['link']."\"><img src=\"".$photos[$j]
['src_small']."\" alt=\"Photo\" border=\"0\" /></a>";
}
}
echo "<br>";
}

but i keeps showing me all of the pictures, even if the aids are
different. I looked in the manual, but I do not know what I am doing
wrong. Can someone please help me?

Where do you think you are "selecting" a special aid? This code goes
through all of the albums, and for each album, it prints all of the
pictures from that album. Translation: print all of the pictures.
--
Tim Roberts, ti**@probo.com
Providenza & Boekelheide, Inc.

[Captain Paralytic]

On 24 Jun, 05:33, [e-mail] (axlq) wrote:

In article <g3pnje$[e-mail]1>,
Iván Sánchez Ortega *<[e-mail].-..orgwrote:

O(n^2) search algorithm.

First, always use foreach() to iterate through an array.

foreach() is also O(n^2), for a 2-D array.

-A

Yes, but Iván was only offering that part as a general piece of advice.

[DavidSeck.com]

On 24 Jun., 06:37, Tim Roberts <[e-mail]:

[e-mail] wrote:

There is this aid, it is the album id, and I wanted to only show
pictures of a special aid, so I wrote this:

for($i=0;$i<count($albums);$i++){
* *echo "<br><b>".$albums[$i]['name']."</b><br>";
* *$album_aid = $albums[$i]['aid'];
* *for($j=0;$j<count($photos);$j++){
* * * * * *$photo_aid = $photos[$j]['aid'];
* * * * * *if($photo_aid == $album_aid){
* * * * * * * * * *echo $album_aid." ".$photo_aid;
* * * * * * * * * *echo "<a href=\"".$photos[$j]['link']."\"><img src=\"".$photos[$j]
['src_small']."\" alt=\"Photo\" border=\"0\" /></a>";
* * * * * *}
* *}
* *echo "<br>";
}

but i keeps showing me all of the pictures, even if the aids are
different. I looked in the manual, but I do not know what I am doing
wrong. Can someone please help me?

Where do you think you are "selecting" a special aid? *This code goes
through all of the albums, and for each album, it prints all of the
pictures from that album. *Translation: print all of the pictures.
--
Tim Roberts, [e-mail]
Providenza & Boekelheide, Inc.

and this:
if($photo_aid == $album_aid){
? There I check if the photo belongs to the current album, if it does
I print it. But it keeps printing all of the pictures even if aid is
different

[DavidSeck.com]

please I need help!
I tried this foreach but it doesn't change anything :(

[DavidSeck.com]

still need help :(

pleaseeee

[Tim Roberts]

Da***********@gmail.com wrote:

>On 24 Jun., 06:37, Tim Roberts <[e-mail]:

>[e-mail] wrote:

>but i keeps showing me all of the pictures, even if the aids are
different. I looked in the manual, but I do not know what I am doing
wrong. Can someone please help me?

Where do you think you are "selecting" a special aid? *This code goes
through all of the albums, and for each album, it prints all of the
pictures from that album. *Translation: print all of the pictures.

and this:
if($photo_aid == $album_aid){
? There I check if the photo belongs to the current album, if it does
I print it. But it keeps printing all of the pictures even if aid is
different

The inner loop prints all of the pictures in the album "album_aid" --
that's true. However, because of the outer loop, that loop will be run
once for each album. So, by the time that chunk is done, every picture in
every album will be printed.

Are you saying that you see every picture multiple times?
--
Tim Roberts, ti**@probo.com
Providenza & Boekelheide, Inc.

[sheldonlg]

Tim Roberts wrote:

Da***********@gmail.com wrote:

>On 24 Jun., 06:37, Tim Roberts <[e-mail]:

>>[e-mail] wrote:

but i keeps showing me all of the pictures, even if the aids are
different. I looked in the manual, but I do not know what I am doing
wrong. Can someone please help me?
Where do you think you are "selecting" a special aid? This code goes
through all of the albums, and for each album, it prints all of the
pictures from that album. Translation: print all of the pictures.

and this:
if($photo_aid == $album_aid){
? There I check if the photo belongs to the current album, if it does
I print it. But it keeps printing all of the pictures even if aid is
different

The inner loop prints all of the pictures in the album "album_aid" --
that's true. However, because of the outer loop, that loop will be run
once for each album. So, by the time that chunk is done, every picture in
every album will be printed.

Are you saying that you see every picture multiple times?

Tim,I may be missing something as well, but as I see it the outer loop
only prints the names of all the albums, not their contents. The inner
loop should be selecting which content to print. See the comments below
inserted into his original code.

for($i=0;$i<count($albums);$i++){ // loop over all albums
echo "<br><b>".$albums[$i]['name']."</b><br>"; // Name is printed
$album_aid = $albums[$i]['aid']; // album linking parameter
for($j=0;$j<count($photos);$j++){ // loop over all photos
$photo_aid = $photos[$j]['aid']; // photo linking parameter
if($photo_aid == $album_aid){ // link parameters match test
echo $album_aid." ".$photo_aid; // show that they match
echo "<a href=\"".$photos[$j]['link']."\" // photo printed
<img src=\"".$photos[$j]
['src_small']."\" alt=\"Photo\" border=\"0\" /></a>";
}
}
echo "<br>";
}

Ican't find why this is happening to you, but from this logic that you
present it seems that a photo can belong to one, and only one, album.
So, why bother having two arrays? Why not simply have:
$albums[$i]['photo_list'] = $photos
where $photos is the array of photos belonging to that album? Do that
makes it that there is no need for looping over the inner array.

Also, as a suggestion for cleaner looking code why not try:

echo '<a href="' . $photos[$j]['link'] . '">' .
'<img src="' . $photos[$j]['src_small'] . '" ' .
'alt="Photo" border="0" /></a>';

That way you can avoid all those ugly quoting characters and it makes it
easier to find a missing or extra " in your code.

[Curtis]

On Tue, 24 Jun 2008 13:36:22 -0700, <Da***********@gmail.comwrote:

please I need help!
I tried this foreach but it doesn't change anything :(

What did you change? You failed to post the code. I don't think you
understand, Iván pointed out an entirely different (better) approach, not
just changing to foreach.

--
Curtis, http://dyersweb.com