"Svenne Krap" <usenet@krap.dk> skrev i en meddelelse
news:su2b3u872pvk6t9h6cg7apsjsfd3j1ll9s@4ax.com...
> On Fri, 4 Jan 2002 01:40:28 +0100, "Jesper N. Henriksen"
> <ccc60350@vip.cybercity.dk> wrote:
>
> >Jeg har lige skrevet
> >
http://www.geekarea.dk/q-data/admin.phps
>
> Uha.
>
> Der er mange issues med den side.
> Det kan være, at den stadig kun er på skitse plan eller det kan også
> være, at du endnu ikke er så rutineret. Derfor kommer der lige de par
> ting, der springer mig i øjnene:
>
> - nu er det en smag sag, men jeg mener den der $action filosofi er
> mildt sagt tåbelig. Brug lidt flere filer på det og få meget større
> overskuelighed. Det betaler sig på lang sigt.
> - logininformationer til databasen bør ikke hardcodes på hver side men
> lægges i en include fil. Det gælder fx. host, user,database og
> password
> - hvorfor har du tabel som en variabel ? er det noget, der skal ændre
> sig undervejs ? Ellers er det ikke hensigtsmæssigt.
> - hvad med fejlhåndtering i fald af, at du ikke kan connecte eller
> selecte database ?
> - og til slut den helt grimme. Hvad med kontrol (validering) og
> håndtering af manglede eller ugyldigt (dvs. invalidt) input til de to
> opdaterende queries (insert/update)
> - og hvorfor fyrrer du disse queries af "SELECT id, dato, type,
> specifikation, pris FROM $tabel" ? Du overskriver alligevel
> resultatet, når du ryger ud af switch-konstruktionen.
>
Hej Svenne
Jeg sad bare og kedede mig, så jeg tænkte at jeg lige ville bikse noget
sammen... det er snart et halvt år siden jeg sidst har lavet noget i php, så
det var bare et forsøg på at huske!!! ... da jeg så fik den der fejl...
kunne jeg ikke umiddelbart finde ud af hvorfor... ville så bare lige
spørge... men jeg giver dig helt ret!!!
Jesper