hjælp til at slette fra en tabel

Du skal til at starte med have sat et stel-link på:

$foresp = mysql_query("SELECT id, overskrift FROM artikler_avis ORDER BY id");
while ($data = mysql_fetch_array($foresp))
{
    echo "<b>" . $data["overskrift"] . "</b> <a href="slet.php?id=" . $data["id"] . "</a><br>";
}

Derefter skal du have lavet en slet.php som kan se nogenlunde sådan her ud:

$id = $_GET["id"];
if (isset($id))
{
    $sql = "DELETE FROM artikler_avis WHERE id=" . $id;
    mysql_query($sql) or die(mysql_error());
}

kan du ikke lave en hel kode? er ikke vildt god til php:P

Ok, men du bliver selv nødt til at fylde ud med password og den slags oplysninger...

Jeg gætter på at den del af koden findes i "config.php".

Så burde dette være nok:

<?
include("config.php");

// Slet her.

$id = $_GET["id"];
if (isset($id))
{
    $sql = "DELETE FROM artikler_avis WHERE id=" . $id;
    mysql_query($sql) or die(mysql_error());
}

// Vis data her

$sql = "SELECT overskrift FROM artikler_avis ORDER BY id";
$result = mysql_query($sql) or die(mysql_error());

while ($data = mysql_fetch_assoc($result))
{
    echo "<b>" . $data["overskrift"] . "</b> <a href='" . $_SERVER['PHP_SELF'] . "?id=" . $data["id"] . "'>Slet</a><br>";
}
?>

Altså du gør bare som nielle siger, altså opret slet.php:

<?
$id = $_GET["id"];
if (isset($id))
{
    $sql = "DELETE FROM artikler_avis WHERE id=" . $id;
    mysql_query($sql) or die(mysql_error());
}

$foresp = mysql_query("SELECT id, overskrift FROM artikler_avis ORDER BY id");
while ($data = mysql_fetch_array($foresp))
{
    echo "<b>" . $data["overskrift"] . "</b> <a href="slet.php?id=" . $data["id"] . "</a><br>";
}
?>

Jeg ved ikke lige hvordan du kører dine sider mht. connect osv.

nielle - jeg får bare denne her meddelelse
You have an error in your SQL syntax near '' at line 1

sååh det virker ikke?

Det lyder som om at der er en fejl i din config.php fil...

neej tror at der er fejl i din kode

den skriver nemlig ikke noget tal efter blabla.php?id=

skal den ikke det?

Jeg tvivler, men du kan da udvide koden med lidt debug:

<?
echo "DEBUG #0<br>";
include("config.php");

// Slet her.

$id = $_GET["id"];
if (isset($id))
{
    $sql = "DELETE FROM artikler_avis WHERE id=" . $id;
    echo "DEBUG #1: $sql<br>";
    mysql_query($sql) or die(mysql_error());
}

// Vis data her

$sql = "SELECT overskrift FROM artikler_avis ORDER BY id";
echo "DEBUG #2: $sql<br>";
$result = mysql_query($sql) or die(mysql_error());

while ($data = mysql_fetch_assoc($result))
{
    echo "<b>" . $data["overskrift"] . "</b> <a href='" . $_SERVER['PHP_SELF'] . "?id=" . $data["id"] . "'>Slet</a><br>";
}
?>

- og så kan du fortælle mig hvad den skriver.

Hvis der er fejl i config.php så stopper den inde at denm når at skrive noget ud. Så jo det kan godt være der den er gal.

den når at sprøge hvilken jeg vil slette og hente data fra databasen men når jeg trykker slet så skriver den bare:

DEBUG #0
DEBUG #1: DELETE FROM artikler_avis WHERE id=
You have an error in your SQL syntax near '' at line 1

sprørge*

spørge*

<?
echo "DEBUG #0<br>";
include("config.php");

// Slet her.

$id = $_GET["id"];
if (isset($id) && trim($id) != "")
{
    $sql = "DELETE FROM artikler_avis WHERE id=" . $id;
    echo "DEBUG #1: $sql<br>";
    mysql_query($sql) or die(mysql_error());
}

// Vis data her

$sql = "SELECT overskrift FROM artikler_avis ORDER BY id";
echo "DEBUG #2: $sql<br>";
$result = mysql_query($sql) or die(mysql_error());

while ($data = mysql_fetch_assoc($result))
{
    echo "<b>" . $data["overskrift"] . "</b> <a href='" . $_SERVER['PHP_SELF'] . "?id=" . $data["id"] . "'>Slet</a><br>";
}
?>

nu sker der ingenting når jeg trykker slet...

Prøv at smide resultatet af en Vis Kilde her.

