Sortering af output fra mysql

Da $sql bare en tekststring kan du lade ORDER BY delen blive tilsat af en if på din nye sorteringsoption.

Siden vil dog reloade for at SQL'en kan køre igen, og dermed skal du kopiere andre relevante variable op i din nye form som hidden felter.
f.eks mobile_id

Lidt kort skrevet af mig, men kan du se ideen?

Nej jeg er desværre ikke helt med, er stadig newbie til PHP.

Kan du evt. lave eller linke til et lille eksempel, så tror jeg nemmere jeg forstår hvad du mener :)

Jeg var godt klar over at siden blev nød til at reloade for at hente oplysningerne igen, men jeg er ikke helt med på det der med hidden felter osv.

Det med hidden felt er fordi du har lagt op til en ny lille form:
Når din side bliver kaldt anden gang af denne form vil du mangle input i $_GET['mobile_id']

Derfor skal din ekstra form udvides med;

Når du danner $sql skal du teste på om den alternative sortering er valgt:

Der testes på om get-variabel sorter findes og om den er = "m" for at give den nye sortering på sql'en. Ellers gives den normale sortering.

Nu har jeg jeg lige prøvet at sammensætte min kode med det du skrev, så den bliver således:

<?php

mysql_connect("localhost", "********", "********") or die(mysql_error());
mysql_select_db("database") or die(mysql_error());

// Sanisering af bruger input
$id = (int) $_GET['mobile_id'];

// SQL
$sql = 'SELECT
mobiledeals.*, mobileplans.*, mobilenetworks.*
FROM mobiledeals
INNER JOIN mobileplans ON mobileplans.id = mobiledeals.mobileplans_id
INNER JOIN mobilenetworks ON mobilenetworks.id = mobileplans.network_id
WHERE mobiledeals.mobilephone_id = '.$id;

if (isset($_GET['sorter']) && $_GET['sorter'] == 'm') {
    $sql += ' ORDER BY (mobileplans.abonnement*mobileplans.binding)+mobilenetworks.oprettelse+mobiledeals.price ASC';
} else {
    $sql += ' ORDER BY mobiledeals.price, (mobileplans.abonnement*mobileplans.binding)+mobilenetworks.oprettelse+mobiledeals.price ASC';
}
// Query
$query = mysql_query($sql) or die(mysql_error());

// Generer resultat
$result = array();
while($row = mysql_fetch_assoc($query)) {
$result[] = $row;
}
?>

<div class="vaelg"><form action="" method="GET"><input type="hidden" name="mobile_id" value="<?php echo $id; ?>" /><select onchange="this.form.submit();" name="sorter"><option value="">Vælg sortering</option><option value="m">Sorter på mindstepris</option><option value="b">Sorter på købspris</option></select></form></div>

<!-- HTML Template -->
<table class="data" cellpadding="0" cellspacing="0" border="0">
<tr>
<td class="hd" style="width: 130px;"></td>
<td class="hd" style="width: auto;"></td>
<td class="hd" style="width: 145px;"></td>
</tr>
<?php $bgcolor="#f1f1f1";?>
<?php foreach($result as $row): ?>
<tr onclick="window.open('<?php echo $row['link1']; ?>');" class="trhand" bgcolor="<?php echo $bgcolor; ?>" onmouseover="this.bgColor = '#e3f4f8';" onmouseout ="this.bgColor = '<?php echo $bgcolor; ?>';return true;">
<?php $bgcolor = ($bgcolor=="#f1f1f1" ? "#ffffff" : "#f1f1f1"); ?>
<td class="dataclass_udbyder">
<img src="img/teleselskaber/<?php echo $row['logo']; ?>" alt="<?php echo $row['network_name']; ?>" border="1" />
</td>
<td class="dataclass">
<div class="dataover"><? echo $row['name']; ?></div>
<div class="dataclass2"><b>Minuttakst:</b> DKK <?php echo $row['minuttakst']; ?><br /><b>Taksering:</b> <?php echo $row['taksering']; ?><br /><b>Opkaldsafgift:</b> DKK <?php echo $row['opkaldsafgift']; ?></div>
<div class="dataclass2"><b>SMS pris:</b> DKK <?php echo $row['sms']; ?><br /><b>MMS pris:</b> DKK <?php echo $row['mms']; ?><br /><b>Data pris:</b> DKK <?php echo $row['data']; ?></div>
<div class="dataclass2"><b>Abonnement:</b> DKK <?php echo $row['abonnement']; ?>,-/md.<br /><b>Binding:</b> <?php echo $row['binding']; ?> mdr.<br /><b>Oprettelse:</b> DKK <?php echo $row['oprettelse']; ?>,-</div>
</td>
<td class="dataclass_koeb rh">
<div class="pris">Pris DKK <?php echo $row['price']; ?>,-</div><img src="img/koebnu.png" alt="Køb nu!" /><br /><div class="mindstepris">Mindstepris DKK <?php $sum_total = ( $row['abonnement'] * $row['binding'] ) + $row['oprettelse'] + $row['price']; print ($sum_total); ?>,-</div>
</td>
</tr>
<?php endforeach; ?>
</table>

