[yii] PHP notice Undefined variable Opcje

[Marek607]

Witam,

Nie wiem czy to bardziej do przedszkola czy do działu o frameworkach, ale dałem tutaj - jak zły dzial to poprosze o przeniesienie.
Kupiłem książke z helionu o yii framework i mam dziwny problem - nie chce mi wykonać prostego kodu, tylko wyświetla informacje - przykładowe 2 kody:

[PHP] pobierz, plaintext 
<?php
class FormularzdaneController extends CController
{
public function actionIndex()
{
if($_POST['wyslano'] == 'tak')
	{
	$Polaczenie = Yii::app()->db;
	$Zapytanie = $Polaczenie->createCommand('INSERT INTO osoby (imie,nazwisko) VALUES (:imie,:nazwisko)');
	$Zapytanie->bindParam(":imie",$_POST['imie'],PDO::PARAM_STR);
	$Zapytanie->bindParam(":nazwisko",$_POST['nazwisko'],PDO::PARAM_STR);
	$Zapytanie->execute();
	echo '<div>Dane zostały umieszczone w tabeli.</div>';
}
echo '<form action="formularzdane" method="post">
Imię: <input type="text" name="imie" /><br />
Nazwisko: <input type="text" name="nazwisko" /><br />
<input type="hidden" name="wyslano" value="tak" />
<input type="submit" value="Dodaj" />
</form>';
}
}
?>
[PHP] pobierz, plaintext 

błąd: Undefined index: wyslano z oznaczeniem na czerwono: if($_POST['wyslano'] == 'tak')
Po wyłączeniu trybu developerskiego mam error 500 z tym samym kodem.

Wiem czego tyczy błąd jednak nigdy brak zmiennej (bo będzie ona dopiero po wypelnieniu formularza) nie powodowało że nie wyświetał się formularz:
http://scr.hu/0anw/4n6g0

Doszłem już do 2 formularza i jest taki sam problem, więc proszę o pomoc.

Ten post edytował Marek607 5.01.2014, 14:58:08

[Turson]

Ogólnie kod fatalny Całkowicie zgubiłeś sens MVC

Kontroler

[PHP] pobierz, plaintext 
<?php
class FormularzdaneController extends CController
{
	public function actionIndex()
	{
		if(isset($_POST)){
			$Polaczenie = Yii::app()->db;
			$Zapytanie = $Polaczenie->createCommand('INSERT INTO osoby (imie,nazwisko) VALUES (:imie,:nazwisko)');
			$Zapytanie->bindParam(":imie",$_POST['imie'],PDO::PARAM_STR);
			$Zapytanie->bindParam(":nazwisko",$_POST['nazwisko'],PDO::PARAM_STR);
			$Zapytanie->execute();
			return $zapisano;
		}
		$this->render('index', array(
			'zapisano'=>$zapisano,
		));
	}
}
[PHP] pobierz, plaintext 

Widok:

[PHP] pobierz, plaintext 
<?PHP if(!isset($zapisano)):?>
//Wyświetlenie formularza
<?PHP endif;?>
[PHP] pobierz, plaintext 

Ogólnie formularze tworzy się inaczej: http://www.yiiframework.com/doc/guide/1.1/pl/form.view

Po trzecie zapisywanie danych w kontrolerze wygląda nieco inaczej. Używasz $model->attributes, $model->save

Po czwarte, Yii ma własny sposób zapytań do bazy w modelu, w typie:

[PHP] pobierz, plaintext 
$cmd = Yii::app()->db->createCommand();
$row = $cmd->from($this->tableName())
->select('*')
->where('id=:id', array('id'=>$id))
->queryRow();
[PHP] pobierz, plaintext 

Ten post edytował Turson 5.01.2014, 15:06:52

[Marek607]

Przykłady są podane bezpośrednio kopiuj-wklej z książki http://helion.pl/ksiazki/yii-framework-luk...osna,yiifra.htm - podpowiesz mi jeszce co poprawić w tym kontrolerze?

