Vinnaren i pepparkakshustävlingen!
  • 2
  • 3
2008-12-12, 11:33
  #25
Medlem
CamoDrakes avatar
Kod:
<?php

//Ta med $cid i select_db
mysql_select_db($db$cid)

//och när du kör querys:

if (ctype_digit($annonsid)){    
  
$query "SELECT * FROM ads WHERE id = '$annonsid'";   
  
$lolzor mysql_query($query$cid)or die(mysql_error());   
} else {   
  die (
"Ett fel uppstod!");   
}

?>

/Camodrake
Citera
2008-12-14, 17:47
  #26
Medlem
Hasztexs avatar
Okey.
Kolla så jag har förstått rätt, för jag får bara:
"Ett fel uppstod!"

Det som du har skrivit in i "else".

Här är hela koden:

Kod:
<title>Prylpiraten.se</title>
<link href="stylesheet.css" rel="stylesheet" type="text/css" />
<?
    $usr 
"usr";
    
$pwd "passwd";
    
$db "ad_database";
    
$host "localhost";

    
# connect to database
    
$cid mysql_connect($host,$usr,$pwd);
    
mysql_select_db($db$cid);
    if (
mysql_error()) { print "Database ERROR: " mysql_error(); }

?> 
<table width="1001" border="0" cellpadding="0" cellspacing="0" class="content">
  <tr>
    <td width="852" height="100" class="top"><img src="img/logo_small.png" width="350" height="100" /></td>
    <td width="178" class="menu">&nbsp;</td>
    <td width="12">&nbsp;</td>
  </tr>
  <tr>
    <td height="14" class="searchfield">&nbsp;</td>
    <td class="menu">&nbsp;</td>
    <td>&nbsp;</td>
  </tr>
<?php
$annonsid 
$_GET['ID'];

if (
ctype_digit($annonsid)){    
  
$query "SELECT * FROM ads WHERE id = '$annonsid'";   
  
$lolzor mysql_query($query$cid)or die(mysql_error());   
} else {   
  die (
"Ett fel uppstod!");   
}

$resultat mysql_fetch_assoc($lolzor);

$resultat['titel'];
$resultat['beskrivning'];
?>   
<tr>  
    <td height="699" valign="top" class="ads">  
      <? echo $resultat['titel']; ?>  
    </td>   
    <td valign="top" class="menu">&nbsp;• <a href="add.php"><font color="red">Lägg in din annons!</font></a><br />
    &nbsp;• <a href="index.php">Säljes</a><br />
    &nbsp;• <a href="index.php">Köpes</a><br />
    &nbsp;• <a href="#">Om</a></td>
    <td>&nbsp;</td>
  </tr>
  <tr>
    <td class="copyright"><p>&nbsp;Ansvarig utgivare: <br />
      &nbsp;&copy; Annons</p>    </td>
    <td>&nbsp;</td>
    <td>&nbsp;</td>
  </tr>
</table>
Citera
2008-12-15, 14:40
  #27
Medlem
Hasztexs avatar
Ska

Kod:
$resultat['titel'];
$resultat['beskrivning']; 

stå inom { } eller vad är det som är fel. Jag kan verkligen inte fatta vad som är fel.
Citera
2008-12-15, 15:06
  #28
Medlem
TMSs avatar
Citat:
Ursprungligen postat av Hasztex
Ska

Kod:
$resultat['titel'];
$resultat['beskrivning']; 

stå inom { } eller vad är det som är fel. Jag kan verkligen inte fatta vad som är fel.

Nej det ska dem inte, test sätt "echo" före dem.

Kod:
echo $resultat['titel'];
echo 
$resultat['beskrivning']; 

Om du bara får upp "Ett fel uppstod" så får du börja felsöka, spotta ut $_GET['ID']; och se vad den innehåller, är det verkligen en siffra? Testa ta bort if-satsen helt om det är en siffra etc etc.

Sedan bör du flytta hela detta
Kod:
$annonsid $_GET['ID'];

