zagnieżdżanie html w klasie?, przykład z książki! Opcje

[bl4ck_b0x]

Witam. Przeglądając stronę Heliona (tak wiem, wielu narzeka na te wydawnictwo - ale nie o to chodzi) trafiłem na książkę "Programowanie obiektowe w PHP 5", korzystając z przykładów zamieszczonych na ftp (które są w tej książce), ściągnąłem, rozpakowałem archiwum, otworzyłem rozdział 1, zawiera się w nim jedna klasa - emailer.

[PHP] pobierz, plaintext 
<?
// class.emailer.php
class Emailer
{
   private $sender;
   private $recipients;
   private $subject;
   private $body;
   function __construct($sender)
   {
      $this->sender = $sender;
      $this->recipients = array();
   }
 
   public function addRecipients($recipient)
   {
      array_push($this->recipients, $recipient);
   }
   public function setSubject($subject)
   {
      $this->subject = $subject;
   }
   public function setBody($body)
   {
      $this->body = $body;
   }
   public function sendEmail()
   {
      foreach ($this->recipients as $recipient)
      {
         $result = mail($recipient, $this->subject, $this->body,
            "From: {$this->sender}r\n");
         if ($result) echo "Wiadomość została wysła do
            {$recipient}<br/>";
      }
   }
}
?>
[PHP] pobierz, plaintext 

Pytanie moje jest następujące: czy dozwolonym, etycznym i poprawnym jest zamieszczanie kodu HTML w samej klasie? Kiedyś przeglądając forum trafiłem na klasę pisaną przez amatora która była przepełniona HTMLem i o mało nie został tutaj zlinczowany ;P Jako, że nie piszę obiektowo (ciągle się do tego przymierzam) stąd moje pytanie - aby nie popełniać błędów (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

Drugie pytanie: jeśli używanie HTMLa w samej klasie jest błędem jak zatem informować o czymś użytkownika nie wpisując tego w klasie? Stosować true i false a "po stronie użytkownika" (brakuje mi słowa które niedawno wyczytałem; chodzi o wykorzystanie klasy... mam nadzieję, że zrozumiecie o co chodzi) stosować:

[PHP] pobierz, plaintext 
<?
...
if (true)
{
 echo 'wyslano';
}
else
{
 echo 'nie wyslano';
}
..
?>
[PHP] pobierz, plaintext 

Proszę o odpowiedzi (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

P.S.
Jeśli macie jakieś rady dla kogoś raczkującego w temacie programowania zorientowanego obiektowo to również możecie pisać (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Ten post edytował bl4ck_b0x 23.06.2009, 22:42:28

[erix]

Pytanie moje jest następujące: czy dozwolonym, etycznym i poprawnym jest zamieszczanie kodu HTML w samej klasie?

Jeśli działasz w oparciu o model MVC - jest niedozwolone, nieetyczne i niepoprawne. (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)

Drugie pytanie: jeśli używanie HTMLa w samej klasie jest błędem jak zatem informować o czymś użytkownika nie wpisując tego w klasie? Stosować true i false a "po stronie użytkownika" (brakuje mi słowa które niedawno wyczytałem; chodzi o wykorzystanie klasy... mam nadzieję, że zrozumiecie o co chodzi) stosować:

HTML a komunikat, to nieco inne sprawy. Możesz przecież dorzucać kolejną zmienną akcji, wywołać widok komunikatu i wypluć informację.

[Rafal Filipek]

Pytanie moje jest następujące: czy dozwolonym, etycznym i poprawnym jest zamieszczanie kodu HTML w samej klasie?

zalezy pewnie od klasy, czasami jest konieczne umieszczanie fragmentow html'a ( np. klasy odpowiedzialne za widok [View] we frameworkach albo odpowiedzialne za prasowanie np. texile )
co do "informowanie uzytkownika"

Kod

<?php
class Foo{
  public function czyMoge(){
    return true;
  }
}
$f = new Foo();
?>

//jakis tam html

<?php if ($f->czyMoge()): ?>
<p>tak, możesz</p>
<?php else: ?>
<p>nie możesz</p>
<?php endif; ?>

ale to jest tylko przyklad (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) wszystko zalezy od konkretnego przypadku (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[erix]

( np. klasy odpowiedzialne za widok [View] we frameworkach albo odpowiedzialne za prasowanie np. texile )

Za widok? A ja myślałem, że od tego jest skeleton (szkielet strony, ale też z szablonów)... Textile? No to jest już bardziej powiązane z widokiem, to nie ma rady - musi być.