TestMax.dk Paster
  • Forside
  • Print
  • Bookmark
  • Kontakt
  • SiteMap


infobox infobox
 

Feedback (:

29-05-2006 13:45:00
 
 
Muligheder (Anmeld)
Skrevet af Gudiik
Kliks 671
Kategori PHP Feedback
Hejsa, vil hører hvad i syntes om min hjemmeside, I skal ikke kigge på design, for det er der slet ikke, vil bare lige hører hvad i syntes om min php del (: - bare kom med ris og ros.

http://www.bydam.dk/Gudiik
500 deltagere - 100% action - VHCamp
 
infobox infobox


infobox infobox
 
#1Panda(Anmeld) 29-05-2006 14:26:00
 
 
1) Menuen ændrer sig når man skifter fra index til nyheder
2) Du har ingen ifs til at checke når man skal oprette en bruger om man overhovedet har skrevet noget i de inputs
3) Man kan stadig se Login når man er logget ind.. Man kan ikke logge ud?
4) Når man har skrevet i gæstebogen får man beskeden:
Nyheder:
Du har nu skrevet i gæstebogen

Nyheder? :D
5) Du har ingen ifs til når man skal skrive i gæstebogen

Det var hvad jeg kunne finde af store fejl
Portfolio: http://dumpen.dk

Panda's signatur billede
 
infobox infobox

infobox infobox
 
