Class Noticias

Queria que os coders analiza-se e me disse-se se tem algum erro...

arquivo news.class.php

Código PHP:

---

<?php


class news { 
    function insertnews()
    {
         $code = new code();
         $titulo    = $_POST['titulo'];
         $corpo     = $_POST['corpo'];
         $data      = $_POST['DATA'];
         $hora      = $_POST['hora'];
         $login        = $_SESSION['login'];

         # Insere os novos dados no Banco de Dados
         $sql = "insert into news (titulo, Noticia, DATA, hora, autor) Values ('$titulo','$corpo','$data','$hora','$login')";

         $query = mysql_query($sql);

         # Após inserir os dados
         echo $code->bbcode('<br />[sucess]Sua notícia foi inserida com sucesso.[/sucess]');


    }
   function addnews()
    { 
         $data = date ("d/m/Y",time());
         $hora = strftime("%H:%M:%S");
         
         # Mostra formulário para escrever os dados
         echo "<form method=\"post\" action=\"$PHP_SELF\">
         <p>Titulo da notícia:<br><input type=\"text\" name=\"titulo\" size=\"30\"></p>
         <p>Corpo da notícia:<br><textarea name=\"corpo\" rows=\"6\" cols=\"40\"></textarea></p>
         <p><input type='hidden' name='DATA' value='$data'>
         <input type='hidden' name='hora' value='$hora'>
         <input type='hidden' name=\"pass\" value=\"$senha\">
         <input type=\"submit\" name=\"inserir\" value=\"Postar\"></p>
         </form>";

    }
    function deletenews()
    {    
         $code = new code();
         $id = $_POST['id'];
         
         # Deleta os dados no Banco de Dados
         $sql = mysql_query("delete from news where id = $id");

        
         echo $code->bbcode('<br />[sucess]Sua notícia foi deletada com sucesso[/sucess]');
    }
    function updatenews()
    {
         $code = new code();
         $id        = $_POST['id'];
         $titulo    = $_POST['titulo'];
         $corpo     = $_POST['corpo'];
         $data      = $_POST['DATA'];
         $hora      = $_POST['hora'];

         # Atualiza os dados no Banco de Dados
         $sql = "update news set titulo = '$titulo', Noticia = '$corpo', DATA = '$data', hora = '$hora' where id = $id";

         $query = mysql_query($sql);

       
         echo $code->bbcode('<br />[sucess]Sua notícia foi atualizada com sucesso[/sucess]');
    }
    function recuperar()
    {
         $id  = $_POST['id'];
         
         # Procura os dados no Banco de Dados
         $sql = "select * from news where id = $id";
         $query = mysql_query($sql);

         $linha = mysql_fetch_array($query);
    {
         # Mostra os dados
         echo "<form method=\"post\" action=\"$PHP_SELF\">
         <p>Titulo da notícia:<br><input type=\"text\" name=\"titulo\" size=\"30\" value=\"" . $linha["titulo"] . "\"></p>
         <p>Corpo da notícia:<br><textarea name=\"corpo\" rows=\"6\" cols=\"40\">" . $linha["corpo"] . "</textarea></p>
         <p><input type='hidden' name=\"id\" value=\"" . $linha["id"] . "\">
         <input type='hidden' name=\"pass\" value=\"$senha\">
         <input type='hidden' name=\"DATA\" value=\"" . $linha["DATA"] . "\">
         <input type='hidden' name=\"hora\" value=\"" . $linha["hora"] . "\">
         <input type=\"submit\" name=\"update\" value=\"Postar\"></p>
         </form>";
    }
    }
}

?>

---

aceitando dicas :D sou leigo em php mais to querendo aprender então .. :)

Você ta encontrando algum erro?
Ou quer apenas dicas...
Pelo que li está tudo Ok e muito bom, sempre busque aprender coisas novas ;)

bom queria umas dicas...
to pensando em usar ele em um form com GET :D

Sei que o tópico está meio antigo mas não custa ajudar =)
No caso esse script tem uma parte vulnerável, sem filtro no $_POST['id'] realizando a consulta diretamente no MySQL, sem filtro nenhum.
Caso a variável seja numérica, coloque $id = (int)$_POST['id'] caso seja string $id = addslashes($_POST['id'])
No caso do form que você criou:
<form method=\"post\" action=\"$PHP_SELF\">
Não é preciso deixar o $PHP_SELF no action, se deixar vazio ele já manda automaticamente pra pagina atual que você está, só é usado o action="página.php" quando se manda os dados para uma página de fora.
E lembrando, NUNCA faça uma consulta vindo do site no MySQL ou em qualquer banco de dados sem filtrar este dado.
Espero ter ajudado.