24. august 2005 - 17:14Der er
11 kommentarer og 1 løsning
Er der en bedre/nemmere måde at lave dette på
Hejsa.. jeg har lavet et forum (udelukkene kun for at lære lidt mere om php) jeg er så blevet færdig med det og vil gerne lige høre om der er nogle ting som kunne finpudses ? Altså er der nemmere måder eller bedre måder at lave de forskellige ting på..
Jeg har 3 filer:
index.php laes.php opret.php
Hvis i vil hjælpe så skriv lige.. så skriver jeg mine koder..
I dette særtema ser vi på, hvordan cloud og AI bliver fundamentet for virksomhedernes digitale forretning, og hvordan de nye muligheder for automatisering og forretningsværdi kan udnyttes uden at miste overblik, sikkerhed og menneskelig kontrol.
<? //Her inkludere vi filen der indenholder database oplysningerne
include "mysql.php";
// Her tjekker vi om man overhovdet har trykket på submit knappen
if($_POST[submit]) {
// Her tjekker vi om man overhovedet har skrevet noget i felterne
if($_POST[overskrift] == "" OR $_POST[besked] == "") { print "Et eller flere felter mangler at blive udfyldt!"; print "<br/><br/>";
}else{
//hvis de alt ovenstående er opfyldt, indsætter den teksten i mysql databasen
mysql_query("INSERT INTO forum (overskrift, besked) VALUES ('$_POST[overskrift]', '$_POST[besked]')") or die(mysql_error()); print "Beskeden blev nu gemt!"; print "<br/><br/>";
<? //Her inkludere vi filen der indenholder database oplysningerne
include "mysql.php";
// Her tjekker vi om man overhovdet har trykket på submit knappen
if($_POST[submit]) {
// Her tjekker vi om man overhovedet har skrevet noget i felterne
if($_POST[overskrift] == "" OR $_POST[besked] == "") { print "Et eller flere felter mangler at blive udfyldt!"; print "<br/><br/>";
}else{
//hvis de alt ovenstående er opfyldt, indsætter den teksten i mysql databasen
mysql_query("INSERT INTO forum (overskrift, besked) VALUES ('$_POST[overskrift]', '$_POST[besked]')") or die(mysql_error()); print "Beskeden blev nu gemt!"; print "<br/><br/>";
<? //Her inkludere vi filen der indenholder database oplysningerne include "mysql.php";
//Her indsætter vi alle vores overskrifter og laver dem om til links //Derefter gør vi sådan at når man trykker på dem, kommer selve indlægget frem
if (isset($_GET['id'])) { $query = mysql_query("select id, overskrift, besked from forum where id='" . $_GET['id'] . "'") or die(mysql_error()); /* i ovenstående mysql_query() tillader du et hul, i det at det er muligt at sende uønskede tegn ind i forespørgelsen.
brug mysql_real_escape_string() eller mysql_escape_string() for at lukke hullet:
mysql_query("select id, overskrift, besked from forum where id='" . mysql_escape_string( $_GET['id'] ). "'") */
while ($foo = mysql_fetch_array($query)) { echo $foo['overskrift'] . "<br><hr><br>" . $foo['besked']; } } else { $query = mysql_query("select id, overskrift, besked from forum") or die(mysql_error()); while ($row = mysql_fetch_array($query)) { echo "<a href=\"laes.php?id=" . strip_tags($row['id']) . "\">" . strip_tags($row['overskrift']) . "</a><br><hr><br>"; /* personligt foretrækker jeg at bruge enkelt-quotes når jeg skriver html, for at undgå at escape dobbelt-quotes i selve html'et
<? //Her inkludere vi filen der indenholder database oplysningerne
include "mysql.php";
// Her tjekker vi om man overhovdet har trykket på submit knappen
if($_POST[submit]) {
// Her tjekker vi om man overhovedet har skrevet noget i felterne
if($_POST[overskrift] == "" OR $_POST[besked] == "") { /* brug trim() for at undgå at en bruger blot skriver et mellemrum i overskrift og besked if(trim($_POST[overskrift]) == "" OR trim($_POST[besked]) == "") */ print "Et eller flere felter mangler at blive udfyldt!"; print "<br/><br/>";
}else{
//hvis de alt ovenstående er opfyldt, indsætter den teksten i mysql databasen
mysql_query("INSERT INTO forum (overskrift, besked) VALUES ('$_POST[overskrift]', '$_POST[besked]')") or die(mysql_error()); /* igen, mysql_escape_string()
mysql_query("INSERT INTO forum (overskrift, besked) VALUES ('".mysql_escape_string($_POST[overskrift])."', '".mysql_escape_string($_POST[besked])."')") */
print "Beskeden blev nu gemt!"; print "<br/><br/>";
<? //Her inkludere vi filen der indenholder database oplysningerne
include "mysql.php";
// Her tjekker vi om man overhovdet har trykket på submit knappen
if($_POST[submit]) {
// Her tjekker vi om man overhovedet har skrevet noget i felterne
if($_POST[overskrift] == "" OR $_POST[besked] == "") { /* trim() */ print "Et eller flere felter mangler at blive udfyldt!"; print "<br/><br/>";
}else{
//hvis de alt ovenstående er opfyldt, indsætter den teksten i mysql databasen
mysql_query("INSERT INTO forum (overskrift, besked) VALUES ('$_POST[overskrift]', '$_POST[besked]')") or die(mysql_error()); /* mysql_escape_string() */ print "Beskeden blev nu gemt!"; print "<br/><br/>";
Umiddelbart ser det jo pænt ud, dog ville jeg selv gøre lidt mere ved opsætningen af HTML, her tænker jeg på tekst-layoutet. (tabulator er en fantastisk ting :-)
leas.php og opret.php har næsten samme layout og samme validering - du kunne måske nøjes med kun ene, der hed skriv.php?
Hmm.. nej.. så ville jeg få problemer når man skal kommentere spørgsmålene.. for så ville kommentar textarea'et også kommer frem når man opretter et indlæg.. engang var det to filer, men det har jeg lavet om ;) Læg et svar.. jeg har sku lært meget :)
Til næste gang du vil have din kode nederst på sitet, skal du blot skrive:
show_source(__FILE__);
Synes godt om
Ny brugerNybegynder
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.