#2Gudiik(Anmeld) 29-05-2006 15:29:00
 
 
tak skal du have panda, men når man sidder og laver det tænker jeg aldrig over sådan noget. og det med index, er fordi jeg ikke har fået den uploadet. men jeg ændre det lige (:
500 deltagere - 100% action - VHCamp
 
infobox infobox

infobox infobox
 
#3Gudiik(Anmeld) 29-05-2006 15:53:00
 
 
Sidder lige med det "if", men kan ikke få det til at funge, bruger det her kode:
       if(empty($brugernavn) || empty($password) || empty($fornavn) || empty($efternavn) || empty($alder) || empty($land)) {
echo "Ufyld alle felterne tak :)";


Men hvor skal jeg have det placeret i det her kode: ?

<head>
<meta http-equiv="Content-Language" content="da">
</head>

<body link="#000000" vlink="#000000" alink="#000000">

<table border="0" cellpadding="0" cellspacing="0" width="1055" height="799">
       <!-- MSTableType="layout" -->
       <tr>
              <td valign="top" colspan="2">
              <!-- MSCellType="DecArea" -->
              &nbsp;</td>
       </tr>
       <tr>
              <td valign="top" width="181">
              <!-- MSCellType="NavBody" -->
              <b><u>Menu:
              </u></b><br><a href="nyheder.php"><font color="#000000">
              <span style="text-decoration: none">Forside</span></font></a><br>
              <a href="opret_bruger.php"><font color="#000000">
              <span style="text-decoration: none">Opret Bruger</span></font></a><br>
                            <a href="profiler.php"><font color="#000000">
              <span style="text-decoration: none">Profiler</span></font></a><br>
                            <a href="statistik.php"><font color="#000000">
              <span style="text-decoration: none">Statistik</span></font></a><br>
              <a href="mg.php"><font color="#000000">
              <span style="text-decoration: none">Login</span></font></a>              <?php
session_start();
if (isset($_SESSION["s_navn"]))
{
$id = $_SESSION[s_id];
?><p><b><u>Din
              Menu:</u></b><br>
              <a href="profilmere.php?id=<? echo $id; ?>"><span style="text-decoration: none">Din Profil</span></a><br>
              <a href="ret.php?id=<? echo $id; ?>"><span style="text-decoration: none">Ret Profil</span></a><br>
              <a href="bog.php?id=<? echo $id; ?>"><span style="text-decoration: none">Gæstebog</span></a>
              <?php
session_start();
if (isset($_SESSION['s_rang']) && $_SESSION['s_rang'] == "admin")

{
?>
<p><b><u>Admin Menu:</u></b><br>
<a href="add_news.php"><span style="text-decoration: none">Lav Nyhed</span></a><br>
<a href="del_news.php"><span style="text-decoration: none">Slet Nyhed</span></a><p>&nbsp;</td>
<?
}
}
?>
              <td valign="top" height="691" width="874">
              <?
if (isset($_SESSION["s_navn"]))
{
       echo "Du er logget ind, så du kan ikke oprette en bruger mere :)";
}
else
{?>
              <?php
       include("sql/db_ind.inc");
$brugernavn = $_REQUEST['brugernavn'];
$result = mysql_query("SELECT * FROM login3 WHERE brugernavn='$brugernavn'");
if (mysql_num_rows($result) == "1")
{
       echo("Brugernavnet er optaget, vælg et andet tak :)");
}
else
{
if (isset($_POST['password'])) {
$header = $_POST['header'];
$brugernavn = $_POST['brugernavn'];
$password = $_POST['password'];
$fornavn = $_POST['fornavn'];
$efternavn = $_POST['efternavn'];
$alder = $_POST['alder'];
$land = $_POST['land'];
mysql_query("INSERT INTO login3 (brugernavn, password, fornavn, efternavn, alder, land) VALUES ('$brugernavn','$password','$fornavn','$efternavn','$alder','$land')") or die (mysql_error());
echo "Din bruger er nu oprettet.";
} else {

?>

<form action="<?php echo $SERVER[PHP_SELF]; ?>" method="post">
<table><tr><td>Brugernavn</td><td><input size="35" name="brugernavn"></td></tr>
<tr><td>Password</td><td><input type="password" size="35" name="password"></tr>
<table><tr><td>Fornavn</td><td><input size="35" name="fornavn"></td></tr>
<table><tr><td>Efternavn</td><td><input size="35" name="efternavn"></td></tr>
<table><tr><td>Alder</td><td><input size="35" name="alder"></td></tr>
<table><tr><td>Land</td><td><input size="35" name="land"></td></tr></table>
<br><input type="submit" value="Opret"></td></tr>
</form></td>
              <!-- MSCellType="ContentBody" -->


<?
}
}
}
?></td></tr></table>
500 deltagere - 100% action - VHCamp
 
infobox infobox

infobox infobox
 
#4Gudiik(Anmeld) 29-05-2006 16:11:00
 
 
Panda, nu har jeg rettet:
1 & 4 mangler lige lidt hjælp til de if'er
500 deltagere - 100% action - VHCamp
 
infobox infobox

infobox infobox
 
#5Panda(Anmeld) 29-05-2006 16:44:00
 
 
f (isset($_POST['password'])) {
if(empty($brugernavn) || empty($password) || empty($fornavn) || empty($efternavn) || empty($alder) || empty($land)) {
echo "Ufyld alle felterne tak :)";
} else {
$header = $_POST['header'];
$brugernavn = $_POST['brugernavn'];
$password = $_POST['password'];
$fornavn = $_POST['fornavn'];
$efternavn = $_POST['efternavn'];
$alder = $_POST['alder'];
$land = $_POST['land'];
mysql_query("INSERT INTO login3 (brugernavn, password, fornavn, efternavn, alder, land) VALUES ('$brugernavn','$password','$fornavn','$efternavn','$alder','$land')") or die (mysql_error());
echo "Din bruger er nu oprettet.";
}
} else {


Sådan ville jeg tro :D
Portfolio: http://dumpen.dk

Panda's signatur billede
 
infobox infobox

infobox infobox
 
#6Mulle(Anmeld) 29-05-2006 16:59:00
 
 
Panda vi ved jo begge at empty er aldrig den bedste til og check den form med, kan du ikke huske det gæstebog? :D

include("sql/db_ind.inc");
$header = $_POST['header'];
$brugernavn = $_POST['brugernavn'];
$password = $_POST['password'];
$fornavn = $_POST['fornavn'];
$efternavn = $_POST['efternavn'];
$alder = $_POST['alder'];
$land = $_POST['land'];
$result = mysql_query("SELECT * FROM login3 WHERE brugernavn='$brugernavn'");
if (mysql_num_rows($result) == "1") {
echo("Brugernavnet er optaget, vælg et andet tak :)");
} else {
if(!ereg("[A-z0-9]+", $brugernavn) || !ereg("[A-z0-9]+", $password) || !ereg("[A-z0-9]+", $fornavn) || !ereg("[A-z0-9]+", $efternavn) || !ereg("[A-z0-9]+", $alder) || !ereg("[A-z0-9]+", $land)) {
echo "Udfyld alle felterne tak";
} elseif(isset($_POST['password'])) {
mysql_query("INSERT INTO login3 (brugernavn, password, fornavn, efternavn, alder, land) VALUES ('$brugernavn','$password','$fornavn','$efternavn','$alder','$land')") or die (mysql_error());
echo "Din bruger er nu oprettet.";
}


sådan ville jeg lave det ^^
There is a lot of things to think about, but nothing to worry about.
Everybody is special to your life!
Its nice to be important but it is more important to be nice.
Love is Forever but Friends are for Eternity.
Were you born to resist or be abused?

Mulle's signatur billede
 
infobox infobox

infobox infobox
 
#7Gudiik(Anmeld) 29-05-2006 22:06:00
 
 
Tak begge 2, men mulle når jeg bruger dit script sådan her:


<head>
<meta http-equiv="Content-Language" content="da">
</head>

<body link="#000000" vlink="#000000" alink="#000000">

<table border="0" cellpadding="0" cellspacing="0" width="1055" height="799">
       <!-- MSTableType="layout" -->
       <tr>
              <td valign="top" colspan="2">
              <!-- MSCellType="DecArea" -->
              &nbsp;</td>
       </tr>
       <tr>
              <td valign="top" width="181">
              <!-- MSCellType="NavBody" -->
              <b><u>Menu:
              </u></b><br><a href="nyheder.php"><font color="#000000">
              <span style="text-decoration: none">Forside</span></font></a><br>
              <a href="opret_bruger.php"><font color="#000000">
              <span style="text-decoration: none">Opret Bruger</span></font></a><br>
                            <a href="profiler.php"><font color="#000000">
              <span style="text-decoration: none">Profiler</span></font></a><br>
                            <a href="statistik.php"><font color="#000000">
              <span style="text-decoration: none">Statistik</span></font></a><br>
              <a href="mg.php"><font color="#000000">
              <span style="text-decoration: none">Login</span></font></a>              <?php
session_start();
if (isset($_SESSION["s_navn"]))
{
$id = $_SESSION[s_id];
?><p><b><u>Din
              Menu:</u></b><br>
              <a href="profilmere.php?id=<? echo $id; ?>"><span style="text-decoration: none">Din Profil</span></a><br>
              <a href="ret.php?id=<? echo $id; ?>"><span style="text-decoration: none">Ret Profil</span></a><br>
              <a href="bog.php?id=<? echo $id; ?>"><span style="text-decoration: none">Gæstebog</span></a>
              <?php
session_start();
if (isset($_SESSION['s_rang']) && $_SESSION['s_rang'] == "admin")

{
?>
<p><b><u>Admin Menu:</u></b><br>
<a href="add_news.php"><span style="text-decoration: none">Lav Nyhed</span></a><br>
<a href="del_news.php"><span style="text-decoration: none">Slet Nyhed</span></a><p>&nbsp;</td>
<?
}
}
?>
              <td valign="top" height="691" width="874">
              <?
if (isset($_SESSION["s_navn"]))
{
       echo "Du er logget ind, så du kan ikke oprette en bruger mere :)";
}
else
{?>
              <?php
include("sql/db_ind.inc");
$header = $_POST['header'];
$brugernavn = $_POST['brugernavn'];
$password = $_POST['password'];
$fornavn = $_POST['fornavn'];
$efternavn = $_POST['efternavn'];
$alder = $_POST['alder'];
$land = $_POST['land'];
$result = mysql_query("SELECT * FROM login3 WHERE brugernavn='$brugernavn'");
if (mysql_num_rows($result) == "1") {
echo("Brugernavnet er optaget, vælg et andet tak :)");
} else {
if(!ereg("[A-z0-9]+", $brugernavn) || !ereg("[A-z0-9]+", $password) || !ereg("[A-z0-9]+", $fornavn) || !ereg("[A-z0-9]+", $efternavn) || !ereg("[A-z0-9]+", $alder) || !ereg("[A-z0-9]+", $land)) {
echo "Udfyld alle felterne tak";
} elseif(isset($_POST['password'])) {
mysql_query("INSERT INTO login3 (brugernavn, password, fornavn, efternavn, alder, land) VALUES ('$brugernavn','$password','$fornavn','$efternavn','$alder','$land')") or die (mysql_error());
echo "Din bruger er nu oprettet.";
} else {

?>

<form action="<?php echo $SERVER[PHP_SELF]; ?>" method="post">
<table><tr><td>Brugernavn</td><td><input size="35" name="brugernavn"></td></tr>
<tr><td>Password</td><td><input type="password" size="35" name="password"></tr>
<table><tr><td>Fornavn</td><td><input size="35" name="fornavn"></td></tr>
<table><tr><td>Efternavn</td><td><input size="35" name="efternavn"></td></tr>
<table><tr><td>Alder</td><td><input size="35" name="alder"></td></tr>
<table><tr><td>Land</td><td><input size="35" name="land"></td></tr></table>
<br><input type="submit" value="Opret"></td></tr>
</form></td>
              <!-- MSCellType="ContentBody" -->


<?
}
}
}
?></td></tr></table>

Skriver den med det samme , brugernavn er optaget vælg et andet tak :)
Hvordan skal det se ud ?
500 deltagere - 100% action - VHCamp
 
infobox infobox

infobox infobox
 
#8Mulle(Anmeld) 30-05-2006 08:02:00
 
 
<head>
<meta http-equiv="Content-Language" content="da">
</head>

<body link="#000000" vlink="#000000" alink="#000000">

<table border="0" cellpadding="0" cellspacing="0" width="1055" height="799">
<!-- MSTableType="layout" -->
<tr>
<td valign="top" colspan="2">
<!-- MSCellType="DecArea" -->
&nbsp;</td>
</tr>
<tr>
<td valign="top" width="181">
<!-- MSCellType="NavBody" -->
<b><u>Menu:
</u></b><br><a href="nyheder.php"><font color="#000000">
<span style="text-decoration: none">Forside</span></font></a><br>
<a href="opret_bruger.php"><font color="#000000">
<span style="text-decoration: none">Opret Bruger</span></font></a><br>
<a href="profiler.php"><font color="#000000">
<span style="text-decoration: none">Profiler</span></font></a><br>
<a href="statistik.php"><font color="#000000">
<span style="text-decoration: none">Statistik</span></font></a><br>
<a href="mg.php"><font color="#000000">
<span style="text-decoration: none">Login</span></font></a> <?php
session_start();
if (isset($_SESSION["s_navn"]))
{
$id = $_SESSION[s_id];
?><p><b><u>Din
Menu:</u></b><br>
<a href="profilmere.php?id=<? echo $id; ?>"><span style="text-decoration: none">Din Profil</span></a><br>
<a href="ret.php?id=<? echo $id; ?>"><span style="text-decoration: none">Ret Profil</span></a><br>
<a href="bog.php?id=<? echo $id; ?>"><span style="text-decoration: none">Gæstebog</span></a>
<?php
session_start();
if (isset($_SESSION['s_rang']) && $_SESSION['s_rang'] == "admin")

{
?>
<p><b><u>Admin Menu:</u></b><br>
<a href="add_news.php"><span style="text-decoration: none">Lav Nyhed</span></a><br>
<a href="del_news.php"><span style="text-decoration: none">Slet Nyhed</span></a><p>&nbsp;</td>
<?
}
}
?>
<td valign="top" height="691" width="874">
<?
if (isset($_SESSION["s_navn"]))
{
echo "Du er logget ind, så du kan ikke oprette en bruger mere :)";
}
else
{?>
<?php
include("sql/db_ind.inc");
$header = $_POST['header'];
$brugernavn = $_POST['brugernavn'];
$password = $_POST['password'];
$fornavn = $_POST['fornavn'];
$efternavn = $_POST['efternavn'];
$alder = $_POST['alder'];
$land = $_POST['land'];
$result = mysql_query("SELECT * FROM login3 WHERE brugernavn='$brugernavn'");
if(mysql_num_rows($result) == "1") {
echo("Brugernavnet er optaget, vælg et andet tak :)");
} elseif(mysql_num_rows($result) == "0") {
if(!ereg("[A-z0-9]+", $brugernavn) || !ereg("[A-z0-9]+", $password) || !ereg("[A-z0-9]+", $fornavn) || !ereg("[A-z0-9]+", $efternavn) || !ereg("[A-z0-9]+", $alder) || !ereg("[A-z0-9]+", $land)) {
echo "Udfyld alle felterne tak";
} elseif(isset($_POST['password'])) {
mysql_query("INSERT INTO login3 (brugernavn, password, fornavn, efternavn, alder, land) VALUES ('$brugernavn','$password','$fornavn','$efternavn','$alder','$land')") or die (mysql_error());
echo "Din bruger er nu oprettet.";
}
} else {
?>

<form action="<?php echo $SERVER[PHP_SELF]; ?>" method="post">
<table><tr><td>Brugernavn</td><td><input size="35" name="brugernavn"></td></tr>
<tr><td>Password</td><td><input type="password" size="35" name="password"></tr>
<table><tr><td>Fornavn</td><td><input size="35" name="fornavn"></td></tr>
<table><tr><td>Efternavn</td><td><input size="35" name="efternavn"></td></tr>
<table><tr><td>Alder</td><td><input size="35" name="alder"></td></tr>
<table><tr><td>Land</td><td><input size="35" name="land"></td></tr></table>
<br><input type="submit" value="Opret"></td></tr>
</form></td>
<!-- MSCellType="ContentBody" -->


<?
}
}
?></td></tr></table>


sådan det vi havde glemt var at lukke den første else efter num_rows så derfor laver den det ^^
There is a lot of things to think about, but nothing to worry about.
Everybody is special to your life!
Its nice to be important but it is more important to be nice.
Love is Forever but Friends are for Eternity.
Were you born to resist or be abused?

Mulle's signatur billede
 
infobox infobox

infobox infobox
 
#9Gudiik(Anmeld) 30-05-2006 11:53:00
 
 
Hmm mulle Nu skriver den "udfyld alle felterne tak" men sidste kan have jeg også sådan at det stod oven over det her:
$header = $_POST['header'];
$brugernavn = $_POST['brugernavn'];
$password = $_POST['password'];
$fornavn = $_POST['fornavn'];
$efternavn = $_POST['efternavn'];
$alder = $_POST['alder'];
$land = $_POST['land'];
500 deltagere - 100% action - VHCamp
 
infobox infobox

infobox infobox
 
#10Gudiik(Anmeld) 30-05-2006 12:06:00
 
 
Det virker nu med det panda skrev :) tak begge 2.
500 deltagere - 100% action - VHCamp
 
infobox infobox

Påmindelse ved nyt svar

Ved at tilmelde dig automatisk påmindelse, modtager du en email så snart denne tråd bliver besvaret, dog sender vi kun en mail til dig, også selvom der er kommet flere svar, simpelthen for at spare dig for spam i din indbakke.


infobox infobox
  Svar på tråden  
   
 
 
infobox infobox

infobox infobox
  Læs dette før du sender dit indlæg!  
 
Ved tryk på send knappen accepterer du vores regelsæt som kan læses her
Praktiske informationer
Type Eksempel Resultat
Fed [b]din tekst[/b] din tekst
Kursiv [k]din tekst[/k] din tekst
Understreg [u]din tekst[/u] din tekst
Kode felt [code]din tekst[/code]
din tekst
infobox infobox