Forum PHP.pl

Mam formularz który dodaje rekordy do bazy. W nim prawie 80 pól z których każde jest inne więc trzeba je ponazywać, dopisać do widoku i umieścić jeszcze w widoku edycji. Sam plik ma ponad 1 000 linijek i gdy próbuję coś zmienić muszę edytować na raz 2 pliki (dodawanie i edycja). Myślę więc jak to powinno się najlepiej rozwiązać. Zrobić jeden widok dla dodawania i edycji i w nim warunki czy zostawić to tak jak jest ?
Gdybym miał drugi widok musiałbym
* zrobić tablice z pustymi wartościami z wszystkich pól które będą zapisane (żeby nie wywaliło w widoku że zmienna nie istnieje)
* zrobić warunki na akcje formularza (add/edit) oraz na submit formularza (dodaj/zapisz zmiany)
tylko czy to jest dobry pomysł takie faszerowanie widoku pustymi zmiennymi oraz warunkami ? Niby bym miał łatwiej ale plik będzie bardziej zamotany chyba. Jak wy to robicie dla takich dużych formularzy ?

Dokładnie tak

Ten post edytował Turson 26.02.2014, 11:37:43

a co robicie z pustymi zmiennymi ? isset czy generowanie ich masowo w controler'ach. Probuje znalezc najlepszy sposob a jak sie czegos naucze pozniej nawykow trudno jest sie pozbyc wiec najlepiej nabyc ich jak najlepszych (IMG:style_emoticons/default/wink.gif)

Te 80 pól różni się od siebie bardzo? Mógłbyś podać jakieś przykłady?

W samym frameworku powinien istnieć sposób na to, że albo wartość jest pusta (dodawanie), albo pobiera ją z bazy jeżeli istnieje (istnieje). Wydaje mi się, że CI powinno mieć coś takiego.

