Mit dokument vil ikke virke ?

Kalder du dine funktioner?

Altså fx:

update__or_insert_user_info($_POST['mail'], $_POST['hjemmeside'], $_POST['interesser'], $_POST['alder']);

Har som sagt ikke selv lavet det så jo der var jo et eller andet der skulle gå galt :)

Hvad kan jeg så gøre for at kalde funktionerne så det kommer til at virke ?

Skrive det jeg skriver - hvis det giver mening.

Man kan definere funktioner li'så tosset man vil. Og man bør egentlig definere dem før man begynder at kigge på det, der skal udføres: altså, flyt dine to funktioner ud før din if-sætning, måske endda i toppen af dokumentet, måske endda på en separat fil, du siger include til - så kan de bruges andre steder.

Og så kalder du en eller flere funktioner, sådan ca. som jeg skriver - i dette tilfælde inden i din if-sætning.

Sådan her ?

<?
function update__or_insert_user_info($mail, $hjemmeside, $interesser, $alder) {
    // Denne funktion returnerer det id som blev behandlet.
    // Hvis ny bruger blev insat, returneres nye brugers id.
    // Returnerer -1 hvis noget gik galt.
    $sql = "SELECT id FROM brugere WHERE brugernavn = '" . $brugernavn . "'";
    $res = mysql_query($sql);
    if (mysql_num_rows($res) > 0) {
        $row = mysql_fetch_array($res);
        $sql = "UPDATE brugere SET mail = '" . $mail . "', hjemmeside = '" . $hjemmeside . "', interesser = '" . $interesser . "', alder = '" . $alder . "' WHERE id = " . $row['id'];
        $res = mysql_query($sql);
        if ($res) {
            $done = $row['id'];
        } else {
            $done = 0;
        }
    } else {
        $sql = "INSERT INTO brugere (mail, hjemmeside, interesser, alder) VALUES ('" . $mail . "', '" . $hjemmeside . "', '" . $interesser . "', '" . $alder . "')";
        $res = mysql_query($sql);
        if ($res) {
            $done = mysql_insert_id();
        } else {
            $done = -1;
        }
    }
    return $done;
}


function replace_user_info($brugernavn, $kodeord) {
    // Denne funktion returnerer TRUE hvis sql gik godt, eller FALSE.
    $sql = "SELECT id FROM brugere WHERE brugernavn = '" . $brugernavn . "'";
    $res = mysql_query($sql);
    $row = mysql_fetch_array($res);
    $id = ($row['id'] > 0) ? $row['id'] : 'NULL';
    $sql = "REPLACE INTO brugere (mail, hjemmeside, interesser, alder) VALUES ('" . $mail . "', '" . $hjemmeside . "', '" . $interesser . "', '" . $alder . "')";
    $res = mysql_query($sql);
    if ($res) {
        $done = true;
    } else {
        $done = false;
    }
    return $done;
}
?>

E-mail: <input type="text" name="mail"><br>
Hjemmeside: <input type="text" name="hjemmeside"><br>
Interesser: <input type="text" name="interesser"><br>
Alder: <input type="text" name="alder"><br>
<input type="submit" value="Gem" name="submit">

<?php
include "mysql.php";

if ($_POST[submit]) {
update__or_insert_user_info($_POST['mail'], $_POST['hjemmeside'], $_POST['interesser'], $_POST['alder']);
}
?>

Det er da lidt bedre.

1) Du bruger en $brugernavn, der ikke kommer nogen steder fra.
2) update__or_insert_user_info returnerer en værdi, som du ikke bruger (det behøver du heller ikke, men der er vel en grund til det)

1) hmm, ja han havde indsat nogle felter til at starte med, men så rettede jeg til mine egne, som du kan se i html delen, så er det kun de felter der bruges, da ovenstående ER hele dokumentet. Men hvis du gad at hjælpe mig med at få det ordnet, er jeg villig til at smide flere point oven i hvis du vil ? selvom du nok ikke går så meget efter point, men hva' fa'n det er det eneste jeg kan give :)

2) snakker du om denne del ?
    if ($res) {
        $done = true;
    } else {
        $done = false;
    }
    return $done;
}

den er jo til for at se om det gik rigtigt til, men jo helt unødvendig, kan jeg roligt slette den ?

