wyswietlanie formularza i zapis danych w jednej funkcji Opcje

[emil_seba]

Witam mam taki problem otóż mam klasę

[PHP] pobierz, plaintext 
//plik class.Czlowiek.php
class czlowiek
{
private $Imie;
private $nazwisko;
 
public function wypelnijDane()
{
 echo '<form action="'.htmlentities($_SERVER['PHP_SELF']).'" method= "POST">
<input type = "text" name = "imie"/>
<input type = "text" name = "nazwisko"/>
<input type ="submit" Value"dodaj!!">
</form>';
}
public function ustaw()
{
 $this->Imie = $_POST['imie'];
$this->Nazwisko = $_POST['nazwisko'];
}
public function dodaj()
{
 $query = "INSERT INTO uzytkownicy SET imie ='$this->Imie', nazwisko = '$this->nazwisko'";
$result = mysql_query($query);
if($result) echo 'wstawiono uzytkownika';
else echo 'blad'.mysql_error(); 
}
 
[PHP] pobierz, plaintext 

I teraz chcial bym utworzyc odnosnik do wstawiania uzytkownika:

[PHP] pobierz, plaintext 
//plik index.php
$require_once('class.Czlowiek.php');
$pomoc = 1;
echo "<a href = \"?show=".$pomoc."\">Utworz ucznia</a><br />"
if(isset($_GET['show']))
{
 $czlowiek =  new Czlowiek();
$czlowiek -> wypelnijDane();
$czloweik -> ustaw();
$czlowiek -> dodaj()
}
[PHP] pobierz, plaintext 

Problem w tym ze jezeli tak to wywolam to tylko wyswietli sie formularz nie ustawiajac zmiennych Imie i Nazwisko gdyz tablica $_POST jest pusta.
Nie chcial bym do kazdej funkcji tworzyc osobnego odnosnika by sie wywolala. Wiem ze jezeli funkcje ustaw i dodaj wywale poza ifa to bedzie dzialac, ale co zrobic gdy np mam inna klase i w niej mam funkcje dodajCzlowieka ktora zawiera wywolanie tych 3 funkcji klasy czlowiek

[PHP] pobierz, plaintext 
require_once('class.Czlowiek.php')
class nadCzlowiek
{
$czlowiek;
public function dodajCzlowieka
{
$this-> czlowiek = new Czlowiek;
$this->czlowiek ->wypelnijDane();
$this->czlowiek->ustaw();
$this->czlowiek->dodaj();
}
}
 
 
//plik index.php
$nadczlowiek = new nadCzlowiek();
$nadczlowiek -> dodajCzlowieka()
[PHP] pobierz, plaintext 

Bardzo prosze o pomoc bo nie wiem jak sobie poradzic z tym problemem

[Cysiaczek]

Słowa klucze - enkapsulacja i parametryzacja
Błędy:
- echo w ciele metod - niczym nieuzasadnione
- automatyczne przypisywanie wartości z $_POST do atrybutów obiektu

Jeśli rozwiążesz te dwa problemy to Twój problem się rozwiąże sam.

[emil_seba]

możesz wyjaśnić mi o co rozchodzi się z tym automatycznym przypisywaniem wartości $_POST atrybutom obiektów ?

[jaslanin]

jest:

[PHP] pobierz, plaintext 
 
public function ustaw()
{
 
$this->Imie = $_POST['imie'];
$this->Nazwisko = $_POST['nazwisko'];
 
}
[PHP] pobierz, plaintext 

a powinno być

[PHP] pobierz, plaintext 
public function ustaw($imie, $nazwisko)
{
 
$this->Imie = $imie;
$this->Nazwisko = $nazwisko;
 
}
[PHP] pobierz, plaintext 

Ten post edytował jaslanin 26.05.2010, 11:42:00

[emil_seba]

hmmm.. ale gdzie jest przypisanie do zmiennej np imie watrosci $_POST['imie']

[nospor]

Tam, gdzie będziesz wywoływał metodę ustaw()

[emil_seba]

czyli moja funkcja powinna wyglądać tak :

[PHP] pobierz, plaintext 
public function ustaw($imie, $nazwisko)
{
$imie = $_POST['imie'];
$nazwisko = $_POST['nazwisko'];
$this->Imie = $imie;
$this->Nazwisko = $nazwisko;
}
[PHP] pobierz, plaintext 

nie rozumiem.. to po co te argumenty w funkcji?? Proszę o wyjaśnienie

[nospor]

Wywołanie funkcji to nie jest deklaracja funkcji.
Zapoznaj się z podstawami
http://pl2.php.net/manual/pl/language.functions.php

[emil_seba]

tak wiem, ale nie rozumiem jak mam napisać taką funkcje ? Wyżej napisano że powinienem napisać funkcje z dwoma argumentami, oraz że powinienem w tej funkcji przypisać dane z $_POST do zmiennych, wiec nie wiem co maja zawierać te argumenty w funkcji. Jedyne co mi przychodzi do głowy to wywołanie funkcji z argumentami $_POST

[PHP] pobierz, plaintext 
definicja funkcji 
public function ustaw($imie, $nazwisko)
{
$this->Imie = $imie;
$this->Nazwisko = $nazwisko;
}
 
wywolanie funkcji
$obiekt -> ustaw($_POST['imie'], $_POST['nazwisko'])
[PHP] pobierz, plaintext 

ale nie działa tak jak chce.

[nospor]

odpowiadalem jedynie na Twoj ostatni post.
Teraz przeczytalem temat i masz inne bledy

Problem w tym ze jezeli tak to wywolam to tylko wyswietli sie formularz nie ustawiajac zmiennych Imie i Nazwisko gdyz tablica $_POST jest pusta.
Nie chcial bym do kazdej funkcji tworzyc osobnego odnosnika by sie wywolala. Wiem ze jezeli funkcje ustaw i dodaj wywale poza ifa to bedzie dzialac, ale co zrobic gdy np mam inna klase i w niej mam funkcje dodajCzlowieka ktora zawiera wywolanie tych 3 funkcji klasy czlowiek

No bo chcesz pójść po najmniejszej linii oporu i wszystko zawrzec w 3 funkcjach w ktorych masz burdel na kółkach.
Owszem, możesz tak zrobić, nikt ci nie broni. Musisz wiec przed dodawaniem rekordu sprawdzac czy POST poszedl (np. isset($_POST['imie'])). Wowczas bedzie sie dodwalao wtedy kiedy trzeba.

[emil_seba]

tylko ze próbuje już 2 dzień i wniosek mam jeden jeżeli tak wywołam jak wyżej to te dane nie zostaną ustawione sama funkcja ustaw się wykona ale jak mam w niej warunek if(!empty($_POST['imie'])) to nie ustawia mi zadnych zmiennych bo o ile dobrze rozumiem to on wchodzi do funkcji

[PHP] pobierz, plaintext 
public function dodajCzlowieka()
{
public function wypelnijDane();
public function ustaw()
public funciton dodaj()
}
[PHP] pobierz, plaintext 

wykonuje wszystkie 3 funkcje na raz czyli w funkcji WypelnijDane nie zostalo jeszcze nic wyslane w $_POST a pozostałe już sie wykonały, jezeli ustawie warunek w ustaw() sprawdzajacy czy cos jest w $_POST to to tez nic nie da bo $_POST bedzie puste dopoki nic nie wysle a jak juz wysle to funkcja ustaw() dawno sie juz wykonala.

Ma ktos moze pomysl jak to inaczej rozwiac tzn jak utworzyc w funkcji klasy jednego obiektu obiekt innej klasy?

[jaslanin]

[PHP] pobierz, plaintext 
 
public function dodajCzlowieka
 
{
$this-> czlowiek = new Czlowiek;
$this->czlowiek ->wypelnijDane();
 
if ( /* formularz został wysłany */) {
  $this->czlowiek->ustaw();
  $this->czlowiek->dodaj();
}
 
}
[PHP] pobierz, plaintext 

ale ładniej byś to zrobił jakbyś rozbił na dwie osobne rzeczy wyświetlanie formularza, i ustawianie/dodawanie człowieka

Ten post edytował jaslanin 26.05.2010, 13:18:42

[emil_seba]

no tylko tak nie chce mi działać jeżeli w ifie zrobię warunek if(!empty($_POST['imie'])) to funkcje ustaw i dodaj nie wykonają się a innego pomysłu na sprawdzenie czy formularz został wysłany nie mam

Ten post edytował emil_seba 27.05.2010, 12:11:53

[jaslanin]

dodaj pole input:

Kod

<input type="hidden" name="formularzWyslany" value="1" />

i sprawdzaj odpowiednio czy formularzWyslany ma wartość 1

a ten if jest po to żeby te funkcje się nie wykonały, jeżeli nie masz wypełnionego formularza to skąd weźmiesz imię i nazwisko ?

Ten post edytował jaslanin 27.05.2010, 12:46:27

[emil_seba]

tak ja to rozumiem ale po prostu obojętnie czy ustawie ifa czy nie to nie działa tak jak chce ponieważ te 3 funkcje są wykonywane w 1 a wykonanie pierwsze potrzebuje chyba "odświeżenia strony". Na mój chłopski rozum to on wykonuje pierwsza (nie odświeża sobie tablicy $_POST) i wykonuje następne ponieważ wykonywane są w 1 funkcji dopiero po wyjściu z funkcji "odświeża tablice"

[jaslanin]

masz pusty formularz, to co jest w instrukcji warunkowej się nie wykonuje, wypełniasz formularz, przyciskasz wyślij (to jest pewnie to twoje odświeżenie o którym piszesz), tablica $_POST jest uzupełniana, warunek zachodzi, ustaw i dodaj się wykonują.

[emil_seba]

ale wstaw te 3 funkcje w 1 i wywołaj ją.. mi za każdym razem nie działa... zrobiłem to inaczej formularz zrobiłem jako osobna stronę wykonywana po kliknięciu na link i mającą pole ukryte (jak ktoś wyżej polecił) wtedy sprawdzam czy te pole ukryte ma wartość jeżeli tak to wykonuje funkcje ustaw() i dodaj(), może ktoś mi poradzić jak otworzyć stronę z formularzem w stronie glownej(idzie tak wogole?) aby zachowac caly layout strony bez ponownego definiowania go w stronie z formularzem, z góry dziękuje za odpowiedź i poprzednie odpowiedzi