@markuz baza wyglada mniej wiecej tak (sory ze w screenie):
(IMG:http://my.jetscreenshot.com/demo/20140226-zr74-62kb.jpg)

@Turson o to chodzi ze ucze się Code Igniter i właśnie szukam na to rozwiązania bo nie wiem jak to zrobić :/ nie natknąłem się nigdzie na takie rozwiązanie, ale jak dam zmienną która nie istnieje to oczywiście jest błąd.

[PHP] pobierz, plaintext 
# Przykładowy kontroler
 
class TestController extends CI_Controller {
 
    private $fields = array(
        array(
            'label' => 'Nazwa',
            'field' => 'name',
            'rules' => 'required|trim|xss_clean',
            'value' => NULL,
            'type'  => 'input'
        ),
        array(
            'label' => 'Lokalizacja tekstu?',
            'field' => 'location_text',
            'rules' => 'required|trim|xss_clean',
            'value' => NULL,
            'type'  => 'textarea'
        ),
        array(
            'label' => 'Is comm',
            'field' => 'is_comm',
            'rules' => 'required|trim|xss_clean',
            'value' => NULL,
            'type'  => 'checkbox'
        ),
    );
 
    public function add()
    {
        foreach($this->fields as $field)
        {
            $this->form_validation->set_rules($field['label'], $field['field'], $field['rules']);
        }
 
        if($this->form_validation->run() == TRUE)
        {
            foreach($this->fields as $key => $field)
            {
                $this->fields[$key]['value'] = $this->input->post($field['field']);
            }
 
            $this->ExampleModel->insert($this->fields);
        }
 
        $this->load->view('add', array(
            'fields' => $this->fields,
        ));
    }
 
    public function edit($id)
    {
        foreach($this->fields as $field)
        {
            $this->form_validation->set_rules($field['label'], $field['field'], $field['rules']);
        }
 
        if($this->form_validation->run() == TRUE)
        {
            foreach($this->fields as $key => $field)
            {
                $this->fields[$key]['value'] = $this->input->post($field['field']);
            }
 
            $this->ExampleModel->update($this->fields, $id);
        }
 
        $this->load->view('update', array(
            'fields' => $this->fields,
        ));
    }
 
}
 
# Widok add.php
 
foreach($fields as $field)
{
    switch($field['type'])
    {
        case 'textarea':
            echo form_textarea($field['field'], set_value($field['field']));
        break;
 
        case 'checkbox':
            echo form_checkbox($field['field'], set_value($field['field']));
        break;
 
        default:
            echo form_input($field['field'], set_value($field['field']));
        break;
    }
 
}
echo form_submit('submit');
 
# Widok edit.php
 
foreach($fields as $field)
{
    switch($field['type'])
    {
        case 'textarea':
            echo form_textarea($field['field'], set_value($field['field'], $field['value']));
        break;
 
        case 'checkbox':
            echo form_checkbox($field['field'], set_value($field['field'], $field['value']));
        break;
 
        default:
            echo form_input($field['field'], set_value($field['field'], $field['value']));
        break;
    }
}
echo form_submit('submit');
[PHP] pobierz, plaintext 

CI to bardzo oporny framework, pracuję na nim już od 2 lat i czasami trzeba wymyśleć coś po swojemu żeby sobie zaoszczędzić pracy. Nie wiem czy kod jest prawidłowy bo pisałem z palca i nie sprawdzałem aczkolwiek myślę, że ogólny sens zrozumiałeś (IMG:style_emoticons/default/wink.gif)

Dziękuję za kod, później go jeszcze dokładniej przeanalizuję z verification, sporo się z niego nauczę. Nie rozumiem jednak jednego. Po co to set_value skoro 2 parametr dla np. input i tak przyjmuje wartosc "value" wiec nwaet jak sie da bez set_value to bedzie to samo ...

Nie jestem pewien czy dobrze zrozumiałem ale w Twoim kodzie chyba i tak jest podzielone na 2 pliki widok add/edit a mam tam już ponad 1 000 linijek razem z foundation posegregowanych w tabulatory i zmienianie tego na tym etapie to prawie jak pisanie tego od nowa (IMG:style_emoticons/default/sad.gif)

Zazwyczaj używam trochę inaczej form_helper'a bo muszę dać więcej opcji więc przykładowo wygląda u mnie to tak:

[PHP] pobierz, plaintext 
<?=form_label('Name', 'name')?>
<?=form_input(array('name'=>'name', 'id'=>'name', 'placeholder'=>'Obiect name', 'value'=> $object['name'] ))?>
[PHP] pobierz, plaintext 

i myślę czy nie szybciej w tym momencie byłoby w widoku dać jakiś isset albo przynajmniej cos w stylu:

[PHP] pobierz, plaintext 
($object['name'] != '' ? $object['name'] : '') 
[PHP] pobierz, plaintext 

ale jak do tablicy dodać takie cos ?
Mogę dla każdego z 80 elementów dać dodatkowa zmienna np.:

[PHP] pobierz, plaintext 
($object['name'] != '' ? $element_name = $object['name'] : $element_name = '') 
[PHP] pobierz, plaintext 

ale pisać to znów 80 razy ?

Drugi parametr dla input to jego value. Jak go nie podasz to value = "". Jak dodasz do tego set_value('pole') to wtedy value = $_POST['pole']. Jak zrobisz set_value('pole', 'pole2') wtedy jeśli $_POST['pole'] jest puste to pobiera pole2 (w moim przypadku pobierze go z modelu).

Równie dobrze możesz w tablicy fields dodać elementy takie jak placeholder czy id i wyświetlać je jeśli element != NULL

Dodałem widoki edit i add bo takie rozwiązanie jest według mnie lepsze. Ogólnie chciałem Ci przedstawić koncepcję a jeśli chodzi o wykonanie to już twoja sprawa (IMG:style_emoticons/default/wink.gif) Nie chcesz poprawiać - nie musisz. To zależy jak dużo zmian będziesz musiał potem wprowadzać i co zajmie Ci więcej czasu.