if (
ctype_digit($annonsid)){    
  
$query "SELECT * FROM ads WHERE id = '$annonsid'";   
  
$lolzor mysql_query($query$cid)or die(mysql_error());   
} else {   
  die (
"Ett fel uppstod!");   
}

$resultat mysql_fetch_assoc($lolzor); 

ovanför html-koden åtminstone.

Hela koden ser jäkligt slafsigt ihop satt ut, ser ut som du klippt och klistrat hej vilt utan tanke bakom? Vart är <body> taggen etc?
__________________
Senast redigerad av TMS 2008-12-15 kl. 15:09.
Citera
2008-12-15, 17:47
  #29
Medlem
Hasztexs avatar
Citat:
Ursprungligen postat av TMS
Nej det ska dem inte, test sätt "echo" före dem.

Kod:
echo $resultat['titel'];
echo 
$resultat['beskrivning']; 

Om du bara får upp "Ett fel uppstod" så får du börja felsöka, spotta ut $_GET['ID']; och se vad den innehåller, är det verkligen en siffra? Testa ta bort if-satsen helt om det är en siffra etc etc.

Sedan bör du flytta hela detta
Kod:
$annonsid $_GET['ID'];

if (
ctype_digit($annonsid)){    
  
$query "SELECT * FROM ads WHERE id = '$annonsid'";   
  
$lolzor mysql_query($query$cid)or die(mysql_error());   
} else {   
  die (
"Ett fel uppstod!");   
}

$resultat mysql_fetch_assoc($lolzor); 

ovanför html-koden åtminstone.

Hela koden ser jäkligt slafsigt ihop satt ut, ser ut som du klippt och klistrat hej vilt utan tanke bakom? Vart är <body> taggen etc?

Citat:
Om du bara får upp "Ett fel uppstod" så får du börja felsöka, spotta ut $_GET['ID']; och se vad den innehåller, är det verkligen en siffra? Testa ta bort if-satsen helt om det är en siffra etc etc.

Vad menar du med det?
Det är bara siffror ja.
Citat:
Hela koden ser jäkligt slafsigt ihop satt ut, ser ut som du klippt och klistrat hej vilt utan tanke bakom? Vart är <body> taggen etc?

Jag vet.
Jag prioriterar inte stylen för tillfället. Vill helst att php koderna ska fungera först. Grafiken klarar jag själv. Webbdesign har jag gått på gymnasiet. Dock inte php-programmering.
Förstår lite php, men inte tillräckligt
Citera
2008-12-15, 18:32
  #30
Medlem
Traktes avatar
Citat:
Ursprungligen postat av Hasztex
Jag vet.
Jag prioriterar inte stylen för tillfället. Vill helst att php koderna ska fungera först. Grafiken klarar jag själv. Webbdesign har jag gått på gymnasiet.

Jag tror inte det var det han menade.
Tror han snarare menade ordningen på koden, bra ordning på koden och fullständig kod gör det lättare att hitta fel när man felsöker allt om något går galet.
Jag har för vana att kommentera ganska mycket i min kod och dela in det i olika stycken osv.
Citera
2008-12-15, 18:42
  #31
Medlem
Citat:
Ursprungligen postat av Hasztex
Vad menar du med det?
Det är bara siffror ja.
Vad står i adressraden när du laddar in sidan i webläsaren? Exakta, fullständiga adressen alltså.
Citera
2008-12-15, 18:52
  #32
Medlem
Hasztexs avatar
Citat:
Ursprungligen postat av Koenigsegg
Vad står i adressraden när du laddar in sidan i webläsaren? Exakta, fullständiga adressen alltså.

http://adress.com/annons/view.php?id=29

Sen självklart står det andra ID när jag klickar på andra annonser
Citera
  • 2
  • 3

Stöd Flashback

Flashback finansieras genom donationer från våra medlemmar och besökare. Det är med hjälp av dig vi kan fortsätta erbjuda en fri samhällsdebatt. Tack för ditt stöd!

Stöd Flashback