Code Bewerten

Hallo Zusammen


Darf ich Euch fragen wie Ihr den Code findet? Mache ich das kompliziert, nicht korrekt oder ist das ok so?

Danke für das Feedback.




Controller

public function actionProject() {

        $data = new TblProjekt;
  $form = new TblStatus;
  $modelNachricht = new Tbl_Nachrichten;

        // Gast und $_GET ID prüfen 
        if (isset($_GET['id']) && !Yii::app()->user->isGuest) {

            $data = TblProjekt::model()->findAllByPk($_GET['id']);
   
        } else {
            $data = TblProjekt::model()->findAll();
        }
  
  // Wenn der Benutzer Gast ist und sich bewirbt
        if (Yii::app()->user->isGuest) {

            $msg = "Bitte zuerst anmelden.";
            $this->render('mobile', array('var1' => $msg, 'typ' => 'alert alert-error', 'data' => $data));  
   
        } else {
  
   if (isset($_POST['TblStatus'])) { 
  
    // Wenn der Benutzer angemeldet ist und sich bewirbt
    $form->setAttributes($_POST['TblStatus']);
    $modelNachricht->setAttributes($_POST['Tbl_Nachrichten']);
    $modelNachricht->idfs_User = Yii::app()->user->id;
    $modelNachricht->idfs_Projekt = $_GET['id'];
    $modelNachricht->save();
    $form->idfs_Bewerber = Yii::app()->user->id;
    $form->idfs_Faehigkeit = $form->idfs_Faehigkeit;
    $form->idfs_Projekt = $_GET['id'];
    $form->Datum = time();
    $form->Status = "Anfrage offen";
    $form->Save();
    
    
     if ($form->validate()) {
     
     // Mail versenden
     $Projekt = TblProjekt::model()->findByPK($_GET['id']);
     $User = User::model()->findByPK($Projekt->Benutzer);
     $AnbieterEmail = $User->username;
     Yii::app()->Mail->Send_Bewerbung($AnbieterEmail, $Projekt);
     
     $msg = "Die Bewerbung wurde erfolgreich an den Projektleiter versendet.";
     $data = TblProjekt::model()->findAll();
     $this->render('mobile', array('var1' => $msg, 'typ' => 'alert alert-success', 'data' => $data,'model'=>$form, 'modelNachricht'=>$modelNachricht));
     }
      
   } else {
   
   // Wenn Formular leer ist
   
   $msg = "Bitte wählen Sie die Fähigkeit aus";
   $this->render('mobile', array('var1' => $msg, 'typ' => 'alert alert-info', 'data' => $data,'model'=>$form, 'modelNachricht'=>$modelNachricht));
   }
    }
 }

View

<div> <?
  if (isset($var1)) {
   Echo "<div class='". $typ ."'>";
   echo $var1;
   echo "</div>";
  } ?>
 </div>

 <?
 
 // Wenn es um eine Bewerbung geht
 if( isset($model)) {
  
  // Pinboard anzeigen
  Yii::app()->Pin->CreatePin($data); 
  
  // Fähigkeiten auswählen
  $this->renderPartial('myform', array('model' => $model, 'modelNachricht' => $modelNachricht));  
  
 } else {
 
  // Pinboard erstellen
  Yii::app()->Pin->CreatePin($data);
 }

 $model = TblProjekt::model()->FindByPk(29); ?>