PHP help

[dc1pop]

Hi all,

Quite new to php and learning more and more each day. Ive now come across a situation i need help with.

I have a list of hyper links (single letters). What i want to happen is that when the user clicks his chosen letter (link) for it to query on the database showing only match's for the said letter.

So basically im guessing i somehow need to parse the URL link somehow so i can do this.

Thanks,
Darren

 

[Barrucadu]

Assuming that you are using MySQL:

<?php
mysql_connect('username', 'password', 'server');
mysql_select_db('database');
$results = mysql_query('SELECT * FROM `table` WHERE `letter` = "'.$_GET['value'].'"')or die(mysql_error());
mysql_close();

$i = 0;
while($i < mysql_num_rows($result)){
     echo mysql_result($result, $i, 'field');
     $i++;
}

 

[dc1pop]

Indeed im using mySQL. I'll see what i can do with that code

Thanks ever so much for the help!

Darren

 

[Peter]

Although in essence what mikor has done is indeed correct, he has not validated the input and is just using it straight into the mysql query. Doing this is a quick way to get your application hacked.

Always validate something the user has supplied.

 

[dc1pop]

Although in essence what mikor has done is indeed correct, he has not validated the input and is just using it straight into the mysql query. Doing this is a quick way to get your application hacked.

Always validate something the user has supplied.

Thanks i guess il have to look up some validation aswell.

 

[Ray]

And the querie used to acess that connection would be the following:


Link = page.php?letter=a

<?php 
$value=$_GET[letter];

//MYSQL QUERIE HERE

?>

 

[dc1pop]

now im really confused.

When i set the hyperlink value what shall i put?

Also does this code look ok for pulling the data on the page that will show the rest of the data?

<?php

//Gets the database information to carry out SQL tasks
include ("includes/config.php");

//Includes functions for menu items
include ("includes/functions.php");


	//SQL query to gather data
	$sql = "SELECT * FROM `cprofile` WHERE `club` = "'.$_GET['value'].'";
	$result = mysql_query ($sql); 

	$i = 0;
		while($i < mysql_num_rows($result)){
			echo mysql_result($result, $i, 'field');
			
			data selection and display code would go here?
			
			$i++;
} 

?>

Thanks!

 

[Dan]

Your query is messing up the rest of the code. You set the quotes up wrong.

$sql = "SELECT * FROM `cprofile` WHERE `club` = '".$_GET['value']."'";

 

[dc1pop]

Your query is messing up the rest of the code. You set the quotes up wrong.

$sql = "SELECT * FROM `cprofile` WHERE `club` = '".$_GET['value']."'";

Havent actually tested this code yet so thanks for pointing it out so i can ajdust it.

Ive coded most of the other stuff i can do. Im only a newbie so learning everything i come across and need to create the site.

If anyone knows of any tutorials on the following subject also be grateful

- Splitting a page up so results are spread over a number of pages

Thanks to all!

Regards,
Darren

 

[Dan]

http://www.pixel2life.com/search/10/pagination/1/
Some of those should help you with making it into multiple pages.

 

[dc1pop]

http://www.pixel2life.com/search/10/pagination/1/
Some of those should help you with making it into multiple pages.

Excellent! Once again thanks to everyone. Just goes to show what a great community NPs is

 

[dc1pop]

---


Right ive just tried the following code:

function teamlist() {

	//Get letter of the club to display the
	//relevant page
	$value=$_GET[letter]; 


	//SQL query to gather data including the id
	//to pull out the correct data
	$sql = "SELECT * FROM `cprofile` WHERE `club` = '".$_GET['value']."'";  
	$result = mysql_query ($sql); 


	//While there are more results fill
	//the following variables
	while ($row = mysql_fetch_array($result))
	{
		$clubinfo= $row["club"];

		//Display the information as needed on the page
	echo '<table>';
	echo '<tr>';
	echo '<td> '.$clubinfo.' </td>';
	echo '</tr>';
	echo '</table>';
	}
}

It dont seem to work im not sure if the URL is passing the letter correctly. To get to this page ive set the url on the page before as:

listclub.php?letter=A

Thanks again.
Darren

 

[Keral_Patel]

listclub.php?letter=A

Make it listclub.php?A

And it will start working. Bcoz the value right now is coming as "letter=A"

And you are running the query with that value. While you only need to run it for "A"

 

[Dan]

Make it listclub.php?A

And it will start working. Bcoz the value right now is coming as "letter=A"

And you are running the query with that value. While you only need to run it for "A"

How will it know what A is if it's just there and doesn't have a variable name?

Don't put that code inside of a function. Your MySQL connection is probably not being passed into it because functions are annoying like that.

 

[Keral_Patel]

<?php
echo "<a href="listclub.php?A">A</a><a href="listclub.php?B">B</a>";
?>

Below the code for listclub.php

<?php

$pageid = $QUERY_STRING;
if ($QUERY_STRING == "")
{
$pageid = A;
}

echo "The passed variablee is $pageid";
//Now process the query here


?>

By default if nothing is passed it will take A

 

[Dan]

nick_mayhem, both of those codes have errors and that doesn't change anything about his problem.

dc1pop, there's really no reason to have that inside of a function and that's what's messing it up. Your MySQL resource handle is outside of the function and never passed to it. Take a look at variable scope.

 

[Keral_Patel]

Got it. Bcoz my PHP REGISTER GLOBALS = ON

 

[Peter]

in your function you have:-

$value=$_GET[letter];

yet when you use it within the sql you are still using $_GET[letter], you should simply be using $value. Also as stated before ensure you validate the input, if you only expect it to be 1 letter ensure it is in fact only 1 letter.

 

[dc1pop]

in your function you have:-

$value=$_GET[letter];

yet when you use it within the sql you are still using $_GET[letter], you should simply be using $value. Also as stated before ensure you validate the input, if you only expect it to be 1 letter ensure it is in fact only 1 letter.

Thanks for all your help. As the site is only on my localhost i'll add all validation rules to it later so i can get the basic script up and running to see if ti all works ok.

Thanks again all!