Hjælp mig med at blive bedre.

Tags:    php

Jeg prøver på at lære PHP og tænkte jeg ville prøve at poste lidt kode herinde og se om jeg kunne få nogle råd til hvordan jeg kan gøre det bedre og om jeg gør noget helt helt forkert.

Jeg har forsøgt at lavet en Class til en MySQL database for at lære lidt. Meningen er at det skal være let at bruge og at fejl skal skrives i en fejl log:


<?php
Class database {

var $connection;

function clean($zap) {
$zap = strip_tags($zap, '/n');
return $zap;
}
//----------------------------------------------------------

function database() {
$this->connection = @mysql_connect($dbserver, $bruger, $pass);
@mysql_select_db($database,$this->connection);
}
//----------------------------------------------------------

function query($sql) {
$result = @mysql_query($sql, $this->connection);
if(!$result) {
$this->errorlog(mysql_errno($this->connection), mysql_error($this->connection));
return FALSE;
}else{
$num = @mysql_num_rows($result);
if(!$num) {
$num = '0';
}
return array($result, $num);
}
}

//----------------------------------------------------------
function errorlog($error_num, $error_txt) {
$dato = date("\\tG:i d/m-y");
$error_num = addslashes($error_num);
$error_txt = addslashes($error_txt);
$user_ip = $_SERVER['REMOTE_ADDR'];
$sql = "INSERT INTO errorlog (error_num, error_txt, time user_ip) VALUES ('$error_num', '$error_txt', '$dato', '$user_ip')";
$this->query($sql);
}
}
?>






Den 07-08-05 20:50 skrev Dust følgende:
-----------------------------------------------------------------------------
jeg ville foreslå at du startede variabler i klasser men f og variabler i
functioner med a.
Fold kodeboks ind/udKode 


tror det ret upraktisk at din classe connecter til db når den bliver createt, i stedet for ville jeg lade de funktioner der interagere med databasen connecte og disconnecte.

Fold kodeboks ind/udKode 


end videre mangler der såvidt jeg kan set et komma i din insert query nede i error log mellem time og user_ip i fieldsne.


-----------------------------------------------------------------------------

Min side ->www.the-hive.dk/~donp/



Tak for dit svar.
Det prøver jeg lige at kigge på.

Forresten, hvordan laver man det så kodeeksempler bliver foldet sammen?

[Redigeret d. 07/08-05 23:27:17 af Dust]



du laver et pre tag rundt om dem men med [ i stedet for < og ] i stedet for >
-----------------------------------------------------------------------------

Min side ->www.the-hive.dk/~donp/



t