On May 21, 2007, at 15:29, Werner Laude wrote:
Bildergalerie zum durchklicken..
am Ende will ich wieder das erst Bild laden lassen.
def next_pic
unless @photo = Bilder.find(params[:id]).blank?
@photo = Bilder.find(params[:id])
render :action => 'show'
Bis hier hin scheint es zu funktionieren.
else
@photo = Bilder.find(:first, :conditions => "id = '#{12}'")
[EMAIL PROTECTED] = Bilder.find(:first, :conditions => {:id => "12"})
render :action => 'show'
end
end
Bilder.find(params[:id]).blank? ist in mehrfacher hinsicht nicht
sinnvoll..
1. nicht gefundene objekte werfen exceptions und geben keine leeren
objekte zurueck
2. Bilder ist ein unguenstiger Model-Name.. wenn ueberhaupt, dann
Bild, noch besser aber Picture
3. conditions unterstuetzen arrays, um sql injections vorzubeugen
4. api.rubyonrails.com bietet einige grossartige beispiele, wie man
es machen sollte
5. Das Kapitel 17 Active Records im Agile Web Development with Rails
[1] kann all diese Punkte beantworten.
Gruss,
Ben
[1] http://www.pragmaticprogrammer.com/titles/rails/ - liegt auch in
deutsch vor
_______________________________________________
rubyonrails-ug mailing list
[email protected]
http://mailman.headflash.com/mailman/listinfo/rubyonrails-ug
