CIO Tech Eksperten IT-JOB IT-Kurser Events Podcast Søg
Cookie ikon
Avatar billede techboy992 Praktikant
05. oktober 2012 - 23:59 Der er 6 kommentarer og
1 løsning

Hvordan Kan jeg lave et link fra en mysql database

Det jeg gerne vil have det til at goere er at:
Lave et link fra tre felter
Mine felter i databasen er:
link, name, active

Mit script til nu er:
<?php
$con = mysql_connect("host","database","password");
if (!$con)
  {
  die('Could not connect: ' . mysql_error());
  }
$row = mysql_fetch_assoc($result);
echo '<a href="'.$row["link"].'">'.$row["name"].'</a>';
?>

Men giver fejl:
Warning: mysql_fetch_assoc(): supplied argument is not a valid MySQL result resource in /home/a3485397/public_html/topsites/view.php on line 7


Det jeg gerne vil have det til er at vise en raekke links til topsites.

Men skal kun vise links hvis active er = YES
som standart i min form til at adde active er sat til NO
Avatar billede erikjacobsen Ekspert
06. oktober 2012 - 00:16 #1
Ser ud som om du mangler SQL-sætningen
Avatar billede techboy992 Praktikant
06. oktober 2012 - 00:58 #2
Jeg fik til at virke med foelgende:
$con = mysql_connect("host","database","password");
if (!$con)
  {
  die('Could not connect: ' . mysql_error());
  }
mysql_select_db("database", $con);

$result = mysql_query("SELECT * FROM table");

while($row = mysql_fetch_array($result))
  {
echo '| <a href="'.$row["link"].'">'.$row["name"].'</a> |';
  }

mysql_close($con);
?>

Mit eneste problem nu er at faa det til kun at vise aktive link fra feltet active

Jeg har proevet foelgende:
mysql_select_db("database", $con);

$result = mysql_query("SELECT * FROM Topsites WHERE active = YES");

while($row = mysql_fetch_array($result))
  {
echo '| <a href="'.$row["link"].'">'.$row["name"].'</a> |';
  }

mysql_close($con);
?>

Men siger unespectet t_strin on line 9
Avatar billede olebole Juniormester
06. oktober 2012 - 01:47 #3
<ole>

Du mangler apostroffer omkring YES i SQL'en. Det er en strengværdi og skal derfor escapes.

Det lader til, at du lige har taget fat på MySQL. I så fald giver det ikke mening at rode med det gamle MySQL-API. Det er forældet og afløst af MySQLI.

Du kan prøve at kikke på denne guide om prepared statements under MySQLI

/mvh
</bole>
Avatar billede techboy992 Praktikant
06. oktober 2012 - 05:47 #4
Ja det er faktisk mit foerste script i det hele taget i PHP og Mysql.

Jeg fandt ud af det selv efter 4 timers search i google og nu virker det fint, men fordi du var her foer mig saa skal du alligevel have dine point.

Kan du sende et indlaeg saa du kan faa dem
Avatar billede techboy992 Praktikant
06. oktober 2012 - 05:54 #5
Det faerdig script:
form.html:
<html>
<body>

<form action="insert.php" method="post">
Link:        <input type="text" name="link" /><br>
Website name: <input type="text" name="name" /><br>
Secret:      <input type="password" name="active" /><br>
<input type="submit" value="Post"/>
</form>

</body>
</html>

insert.php:
<?php
$con = mysql_connect("host","Database","password");
if (!$con)
  {
  die('Could not connect: ' . mysql_error());
  }

mysql_select_db("database", $con);

$sql="INSERT INTO Topsites (link, name, active)
VALUES
('$_POST[link]','$_POST[name]','$_POST[active]')";

if (!mysql_query($sql,$con))
  {
  die('Error: ' . mysql_error());
  }
echo "1 record added";

mysql_close($con);
?>

view.php:
<?php
$con = mysql_connect("host","database","password");
if (!$con)
  {
  die('Could not connect: ' . mysql_error());
  }
mysql_select_db("database", $con);

$result = mysql_query("SELECT * FROM Topsites where active='1011'");

while($row = mysql_fetch_array($result))
  {
echo '| <a href="'.$row["link"].'" target="_blank">'.$row["name"].'</a> |';
  }

mysql_close($con);
?>
Avatar billede olebole Juniormester
06. oktober 2012 - 13:26 #6
Ellers tak, jeg samler ikke point. Du accepterer bare dit eget svar, så tråden lukkes (Erik samler heller ikke)  =)

