From 20fce47f8f7788c02a5ffa2d28b09f4be7881a5e Mon Sep 17 00:00:00 2001 From: Alisson Oshiro Date: Thu, 12 Dec 2013 09:33:40 -0300 Subject: [PATCH 01/19] Adicionando dicas --- README.md | 59 ++++++++++++++++++++++++++++++++++++++++++++++++++++++- 1 file changed, 58 insertions(+), 1 deletion(-) diff --git a/README.md b/README.md index 2180d0e..be304bb 100644 --- a/README.md +++ b/README.md @@ -1,4 +1,61 @@ best-practices ============== -Compilation of best practices and standards for coding in using at Radig \ No newline at end of file +Compilation of best practices and standards for coding in using at Radig + +-> Geral + - Vai fazer um CRUD? cria tudo de uma vez.. não fica enrolando pra criar os testes.. + - Ta criando método novo? Já cria a referência nos testes.. pelo menos quando rodar o teste já vai mostrar a baixa procentagem.. + - Usa a porra do Localize, Unlocalize se tiver. já que tem usa, não fica fazendo strrepalce.. + - Cara.. use callBacks.. + http://book.cakephp.org/2.0/en/models/callback-methods.html + + - Comentário de método é grudado na margem, de classe tbem.. + - Se você achou que seu código ta uma merda de entender, coloca um comentário nele.. + - Ainda não sei se é uma boa fazer isso $this->__setFormData(); Se só o add e o edit vão usar os dados, acho que é uma boa usar ele. + - Para coisas bem especificas, algo que o beforeRender não precise fazer. + - No Form->input, se 'options' então 'empty'. + - Use else com cuidado, no geral, não precisa! + + +-> Sobre Controllers + - Controller não salva, nãp carrega muitos dados não faz muita coisa. + - Faça verificação de empty do request->data no controller.. você já faz isso antes de mandar pro modelo.. não precisa verificar de novo no modelo. + - Prefixo ajax em métodos ajax.. ok? + Ex: + public function ajaxLalala() + { + $this->request->onlyAllow(['ajax']); + + ... + + $this->set('data', $data); + $this->set('_serialize', 'data'); + } + - Sobre $this->request->onlyAllow(string|array $methods); + http://api.cakephp.org/2.4/class-CakeRequest.html#_onlyAllow + + +-> Sobre Models + - Modelo faz a merda toda, ele se fode pra resolver QUASE tudo. + - Disparar evento no modelo, não salva dados que deveriam ser de outro modelo. + Ex: Model A (só salva coisas do model A) -> evento -> Model B (só salva coisas do model B) + - Se você precisa salvar model, evento e depois o model de novo, você precisa rever a associação. + - Se seu campo data não precisa ser muito especifico, use type date, senão é datetime e cuide disso. + - Vou usar só type number pra float, azar.. + - Mantenha o fixture e o model sempre atualizados. + + +-> Sobre Views + - View, só mostra msm, no muito faz umas contas de somar.. + + +-> Sobre Formatação de códigos + - Nome de método protected começa com 2 __ + - Nome de método private começa com 1 _ + - Código de debug grudado na margem e com separação de duas linhas acima e abaixo, ajuda pra caralho no find all.. + - Usa espaço e não tab. + + + + From ed84302b421bc532f9952675c496d760a1a65420 Mon Sep 17 00:00:00 2001 From: Alisson Oshiro Date: Thu, 12 Dec 2013 09:35:34 -0300 Subject: [PATCH 02/19] =?UTF-8?q?Precisa=20de=20formata=C3=A7=C3=A3o?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- README.md | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/README.md b/README.md index be304bb..8f05cab 100644 --- a/README.md +++ b/README.md @@ -1,7 +1,6 @@ -best-practices +Boas Práticas ============== -Compilation of best practices and standards for coding in using at Radig -> Geral - Vai fazer um CRUD? cria tudo de uma vez.. não fica enrolando pra criar os testes.. From dcd9d6e2dd4728aa0ffaf1c702766c4a4a302489 Mon Sep 17 00:00:00 2001 From: Alisson Oshiro Date: Thu, 12 Dec 2013 10:41:10 -0300 Subject: [PATCH 03/19] =?UTF-8?q?Um=20pouco=20de=20formata=C3=A7=C3=A3o?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- README.md | 67 +++++++++++++++++++++++++++---------------------------- 1 file changed, 33 insertions(+), 34 deletions(-) diff --git a/README.md b/README.md index 8f05cab..fb27f8a 100644 --- a/README.md +++ b/README.md @@ -2,25 +2,25 @@ Boas Práticas ============== --> Geral - - Vai fazer um CRUD? cria tudo de uma vez.. não fica enrolando pra criar os testes.. - - Ta criando método novo? Já cria a referência nos testes.. pelo menos quando rodar o teste já vai mostrar a baixa procentagem.. - - Usa a porra do Localize, Unlocalize se tiver. já que tem usa, não fica fazendo strrepalce.. - - Cara.. use callBacks.. - http://book.cakephp.org/2.0/en/models/callback-methods.html - - - Comentário de método é grudado na margem, de classe tbem.. - - Se você achou que seu código ta uma merda de entender, coloca um comentário nele.. - - Ainda não sei se é uma boa fazer isso $this->__setFormData(); Se só o add e o edit vão usar os dados, acho que é uma boa usar ele. - - Para coisas bem especificas, algo que o beforeRender não precise fazer. - - No Form->input, se 'options' então 'empty'. - - Use else com cuidado, no geral, não precisa! - - --> Sobre Controllers - - Controller não salva, nãp carrega muitos dados não faz muita coisa. - - Faça verificação de empty do request->data no controller.. você já faz isso antes de mandar pro modelo.. não precisa verificar de novo no modelo. - - Prefixo ajax em métodos ajax.. ok? +## Geral + * Vai fazer um CRUD? cria tudo de uma vez.. não fica enrolando pra criar os testes.. + * Ta criando método novo? Já cria a referência nos testes.. pelo menos quando rodar o teste já vai mostrar a baixa procentagem.. + * Usa a porra do Localize, Unlocalize se tiver. já que tem usa, não fica fazendo strrepalce.. + * Cara.. use [callBacks](http://book.cakephp.org/2.0/en/models/callback-methods.html).. + + + * Comentário de método é grudado na margem, de classe tbem.. + * Se você achou que seu código ta uma merda de entender, coloca um comentário nele.. + * Ainda não sei se é uma boa fazer isso $this->__setFormData(); Se só o add e o edit vão usar os dados, acho que é uma boa usar ele. + + Para coisas bem especificas, algo que o beforeRender não precise fazer. + * No Form->input, se 'options' então 'empty'. + * Use else com cuidado, no geral, não precisa! + + +## Sobre Controllers + * Controller não salva, nãp carrega muitos dados não faz muita coisa. + * Faça verificação de empty do request->data no controller.. você já faz isso antes de mandar pro modelo.. não precisa verificar de novo no modelo. + * Prefixo ajax em métodos ajax.. ok? Ex: public function ajaxLalala() { @@ -31,29 +31,28 @@ Boas Práticas $this->set('data', $data); $this->set('_serialize', 'data'); } - - Sobre $this->request->onlyAllow(string|array $methods); - http://api.cakephp.org/2.4/class-CakeRequest.html#_onlyAllow + * Sobre [$this->request->onlyAllow(string|array $methods)](http://api.cakephp.org/2.4/class-CakeRequest.html#_onlyAllow). --> Sobre Models - - Modelo faz a merda toda, ele se fode pra resolver QUASE tudo. - - Disparar evento no modelo, não salva dados que deveriam ser de outro modelo. +## Sobre Models + * Modelo faz a merda toda, ele se fode pra resolver QUASE tudo. + * Disparar evento no modelo, não salva dados que deveriam ser de outro modelo. Ex: Model A (só salva coisas do model A) -> evento -> Model B (só salva coisas do model B) - - Se você precisa salvar model, evento e depois o model de novo, você precisa rever a associação. - - Se seu campo data não precisa ser muito especifico, use type date, senão é datetime e cuide disso. - - Vou usar só type number pra float, azar.. - - Mantenha o fixture e o model sempre atualizados. + * Se você precisa salvar model, evento e depois o model de novo, você precisa rever a associação. + * Se seu campo data não precisa ser muito especifico, use type date, senão é datetime e cuide disso. + * Vou usar só type number pra float, azar.. + * Mantenha o fixture e o model sempre atualizados. --> Sobre Views +## Sobre Views - View, só mostra msm, no muito faz umas contas de somar.. --> Sobre Formatação de códigos - - Nome de método protected começa com 2 __ - - Nome de método private começa com 1 _ - - Código de debug grudado na margem e com separação de duas linhas acima e abaixo, ajuda pra caralho no find all.. - - Usa espaço e não tab. +## Sobre Formatação de códigos + * Nome de método protected começa com 2 __ + * Nome de método private começa com 1 _ + * Código de debug grudado na margem e com separação de duas linhas acima e abaixo, ajuda pra caralho no find all.. + * Usa espaço e não tab. From 7d453cc519d451b5cff7dba1b1e171adc4b6aa2a Mon Sep 17 00:00:00 2001 From: Alisson Oshiro Date: Thu, 12 Dec 2013 13:28:21 -0300 Subject: [PATCH 04/19] Teste de estilo --- README.md | 29 +++++++++-------------------- 1 file changed, 9 insertions(+), 20 deletions(-) diff --git a/README.md b/README.md index fb27f8a..62a350f 100644 --- a/README.md +++ b/README.md @@ -1,21 +1,17 @@ Boas Práticas ============== - ## Geral - * Vai fazer um CRUD? cria tudo de uma vez.. não fica enrolando pra criar os testes.. - * Ta criando método novo? Já cria a referência nos testes.. pelo menos quando rodar o teste já vai mostrar a baixa procentagem.. - * Usa a porra do Localize, Unlocalize se tiver. já que tem usa, não fica fazendo strrepalce.. - * Cara.. use [callBacks](http://book.cakephp.org/2.0/en/models/callback-methods.html).. - - - * Comentário de método é grudado na margem, de classe tbem.. - * Se você achou que seu código ta uma merda de entender, coloca um comentário nele.. - * Ainda não sei se é uma boa fazer isso $this->__setFormData(); Se só o add e o edit vão usar os dados, acho que é uma boa usar ele. +* Vai fazer um CRUD? cria tudo de uma vez.. não fica enrolando pra criar os testes.. +* Ta criando método novo? Já cria a referência nos testes.. pelo menos quando rodar o teste já vai mostrar a baixa procentagem.. +* Usa a porra do Localize, Unlocalize se tiver. já que tem usa, não fica fazendo strrepalce.. +* Cara.. use [callBacks](http://book.cakephp.org/2.0/en/models/callback-methods.html).. +* Comentário de método é grudado na margem, de classe tbem.. +* Se você achou que seu código ta uma merda de entender, coloca um comentário nele.. +* Ainda não sei se é uma boa fazer isso $this->__setFormData(); Se só o add e o edit vão usar os dados, acho que é uma boa usar ele. + Para coisas bem especificas, algo que o beforeRender não precise fazer. - * No Form->input, se 'options' então 'empty'. - * Use else com cuidado, no geral, não precisa! - +* No Form->input, se 'options' então 'empty'. +* Use else com cuidado, no geral, não precisa! ## Sobre Controllers * Controller não salva, nãp carrega muitos dados não faz muita coisa. @@ -33,7 +29,6 @@ Boas Práticas } * Sobre [$this->request->onlyAllow(string|array $methods)](http://api.cakephp.org/2.4/class-CakeRequest.html#_onlyAllow). - ## Sobre Models * Modelo faz a merda toda, ele se fode pra resolver QUASE tudo. * Disparar evento no modelo, não salva dados que deveriam ser de outro modelo. @@ -43,17 +38,11 @@ Boas Práticas * Vou usar só type number pra float, azar.. * Mantenha o fixture e o model sempre atualizados. - ## Sobre Views - View, só mostra msm, no muito faz umas contas de somar.. - ## Sobre Formatação de códigos * Nome de método protected começa com 2 __ * Nome de método private começa com 1 _ * Código de debug grudado na margem e com separação de duas linhas acima e abaixo, ajuda pra caralho no find all.. * Usa espaço e não tab. - - - - From e63cfe60a6d2863cc728197c31ec8f7ae2f93344 Mon Sep 17 00:00:00 2001 From: Alisson Oshiro Date: Thu, 12 Dec 2013 13:32:48 -0300 Subject: [PATCH 05/19] =?UTF-8?q?Mais=20estiliza=C3=A7=C3=B5es?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- README.md | 46 +++++++++++++++++++++++----------------------- 1 file changed, 23 insertions(+), 23 deletions(-) diff --git a/README.md b/README.md index 62a350f..8540eae 100644 --- a/README.md +++ b/README.md @@ -14,35 +14,35 @@ Boas Práticas * Use else com cuidado, no geral, não precisa! ## Sobre Controllers - * Controller não salva, nãp carrega muitos dados não faz muita coisa. - * Faça verificação de empty do request->data no controller.. você já faz isso antes de mandar pro modelo.. não precisa verificar de novo no modelo. - * Prefixo ajax em métodos ajax.. ok? - Ex: - public function ajaxLalala() - { - $this->request->onlyAllow(['ajax']); +* Controller não salva, nãp carrega muitos dados não faz muita coisa. +* Faça verificação de empty do request->data no controller.. você já faz isso antes de mandar pro modelo.. não precisa verificar de novo no modelo. +* Prefixo ajax em métodos ajax.. ok? + Ex: + public function ajaxLalala() + { + $this->request->onlyAllow(['ajax']); - ... + ... - $this->set('data', $data); - $this->set('_serialize', 'data'); - } - * Sobre [$this->request->onlyAllow(string|array $methods)](http://api.cakephp.org/2.4/class-CakeRequest.html#_onlyAllow). + $this->set('data', $data); + $this->set('_serialize', 'data'); + } +* Sobre [$this->request->onlyAllow(string|array $methods)](http://api.cakephp.org/2.4/class-CakeRequest.html#_onlyAllow). ## Sobre Models - * Modelo faz a merda toda, ele se fode pra resolver QUASE tudo. - * Disparar evento no modelo, não salva dados que deveriam ser de outro modelo. +* Modelo faz a merda toda, ele se fode pra resolver QUASE tudo. +* Disparar evento no modelo, não salva dados que deveriam ser de outro modelo. Ex: Model A (só salva coisas do model A) -> evento -> Model B (só salva coisas do model B) - * Se você precisa salvar model, evento e depois o model de novo, você precisa rever a associação. - * Se seu campo data não precisa ser muito especifico, use type date, senão é datetime e cuide disso. - * Vou usar só type number pra float, azar.. - * Mantenha o fixture e o model sempre atualizados. +* Se você precisa salvar model, evento e depois o model de novo, você precisa rever a associação. +* Se seu campo data não precisa ser muito especifico, use type date, senão é datetime e cuide disso. +* Vou usar só type number pra float, azar.. +* Mantenha o fixture e o model sempre atualizados. ## Sobre Views - - View, só mostra msm, no muito faz umas contas de somar.. +* View, só mostra msm, no muito faz umas contas de somar.. ## Sobre Formatação de códigos - * Nome de método protected começa com 2 __ - * Nome de método private começa com 1 _ - * Código de debug grudado na margem e com separação de duas linhas acima e abaixo, ajuda pra caralho no find all.. - * Usa espaço e não tab. +* Nome de método protected começa com 2 __ +* Nome de método private começa com 1 _ +* Código de debug grudado na margem e com separação de duas linhas acima e abaixo, ajuda pra caralho no find all.. +* Usa espaço e não tab. From da19478cc7b3ef9589c553ddf3ac66a5c0a12055 Mon Sep 17 00:00:00 2001 From: Alisson Oshiro Date: Thu, 12 Dec 2013 14:00:31 -0300 Subject: [PATCH 06/19] =?UTF-8?q?teste=20de=20bloco=20de=20c=C3=B3digo?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- README.md | 16 +++++++++------- 1 file changed, 9 insertions(+), 7 deletions(-) diff --git a/README.md b/README.md index 8540eae..232f914 100644 --- a/README.md +++ b/README.md @@ -18,15 +18,17 @@ Boas Práticas * Faça verificação de empty do request->data no controller.. você já faz isso antes de mandar pro modelo.. não precisa verificar de novo no modelo. * Prefixo ajax em métodos ajax.. ok? Ex: - public function ajaxLalala() - { - $this->request->onlyAllow(['ajax']); +```php +public function ajaxLalala() +{ + $this->request->onlyAllow(['ajax']); - ... + ... - $this->set('data', $data); - $this->set('_serialize', 'data'); - } + $this->set('data', $data); + $this->set('_serialize', 'data'); +} +``` * Sobre [$this->request->onlyAllow(string|array $methods)](http://api.cakephp.org/2.4/class-CakeRequest.html#_onlyAllow). ## Sobre Models From cce6abf30f70072836e0118cc4b793d27cc4757a Mon Sep 17 00:00:00 2001 From: Alisson Oshiro Date: Thu, 12 Dec 2013 14:02:19 -0300 Subject: [PATCH 07/19] =?UTF-8?q?Teste=20de=20bloco=20de=20c=C3=B3digo?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- README.md | 16 +++++++--------- 1 file changed, 7 insertions(+), 9 deletions(-) diff --git a/README.md b/README.md index 232f914..ee01a6c 100644 --- a/README.md +++ b/README.md @@ -18,17 +18,15 @@ Boas Práticas * Faça verificação de empty do request->data no controller.. você já faz isso antes de mandar pro modelo.. não precisa verificar de novo no modelo. * Prefixo ajax em métodos ajax.. ok? Ex: -```php -public function ajaxLalala() -{ - $this->request->onlyAllow(['ajax']); + public function ajaxLalala() + { + $this->request->onlyAllow(['ajax']); - ... + ... - $this->set('data', $data); - $this->set('_serialize', 'data'); -} -``` + $this->set('data', $data); + $this->set('_serialize', 'data'); + } * Sobre [$this->request->onlyAllow(string|array $methods)](http://api.cakephp.org/2.4/class-CakeRequest.html#_onlyAllow). ## Sobre Models From dc9df0f59a65460e797ed503b78746c491d8144e Mon Sep 17 00:00:00 2001 From: Alisson Oshiro Date: Thu, 12 Dec 2013 14:03:46 -0300 Subject: [PATCH 08/19] blocos --- README.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/README.md b/README.md index ee01a6c..958b01b 100644 --- a/README.md +++ b/README.md @@ -18,15 +18,15 @@ Boas Práticas * Faça verificação de empty do request->data no controller.. você já faz isso antes de mandar pro modelo.. não precisa verificar de novo no modelo. * Prefixo ajax em métodos ajax.. ok? Ex: +''' public function ajaxLalala() { $this->request->onlyAllow(['ajax']); - ... - $this->set('data', $data); $this->set('_serialize', 'data'); } +''' * Sobre [$this->request->onlyAllow(string|array $methods)](http://api.cakephp.org/2.4/class-CakeRequest.html#_onlyAllow). ## Sobre Models From 298ea7f903f4b9b7e24b966fa3e5e0282985cd84 Mon Sep 17 00:00:00 2001 From: Alisson Oshiro Date: Thu, 12 Dec 2013 14:04:26 -0300 Subject: [PATCH 09/19] blocos --- README.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/README.md b/README.md index 958b01b..3866303 100644 --- a/README.md +++ b/README.md @@ -18,7 +18,7 @@ Boas Práticas * Faça verificação de empty do request->data no controller.. você já faz isso antes de mandar pro modelo.. não precisa verificar de novo no modelo. * Prefixo ajax em métodos ajax.. ok? Ex: -''' + public function ajaxLalala() { $this->request->onlyAllow(['ajax']); @@ -26,7 +26,7 @@ Boas Práticas $this->set('data', $data); $this->set('_serialize', 'data'); } -''' + * Sobre [$this->request->onlyAllow(string|array $methods)](http://api.cakephp.org/2.4/class-CakeRequest.html#_onlyAllow). ## Sobre Models From 527d8b2d6b9f3aa547a65492b5aa3580742349f0 Mon Sep 17 00:00:00 2001 From: Alisson Oshiro Date: Thu, 12 Dec 2013 14:08:08 -0300 Subject: [PATCH 10/19] lala --- README.md | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/README.md b/README.md index 3866303..17b204b 100644 --- a/README.md +++ b/README.md @@ -18,15 +18,15 @@ Boas Práticas * Faça verificação de empty do request->data no controller.. você já faz isso antes de mandar pro modelo.. não precisa verificar de novo no modelo. * Prefixo ajax em métodos ajax.. ok? Ex: +``` +public function ajaxLalala() +{ + $this->request->onlyAllow(['ajax']); - public function ajaxLalala() - { - $this->request->onlyAllow(['ajax']); - ... - $this->set('data', $data); - $this->set('_serialize', 'data'); - } - + $this->set('data', $data); + $this->set('_serialize', 'data'); +} +``` * Sobre [$this->request->onlyAllow(string|array $methods)](http://api.cakephp.org/2.4/class-CakeRequest.html#_onlyAllow). ## Sobre Models From b9401162ea276737badee9d644e2a96399716435 Mon Sep 17 00:00:00 2001 From: Alisson Oshiro Date: Thu, 12 Dec 2013 14:09:44 -0300 Subject: [PATCH 11/19] test tabs --- README.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/README.md b/README.md index 17b204b..9ccc51e 100644 --- a/README.md +++ b/README.md @@ -21,7 +21,7 @@ Boas Práticas ``` public function ajaxLalala() { - $this->request->onlyAllow(['ajax']); +⋅⋅⋅$this->request->onlyAllow(['ajax']); $this->set('data', $data); $this->set('_serialize', 'data'); From 25f09659fb0514b8c83f15707811ca8e2361722f Mon Sep 17 00:00:00 2001 From: Alisson Oshiro Date: Thu, 12 Dec 2013 09:16:39 -0800 Subject: [PATCH 12/19] =?UTF-8?q?Tab=20=C3=A9=20com=20tab=20mesmo.?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- README.md | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/README.md b/README.md index 9ccc51e..794bbb3 100644 --- a/README.md +++ b/README.md @@ -21,10 +21,10 @@ Boas Práticas ``` public function ajaxLalala() { -⋅⋅⋅$this->request->onlyAllow(['ajax']); + $this->request->onlyAllow(['ajax']); - $this->set('data', $data); - $this->set('_serialize', 'data'); + $this->set('data', $data); + $this->set('_serialize', 'data'); } ``` * Sobre [$this->request->onlyAllow(string|array $methods)](http://api.cakephp.org/2.4/class-CakeRequest.html#_onlyAllow). From 7327c42938a675ec08b0b4ea3ed462c507285f26 Mon Sep 17 00:00:00 2001 From: Alisson Oshiro Date: Thu, 12 Dec 2013 14:17:38 -0300 Subject: [PATCH 13/19] =?UTF-8?q?teste=20de=20tab=208=20espa=C3=A7os?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- README.md | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/README.md b/README.md index 794bbb3..e7b6418 100644 --- a/README.md +++ b/README.md @@ -21,10 +21,11 @@ Boas Práticas ``` public function ajaxLalala() { - $this->request->onlyAllow(['ajax']); + $this->request->onlyAllow(['ajax']); $this->set('data', $data); $this->set('_serialize', 'data'); + $this->set('_serialize', 'data'); } ``` * Sobre [$this->request->onlyAllow(string|array $methods)](http://api.cakephp.org/2.4/class-CakeRequest.html#_onlyAllow). From b71bf3844dd036d77b8c11386809a70844b0abba Mon Sep 17 00:00:00 2001 From: Alisson Oshiro Date: Thu, 12 Dec 2013 14:21:21 -0300 Subject: [PATCH 14/19] Removendo linha de teste de tab --- README.md | 1 - 1 file changed, 1 deletion(-) diff --git a/README.md b/README.md index e7b6418..20f5771 100644 --- a/README.md +++ b/README.md @@ -25,7 +25,6 @@ public function ajaxLalala() $this->set('data', $data); $this->set('_serialize', 'data'); - $this->set('_serialize', 'data'); } ``` * Sobre [$this->request->onlyAllow(string|array $methods)](http://api.cakephp.org/2.4/class-CakeRequest.html#_onlyAllow). From b3815934b4558b794ed6552ea3e0f162c524bc26 Mon Sep 17 00:00:00 2001 From: Alisson Oshiro Date: Thu, 12 Dec 2013 18:32:12 -0300 Subject: [PATCH 15/19] corrigindo cosas --- README.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/README.md b/README.md index 20f5771..05fc379 100644 --- a/README.md +++ b/README.md @@ -28,6 +28,7 @@ public function ajaxLalala() } ``` * Sobre [$this->request->onlyAllow(string|array $methods)](http://api.cakephp.org/2.4/class-CakeRequest.html#_onlyAllow). +* Usar $this->request->onlyAllow() somente para métodos que não tem view. ## Sobre Models * Modelo faz a merda toda, ele se fode pra resolver QUASE tudo. @@ -42,7 +43,6 @@ public function ajaxLalala() * View, só mostra msm, no muito faz umas contas de somar.. ## Sobre Formatação de códigos -* Nome de método protected começa com 2 __ -* Nome de método private começa com 1 _ +* Nome de método protected começa com 1 _ * Código de debug grudado na margem e com separação de duas linhas acima e abaixo, ajuda pra caralho no find all.. * Usa espaço e não tab. From 1d2b82f6141c8ae19e2fb5c0ef966749348e8154 Mon Sep 17 00:00:00 2001 From: Alisson Oshiro Date: Wed, 25 Dec 2013 22:35:43 -0300 Subject: [PATCH 16/19] Melhorando texto. --- README.md | 31 +++++++++++++++++-------------- 1 file changed, 17 insertions(+), 14 deletions(-) diff --git a/README.md b/README.md index 05fc379..591cf4b 100644 --- a/README.md +++ b/README.md @@ -4,25 +4,26 @@ Boas Práticas ## Geral * Vai fazer um CRUD? cria tudo de uma vez.. não fica enrolando pra criar os testes.. * Ta criando método novo? Já cria a referência nos testes.. pelo menos quando rodar o teste já vai mostrar a baixa procentagem.. -* Usa a porra do Localize, Unlocalize se tiver. já que tem usa, não fica fazendo strrepalce.. +* Usa a porra do Localize, Unlocalize se tiver. Já que tem usa, não fica fazendo strrepalce.. * Cara.. use [callBacks](http://book.cakephp.org/2.0/en/models/callback-methods.html).. -* Comentário de método é grudado na margem, de classe tbem.. -* Se você achou que seu código ta uma merda de entender, coloca um comentário nele.. -* Ainda não sei se é uma boa fazer isso $this->__setFormData(); Se só o add e o edit vão usar os dados, acho que é uma boa usar ele. - + Para coisas bem especificas, algo que o beforeRender não precise fazer. -* No Form->input, se 'options' então 'empty'. +* Comentário de método é grudado na margem, de classe tbem.. Dá uma olhada no padrão do cakphp. +* Se você achou que seu código ta uma ruim de entender, coloca um comentário nele.. +* Ainda não sei se é uma boa fazer isso $this->__setFormData(); Se só o *add()* e o *edit()* vão usar os dados, acho que é uma boa usar ele. + + Para coisas bem especificas, algo que o *beforeRender()* não precise fazer. +* No Form->input, se *options* então *empty*. * Use else com cuidado, no geral, não precisa! +* Padronize o nome/tipo dos campos por TODO o projeto.. Quando vpcê ler uma variável vai saber o que é não importa onde esteja. ## Sobre Controllers * Controller não salva, nãp carrega muitos dados não faz muita coisa. * Faça verificação de empty do request->data no controller.. você já faz isso antes de mandar pro modelo.. não precisa verificar de novo no modelo. -* Prefixo ajax em métodos ajax.. ok? +* Prefixo *ajax* em métodos ajax.. ok? Ex: ``` public function ajaxLalala() { $this->request->onlyAllow(['ajax']); - + ... $this->set('data', $data); $this->set('_serialize', 'data'); } @@ -31,18 +32,20 @@ public function ajaxLalala() * Usar $this->request->onlyAllow() somente para métodos que não tem view. ## Sobre Models -* Modelo faz a merda toda, ele se fode pra resolver QUASE tudo. +* Modelo faz a mágica toda, ele resolver QUASE tudo pra você. * Disparar evento no modelo, não salva dados que deveriam ser de outro modelo. Ex: Model A (só salva coisas do model A) -> evento -> Model B (só salva coisas do model B) * Se você precisa salvar model, evento e depois o model de novo, você precisa rever a associação. -* Se seu campo data não precisa ser muito especifico, use type date, senão é datetime e cuide disso. -* Vou usar só type number pra float, azar.. -* Mantenha o fixture e o model sempre atualizados. +* Se seu campo data não precisa ser muito especifico, use *type date*, senão é *datetime* e cuide disso. +* Vou usar só *type number* pra *float*, azar.. +* Mantenha o schema do fixture e do model sempre atualizados. ## Sobre Views * View, só mostra msm, no muito faz umas contas de somar.. ## Sobre Formatação de códigos * Nome de método protected começa com 1 _ -* Código de debug grudado na margem e com separação de duas linhas acima e abaixo, ajuda pra caralho no find all.. -* Usa espaço e não tab. + Ex: potected function _methodProtected(){} +* Código de debug grudado na margem e com separação de pelomenos uma linha acima e abaixo, ajuda pra muito no find all.. + + Serve também pra pr(), console.log(), die(), die; e outros. +* Usa espaço e não tab, consifgure isso no seu editor. From 0a28c77594412faad8dd4dd4504133c673582641 Mon Sep 17 00:00:00 2001 From: Alisson Oshiro Date: Wed, 25 Dec 2013 22:39:37 -0300 Subject: [PATCH 17/19] =?UTF-8?q?Corrigindo=20formata=C3=A7a=C3=B5?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- README.md | 15 ++++++++------- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git a/README.md b/README.md index 591cf4b..5ec4676 100644 --- a/README.md +++ b/README.md @@ -4,15 +4,16 @@ Boas Práticas ## Geral * Vai fazer um CRUD? cria tudo de uma vez.. não fica enrolando pra criar os testes.. * Ta criando método novo? Já cria a referência nos testes.. pelo menos quando rodar o teste já vai mostrar a baixa procentagem.. -* Usa a porra do Localize, Unlocalize se tiver. Já que tem usa, não fica fazendo strrepalce.. +* Use o Localize, Unlocalize se tiver. Já que tem usa, não fica fazendo strrepalce.. * Cara.. use [callBacks](http://book.cakephp.org/2.0/en/models/callback-methods.html).. * Comentário de método é grudado na margem, de classe tbem.. Dá uma olhada no padrão do cakphp. * Se você achou que seu código ta uma ruim de entender, coloca um comentário nele.. * Ainda não sei se é uma boa fazer isso $this->__setFormData(); Se só o *add()* e o *edit()* vão usar os dados, acho que é uma boa usar ele. - + Para coisas bem especificas, algo que o *beforeRender()* não precise fazer. + + Para coisas bem especificas, algo que o *beforeRender()* não precise fazer. * No Form->input, se *options* então *empty*. * Use else com cuidado, no geral, não precisa! * Padronize o nome/tipo dos campos por TODO o projeto.. Quando vpcê ler uma variável vai saber o que é não importa onde esteja. +* Use *// TODO descrição do problema* Use uma descrição rápida e direta na primeira linha. No findal all ja é possçĩvel saber do que se trata o *TODO* ## Sobre Controllers * Controller não salva, nãp carrega muitos dados não faz muita coisa. @@ -22,10 +23,10 @@ Boas Práticas ``` public function ajaxLalala() { - $this->request->onlyAllow(['ajax']); - ... - $this->set('data', $data); - $this->set('_serialize', 'data'); + $this->request->onlyAllow(['ajax']); + ... + $this->set('data', $data); + $this->set('_serialize', 'data'); } ``` * Sobre [$this->request->onlyAllow(string|array $methods)](http://api.cakephp.org/2.4/class-CakeRequest.html#_onlyAllow). @@ -47,5 +48,5 @@ public function ajaxLalala() * Nome de método protected começa com 1 _ Ex: potected function _methodProtected(){} * Código de debug grudado na margem e com separação de pelomenos uma linha acima e abaixo, ajuda pra muito no find all.. - + Serve também pra pr(), console.log(), die(), die; e outros. + + Serve também pra pr(), console.log(), die(), die; e outros. * Usa espaço e não tab, consifgure isso no seu editor. From 1a293651eb5a58ee8aaab7c789dce9450f918247 Mon Sep 17 00:00:00 2001 From: Alisson Oshiro Date: Wed, 25 Dec 2013 17:47:23 -0800 Subject: [PATCH 18/19] =?UTF-8?q?teste=20de=20formata=C3=A7=C3=A3o?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- README.md | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/README.md b/README.md index 5ec4676..0b3db0d 100644 --- a/README.md +++ b/README.md @@ -9,7 +9,7 @@ Boas Práticas * Comentário de método é grudado na margem, de classe tbem.. Dá uma olhada no padrão do cakphp. * Se você achou que seu código ta uma ruim de entender, coloca um comentário nele.. * Ainda não sei se é uma boa fazer isso $this->__setFormData(); Se só o *add()* e o *edit()* vão usar os dados, acho que é uma boa usar ele. - + Para coisas bem especificas, algo que o *beforeRender()* não precise fazer. + * Para coisas bem especificas, algo que o *beforeRender()* não precise fazer. * No Form->input, se *options* então *empty*. * Use else com cuidado, no geral, não precisa! * Padronize o nome/tipo dos campos por TODO o projeto.. Quando vpcê ler uma variável vai saber o que é não importa onde esteja. @@ -24,7 +24,7 @@ Boas Práticas public function ajaxLalala() { $this->request->onlyAllow(['ajax']); - ... + $this->set('data', $data); $this->set('_serialize', 'data'); } @@ -35,7 +35,7 @@ public function ajaxLalala() ## Sobre Models * Modelo faz a mágica toda, ele resolver QUASE tudo pra você. * Disparar evento no modelo, não salva dados que deveriam ser de outro modelo. - Ex: Model A (só salva coisas do model A) -> evento -> Model B (só salva coisas do model B) + * Ex: Model A (só salva coisas do model A) -> evento -> Model B (só salva coisas do model B) * Se você precisa salvar model, evento e depois o model de novo, você precisa rever a associação. * Se seu campo data não precisa ser muito especifico, use *type date*, senão é *datetime* e cuide disso. * Vou usar só *type number* pra *float*, azar.. @@ -46,7 +46,7 @@ public function ajaxLalala() ## Sobre Formatação de códigos * Nome de método protected começa com 1 _ - Ex: potected function _methodProtected(){} + * Ex: potected function _methodProtected(){} * Código de debug grudado na margem e com separação de pelomenos uma linha acima e abaixo, ajuda pra muito no find all.. - + Serve também pra pr(), console.log(), die(), die; e outros. + * Serve também pra pr(), console.log(), die(), die; e outros. * Usa espaço e não tab, consifgure isso no seu editor. From 928b7eb5c85134f6312667c9bc86f19c438a7a0c Mon Sep 17 00:00:00 2001 From: Alisson Oshiro Date: Wed, 25 Dec 2013 22:47:38 -0300 Subject: [PATCH 19/19] =?UTF-8?q?corre=C3=A7=C3=A3o=20de=20palavra?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- README.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/README.md b/README.md index 0b3db0d..da0be16 100644 --- a/README.md +++ b/README.md @@ -24,7 +24,7 @@ Boas Práticas public function ajaxLalala() { $this->request->onlyAllow(['ajax']); - + $this->set('data', $data); $this->set('_serialize', 'data'); } @@ -49,4 +49,4 @@ public function ajaxLalala() * Ex: potected function _methodProtected(){} * Código de debug grudado na margem e com separação de pelomenos uma linha acima e abaixo, ajuda pra muito no find all.. * Serve também pra pr(), console.log(), die(), die; e outros. -* Usa espaço e não tab, consifgure isso no seu editor. +* Usa espaço e não tab, configure isso no seu editor.