Action class

Heb je wel eens gehoord van CRUD? dat staat voor Create Read Update & Delete.
Dat staat vaak in 1 klasse ik weet niet of je een design pattern gebruikt of niet, maar het is uiteindelijk je eigen keuze hoe je het opdeelt.
Als ik jou was zou ik de bewerken "update" functie gewoon toevoegen aan je ArticleAction.php.

Ik zal daar eens na gaan zoeken.

De action class heeft nu enkel een execute methode.

<?
$articleAction = new ArticleAction();
if ( $articleAction->execute() )
{
 echo 'GELUKT';
}
else
{
 echo 'FORMULIER';
}
?>

Volgens mij zie je het verkeerd. Een article is een object.

<?php
$article = new Article();
?>
Dit object (het artikel) bevat meerdere functies (in een class ook wel "methods" genoemd) en properties (eigenschappen).

Je moet dan bijvoorbeeld aan zoiets denken:

<?php
class Article {

private $author;
private $text;
private $title;

public function __construct($author) {
$this->author = $author;
}

public function getAuthor() {
return $this->author;
}

public function setText($text) {
$this->text = $text;
}

public function getText() {
return $this->text;
}

public function editText($text) {
$this->text = $text;
}

}
?>
En dan gaan we een artikel aanmaken:

<?php
$article = new Article('Tom Swinkels');
$article->setText('Het is bijna lente!');
echo 'Auteur is: ' . $article->getAuthor();
echo 'Tekst: ' . $article->getText();
?>
Dit is maar even een simpel voorbeeld, maar het gaat vooral om het idee hoe OOP werkt.

@ozzie:

Klopt, dit gedeelte heb ik al ;) Dit gaat over de Article class, ik heb het nu over een Action class.

Een action class word bijvoorbeeld uitgevoerd bij een submit.

Voorbeeld van een action class

<?php
class ArticleAction
{
    private $_articleMapper;

    public function __construct( ArticleMapper $articleMapper )
    {
        $this->_articleMapper = $articleMapper;
    }

    public function execute( Request $request )
    {
        $this->_error = new Error();

        $emptyValidator = new EmptyValidator();

        if( !$emptyValidator->valideer( $request->post( 'submit' ) ) )
        {
            if( $emptyValidator->valideer( $request->post( 'title' ) ) )
            {
                $this->_error->setError( 'Geen titel ingevuld' );
            }
            if( $emptyValidator->valideer( $request->post( 'content' ) ) )
            {
                $this->_error->setError( 'Geen content ingevuld' );
            }
            elseif( !$this->_error->checkError() )
            {
				$article = new Article();
				$article->setTitle( $request->post( 'title' ) );
				$article->setContent( $request->post( 'content' ) );
				
				$this->_articleMapper( $article );
				
				return true;
			}
		}
		return false;
	}
}
?>

[size=xsmall]Toevoeging op 18/03/2013 15:20:36:[/size]

Krijg mijn vorige post niet gewijzigd, klein foutje $this->_articleMapper( $article ); word -> $this->_articleMapper->create( $article );.

Maar je hoeft toch niet een aparte action class te maken? Je kan toch een update/edit method maken in je Article class? Ik zie niet in waarom je een aparte action class zou maken.

Werk je ook met een controller? Anders zou de execute met een andere naam meer een methode hiervan zijn.