[CI] Edycja danych z bazy a form_validation Opcje

[abort]

W kontrolerze definiuję sobie za pomocą $this->form_validation->set_rules, jakie mogą być dopuszczalne wartości pól w formatce, następnie wywołuję widok:

[PHP] pobierz, plaintext 
$hw = $this->sprzet->get_hwdata ($id);  // id to numer indeksu w bazie (auto_increment)
$data['hw'] = $hw;
if ($this->form_validation->run() == FALSE) {
	$this->load->view('/hardware/edit', $data);
} else {
	[...]
}
[PHP] pobierz, plaintext 

I fajnie - do widoku zostały przekazane dane. Chciałbym jednak, aby wprowadzenie błędnych danych skutkowało zmianą np. koloru wyświetlenia elementu "<input type=text>", z zachowaniem (wyświetleniem) danych wprowadzonych przez użytkownika. Wymyśliłem sobie więc, że w widoku wpakuję ciukę kodu, efekt:

[PHP] pobierz, plaintext 
foreach (array ( [tu indeksy przekazywane przez HTTP_POST] ) as $key) {
	if (isset ($_POST [$key])) {
		$hw[$key] = set_value ($key);
	} else {
		$hw[$key] = $db[$key];
	}
}
 
// a dalej w kodzie sprawdzam, co wprowadził user, i jeśli jest źle, to wizualnie mu to sygnalizuję
// (np. przez zmianę backgroundu elementu INPUT TYPE=TEXT)
 
$manufact = array (
	'name'	=> 'manufact',
	'id'		=> 'manufact',
	'value'	=> $hw['manufact'],
	'class'	=> ''
);
if (form_error ('manufact')) {
	$manufact['class'] .= 'error';
}
echo '<div class="manufact">Producent:' . form_input ($manufact) . '</div>';
[PHP] pobierz, plaintext 

Pytania:
1. Czy da się prościej?
2. kłuje w oczy użycie $_POST, ale taka konstrukcja działa, podczas gdy użycie if (isset ($this->input->post($key))) wyrzuca błąd: Fatal error: Can't use method return value in write context in [...] - o co kaman, przecież nic nie zapisuję, tylko testuję, czy "coś" istnieje...
3. Da się (to co chcę) zrobić inaczej, prościej, bardziej elegancko?

P.S.
CodeIgniter 2.1.0, więc całkiem nowy.

[phpion]

1 + 3. Ja stosuję taki mechanizm (pseudokod):

[PHP] pobierz, plaintext 
$form = array();
$errors = array();
 
if (wyslano_postem()) {
	if (dane_sa_poprawne()) {
		// zapis danych do bazy
		// przekierowanie czy cokolwiek
	}
	else {
		$form = przeslane_dane();
		$errors = wykryte_bledy();
	}
}
else {
	// ustawienie poczatkowych wartosci $form
}
 
$view->form = $form;
$view->errors = $errros;
[PHP] pobierz, plaintext 

2. Spróbuj samo:

[PHP] pobierz, plaintext 
if ($this->input->post($key))
[PHP] pobierz, plaintext 

[abort]

