bom galera estou aprendendo PHP, então estou treinando e criei um código para enviar noticias. Segue o código:

Código PHP:
<?php

/**
 * @author Matheus
 * @copyright 2014
 */

class enviarNoticias{
    private 
$titulo;
    private 
$autor;
    private 
$data;
    private 
$hora;
    private 
$categoria;
    private 
$mensagem;
    private 
$error "<span style="border:1px dashed #c00; color:#c00; padding:6px; background-color:#ffebe8;">";
    
    
function enviar(){
        
$this->titulo $_POST['titulo'];
        
$this->autor $_POST['autor'];
        
$this->data $_POST['data'];
        
$this->hora $_POST['hora'];
        
$this->categoria $_POST['categoria'];
        
$this->mensagem $_POST['msg'];
        
       if(empty(
$this->titulo)){
             echo 
$this->error."O campo <strong>Titulo</strong> deve ser preenchido! </span>";
       }
       elseif(empty(
$this->categoria)){
             echo 
$this->error."Selecione uma <strong>Categoria</strong>";
       }
       elseif(empty(
$this->mensagem)){
             echo 
$this->error."O campo <strong>Mensagem</strong> deve ser preenchido! </span>";
       }
    else{
                
$enviaDados mysql_query("INSERT INTO noticia VALUES ('','$this->titulo',' $this->mensagem','$this->autor','$this->data','$this->hora', '$this->categoria')");
                    echo 
'<script>alert("Noticia adicionada com sucesso!");</script>';

       } 
        
    }  

}

?>
queria que vocês que entendem mais sobre PHP deem opiniões. Agradeço desde já