tänk på sqlinjection nielle.

før eller efter at jeg trykker slet?

Nielle, du glemmer at selecte id-kolonnen, derfor finder den aldrig det felt :)

zynzz gør også (indirekte) opmærksom på det med hans query.

DEBUG #0<br>DEBUG #2: SELECT overskrift FROM artikler_avis ORDER BY id<br><b>test</b> <a href='/Geexie/fedesystemer/prove.php?id='>Slet</a><br><b>blabla</b> <a href='/Geexie/fedesystemer/prove.php?id='>Slet</a><br>

Rooolig nu ... en ting af gangen.

Sorry ;o)

04/07-2006 19:10:45> Det havde jeg husket at gøre i 04/07-2006 18:28:44 - men den forsvandt åbenbart igen.

<?
echo "DEBUG #0<br>";
include("config.php");

// Slet her.

$id = $_GET["id"];
if (isset($id) && trim($id) != "")
{
    $sql = "DELETE FROM artikler_avis WHERE id=" . $id;
    echo "DEBUG #1: $sql<br>";
    mysql_query($sql) or die(mysql_error());
}

// Vis data her

$sql = "SELECT overskrift, id FROM artikler_avis ORDER BY id";
echo "DEBUG #2: $sql<br>";
$result = mysql_query($sql) or die(mysql_error());

while ($data = mysql_fetch_assoc($result))
{
    echo "<b>" . $data["overskrift"] . "</b> <a href='" . $_SERVER['PHP_SELF'] . "?id=" . $data["id"] . "'>Slet</a><br>";
}
?>

confused - hvad er de datoer til før?

for*

Datoeren referer til tidligere indlæg i dette her spørgsmål :^)

DET VIRKER! 1000 tak! skriver du lige et svar så du kan få point?

Nej, vi er nemlig ikke helt færdig (04/07-2006 19:10:23). ;^)

hmm hvad mangler?

Der mangler sikkerhed mod at andre kan fuske med URL'en, og potentielt slette hele din database gennem den. :)

Her er det vel bare at bruge denne linie:

if (isset($id) && trim($id) != "" && is_numeric($id))

...og så også lige slette de debug-meddelelser.

jeg lægger den ind i et community og tilføjer en kode der gør at men SKAL være logged ind for at gøre det?

Noget i denne stil:

<?
echo "DEBUG #0<br>";
include("config.php");

// Slet her.

$id = $_GET["id"];
if (isset($id) && preg_match("/^\d+$/", $id))
{
    $sql = "DELETE FROM artikler_avis WHERE id=" . $id;
    echo "DEBUG #1: $sql<br>";
    mysql_query($sql) or die(mysql_error());
}

// Vis data her

$sql = "SELECT overskrift, id FROM artikler_avis ORDER BY id";
echo "DEBUG #2: $sql<br>";
$result = mysql_query($sql) or die(mysql_error());

while ($data = mysql_fetch_assoc($result))
{
    echo "<b>" . $data["overskrift"] . "</b> <a href='" . $_SERVER['PHP_SELF'] . "?id=" . $data["id"] . "'>Slet</a><br>";
}
?>

Det er ikke nok at du kræver login ... for hvordan sikre du at der ikke er en vandal som opretter en konto bare for at lave hærværk mod dit system?

jeg har slettet opret funktionen da det kun er min far der skal kunne lave ting.. men ellers hvad er din idé?

Når man laver den slags ting skal man sikre sig at man kan stole på dem som kan benytte de potentielt destruktive ting. Derfor vil man sædvanligvis lave et login-system hvor at ikke alle brugere nødvendigvis har de samme rettigheder.

F.eks. ville det være naturligt at det kun var din fader (og dig selv) som havde slette-rettigheder.

ja ved godt... men ville denne kode så ikke virke?:

<?

session_start();

if (session_is_registered("brugernavn") AND ("kodeord")) {

include("config.php");

$hent = mysql_query("SELECT * FROM community_avis WHERE brugernavn = '$_SESSION[brugernavn]'") or die(mysql_error());

$vis = mysql_fetch_array($hent);

?>

<?

if($vis[status] == "Ejer")

{

?>

<?
echo "DEBUG #0<br>";
include("config.php");

// Slet her.

$id = $_GET["id"];
if (isset($id) && trim($id) != "")
{
    $sql = "DELETE FROM artikler_avis WHERE id=" . $id;
    echo "DEBUG #1: $sql<br>";
    mysql_query($sql) or die(mysql_error());
}

// Vis data her

$sql = "SELECT overskrift, id FROM artikler_avis ORDER BY id";
echo "DEBUG #2: $sql<br>";
$result = mysql_query($sql) or die(mysql_error());

while ($data = mysql_fetch_assoc($result))
{
    echo "<b>" . $data["overskrift"] . "</b> <a href='" .

$_SERVER['PHP_SELF'] . "?id=" . $data["id"] . "'>Slet</a><br>";
}
?>

Bortset fra at koden slet ikke ville kunne køre... ;^)

... så afhænger det nu lige så meget af hvordan man logger ind på din side.

hvordan ville du foreslå at man tjekkede om man var logged ind?

har du msn? det ville måske være nemmere der?

Nej jeg bruger ikke MSN.

Foretrækker nu også at hjælpe her sådan at alle kan følge med. :^)

