Avatar billede dingkee Nybegynder
14. marts 2007 - 10:18 Der er 4 kommentarer

Kode tjek

Hej alle,

Det er ved at være et godt stykke tid siden jeg har kodet noget seriøst i php...

Så tænkte på om der var nogen der ville kigge det igennem og se om det var helt dårligt det jeg laver??

Koden virker, men kan godt være at det ikke er helt korrekt jo :)

min connect fil:

<?
mysql_connect("localhost","","") OR DIE(  "Kunne ikke etabler en forbindelse til MySQLen.!");
mysql_select_db("") OR DIE(  "Kunne ikke finde databasen!");

$id = $_GET['id'];
$sql = "SELECT * FROM cdkasse WHERE id = '" . $id . "'";
$result = mysql_query($sql) or die (mysql_error());
$r = mysql_fetch_array($result);

$id = $r["id"];
$navn = $r["navn"];
$type = $r["type"];
$nr = $r["nr"];
?>

Fil:
<?PHP include("header.php"); ?>
        <div id="global">
                <div id="menu-1-off">
                        <p class="top-off"><a href="index.php" class="top-off">Titler</a></p>
                </div>
                <div id="menu-2-off">
                        <p class="top-off">Info</p>
                </div>
                <div id="menu-3-off">
                    <p class="top-off"><a href="web.php" class="top-off">Web</a></p>
                </div>
                <div id="menu-4-on">
                    <div id="menu-inden">
                        <p class="top-on"><a href="cdkasse.php" class="top-on">CD-Kasse</a></p>
                    </div>
                </div>
            <div id="bg">
                <div id="bg-inden">
                    <p class="titel">CD-Kasse</p>
                    <ul id="maintab" class="shadetabs">
                        <li class="selected"><a href="#" rel="tcontent1">CD-Kasse</a></li>
                        <li><a href="#" rel="tcontent2">Tilf&oslash;j</a></li>
                    </ul>

                    <div class="tabcontentstyle">

                        <div id="tcontent1" class="tabcontent">
                            <div id="boks" class="alm">
                                <table width="100%" cellspacing="0" cellpadding="0">
                                    <tr>
                                        <td>
                                            <table width="100%" cellspacing="0" cellpadding="0">
                                                <tr>
                                                        <td width="10%">Nr</td>
                                                        <td>Navn</td>
                                                        <td>Type</td>
                                                        <td>Beskrivelse</td>
                                                </tr>
                                            <?PHP
                                                $sql1 = mysql_query("SELECT * FROM cdkasse ORDER BY navn");
                                                    while ($r1 = mysql_fetch_array($sql1)) {
                                                    echo"
                                                    <tr>
                                                        <td width=\"10%\">$r1[nr]</td>
                                                        <td>$r1[navn]</td>
                                                        <td>$r1[type]</td>
                                                        <td>$r1[beskriv]</td>
                                                    </tr>
                                                    ";
                                                  }
                                            ?>
                                            </table>
                                        </td>
                                    </tr>
                                </table>
                            </div>
                        </div>
       
                        <div id="tcontent2" class="tabcontent">
                            <div id="boks" class="alm">
                                <table width="100%" cellspacing="0" cellpadding="0">
                                    <form action="cdkasse.php?mode=tilfoj" method="post" enctype="multipart/form-data">
                                                <tr>
                                                    <td>
                                                        Navn
                                                    </td>
                                                    <td>
                                                        <input type="text" name="navn" size="30" class="input-skrift" />
                                                    </td>
                                                </tr>
                                                <tr>
                                                    <td>
                                                        CD-NR
                                                    </td>
                                                    <td>
                                                        <input type="text" name="nr" size="30" class="input-skrift" />
                                                    </td>
                                                </tr>
                                                <tr>
                                                    <td>
                                                        Type
                                                    </td>
                                                    <td>
                                                        <input type="text" name="type" size="30" class="input-skrift" />
                                                    </td>
                                                </tr>
                                                <tr>
                                                    <td>
                                                        Beskrivelse
                                                    </td>
                                                    <td>
                                                        <input type="text" name="beskriv" size="30" class="input-skrift" />
                                                    </td>
                                                </tr>
                                                <tr>
                                                    <td colspan="2">
                                                        <input type="submit" value="Godkend" class="submit" />
                                                    </td>
                                                </tr>
                                            </form>
                                </table>
                            </div>
                        </div>
                    </div>


<script type="text/javascript">
//Start Tab Content script for UL with id="maintab" Separate multiple ids each with a comma.
initializetabcontent("maintab")
</script>
                </div>
            </div>
           
        </div>
    </body>
</html>

<?
if ($mode == "tilfoj") {
// Smider informationerne ind i MySQLen
$navn = $_POST["navn"];
$nr = $_POST["nr"];
$type = $_POST["type"];
$beskriv = $_POST["beskriv"];

$query = mysql_query("
    INSERT INTO
        cdkasse
        (
            navn,
            nr,
            type,
            beskriv
        )" . "
        VALUES
        (
            '$navn',
            '$nr',
            '$type',
            '$beskriv'
        )
    ");

}
?>


På forhånd tak
Avatar billede dmdisco Nybegynder
14. marts 2007 - 10:39 #1
du skal måske lige ændre if ($mode == "tilfoj") { til if ($_GET['mode'] == "tilfoj")
ellers ser det da nogenlunde fint ud :)
Avatar billede showsource Seniormester
14. marts 2007 - 23:21 #2
Start altid php tags med
<?php
ikke blot
<?

Så er der aldrig tvivl om hvad som startes.

connect filen vile jeg starte med:

<?php
if($_SERVER["REQUEST_METHOD"] == "GET" && isset($_GET["id"]))
{


}
?>
og dertil bruge intval() på get, så du er sikker på det er et tal som bruges i din query
$id = intval($_GET["id"]);
Avatar billede dingkee Nybegynder
15. marts 2007 - 15:19 #3
Tak for jeres besvarelser... Kan se det nok ikk er så meget jeg mangler for det bliver perfekt :)

Men når jeg sætter den understående linie ind, kommer der ikke noget udtræk...:
<?php
if($_SERVER["REQUEST_METHOD"] == "GET" && isset($_GET["id"]))
{

Hvad kan grunden tild det være?
Avatar billede showsource Seniormester
16. marts 2007 - 01:26 #4
Sikkert fordi der intet er i if'en.
Men ellers:

<?php
if($_SERVER["REQUEST_METHOD"] == "GET" && isset($_GET["id"]))
{

mysql_connect("localhost","","") or die ("Kunne ikke etabler en forbindelse til MySQLen.!");
mysql_select_db("") or die ("Kunne ikke finde databasen!");

$id = intval($_GET["id"]);
$sql = "SELECT * FROM `cdkasse` WHERE `id` = " . $id;
$result = mysql_query($sql) or die (mysql_error());

    if(mysql_num_rows($result) == 1)
    {
    $r = mysql_fetch_array($result);

    $id = $r["id"];
    $navn = $r["navn"];
    $type = $r["type"];
    $nr = $r["nr"];
    }else{
    echo"Ups, fejl i request/query ???";
    }
}
?>
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
Vi tilbyder markedets bedste kurser inden for webudvikling

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.

Du kan også logge ind via nedenstående tjenester