Refine constant variables.

app/controllers/admin/cancerpredicts_controller.rb

@@ -324,29 +324,22 @@ class Admin::CancerpredictsController < OrbitAdminController
         @form_to_show.auto_write_predict_js
       end
     end
-    @index = 0
+    @site_locales = Site.last.in_use_locales.each do |locale|
-    Dir.chdir("public") do
+    I18n.with_locale(locale) do
-      while File.exist?("cancerfield_back" + @index.to_s + ".txt")
+      File.open(Cancerpredictfields::ToolTableMap[I18n.locale], "w") do |f|
-        @index += 1
+        tmp_table_texts = create_table(locale)
+        f.write(tmp_table_texts)
       end
     end
-    Dir.chdir("public") do
+    idx = 0
-      @site_locales = Site.last.in_use_locales.each do |locale|
+    while File.exist?("public/cancerfield_back#{idx}.txt")
-        I18n.with_locale(locale) do
+      idx += 1
-          @file_tmp = File.new("#{Cancerpredictfields::ToolTablePrefix}#{I18n.locale}.txt", "w")
-          tmp_table_texts = create_table(locale)
-          @file_tmp.write(tmp_table_texts)
-          @file_tmp.close
-        end
-      end
     end
-    Dir.chdir("public") do
+    File.open("public/cancerfield_back#{idx}.txt", "w") do |f|
-      @file_back = File.open("cancerfield_back" + @index.to_s + ".txt", "w")
+    	f.write(@form_to_show.attributes)
-      @file_back.write(@form_to_show.attributes)
+    end
-      @file_back.close
+    File.open("public/cancerfield_org#{idx}.txt", "w") do |f|
-      @file_org = File.open("cancerfield_org" + @index.to_s + ".txt", "w")
+    	f.write(Cancerpredictfields.where("title" => @app_title + "_back").first.attributes)
-      @file_org.write(Cancerpredictfields.where("title" => @app_title + "_back").first.attributes)
-      @file_org.close
     end
     redirect_to admin_cancerpredicts_path
   end
@@ -637,12 +630,10 @@ class Admin::CancerpredictsController < OrbitAdminController
     #FileUtils.rm_r(dir_path, :force => true) if Dir.exist?(dir_path)
     FileUtils.mkdir dir_path if !Dir.exist?(dir_path) #create dir for storing tmp_file if dir doesn't exist
     @filename = ("#{Time.now.year}_%02s_%02s_export_cancer_tool_record.xlsx" % [Time.now.month, Time.now.day]).gsub(" ", "0")
-    Dir.chdir(dir_path) do
+    File.open("#{dir_path}#{@filename}", "w") do |f|
-      File.open(@filename, "w") do |f|
+      f.write render_to_string(:handlers => [:axlsx], :formats => [:xlsx], :partial => "export_cancer_tool_record.xlsx", :locals => { :results => cancer_records })
-        f.write render_to_string(:handlers => [:axlsx], :formats => [:xlsx], :partial => "export_cancer_tool_record.xlsx", :locals => { :results => cancer_records })
-      end
     end
-    tmp_filename_data = File.read(dir_path + @filename)
+    tmp_filename_data = File.read("#{dir_path}#{@filename}")
     send_data(tmp_filename_data, type: "application/xlsx", disposition: "attachment", filename: @filename)
   end
 
@@ -658,16 +649,12 @@ class Admin::CancerpredictsController < OrbitAdminController
       @form_to_show.title = @app_title
       @form_to_show.save
       @form_to_show = Cancerpredictfields.where("title" => @app_title).first
-      Dir.chdir("public") do
+      @site_locales = Site.last.in_use_locales.each do |locale|
-        @site_locales = Site.last.in_use_locales.each do |locale|
+	      File.open(Cancerpredictfields::ToolTableMap[I18n.locale], "w") do |f|
-          I18n.with_locale(locale) do
+	        tmp_table_texts = create_table(locale)
-            @file_tmp = File.new("#{Cancerpredictfields::ToolTablePrefix}#{I18n.locale}.txt", "w")
+	        f.write(tmp_table_texts)
-            tmp_table_texts = create_table(locale)
+	      end
-            @file_tmp.write(tmp_table_texts)
+	    end
-            @file_tmp.close
-          end
-        end
-      end
     else
       @form_to_show = Cancerpredictfields.where("title" => @app_title).first
     end

app/controllers/cancerpredicts_controller.rb

@@ -150,7 +150,7 @@ class CancerpredictsController < ApplicationController
     uid = OrbitHelper.params[:uid] rescue ""
     tags = OrbitHelper.widget_tags
     categories = OrbitHelper.widget_categories || []
-    @table_str = File.read("#{Cancerpredictfields::ToolTablePrefix}#{I18n.locale}.txt")
+    @table_str = File.read(Cancerpredictfields::ToolTableMap[I18n.locale])
     preidct_js_url = "/assets/cancer_predict.js"
     if File.exist?(Cancerpredictfields::JSFileName)
       js_filename = File.read(Cancerpredictfields::JSFileName)
@@ -171,7 +171,7 @@ class CancerpredictsController < ApplicationController
     uid = OrbitHelper.params[:uid] rescue ""
     tags = OrbitHelper.widget_tags
     categories = OrbitHelper.widget_categories || []
-    @table_str = File.read("#{Cancerpredictfields::ToolTablePrefix}#{I18n.locale}.txt")
+    @table_str = File.read(Cancerpredictfields::ToolTableMap[I18n.locale])
     preidct_js_url = "/assets/cancer_predict.js"
     if File.exist?(Cancerpredictfields::JSFileName)
       js_filename = File.read(Cancerpredictfields::JSFileName)

app/models/cancerpredictfields.rb

@@ -10,7 +10,9 @@ class Cancerpredictfields
   TherapyFields = ["variable","name","hint","comment_text","choice_fields","lpv_impact","active_choice","disable_condition"]
   TherapyOnly = ["lpv_impact","active_choice","disable_condition"]
   JSFileName = "public/cancer_tool_js_filename.txt".freeze
-  ToolTablePrefix = "public/cancer_tool_table_tmp_".freeze
+  ToolTableMap = I18n.available_locales.map do |locale|
+    [locale, "public/cancer_tool_table_tmp_#{locale}.txt".freeze]
+  end.to_h
   field :title ,type:String ,default:""
   field :advance_mode, type: Boolean, default: false
   field :form_show ,  :type=> Hash ,default:{