Men når jeg køre koden nu, får jeg denne fejl, som jeg ikke rigtig forstår:

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '0' at line 1

Men ellers gav forklaringen med et hidden felt god mening, så tak for det :)

Nu har jeg jeg lige prøvet at sammensætte min kode med det du skrev, så den bliver således:

<?php

mysql_connect("localhost", "********", "********") or die(mysql_error());
mysql_select_db("database") or die(mysql_error());

// Sanisering af bruger input
$id = (int) $_GET['mobile_id'];

// SQL
$sql = 'SELECT
mobiledeals.*, mobileplans.*, mobilenetworks.*
FROM mobiledeals
INNER JOIN mobileplans ON mobileplans.id = mobiledeals.mobileplans_id
INNER JOIN mobilenetworks ON mobilenetworks.id = mobileplans.network_id
WHERE mobiledeals.mobilephone_id = '.$id;

if (isset($_GET['sorter']) && $_GET['sorter'] == 'm') {
    $sql += ' ORDER BY (mobileplans.abonnement*mobileplans.binding)+mobilenetworks.oprettelse+mobiledeals.price ASC';
} else {
    $sql += ' ORDER BY mobiledeals.price, (mobileplans.abonnement*mobileplans.binding)+mobilenetworks.oprettelse+mobiledeals.price ASC';
}
// Query
$query = mysql_query($sql) or die(mysql_error());

// Generer resultat
$result = array();
while($row = mysql_fetch_assoc($query)) {
$result[] = $row;
}
?>

<div class="vaelg"><form action="" method="GET"><input type="hidden" name="mobile_id" value="<?php echo $id; ?>" /><select onchange="this.form.submit();" name="sorter"><option value="">Vælg sortering</option><option value="m">Sorter på mindstepris</option><option value="b">Sorter på købspris</option></select></form></div>

<!-- HTML Template -->
<table class="data" cellpadding="0" cellspacing="0" border="0">
<tr>
<td class="hd" style="width: 130px;"></td>
<td class="hd" style="width: auto;"></td>
<td class="hd" style="width: 145px;"></td>
</tr>
<?php $bgcolor="#f1f1f1";?>
<?php foreach($result as $row): ?>
<tr onclick="window.open('<?php echo $row['link1']; ?>');" class="trhand" bgcolor="<?php echo $bgcolor; ?>" onmouseover="this.bgColor = '#e3f4f8';" onmouseout ="this.bgColor = '<?php echo $bgcolor; ?>';return true;">
<?php $bgcolor = ($bgcolor=="#f1f1f1" ? "#ffffff" : "#f1f1f1"); ?>
<td class="dataclass_udbyder">
<img src="img/teleselskaber/<?php echo $row['logo']; ?>" alt="<?php echo $row['network_name']; ?>" border="1" />
</td>
<td class="dataclass">
<div class="dataover"><? echo $row['name']; ?></div>
<div class="dataclass2"><b>Minuttakst:</b> DKK <?php echo $row['minuttakst']; ?><br /><b>Taksering:</b> <?php echo $row['taksering']; ?><br /><b>Opkaldsafgift:</b> DKK <?php echo $row['opkaldsafgift']; ?></div>
<div class="dataclass2"><b>SMS pris:</b> DKK <?php echo $row['sms']; ?><br /><b>MMS pris:</b> DKK <?php echo $row['mms']; ?><br /><b>Data pris:</b> DKK <?php echo $row['data']; ?></div>
<div class="dataclass2"><b>Abonnement:</b> DKK <?php echo $row['abonnement']; ?>,-/md.<br /><b>Binding:</b> <?php echo $row['binding']; ?> mdr.<br /><b>Oprettelse:</b> DKK <?php echo $row['oprettelse']; ?>,-</div>
</td>
<td class="dataclass_koeb rh">
<div class="pris">Pris DKK <?php echo $row['price']; ?>,-</div><img src="img/koebnu.png" alt="Køb nu!" /><br /><div class="mindstepris">Mindstepris DKK <?php $sum_total = ( $row['abonnement'] * $row['binding'] ) + $row['oprettelse'] + $row['price']; print ($sum_total); ?>,-</div>
</td>
</tr>
<?php endforeach; ?>
</table>

Men når jeg køre koden nu, får jeg denne fejl, som jeg ikke rigtig forstår:

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '0' at line 1

Men ellers gav forklaringen med et hidden felt god mening, så tak for det :)

Hov, det var ikke meningen den skulle komme to gange :0)

Jeg tror min PHP er rusten
Så prøv at rette de 2 += til .=

Og da $sql jo bare er en string skal vi kunne bygge den op i flere statements. Så du må prøve dig frem i steps, hvis ovenstående ikke løser det:

Hvis det originale virker:


så prøv at klippe den op i 2:

Hvis det stadig virker kan du bygge if/else omkring det.

Hvis det ikke virker må du skrive $sql til skærmen og kigge nærmere på den:

Derudover tror jeg du skal have noget i action="" i din nye form.

Efter at have ændret fra + til . virker det som det skal. Også selvom jeg intet har i min action="" :)

Men smid lige et svar og mange tak for hjælpen :)

Hermed svar.
Godt du fik det til at virke.