[PHP] pobierz, plaintext 
<?php
class FormularzController extends CController
{
public function actionIndex()
{
$ModelOsoby = new OsobyFormularz;
if(isset($_POST['OsobyFormularz']))
{
$ModelOsoby->attributes = $_POST['OsobyFormularz'];
if($ModelOsoby->validate())
{
$ModelOsoby->save();
$Zapisano = true;
}
}
$this->render('formularz',
array(
'model'=>$ModelOsoby,
'Zapisano'=>$Zapisano
)
);
}
}
?>
[PHP] pobierz, plaintext 

Model:

[PHP] pobierz, plaintext 
<?php
class OsobyFormularz extends CActiveRecord
{
	public $id;
	public $imie;
	public $nazwisko;
	public static function model($className=__CLASS__)
	{
		return parent::model($className);
	}
	public function tableName()
	{
		return 'osoby';
	}
	public function attributeLabels()
	{
		return array(
		'id' => 'ID',
		'imie' => 'Imię',
		'nazwisko' => 'Nazwisko',
		);
	}
	public function rules()
	{
		return array(
		array('imie', 'required'),
		array('nazwisko', 'required'),
		);
	}
}
?>
[PHP] pobierz, plaintext 

Widok:

[PHP] pobierz, plaintext 
<div class="form" style="padding: 15px;">
<?php $form=$this->beginWidget('CActiveForm', array(
'id'=>'osoby-formularz',
'enableAjaxValidation'=>false,
)); ?>
<p class="note">Pola oznaczone <span class="required">*</span> są wymagane.</p>
<?php
if($Zapisano == true)
{
echo '<div style="color: #600000;">Dane zostały zapisane.</div>';
}
?>
<?php echo $form->errorSummary($model); ?>
<div class="row">
<?php echo $form->labelEx($model,'imie'); ?>
<?php echo $form->textField($model,'imie'); ?>
<?php echo $form->error($model,'imie'); ?>
</div>
<div class="row">
<?php echo $form->labelEx($model,'nazwisko'); ?>
<?php echo $form->textField($model,'nazwisko'); ?>
<?php echo $form->error($model,'nazwisko'); ?>
</div>
<div class="row buttons">
<?php echo CHtml::submitButton('Dodaj'); ?>
</div>
<?php $this->endWidget(); ?>
</div>
[PHP] pobierz, plaintext 

mam podobnie jak powyżej, czyli:
http://scr.hu/0anw/3ry6a

Miałem się uczyć z dokumentacji, ale zawsze wolałem książkę - chyba jednak zmienie zdanie skoro twierdzicie że uczy złych nawyków.

Ten post edytował Marek607 5.01.2014, 15:17:37

[Turson]

Szczerze mówiąc nie rozumiem czemu wywaliło Ci błąd, bo sam tak stosuję i jest ok.

[PHP] pobierz, plaintext 
<?php
class FormularzController extends Controller{
	public function actionIndex(){
	$ModelOsoby = new OsobyFormularz;
	if(isset($_POST['OsobyFormularz'])){
		$ModelOsoby->attributes = $_POST['OsobyFormularz'];
		if($ModelOsoby->validate()){
			$ModelOsoby->save();
			$Zapisano = true;
		}
	}
	else{
		$Zapisano = false;
	}
	$this->render('formularz',array(
	'model'=>$ModelOsoby,
	'Zapisano'=>$Zapisano
	));
	}
}
?>
[PHP] pobierz, plaintext 

w widoku if($Zapisano===true)


Przykład z posta #1 jest serio z książki? W takim razie dobre jaja

Ten post edytował Turson 5.01.2014, 15:27:38

[tomxx]

Jeżeli pierwszy raz ładujesz stronę, to w zmiennej $_POST nie ma indeksu ['wyslano'], ponieważ formularz nie został jeszcze wysłany (więc zmienna $_POST jest pusta). Wpisz:

[PHP] pobierz, plaintext 
if(@$_POST['wyslano'] == 'tak')
[PHP] pobierz, plaintext 

Ten post edytował tomxx 5.01.2014, 16:58:13

[Turson]

Zamiast wygłuszać błędy to lepiej isset jak z resztą podałem w kodach wyżej.

[Marek607]

Dzięki Turson - dodanie else pomogło.

Tak, są to przyklady z książki - póki co to się nie zniechęcam tylko śmiało do przodu ide