Du behøver ikke slette de linier. En gang kan det være du vil tjekke om funktionen gik godt.

Jeg kan ikke hjælpe, point eller ej, for jeg ved ikke hvor $brugernavn sku' komme fra.

Mon ikke den er til at tjekke id'et på brugeren ?
altså til at finde ud af hvilken bruger det er det skal indsættes hos ?

jo

Så det burde virke ?
- har nemlig ingen mulighed for at teste det nu, derfor de underlige spørgsmål :)
frac.dk understøtter desværre kun 15 brugere online på ftp af gangen :S glæder mig til systemet er færdig, så lægger jeg det op på en ordenlig side :)

Og forresten så lukkes tråden lige efter det er testet, altså når der er adgang til ftp. :)

Ok ;)  Gratis er gratis. 

Jo, men det er nok hvad $brugernavn bruges til. Men hvor kommer den fra?

brugernavnets id kommer fra adresselinien, så er der en lettere måde at tjekke det på ?

Så vil det nok være sundt at skrive som $_GET['brugernavn']

Men om det så er ok er ikke muligt at vide ud fra de oplysninger du giver. Fx vil det være rart at se hele din php-fil.

Som sagt i 14/11-2005 23:04:05 så ER dette hele dokumentet

Nej, det er det ikke. Du skal også vise din <form> og gerne resten. Og selvfølgelig om der er andet PHP involveret. Ellers er det jo bare gætteri.

Pinligt :/

Havde glemt <form> :S

smider lige en <form action="" method="POST"> ... </form> ind i dokumentet:

så det ser sådan ud nu, vil det virke ? :)

-------------------------------------------------------------------------------------
<form action="" method="POST">

<?
function update__or_insert_user_info($mail, $hjemmeside, $interesser, $alder) {
    // Denne funktion returnerer det id som blev behandlet.
    // Hvis ny bruger blev insat, returneres nye brugers id.
    // Returnerer -1 hvis noget gik galt.
    $sql = "SELECT id FROM brugere WHERE brugernavn = '" . $brugernavn . "'";
    $res = mysql_query($sql);
    if (mysql_num_rows($res) > 0) {
        $row = mysql_fetch_array($res);
        $sql = "UPDATE brugere SET mail = '" . $mail . "', hjemmeside = '" . $hjemmeside . "', interesser = '" . $interesser . "', alder = '" . $alder . "' WHERE id = " . $row['id'];
        $res = mysql_query($sql);
        if ($res) {
            $done = $row['id'];
        } else {
            $done = 0;
        }
    } else {
        $sql = "INSERT INTO brugere (mail, hjemmeside, interesser, alder) VALUES ('" . $mail . "', '" . $hjemmeside . "', '" . $interesser . "', '" . $alder . "')";
        $res = mysql_query($sql);
        if ($res) {
            $done = mysql_insert_id();
        } else {
            $done = -1;
        }
    }
    return $done;
}


function replace_user_info($brugernavn, $kodeord) {
    // Denne funktion returnerer TRUE hvis sql gik godt, eller FALSE.
    $sql = "SELECT id FROM brugere WHERE brugernavn = '" . $brugernavn . "'";
    $res = mysql_query($sql);
    $row = mysql_fetch_array($res);
    $id = ($row['id'] > 0) ? $row['id'] : 'NULL';
    $sql = "REPLACE INTO brugere (mail, hjemmeside, interesser, alder) VALUES ('" . $mail . "', '" . $hjemmeside . "', '" . $interesser . "', '" . $alder . "')";
    $res = mysql_query($sql);
    if ($res) {
        $done = true;
    } else {
        $done = false;
    }
    return $done;
}
?>

E-mail: <input type="text" name="mail"><br>
Hjemmeside: <input type="text" name="hjemmeside"><br>
Interesser: <input type="text" name="interesser"><br>
Alder: <input type="text" name="alder"><br>
<input type="submit" value="Gem" name="submit">

<?php
include "mysql.php";

if ($_POST[submit]) {
update__or_insert_user_info($_POST['mail'], $_POST['hjemmeside'], $_POST['interesser'], $_POST['alder']);
}
?>

</form>
-------------------------------------------------------------------------------------