men hvordan ville du foreslå det?

hvad med at kun éen ip kan komme ind?

eller ville denne kode virke?




<?

session_start();

if (session_is_registered("brugernavn") AND ("kodeord")) {

include("../config.php");

$hent = mysql_query("SELECT * FROM community_avis WHERE brugernavn = '$_SESSION[brugernavn]'") or die(mysql_error());

$vis = mysql_fetch_array($hent);

?>

<?

if($vis[status] == "normal")

{

echo("<font color=red>Ingen adgang!</font>");

}

?>

<?

if($vis[status] == "Ejer")

{

?>

<?

$hent = mysql_query("SELECT * FROM community_avis WHERE brugernavn='$_GET[brugernavn]'");

$vis1 = mysql_fetch_array($hent);

?>



<?
echo "DEBUG #0<br>";
include("config.php");

// Slet her.

$id = $_GET["id"];
if (isset($id) && trim($id) != "")
{
    $sql = "DELETE FROM artikler_avis WHERE id=" . $id;
    echo "DEBUG #1: $sql<br>";
    mysql_query($sql) or die(mysql_error());
}

// Vis data her

$sql = "SELECT overskrift, id FROM artikler_avis ORDER BY id";
echo "DEBUG #2: $sql<br>";
$result = mysql_query($sql) or die(mysql_error());

while ($data = mysql_fetch_assoc($result))
{
    echo "<b>" . $data["overskrift"] . "</b> <a href='" .

$_SERVER['PHP_SELF'] . "?id=" . $data["id"] . "'>Slet</a><br>";
}
?>

<?

}

?>



<?

}else{

print "<font color=red>Ingen Adgang!</font>";

}

?>

Noget i denne stil:

<?
session_start();

$brugernavn = $_SESSION['brugernavn'];
$kodeord = $_SESSION['kodeord'];

if ($brugernavn == "dinFarsBrugernavn" && $kodeord = "dinFarsKodeord")
{
    echo "DEBUG #0<br>";
    include("config.php");

    // Slet her.

    $id = $_GET["id"];
    if (isset($id) && preg_match("/^\d+$/", $id))
    {
        $sql = "DELETE FROM artikler_avis WHERE id=" . $id;
        echo "DEBUG #1: $sql<br>";
        mysql_query($sql) or die(mysql_error());
    }

    // Vis data her

    $sql = "SELECT overskrift, id FROM artikler_avis ORDER BY id";
    echo "DEBUG #2: $sql<br>";
    $result = mysql_query($sql) or die(mysql_error());

    while ($data = mysql_fetch_assoc($result))
    {
        echo "<b>" . $data["overskrift"] . "</b> <a href='" . $_SERVER['PHP_SELF'] . "?id=" . $data["id"] . "'>Slet</a><br>";
    }
}
?>

<?

session_start();

if (session_is_registered("brugernavn") AND ("kodeord")) {

include("../config.php");

$hent = mysql_query("SELECT * FROM community_avis WHERE brugernavn = '$_SESSION[brugernavn]'") or die(mysql_error());

$vis = mysql_fetch_array($hent);

?>

<?

if($vis[status] == "normal")

{

echo("<font color=red>Ingen adgang!</font>");

}

?>

<?

if($vis[status] == "Ejer")

{

?>

<?

$hent = mysql_query("SELECT * FROM community_avis WHERE brugernavn='$_GET[brugernavn]'");

$vis1 = mysql_fetch_array($hent);

?>



<?
echo "DEBUG #0<br>";
include("config.php");

// Slet her.

$id = $_GET["id"];
if (isset($id) && trim($id) != "")
{
    $sql = "DELETE FROM artikler_avis WHERE id=" . $id;
    echo "DEBUG #1: $sql<br>";
    mysql_query($sql) or die(mysql_error());
}

// Vis data her

$sql = "SELECT overskrift, id FROM artikler_avis ORDER BY id";
echo "DEBUG #2: $sql<br>";
$result = mysql_query($sql) or die(mysql_error());

while ($data = mysql_fetch_assoc($result))
{
    echo "<b>" . $data["overskrift"] . "</b> <a href='" .

$_SERVER['PHP_SELF'] . "?id=" . $data["id"] . "'>Slet</a><br>";
}
?>

<?

}

?>



<?

}else{

print "<font color=red>Ingen Adgang!</font>";

}

