echo $html->tableHeaders(
array(
'Product ID',
'Product',
'Description',
'Price'
)
);
foreach($data as $product)
{
echo $html->tableCells(
array(
array(
$product['Product']['product_id'],
$product['Product']['product_name'],
$product['Product']['product_desc'],
$product['Product']['product_price']
)
)
);
}
Zacznę lajtowo bo to nie jest błąd ale moim zdaniem zupełnie zbyteczna rzecz. Tworzenie table za pomocą helperów. Czy naprawdę jest to tak kosmicznie potrzebne? Super, że CakePHP ma taki helper ( ma go pewnie dlatego, że inne frameworki też taki mają ). Ale po kiego grzyba go używać ? Podobnie jest z helperami do tworzenia linków:
echo $htm->link('gdzieś tam','/cos/tam'); - super ale gdzieś tam // echo Router::url('/cos/tam');ani nie jest dłuższe ani bardziej skomplikowane - jest za to szybsze! No i jeszcze można wymienić tworzenie obrazków, list, divów i całej reszty. Nie musisz, nie używaj!
echo $form->input('product_name');Zawsze podawaj nazwę pola wraz z modelem, czyli
Product.product_name ( pomijam fakt bezsensownej nazwy pola ). Przy formularzach dotyczących pojedyńczego modelu może wydawać się to bezsensowne ale jak pojawi się pole id i potem w zapytaniu wyciągane będzie pole id innego modelu to będzie płacz.function add()
{
if(isset($this->data))
{
$this->Product->set($this->data);
if ($this->Product->validates())
{
if($this->Product->save())
$this->Session->setFlash('Data Saved Successfully!');
$this->redirect(array(
'controller' => 'products',
'action' => 'index'
)
);
}
}
}
Zacznijmy od
if(isset($this->data)). Może się nie znam ale jak w klasie zdefiniowane jest
var $data = array();to imho
isset($this->data);wywali TRUE. Dalej przejdźmy do
$this->Product->set($this->data);osobiście użyłbym metody
create, no ale tak też działa. Dalej mamy totalną bzdurę if ($this->Product->validates()). Jeżeli autor tutoriala raczyłby zajrzeć do API to pewnie nie sprawiłoby mu problemu przeczytanie dwu zdaniowego opisu metody save gdzie jest wyraźnie napisane
By default, validation occurs before save.
Nie ma więc najmniejszego sensu dodawanie tej linijki.
Naturalnie nie są to gigantyczne błędy ale uczą one nowych użytkowników złych zasad pisania kodu. Nie da się zrobić tak, żeby ktoś sam nie zagląda do API i book.cakephp.org napisał coś z czego można się czegoś nauczyć.
Dlatego jeszcze raz:
- Czytaj API
- Zaglądaj do book.cakephp.org
- Zaglądaj do tworzonych testów
