[RUBY] Ich habe versucht, ① umzugestalten

Ausführung

・ Ruby 2.5.7 ・ Schienen 5.2.4.3

Überarbeiten Sie den Code für Ihr Portfolio

Zunächst ist die aktuelle Situation wie folgt. Furchtbar,,. Es gibt acht Aktionsnamen, die fast dasselbe schreiben. Lassen Sie uns zunächst die Aktionen von Datum1 bis Datum8 umgestalten. スクリーンショット 2020-08-09 14 27 08 スクリーンショット 2020-08-09 14 27 26 スクリーンショット 2020-08-09 14 27 40

Was ist der Prozess, der in jeden Prozess unterteilt werden soll? Was ist der gleiche Prozess? Denken Sie aus der Perspektive von

Wenn der Verarbeitungsinhalt der Aktionen von Datum1 bis Datum8 grob klassifiziert ist, Bei date1 wird ein neuer Datensatz erstellt. Datum2 bis Datum7 sind der Prozess des Erstellens eines neuen Datensatzes + der Prozess des Empfangens der von der vorherigen Seite übergebenen Spalte und des Speicherns des Datensatzes. Datum8 ist der Vorgang des Empfangens der von der vorherigen Seite übergebenen Spalte und des Speicherns des Datensatzes + der Vorgang des Speicherns eines anderen Datensatzes. Also dachte ich, dass Datum2 bis Datum7 zusammengestellt werden könnten.

Wie man den gleichen Prozess zusammenstellt

In diesem Fall war es die Existenz dieser Seitenspalte, die sie verhinderte, selbst wenn ich sie umgestalten wollte. An den Daten 2 bis 7 wurde nur der Wert der Seite geändert, und der Code war derselbe wie das Umleitungsziel, das bei der nachfolgenden Verarbeitung unterschiedlich war. スクリーンショット 2020-08-09 14 53 04

Erhöhen Sie den Wert jedes Mal, wenn sich die Seite ändert

Anstatt den Wert der Seite selbst zu schreiben, dachte ich, ich sollte ihn so codieren, dass der Wert jedes Mal, wenn sich die Seite ändert, um +1 erhöht wird. Erstellen Sie zu diesem Zweck zunächst eine Instanzvariable mit @conto_page = 1 am Datum1. スクリーンショット 2020-08-09 15 12 13 Als nächstes erhalten Sie den Wert auf der Ansichtsseite und übergeben @conto_page mit conto_page an date2: @conto_page mit form_with. スクリーンショット 2020-08-09 15 18 46 Wenn die empfangenen Parameter [: conto_page] .to_i == 8 sind, leiten Sie zum Datum 8 um, andernfalls erhöht sich conto_page um +1. スクリーンショット 2020-08-09 15 15 10

Refactoring abgeschlossen!

Ich konnte die Aktionen von Datum3 bis Datum7 sicher löschen. Wir konnten die Codemenge um ca. 40% reduzieren.