Übergeben von Daten zwischen den Controllern mit der Sitzung RESTvoll

Question

Ich verwende eine hässliche Muster in meinem Code und ich weiß, es muss eine bessere Möglichkeit, dies zu tun. Hilf mir zu überdenken, was ich tue.

Meine Website ist eine Art Diskussionsforum. Alle Antworten auf Diskussionen werden auf der DiscussionsController # show-Seite inline ausgeführt.

Einige Antworten sind jedoch ungültig. Wenn Sie beispielsweise versuchen, eine Antwort ohne Text einzugeben, werden Sie mit einer Fehlermeldung an DisconsualsController # show zurückgegeben.

Hier ist ein kurzer Überblick, wie ich diesen Workflow implementiert haben:

1. Benutzer geht zu DiscussionsController # show. Diese Vorlage hat ein Antwortformular darauf. Es gibt keine explizite RepliesController # neue Aktion.
2. Der Benutzer sendet ein Antwortformular, das an replys_path gesendet und in RepliesController # create behandelt wird.
3. AntwortenController # create kann die Antwort nicht speichern, da sie ungültig ist (validates_length_of in Reply deaktiviert das Objekt).
4. AntwortenController # create stellt das Antwortobjekt in die Sitzung [: new_reply] und leitet den Diskussionspfad weiter, von dem der Benutzer kam.
5. DiscussionsController # show behandelt das Sitzungsobjekt ...

Wie so:

if session[:new_reply] 
    @new_reply = session[:new_reply] 
    session.delete(:new_reply) 
end 

Und jetzt hat show.html.erb ein neu regenerierten @new_reply Objekt auf Fehler zu überprüfen.

Da ist etwas offensichtlich falsch - Sie sollten nicht ganze Objekte innerhalb der Sitzung speichern. Aber da das Reply-Objekt, das wir in RepliesController # create speichern wollten, nie gespeichert wurde, wie behalte ich es zwischen Controller-Aktionsaufrufen?

Oder wenn es eine größere Design-Lösung gibt, zögern Sie nicht, es zu teilen. Das ist so hässlich, dass es mir weh tut. Vielen Dank.

Answer 1

Wie wäre es statt Umleiten, statt render :action => "discussions/show" statt. Ich würde sagen, Sie müssten die @discussion Variable auch einrichten, aber dies sollte getan werden, da die Antworten verschachtelte Ressourcen innerhalb der Diskussionen sind, richtig?

Answer 2

Statt der Sitzung würde ich die Antwort in der flash speichern, die für das Übergeben von Nachrichten/Objekten über eine Aktion gut ist.

Der Controller würde die Antwort im Flash einfach speichern, wenn es nicht gültig und Umleitung ist:

ein bisschen klüger und sein müßte

if @reply.save 
    ... 
else 
    flash[:reply] = @reply 
end 

Und Ihre Form Logik auf der Diskussionen/show Seite greifen Antwort im Flash oder die neue (die ich nehme an, Sie sind in der Diskussion/show-Aktion erstellen):

form_for(flash[:reply] || @reply) do 
    ... 
end 

Answer 3

ich würde sagen, dass die Art und Weise Sie es tun sinnvoll ist. Nicht das sauberste, aber es ist RESTful.

Der Vorschlag von Radar ist groß, aber in der aktuellen Version von Rails, render :action => "discussions/show" erstellt nicht das richtige Verhalten, zumindest in meinen Versuchen, die ohne verschachtelte Ressourcen Routing waren. Die Verwendung von render :controller => "discussions", :action => "show" ergibt ein anderes Verhalten, das nicht das ist, wonach das Poster sucht.