Discussion :

[gvdmoort]

Bonjour à tous,

Lorsque j'édite le code d'un fichier app/controllers/xxx_controller.rb, je fais appel à des variables ou des portions de code qui sont communes à plusieurs vues.

Par exemple des variables récupérant les valeurs de paramêtres. Et surtout des transformations effectuées sur ces variables avant de les transmettre à des requêtes SQL.

Je pensais pouvoir les placer en début de fichier, avant les blocs 'def index .. end', etc. mais ça ne marche pas.

Pour ne pas ré-écrire dans chaque vue ce code, comment procéder pour qu'il soit partagé par toutes ?

Merci d'avance,

g.

[Taum]

Salut,

Tu peux factoriser du code dans ton contrôleur en créant des méthodes privées, par exemple

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
class SearchController < ApplicationController
  def users
    @list = User.find :all, :conditions => conditions_from_params
  end
  def posts
    @list = Post.find :all, :conditions => conditions_from_params
  end
 
 private
  def conditions_from_params
    # transformation des params en SQL ...
  end 
end

ou même avec before_filter :

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
16
17
18
class SearchController < ApplicationController
 
  before_filter :create_sql
 private
  def create_sql
    @sql = ... # creation d'une requete SQL ...
  end 
 
 public
  # la variable @sql est disponible pour toutes les actions
  def users
    ...
  end
  def posts
    ...
  end
 
end

Attention a ne pas oublier de mettre les méthodes qui ne sont pas des actions (pas accessible par l'url en gros) en private

Je sais pas si mes exemples sont très parlants, sinon montre nous ton code ça sera le plus simple je crois

[gvdmoort]

Salut,

Je sais pas si mes exemples sont très parlants, sinon montre nous ton code ça sera le plus simple je crois

Je crois aussi, je débute, là...

Bon, par exemple j'ai deux actions index et recherche qui débutent exactement de la même manière:

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
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
class MembresController < ApplicationController
 
  def index
 
    @exercices  = params[:exercices]
 
    @conditions=["membre=1"]
 
    @tous_les_exercices = []
    for exercice in Role.tous_les_exercices
      @tous_les_exercices.push(exercice.annee)
    end  
 
    # (du code...)
 
  end  
 
  def recherche
 
    @exercices  = params[:exercices]
 
    @conditions=["membre=1"]
 
    @tous_les_exercices = []
    for exercice in Role.tous_les_exercices
      @tous_les_exercices.push(exercice.annee)
    end  
 
    # (encore du code...)
 
  end
end

Est-ce qu'on ne peut pas reporter ceci:

Code :

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

1
2
3
4
5
6
7
8
9
 
@exercices  = params[:exercices]
 
@conditions=["membre=1"]
 
@tous_les_exercices = []
for exercice in Role.tous_les_exercices
    @tous_les_exercices.push(exercice.annee)
end

en dehors de ces 2 blocs, de manière à ne l'écrire qu'une fois, sachant que j'utiliserai dans ces blocs les variables @exercices, @conditions et @tous_les_exercices

[issueTracker]

Comme l a dis Taum utilise le callback before_filter pour la methode index et recherche

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
before_filter :machin, :only=>[:index,:recherche]
 
def machin
 
@exercices  = params[:exercices]
 
@conditions=["membre=1"]
 
@tous_les_exercices = []
for exercice in Role.tous_les_exercices
    @tous_les_exercices.push(exercice.annee)
end
end

[Taum]

Notes aussi que :

Code :

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

1
2
3
4
@tous_les_exercices = []
for exercice in Role.tous_les_exercices
    @tous_les_exercices.push(exercice.annee)
end

S'écrit en Ruby :

Code :

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

@tous_les_exercices = Role.tous_les_exercices.collect { |e| e.annee }

Voire même grâce aux ajouts apportés par Rails :

Code :

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

@tous_les_exercices = Role.tous_les_exercices.collect(&:annee)

[gvdmoort]

Notes aussi que :

Code :

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

1
2
3
4
5
 
@tous_les_exercices = []
for exercice in Role.tous_les_exercices
    @tous_les_exercices.push(exercice.annee)
end

S'écrit en Ruby :

Code :

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

@tous_les_exercices = Role.tous_les_exercices.collect { |e| e.annee }

De l'inconvénient d'être passé par Perl Merci pour ce genre de conseil qui corrige les mauvaises pratiques.

Il n'est donc pas nécessaire de déclarer au préalable l'objet @tous_les_exercices comme une Array vide ?

[titoumimi]

Il n'est donc pas nécessaire de déclarer au préalable l'objet @tous_les_exercices comme une Array vide ?

non, ce n'est pas la peine, il s'initialisera au passage (le collect ou le map renvoient des tableaux, et vive le duck-typing )