help with html+php

champ_rock · 05-15-2007, 01:57 AM

hi

as always please tell me what am i doing wrong in this?

Code:

<?php

$list = file('list.txt');
srand((float)microtime() * 1000000);
shuffle($list);

echo '<html><br><select multiple name="crap" size="10">';

foreach ($list as $url) {
$u= "<option value='$url' onclick=\"window.location='$url'\"><b>$url</b></option>";
echo $u;
}
echo '</select>';
?>

out put of this http://www.champrock.uk.to/demo/list.php

this outputs the html code as

Code:

<html><br><select multiple name="crap" size="10"><option value='http://mail.yahoo.com
' onclick="window.location='http://mail.yahoo.com
'"><b>http://mail.yahoo.com
</b></option><option value='http://gmail.com
' onclick="window.location='http://gmail.com
'"><b>http://gmail.com
</b></option><option value='http://yahoo.com
' onclick="window.location='http://yahoo.com
'"><b>http://yahoo.com
</b></option><option value='http://airb.us.to
' onclick="window.location='http://airb.us.to
'"><b>http://airb.us.to
</b></option><option value='http://google.com
' onclick="window.location='http://google.com
'"><b>http://google.com
</b></option></select>

if i rearrange this code properly and run it as a html file then it works fine.

therefore i guess the problem is with these arrangement and the new lines that are being created before ' onclick

thanks

paaaaaaaaaa · 05-15-2007, 03:04 AM

those line breaks are breaking the javascript. In your text file don't have the list on new lines or remove line breaks with some php. You could separate each value with a comma and then use implode/explode. Or you could use a string replace to remove line breaks in each url value.

Dan · 05-15-2007, 03:31 AM

Add
$url = trim($url);