?>


virker fint!

Denne linje:

if (session_is_registered("brugernavn") AND ("kodeord")) {

- kan ikke kompilere. Den skal minimum se sådan ud:

if (session_is_registered("brugernavn") AND session_is_registered("kodeord")) {

Men det er jo ikke nok at teste om der er nogget nogen på. Du skal også teste om det er den rigtige som er logget på. Altså din far og ingen anden.

kom med svar for at få points?

Svar :^)

Hvorfor alle de start/slut-tags til PHP?
Nej den ville heller ikke fungerer. Hvis det kun er én (måske to) der skal kunne logge ind, så kig på .htaccess.
www.webcafe.dk

har også gjort? if status = ejer then

<?
session_start();
if (session_is_registered("brugernavn") AND ("kodeord")) {

include("../config.php");

$hent = mysql_query("SELECT * FROM community_avis WHERE brugernavn = '$_SESSION[brugernavn]'") or die(mysql_error());
$vis = mysql_fetch_array($hent);
if($vis[status] == "normal")
{
echo("<font color=red>Ingen adgang!</font>");
}
if($vis[status] == "Ejer")
{
$hent = mysql_query("SELECT * FROM community_avis WHERE brugernavn='$_GET[brugernavn]'");
$vis1 = mysql_fetch_array($hent);
echo "DEBUG #0<br>";
include("config.php");

// Slet her.

$id = $_GET["id"];
if (isset($id) && trim($id) != "")
{
    $sql = "DELETE FROM artikler_avis WHERE id=" . $id;
    echo "DEBUG #1: $sql<br>";
    mysql_query($sql) or die(mysql_error());
}

// Vis data her

$sql = "SELECT overskrift, id FROM artikler_avis ORDER BY id";
echo "DEBUG #2: $sql<br>";
$result = mysql_query($sql) or die(mysql_error());
while ($data = mysql_fetch_assoc($result))
{
    echo "<b>" . $data["overskrift"] . "</b> <a href='" .
$_SERVER['PHP_SELF'] . "?id=" . $data["id"] . "'>Slet</a><br>";
}
}
}else{
print "<font color=red>Ingen Adgang!</font>";
}
?>

sammenlignet med din:

<?

session_start();

if (session_is_registered("brugernavn") AND ("kodeord")) {

include("../config.php");

$hent = mysql_query("SELECT * FROM community_avis WHERE brugernavn = '$_SESSION[brugernavn]'") or die(mysql_error());

$vis = mysql_fetch_array($hent);

?>

<?

if($vis[status] == "normal")

{

echo("<font color=red>Ingen adgang!</font>");

}

?>

<?

if($vis[status] == "Ejer")

{

?>

<?

$hent = mysql_query("SELECT * FROM community_avis WHERE brugernavn='$_GET[brugernavn]'");

$vis1 = mysql_fetch_array($hent);

?>



<?
echo "DEBUG #0<br>";
include("config.php");

// Slet her.

$id = $_GET["id"];
if (isset($id) && trim($id) != "")
{
    $sql = "DELETE FROM artikler_avis WHERE id=" . $id;
    echo "DEBUG #1: $sql<br>";
    mysql_query($sql) or die(mysql_error());
}

// Vis data her

$sql = "SELECT overskrift, id FROM artikler_avis ORDER BY id";
echo "DEBUG #2: $sql<br>";
$result = mysql_query($sql) or die(mysql_error());

while ($data = mysql_fetch_assoc($result))
{
    echo "<b>" . $data["overskrift"] . "</b> <a href='" .

$_SERVER['PHP_SELF'] . "?id=" . $data["id"] . "'>Slet</a><br>";
}
?>

<?

}

?>



<?

}else{

print "<font color=red>Ingen Adgang!</font>";

}

?>

Denne her linje:

if (session_is_registered("brugernavn") AND ("kodeord")) {

- kompilere måske nok, men den fungere ikke efter hensigten.

Deludtrykket ("kodeord"))  vil altid blive sandt, og derfor testes der faktisk slet ikke på om man er logget ind med et korrekt password.

men du er slet ikke logged ind uden et password så det behøves ikke den tjekker jo om man er logged ind...

I så fald er der jo slet ikke nogen grund til at have det delled. :^)

ved ikke :/ er bare glad for svaret :D