Groups | Search | Server Info | Login | Register

Groups > dk.edb.internet.webdesign.serverside.php > #7046

Re: php opdateret

Show all headers | View raw

Her er nogen forslag til dine filer som du kan arbejde med så kigger jeg 
på filen med mysql funktioner imens.
Du kan bare starte med at rette de letter ting først så kan vi se hvad 
der er tilbage.

Hvis du har lyst kan du eventuelt gemme dine filer med UTF-8 encoding. I 
stedet for ISO-8859-1
Det giver mulighed for at man kan skrive flere tegn på hjemmesiden 
direkte f.eks. hvis du har brug for at indsætte noget kinesisk eller 
andre sjove ting.
Så skal du også rette:
<meta http-equiv="Content-Type" content="text/html; charset=ISO-8859-1">
 >
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">

Den her funktion giver vist ikke meget mening i design-funktioner.php:
function top($titel) {

	$html = "";

	return $html;

}

Din kode kunne godt bruge noget indrykning så det er lettere er 
overskue/læse f.eks.:

     <?php
     session_start();
     ?>
     <?php
     require("mysql-funktioner.php");
     ?>
     <? $strTitle="Administrer gæstebogen";?>
     <? include("head.inc"); ?>
     </head>

Kunne være:

     <?php
         session_start();
     ?>

     <?php
         require("mysql-funktioner.php");
     ?>

     <? $strTitle="Administrer gæstebogen";?>
     <? include("head.inc"); ?>
     </head>

Eller noget i den stil. Jeg bruger 4 mellemrumstegn som indrykning. Det 
kan man vælge i det tekstredigeringsprogram man bruger hvor mange 
mellemrum der skal indrykkes med og om det skal være tabulatortegn

Samme med andre koder som CSS:

     body{
     background:white;
     color:black;
     margin:0px;
     }

Mod:

     body{
         background:white;
         color:black;
         margin:0px;
     }

Eller:

     body {
         background: white;
         color: black;
         margin: 0px;
     }

Det gør det lettere for andre at kigge og hjælpe med koden hvis det er 
let at overskrue.

I stedet for require() kan du bruge require_once() som en sikkerhed for 
at filen ikke indlæses mere end én gang.
Se også https://netkoder.dk/netkoder/kode_includeogrequire.php

if(is_array($HTTP_POST_VARS)) {
   $navn = $HTTP_POST_VARS['navn'];
   $epost = $HTTP_POST_VARS['epost'];
   $subject = $HTTP_POST_VARS['subject'];
   $besked = $HTTP_POST_VARS['besked'];

http://php.net/manual/en/reserved.variables.post.php
4.1.0 	Introduced $_POST that deprecated $HTTP_POST_VARS.

Brug blot $_POST
Det er godt nok noget gammel kode du kunne havde fjerne for længst.
Version 4.1.0. blev udgivet 10 Dec 2001.

Samme med HTTP_GET_VARS
http://php.net/manual/en/reserved.variables.get.php
Brug $_GET

Du har en kommentar (måske flere?) hvor ø er skrevet som oe. Du kan 
sagtens skriv æøå direkte.
  * Foerst napper vi variablerne fra miljoet.

Her står:
#  Dette var en simpel gæstebog, konstrueret af claus@webclaus.com  #

Ved ikke hvem det er men hvis du tidligere har fået hjælp kunne det 
måske tænkes at den person vil opdatere koden for dig.

Istedet for at t og f:
$html_p = "f";

Så kan du skrive:
$html_p = true;
$html_p = false;

Det er de rigtige sand/falsk værdier. "f" og "t" det er strenge.

Samme med:
$epost_p = "f";

$admin_epost = "dit.navn@ditdomæne.dk";
Ved ikke hvorfor der står den e-mail-adresse. Så er det blevet blevet 
sendt en e-mail til den addresse hver gang nogen skrev i gæstebogen?

Ligesom der står i loginbeskyttelse.txt så er det sikkert en god ide at 
flytte alle dine loginoplysninger ud i en seperat fil.
På den måde er det lidt lettere når du for eksempel vil vide din kode 
til mig at du skal kun skal huske på at dine adgangskoder kun er i én 
fil så du kan fjerne den fil når du sender noget til andre.
Så vil være en god ide at gå alle dine adgangskoder igennem og ændre dem 
og samle dem ét sted.

Hvis du har lyst kan du opdatere din HTML-kode til HTML 5.
Det handler mest om headeren med <!DOCTYPE...> men det kan du jo lige så 
godt når du er ved at opdatere.
Se også: https://netkoder.dk/netkoder/kode_skabeloner.php

Log mærke til "subject	varchar(100) not null," hvor du kalder det subject.
Det kunne du ændre til "emne" men du skal så også gøre det i databasen 
ved at logge ind osv.
Tænker bare det vel lige så godt kan være på dansk når de andre er det.

Back to dk.edb.internet.webdesign.serverside.php | Previous | Next — Previous in thread | Next in thread | Find similar

Thread

php opdateret Erik Ginnerskov <egin@mail.dk> - 2016-12-11 16:13 +0100
  Re: php opdateret scootergrisen <invalid@invalid.invalid> - 2016-12-11 18:11 +0100
    Re: php opdateret Erik Ginnerskov <egin@mail.dk> - 2016-12-11 18:31 +0100
  Re: php opdateret Jan Hansen <jhjjhjhhansen@gmail.com> - 2016-12-11 17:13 +0000
    Re: php opdateret Erik Ginnerskov <egin@mail.dk> - 2016-12-11 18:40 +0100
      Re: php opdateret scootergrisen <invalid@invalid.invalid> - 2016-12-11 19:03 +0100
        Re: php opdateret Erik Ginnerskov <egin@mail.dk> - 2016-12-11 19:51 +0100
          Re: php opdateret scootergrisen <invalid@invalid.invalid> - 2016-12-11 20:53 +0100
        Re: php opdateret Erik Ginnerskov <egin@mail.dk> - 2016-12-11 19:52 +0100
      Re: php opdateret Jan Hansen <jhjjhjhhansen@gmail.com> - 2016-12-11 18:08 +0000
        Re: php opdateret Erik Ginnerskov <egin@mail.dk> - 2016-12-11 19:41 +0100
          Re: php opdateret scootergrisen <invalid@invalid.invalid> - 2016-12-11 20:00 +0100
            Re: php opdateret Erik Ginnerskov <egin@mail.dk> - 2016-12-11 20:08 +0100
              Re: php opdateret scootergrisen <invalid@invalid.invalid> - 2016-12-11 22:12 +0100
          Re: php opdateret Jan Hansen <jhjjhjhhansen@gmail.com> - 2016-12-11 19:41 +0000
            Re: php opdateret Jan Hansen <jhjjhjhhansen@gmail.com> - 2016-12-11 20:15 +0000
              Re: php opdateret Erik Ginnerskov <egin@mail.dk> - 2016-12-13 18:59 +0100
                Re: php opdateret scootergrisen <invalid@invalid.invalid> - 2016-12-13 19:34 +0100
  Re: php opdateret Krabsen <usenet@krabsen.dk> - 2016-12-11 22:13 +0100

csiph-web