Rada dotycząca pytania 2 pomogła, dzięki.
Natomiast jeśli chodzi o pytania 1 i 3, to mam lekkie wątpliwości: wygląda mi na to, że w CI walidacja wprowadzonych widoków wykonuje się w kontrolerze - wszak w kontrolerze tworzę callbacki do niestandardowej weryfikacji danych (http://codeigniter.com/user_guide/libraries/form_validation.html i poszukać po "Callbacks: Your own Validation Functions"). Widzę też, że Ty rozumujesz podobnie (jeśli dobrze czytam kod): robisz to w kontrolerze, w którym testujesz dane, ustawiasz parametry widoku, czy zapisujesz do bazy (przecież z widoku nikt nie zapisuje do bazy, prawda?)

Natomiast ja swoje proste foreach dopisałem już w samym widoku - i tu mam lekki zgryz, bo z jednej strony jest to krótsze, poza tym sam mój kod odwołuje się (jak by na to nie patrzeć) do PREZENTACJI danych (dane zostały wprowadzone źle, więc wyświetlmy je użytkownikowi w WIZUALNY sposób używając odpowiedniej klasy CSS). Choć z drugiej strony operuję niejako na poprawności danych (choć ich nie sprawdzam, bo tę robotę odwala kontroler).

No i pozostaje pytanie: co lepsze? A może ja nie rozumiem OOP (wszak dopiero zaczynam)

[phpion]

Reguły walidacji możesz przypisywać w modelu, a w kontrolerze jedynie sterować "przepływem" danych w sposób, jaki opisałem. Nie musisz dawać żadnych foreachów ani w widoku, ani w kontrolerze.

W dokumentacji CI nie znalazłem prawdziwego kodu pasującego do mojego pseudokodu, ale spójrz na na ten z Kohany:
http://kohanaframework.org/3.2/guide/kohan...rity/validation (sekcja "A Complete Example")

PS: To, że w dokumentacji większość rzeczy pokazana jest w kontrolerach nie znaczy, że tak musi być. Zapewne chodziło o prostotę przekazu i maksymalnie uproszczenie przedstawianego kodu.

[abort]

Dzięki za wytłumaczenie w prosty sposób.
Niestety, nie programuję zawodowo (i nie zamierzam - za stary jestem, by zmieniać zawód), więc chyba aktualnie nie poradzę sobie z przerobieniem mojego kodu na taki, który zakłada robienie wszystkiego w modelu. Pomimo tego, że czuję, że tak to powinno wyglądać.

Odnośnie Kohany - być może spróbuję poznać tego frameworka, jednak to też jest sprawa "na potem". Bo na razie, skoro już wybrałem CI, to chciałbym skończyć to, co zacząłem - potem w ramach nauki Kohany mogę spróbować zmigrować swój kod z CI na Kohana.

[szok]

Zrób tak:

Plik:
application/librares/MY_Form_validation.php

[PHP] pobierz, plaintext 
<?php  if ( ! defined('BASEPATH')) exit('No direct script access allowed');
/**
 * MY_Form_validation
 *
 * @package		CodeIgniter
 * @subpackage	Libraries
 * @category	form validation
 * @author		Jarolewski Piotr (aka szok) jarolewski.piotr@gmail.com
 * @link		http://www.webcoding.pl
 */
 
class MY_Form_validation extends CI_Form_validation {
 
  /**
 	 * Class Constructor
 	 */
 
   public function __construct()
   {
       parent::__construct();
       /*
        *  Podmiana domyslnego wysweitlania bledow 
        */        
       $this->set_error_delimiters('class="error"', '');
   }
 
   public function error($field = '', $prefix = '', $suffix = '')
	{
		if($prefix == TRUE)
		{
			if ( ! isset($this->_field_data[$field]['error']) OR $this->_field_data[$field]['error'] == '')
			{
				return '';
			}
			else
			{
				return 'error';
			}
		}
		else
		{
			if ( ! isset($this->_field_data[$field]['error']) OR $this->_field_data[$field]['error'] == '')
			{
				return '';
			}
 
			if ($prefix == '')
			{
				$prefix = $this->_error_prefix;
			}
 
			if ($suffix == '')
			{
				$suffix = $this->_error_suffix;
			}
 
			return $prefix.$this->_field_data[$field]['error'].$suffix;
		}
	}
 
}
/* End of file MY_Form_validation.php */
/* Location: ./application/librares/MY_Form_validation.php */
[PHP] pobierz, plaintext 

Następnie w controlerze wszystko tak jak miałeś do tej porty w form validatorze.
W widoku jak tworzysz formularz to:

[PHP] pobierz, plaintext 
<?php echo form_input('nazwa', set_value('nazwa') ,form_error('nazwa', TRUE))); ?>
 
[PHP] pobierz, plaintext 

Funkcja form_error('nazwa', TRUE) z drugim parametrem na TRUE zwraca nazwę klasy CSS zadeklarowaną w: $this->set_error_delimiters('class="error'', '');
Wtedy robisz sobie taki styl w CSS i masz podświetlone.
form_error('nazwapola'); bez drugiego parametru zwróci Ci opis błędu dla danego pola.

Ten post edytował szok 20.08.2012, 07:15:35

[abort]

Trochę nie to, czego potrzebowałem, ale dzięki za naprowadzenie na trop (tropem jest stworzenie klasy pochodnej do CI_Form_validation i wrzucenie mojej logiki własnie tam). Ale tak to jest, jak się programista strukturalny bierze za obiektówkę i zapomina o paradygmatach pisania kodu obiektowego... Myślę, że dalej poradzę sobie sam.