Que os parece mi primer plugin, dont_repeat_controllerself

Community ES
#1

Buenas lista, os pongo la dirección de un primer plugin que he
realizado para que si no es mucha molestia me deis vuestra opinión y
me comentéis que os parece.

Aunque el ingles del readme no es muy bueno espero que se entienda, y
estando el plugin en su versión 0.1 no seáis muy crueles que estoy
aprendiendo.

La dirección http://github.com/alarkspur/dont_repeat_controllerself/tree

Muchas gracias
Saludos

#2

Podria explicar para que sirve…

El día 2 de agosto de 2008 1:12, alarkspur [email protected]
escribió:> Buenas lista, os pongo la dirección de un primer plugin que he

#3

2008/8/2 alarkspur [email protected]:

Buenas lista, os pongo la dirección de un primer plugin que he
realizado para que si no es mucha molestia me deis vuestra opinión y
me comentéis que os parece.

%w(index show new edit create update destroy).each do |m|
define_method :"#{m}" do
case m

      # GET /records
      # GET /records.xml
      when 'index'

Esto… ¿por que lo haces así?, salvando el define_method m.to_sym que
quedaría más limpio… ¿Por que no lo haces con simples

def index
def show
def new… si al fin y al cavo haces un case para cada elemento del
array. No le veo mucho sentido

Por otro lado, el uso del método kclass… en tu caso… ¿En que se
diferencia de usar esa función a poner simplemente:
object.class.find(params[:id])

Y así por encima, otro comentario. ¿Es dry hacer funciones que solo se
llaman una vez? me refiero a respond_destroy

Un Saludo.

#4

Esto… ¿por que lo haces así?, salvando el define_method m.to_sym que
quedaría más limpio… ¿Por que no lo haces con simples
def index
def show

Tienes razón, si en esto si había caído en ello, pero como nunca había
usado define_method por eso lo use…

Por otro lado, el uso del método kclass… en tu caso… ¿En que se
diferencia de usar esa función a poner simplemente:
object.class.find(params[:id])

Pues también tienes
razón.

Y así por encima, otro comentario. ¿Es dry hacer funciones que solo se
llaman una vez? me refiero a respond_destroy

Buen razonamiento, últimamente es que a mi me da por hacer funciones
para todo :stuck_out_tongue:

De verdad, muchas gracias Guillermo por el tiempo que has invertido en
mirar mi “horrible” código, voy a seguir tus consejos y a tenerlos en
cuenta también en el futuro.

Gracias de nuevo.
Un saludo.

#5

On Sat, Aug 2, 2008 at 13:33, alarkspur [email protected] wrote:

dont_repeat_controller_for :users

en el controlador para que generase las acciones index, show, edit,
update, new, create y destroy para en este caso usersController

Esa era la idea pero aun esta verde, aunque funcionar funciona :slight_smile:

Gracias por el interés.
Un saludo.

No te desanímo a crear tu plugin y desarrollarlo más pero si necesitas
(o cualquier otro, claro) esa funcionalidad ahora mismo te
recomendaría mirar make_resourceful http://mr.hamptoncatlin.com/ que
ofrece algo similar a lo que propones y algunas funcionalidades más.

Suerte.

#6

Muchas gracias Daniel, desconocía la existencia de make_resourceful
ademas se le ve muy completo.
Le echare un vistazo (mejor muchos vistazos que seguro algo
aprendo :slight_smile: aunque seguiré “cacharreando” con el mío.

Gracias por la
dirección.Un saludo.

#7

El 02/08/2008, a las 11:14, trancos asd
escribió:

Podria explicar para que sirve…

Ya sabia yo que el readme no se entendería :frowning:

Lo que trataba de hacer con el plugin era el tener que evitar escribir
los métodos básicos para CRUD en cada controlador, por eso con este
plugin bastaria con escribir

dont_repeat_controller_for :users

en el controlador para que generase las acciones index, show, edit,
update, new, create y destroy para en este caso usersController

Esa era la idea pero aun esta verde, aunque funcionar funciona :slight_smile:

Gracias por el interés.
Un saludo.