Så kom ftp'en op og køre igen, men det hele virker altså den indsætter i databasen og det hele, men den indsætter oplysningerne i en anden og tom bruger, dvs at det er funktionen hvor den skal finde hvilken bruger det er det handler om som fejler, kan umiddelbart ikke se hvad det er som er galt ?

Det hjælper lidt :)

Du behøver kun sætte <form...> </form> rundt om selve felterne.

Men hvis der ikke er andet, så har du ikke et HTML-dokument. Om det så virker eller ej, og hvad fejl der opstår og hvordan de rettes er rent lotteri.

Der burde være en checkbox ved spørgsmål om websider her på eksperten.dk

    (Box) Validerer siden hos validator.w3.org

og hvis svaret er "nej", så får man ikke lov til at spørge. Og det er ment alvorligt:
der er ingen grund til at spilde egen (eller andres) tid, hvis de simple formalia ikke er overholdt. Hvis siden er lovlig HTML 4.01 - ja, så kan vi gå på jagt efter fejl.

Hvad mente du med det sidste om lovlig html ? :) overholder min html i w3 og er derfor ulovlig på eksperten eller hvad ? :S

Vender tilbage imorgen, det er ved at blive sent, skal tidligt op i morgen! :)

Der er ikke noget (i den sammenhæng) der er ulovligt - det er bare spild af tid.

Sådan, nu har jeg prøvet selv at lave det, og er efter min mening mere forståeligt, men jeg for dog en fejl i min INSERT sætning:

-------------------------------------------------------
<?
//connecter til databasen
include "mysql.php";

  //finder brugerens id nummer fra adresselinien
  $id = mysql_escape_string($_GET["id"]);
  $query = mysql_query("SELECT brugernavn FROM brugere WHERE id = $id LIMIT 1");

  //opdatere det info man har skrevet
  $sql1 = "UPDATE brugere SET mail = '" . $mail . "', hjemmeside = '" . $hjemmeside . "', interesser = '" . $interesser . "', alder = '" . $alder . "' WHERE id = $id;
  $res1 = mysql_query($sql1);

  //indsætter det info man har skrevet
  $sql2 = mysql_query("INSERT INTO brugere (mail, hjemmeside, interesser, alder) VALUES ('$_POST[mail]','$_POST[hjemmeside]','$_POST[interesser]','$_POST[alder]')") or die(mysql_erorr());
  $res2 = mysql_query($sql2);
?>

<form action="" method="POST">
E-mail: <input type="text" name="mail"><br>
Hjemmeside: <input type="text" name="hjemmeside"><br>
Interesser: <input type="text" name="interesser"><br>
Alder: <input type="text" name="alder"><br>
<input type="submit" value="Gem" name="submit">
</form>

<?php
//connecter til databasen
include "mysql.php";

//hvis man trykker på knappen
if ($_POST[submit]) {
//så tjekker den: hvis der findes nogle brugere
if (mysql_num_rows($query)>0) {
//så skal den opdatere bruger info
mysql_query($sql1)
} else {
//ellers, hvis ikke det findes i databasen så skal den indsætte det nye bruger info
mysql_query($sql2)
}
}
?>

-------------------------------------------------------

er det nu spild af tid ? :)

"eg for dog en fejl i min INSERT" - ja, det er spild af tid, hvis du ikke vil fortælle hvad det er for en fejl.

Men nu har du jo lavet det hele om. Du laver en INSERT, selv om man ikke kommer fra din <form>. Jeg troede vi var enige om at man først med passende if-er skulle se om man kom på siden første gang, eller om man kom fra formen. Det har du nederst, men du laver alligevel altid en INSERT. Dine funktioner var da gode til det de skulle bruges til.

Og når du nederst siger
  mysql_query($sql2)
er det ikke til en sql-sætning, men til resultatet fra den mysql_query med INSERT du allerede har lavet.

Hvad med at gå tilbage til det du havde før?

(Og så har resultatet stadig intet med et HTML-dokument at gøre)

Det virker nu nu :D

Læg et svar :)

Jeg samler slet ikke på point, tak.

Jamen du hjalp mig jo, godt nok virkede lige det der ikke, men jeg fik det da til at virke og du blev ved med at prøve, i sådan nogle spørgsmål hader jeg at tage pointene selv, men så kan jeg da belønne dig med god karma :)