Fedt, du fik det til at virke, men jeg synes stadig, du bør kikke på den guide, jeg linkede til. Den kode, du har nu, står faktisk pivåben for SQL-injections, som er angreb, hvor brugeren skriver noget skadeligt i et formfelt. Det gør, at angriberen kan få vist indholdet af hele eller dele af din database - og/eller ændre/slette den. I dag bruges det gamle API - som er det, du i øjeblikket bruger - ikke til 'nybygninger'.
Avatar billede techboy992 Praktikant
06. oktober 2012 - 21:06 #7
Jeg vil kigge guiden til naeste script :)
Vil helst ikke rode med det nu hvor det virker :)

Men tusinde tak for din hjaelp.
Avatar billede Ny bruger Nybegynder

Din løsning...

Tilladte BB-code-tags: [b]fed[/b] [i]kursiv[/i] [u]understreget[/u] Web- og emailadresser omdannes automatisk til links. Der sættes "nofollow" på alle links.

Loading billede Opret Preview
Kategori
Webudvikling kurser
Vi tilbyder markedets bedste kurser inden for webudvikling
Se alle Webudvikling kurser

Flere spørgsmål fra PHP kategorien

Titel Indlæg Oprettet Seneste aktivitet
Hjælp til PHP ( mener jeg det hedder) Af lurup i PHP 0 12/09/202423:39 -
wp_delete_user virker ikke i Wordpress Af Ravnsen i PHP 2 21/08/202417:53 21/08/202422:03
php driller Af moscov i PHP 1 19/08/202415:09 20/08/202421:24
PHP alfabet Af John i PHP 4 10/08/202415:17 10/08/202417:47
Hent array key Af nemlig i PHP 3 17/05/202415:22 17/05/202416:30
Se alle spørgsmål i kategorien Opret spørgsmål

Log ind eller opret profil

Hov!

For at kunne deltage på Computerworld Eksperten skal du være logget ind.

Det er heldigvis nemt at oprette en bruger: Det tager to minutter og du kan vælge at bruge enten e-mail, Facebook eller Google som login.

Opret Log ind

Du kan også logge ind via nedenstående tjenester



Alle kategorier på Eksperten
Seneste spørgsmål Seneste aktivitet
I dag 13:35 NVIDIA Sheild Pro Af FCK i Musik & videoafspillere
I dag 13:01 Undgå matriksformel? Af Dan Elgaard i Excel
I går 21:33 Ingen reaktion på Kraks Kort Af ErikHg i Søgemaskiner
I går 21:06 iTop Screen Recorder Free Dansk Af eksmojo i Andet software
I går 12:29 Kan ikke fide SSD cards Af JohnVamos i PC
White papers
Flere white papers »


Premium
Teaser billede
Sådan får hackerne med ny fidus adgang til dit system for kun 10 dollar: Næsten umuligt at beskytte sig
Læs mere »
Kommunernes it-forbrug stiger: Sådan har Odense Kommune holdt sine udgifter til it-licenser og it-konsulenter nede
Tidligere Cibicom-topchef Michael Meister har landet nyt job: Skal være administrerende direktør
Styrelse kom med tastefejl til at lægge tre år for meget på it-kontrakt til 350 millioner kroner
Se alle »
Computerworld
Teaser billede
Om lidt går det løs: Så banebrydende bliver iPhone 16
Læs mere »
iPhone 16 og 16 Pro slippes løs: Her er de to afgørende nyheder
Apple ændrer grundlæggende på den måde, som din iPhone skal opdateres på
Test: Her får du lynhurtig wi-fi 7 bredbånd til lavpris
Se alle »
CIO
Teaser billede
Nu kommer næste store version af Copilot
Læs mere »
Syv vigtige teknologier, som alle CIO'er bør forholde sig til - kommer til at sætte dagsordenen
Søstrene Grene skifter Microsoft ud med SAP og rykker for første gang i clouden i stor ERP-transformation
Microsoft har fundet 79 sårbarheder i Windows – du bør opdatere straks
Se alle »
Job & Karriere
Teaser billede
Pludselig exit fra Schultz kom som en overraskelse for Merete Søby: Derfor stoppede samarbejdet
Læs mere »
Merete Søby fratræder med omgående virkning som CEO for Schultz efter blot tre måneder på posten
Professor: "Det er på høje tid at opløse virksomhedernes it-afdelinger"
Kom tæt på Danmarks nye digitaliserings-minister: 33-årige Caroline Stage Olsen er tidligere tobaks-lobbyist - har været aktiv i politik siden hun var helt ung
Se alle »
White paper
Teaser billede
Opdag fordelene ved cloud-baseret backup og recovery
Læs mere »
Nemmere overblik, styring og omkostningskontrol i dit multicloud-miljø
Rapport kortlægger de 13 bedste muligheder for at sætte turbo på din cloud computing
Informationshåndtering på frontlinjen: Sådan optimerer du med automatisering
Se alle »

Events
Flere events »
White papers
Flere white papers »
IT-Kurser
Se alle vores it-kurser »