Clases en php

[miguec04]

Código PHP:

class Empleat{
    private $conexion;
    private $_codi;
    private $_nom;
    private $_dept;
    
    public function __construct($xcodi, $xnom, $xdept){
        $this->_codi=$xcodi;
        $this->_nom=$xnom;
        $this->_dept=$xdept;
    }
 
   public function setConexion($conexion) {
      $this->conexion  = $conexion;
  }
 
    function getCodi(){
        return $this->_codi;
    }
    function getNom(){
        return $this->_nom;
    }
    function getDept(){
        return $this->_dept;
    }
    function guardarBd(){
        $xcodi= $this->getCodi();
        $xnom= $this->getNom();
        $xdept= $this->getDept();
        mysql_query("insert into empleat values($xcodi,'$xnom',$xdept)",$this->conexion);
    }
}
 

Lo primero es que por principio los atributos deben ser o privados o protegidos, protegidos cuando se hereden.

lo siguiente, yo veo que haces el mysql_query pero nunca envias la conexion a la clase para ejecutarla entonces creé un atributo conexion donde por parametro la asignare y poder ejecutar la sentencia.

Código PHP:

session_start();
//no se que tenga aqui en este archivo asi que pienso que trae una funcion llamada conexion(), espero ayudarte con eso.
require 'conexio.php';
require 'class_empleat.php';
 
if(isset($_POST['enviar'])){
 
   //no se como abre la conexion pero pienso que es de alguna de estas maneras
  $conexion = conexion();
 
    $codi=$_POST['codi'];
    $nom=$_POST['nom'];
    $dept=$_POST['dept'];
    
    $e = new Empleat($codi,$nom,$dept);
 
    $e->setConexion($conexion);
 
    if(empty($_SESSION['empleats'])) {
     $_SESSION['empleats'] = array();
   }
    $_SESSION['empleats'][]=$e;
    $e->guardarBd();
}
 

lo siguiente es que debe abrir la conexión en algún lado, no veo donde lo hizo, doy un ejemplo de como yo lo hubiese podido hacer.

otra cosa importante, por principio, el nombre de la clase debe ser el mismo del nombre del archivo, como para que sea mas entendible, mas practico, mas usable.

espero haber ayudado, cualquier cosa estoy pendiente de la respuesta.