After
foreach ($list as $url) {

champ_rock · 05-15-2007, 04:48 AM

thanks guys this trim() works..

also, please tell me how do i open it in a new window on being clicked?

one more thing, i am feeling that i am miserable in php. do u think i should even think about continuing it? every script it create has some bug which i have to get sorted here by your help.

cef · 05-15-2007, 03:10 PM

Quote:

Originally Posted by champ_rock

thanks guys this trim() works..

also, please tell me how do i open it in a new window on being clicked?

In your "onclick" code, instead of

PHP Code:

  window.location='$url'

use

PHP Code:

  window.open('$url');

Quote:

one more thing, i am feeling that i am miserable in php. do u think i should even think about continuing it? every script it create has some bug which i have to get sorted here by your help.

Ah, keep going

I've been doing this for years (decades actually) and I still keep the manual open because I never remember things like the parameter order to str_replace()

Like anything, it gets easier over time. But I know how frustrating it is when just starting out with a new language.

Besides, it seems like you have a head for this stuff, so don't give up; just keep having fun and asking questions.

Barrucadu · 05-15-2007, 03:16 PM

Keep at it, all languages seem hard at first (though I would rather learn another programming language than do my french gcse, yikes!)

Quote:

Originally Posted by cef

I still keep the manual open because I never remember things like the parameter order to str_replace()

Good to know i'm not the only one who forgets parameter order in common functions.

cef · 05-15-2007, 04:26 PM

Yeah, the dirty little secret of our industry, eh?

mikesherov · 05-16-2007, 08:06 AM

parameter order was an issue until I started using a IDE that did code completion... it pops up the parameter order as soon as I type the first "(" in "str_replace("...

gone are the days of memorizing "search,replace,subject"

champ_rock · 05-17-2007, 12:31 AM

Code:

 window.open('$url');

this does not work

please check at http://www.champrock.uk.to/demo/list.php

cef · 05-17-2007, 01:07 AM

Quote:

Originally Posted by champ_rock

Code:

 window.open('$url');

this does not work

please check at http://www.champrock.uk.to/demo/list.php

If you look at the source for your page in the browser, you'll see lines like:

Code:

onclick="window.open='http://gmail.com'"

Which is invalid Javascript syntax. It should look like

Code:

onclick="window.open('http://gmail.com');"

You need to fix the php code which is generating the statements.

champ_rock · 05-17-2007, 01:11 AM

well please check the syntax now.. i think i have rectified it

the php code is

Code:

	$u= "<option value='$url' onclick=\"window.open=('$url')\"><b>$url</b></option>";

.

but it still does not work

cef · 05-17-2007, 07:30 AM

It's still wrong

There's an extra = sign in there. It should be

Code:

	$u= "<option value='$url' onclick=\"window.open('$url')\"><b>$url</b></option>";

champ_rock · 05-17-2007, 08:03 AM

thanks dude it works..

its tough to debug ur own code. i compared those two code lines and i could not find the difference until the third time

cef · 05-17-2007, 03:23 PM

It happens; sometimes your eyes do one thing and your brain another

Glad it worked out.

05-15-2007, 03:04 AM	#2 (permalink)
paaaaaaaaaa NamePros Regular Join Date: Mar 2005 Posts: 412 162.25 NP$ (Donate)	those line breaks are breaking the javascript. In your text file don't have the list on new lines or remove line breaks with some php. You could separate each value with a comma and then use implode/explode. Or you could use a string replace to remove line breaks in each url value. __________________ Please add to my rep points if i was helpful. Thanks. Ripe Website Manager - Steve Parish

05-15-2007, 04:48 AM	#4 (permalink)
champ_rock Senior Member Join Date: Oct 2006 Location: http://akshayjain.org Posts: 2,768 8,037.05 NP$ (Donate)	thanks guys this trim() works.. also, please tell me how do i open it in a new window on being clicked? one more thing, i am feeling that i am miserable in php. do u think i should even think about continuing it? every script it create has some bug which i have to get sorted here by your help. __________________ Akshay Jain ......................

05-17-2007, 12:31 AM	#9 (permalink)
champ_rock Senior Member Join Date: Oct 2006 Location: http://akshayjain.org Posts: 2,768 8,037.05 NP$ (Donate)	Code: window.open('$url'); this does not work please check at http://www.champrock.uk.to/demo/list.php __________________ Akshay Jain ......................

05-17-2007, 01:11 AM	#11 (permalink)
champ_rock Senior Member Join Date: Oct 2006 Location: http://akshayjain.org Posts: 2,768 8,037.05 NP$ (Donate)	well please check the syntax now.. i think i have rectified it the php code is Code: $u= "<option value='$url' onclick=\"window.open=('$url')\"><b>$url</b></option>"; . but it still does not work __________________ Akshay Jain ......................

05-17-2007, 07:30 AM	#12 (permalink)
cef NamePros Regular Join Date: May 2004 Location: NYC Posts: 236 76.50 NP$ (Donate)	It's still wrong There's an extra = sign in there. It should be Code: $u= "<option value='$url' onclick=\"window.open('$url')\"><b>$url</b></option>";

05-15-2007, 03:31 AM	#3 (permalink)
Dan Buy my domains. Join Date: Feb 2006 Posts: 2,801 56.00 NP$ (Donate)	Add $url = trim($url); After foreach ($list as $url) {

05-15-2007, 04:26 PM	#7 (permalink)
cef NamePros Regular Join Date: May 2004 Location: NYC Posts: 236 76.50 NP$ (Donate)	Yeah, the dirty little secret of our industry, eh?

05-16-2007, 08:06 AM	#8 (permalink)
mikesherov NamePros Member Join Date: Apr 2005 Posts: 116 134.00 NP$ (Donate)	parameter order was an issue until I started using a IDE that did code completion... it pops up the parameter order as soon as I type the first "(" in "str_replace("... gone are the days of memorizing "search,replace,subject"

05-17-2007, 08:03 AM	#13 (permalink)
champ_rock Senior Member Join Date: Oct 2006 Location: http://akshayjain.org Posts: 2,768 8,037.05 NP$ (Donate)	thanks dude it works.. its tough to debug ur own code. i compared those two code lines and i could not find the difference until the third time __________________ Akshay Jain ......................

05-17-2007, 03:23 PM	#14 (permalink)
cef NamePros Regular Join Date: May 2004 Location: NYC Posts: 236 76.50 NP$ (Donate)	It happens; sometimes your eyes do one thing and your brain another Glad it worked out.

Currently Active Users Viewing This Thread: 1 (0 members and 1 guests)