Discussion :

[titoumimi]

Bonjour à tous

Je suis en train de faire mumuse avec rails, et dans le cadre d'une relation 1-N, pour le moment, je récupére les infos des autres tables comme suit :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
  def new
    @projets_user = ProjetsUser.new
    @projets = Projet.find_all
    @users = User.find_all
  end
 
  def edit
    @projets_user = ProjetsUser.find(params[:id])
    @projets = Projet.find_all
    @users = User.find_all
  end
...

hors, ça respecte moyennement le principe "Ne pas se répéter", heureusement que je n'ai que deux tables...

Sauriez-vous comment faire en sorte que cette récupération s'applique en une seule déclaration à toutes mes méthodes SVP ?

d'avance

[titoumimi]

PS : j'ai bien essayé de passer ça en variable de classe, mais on dirait que ça le contrarie :p

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
class ProjetsUsersController < ApplicationController
 
  @@projets = Projet.find_all
  @@users = User.find_all

http://localhost:3000/projets_users/new

NameError in Projets_users#new

Showing app/views/projets_users/_form.rhtml where line #6 raised:

uninitialized class variable @@users in ActionView::Base::CompiledTemplates

[titoumimi]

allez hop, auto résolu

Utiliser une variable globale me permet de le faire sans plus de soucis

M'enfin, si vous avez d'autres idées plus propres

[Zfred]

Variables globales -> Bof
La convention Rails, si tu as 2 fois le meme code, c'est d'utiliser les helpers.
Tu mets ton code dans un helper et tu l'appelles quand t'en as besoin.

[titoumimi]

Je te remercie, je n'y avait pas pensé

[patthepat]

Variables globales -> Bof
La convention Rails, si tu as 2 fois le meme code, c'est d'utiliser les helpers.
Tu mets ton code dans un helper et tu l'appelles quand t'en as besoin.

Salut,

Tu peux illustrer ce que tu dis dans ce cas precis ? je suis aussi un nouveau on Rails

Pat

[bolo]

[QUOTE=titoumimi]Bonjour à tous

Je suis en train de faire mumuse avec rails, et dans le cadre d'une relation 1-N, pour le moment, je récupére les infos des autres tables comme suit :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
  def new
    @projets_user = ProjetsUser.new
    @projets = Projet.find_all
    @users = User.find_all
  end
 
  def edit
    @projets_user = ProjetsUser.find(params[:id])
    @projets = Projet.find_all
    @users = User.find_all
  end
...

pourquoi ne pas utiliser before_filter ?

Ex

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
before_filter : find_users_and_projets
 
def new
@projets_user = ProjetsUser.new
end
 
 def edit
    @projets_user = ProjetsUser.find(params[:id])
 end
 
protected
def find_users_and_projets
@projets = Projet.find_all
    @users = User.find_all
end

[SekYo]

Heu j'ai sans doutes mal compris la question, mais si le but c'est de récupérer les users associés à un projet par exemple, pourquoi ne pas faire

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
@projet = Projet.find(params[:id])
@users = @projet.users

en ayant défini un petit has_many dans le model Projet ? ( ou inversement si tu veux récupérer les projets d'un user... )

[bolo]

moi j'avais compris qu'il voulait réduire son code

[patthepat]

La question c'est comment appeler dans plusieurs actions des donnees de la base (pour generer un tag select dans un formulaire par exemple) mais en gardant son code DRY.

[bolo]

Ci c'est cela

il faut utiliser before_filter

[patthepat]

Ci c'est cela

il faut utiliser before_filter

ca me semble aussi une bonne solution.

Je me demandais juste ce que Zfred voulait dire en parlant de Helpers

[bolo]

même question surtt que les helpers c'est pour les vues

[SekYo]

Naon, tu peux utiliser les helpers dans un controller aussi

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
def MonController
  include MonControllerHelper
end

[bolo]

autant pour moi

[SekYo]

En fait y a même deux façons d'inclure le helper :
- soit comme plus haut avec "include"
- soit en utilisant la fonction"helper"

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
def MonController
  helper :mon_helper
end

Par contre je sais pas trop quelle est la différence, a priori la fonction "helper" semble moins brutale, vu qu'elle fait différents test ( selon que tu lui passe un module, un symbole, une string ), mais ensuite elle fait appel à la méthode "add_template_helper" que j'ai pas réussi à trouver sur api.rubyonrails.

J'vais passer à helper, ça me semble moins bourrin que le "include" quand même.