Merge remote-tracking branch 'origin/main' into develop_mh

# Conflicts:
#	app/views/modbus/index.html.erb
#	app/views/modbus/schedule_edit.html.erb

app/controllers/modbus_controller.rb

@@ -1,6 +1,17 @@
 class ModbusController < ApplicationController
   def index
     @schedule = Schedule.all
+    @modbus_running = Modbus::PollingService.running?
+  end
+
+  def start
+    Modbus::PollingService.start
+    redirect_to modbus_index_path
+  end
+
+  def stop
+    Modbus::PollingService.stop
+    redirect_to modbus_index_path
   end
 
   def schedule_edit

app/services/modbus/polling_service.rb

@@ -2,9 +2,9 @@ module Modbus
   class PollingService
     class << self
       def start
-        return if @thread&.alive?
+        return if $modbus_polling_threads.any?(&:alive?)
 
-        @thread = Thread.new do
+        thread = Thread.new do
           puts "[#{Time.current}] Modbus polling service 시작됨"
           last_logged_hour = nil
           loop do
@@ -32,14 +32,17 @@ module Modbus
             end
           end
         end
+        $modbus_polling_threads << thread
       end
 
       def stop
-        if @thread
-          @thread.exit
-          @thread = nil
+        $modbus_polling_threads.each { |t| t.kill if t.alive? }
+        $modbus_polling_threads.clear
         puts "[#{Time.now}] Modbus polling service 중지됨"
       end
+
+      def running?
+        $modbus_polling_threads.any?(&:alive?)
       end
     end
   end

app/views/modbus/index.html.erb

@@ -8,26 +8,11 @@
   </div>
 <% end %>
 
-<div class="flex flex-col h-full divide-y divide-border-table-border">
-  <div class="flex-1 overflow-y-auto p-4">
-    <table class="base-table">
-      <thead>
-      <tr>
-        <th>시간</th>
-        <th>온도</th>
-      </tr>
-      </thead>
-      <tbody>
-      <% @schedule.each do |s| %>
-        <tr>
-          <td><%= s.hour %>시</td>
-          <td><%= s.temperature %> °C</td>
-        </tr>
-      <% end %>
-      </tbody>
-    </table>
+<% @schedule.each do |s| %>
+  <div class="flex justify-items-center gap-x-2">
+    <div><%= s.hour %>시</div>
+    <div><%= s.temperature %> °C</div>
   </div>
-  <div class="flex p-4">
-    <%= link_to "수정", "/modbus/schedule_edit", class: "btn bg-default-slate" %>
-  </div>
-</div>
+<% end %>
+
+<%= link_to "수정", "/modbus/schedule_edit", class: "btn btn-primary" %>

app/views/modbus/schedule_edit.html.erb

@@ -1,24 +1,14 @@
-<%= form_with url: "/modbus/schedule_edit_update", method: :post, class: 'flex flex-col h-full divide-y divide-border-table-border' do %>
-  <div class="flex-1 overflow-y-auto p-4">
-    <table class="base-table">
-      <thead>
-      <tr>
-        <th>시간</th>
-        <th>온도</th>
-      </tr>
-      </thead>
-      <tbody>
+<%= form_with url: "/modbus/schedule_edit_update", method: :post do %>
   <% @schedule.each do |s| %>
-        <tr>
-          <td><%= s.hour %>시</td>
-          <td><%= number_field_tag "schedule[#{s.id}][temperature]", s.temperature, step: "0.1", inputmode: "decimal", class: "border px-2 py-1 rounded" %> °C</td>
-        </tr>
-      <% end %>
-      </tbody>
-    </table>
+    <div class="flex justify-items-center gap-x-2 pb-2">
+      <div><%= s.hour %>시</div>
+      <div>
+        <%= number_field_tag "schedule[#{s.id}][temperature]", s.temperature, step: "0.1", inputmode: "decimal", class: "border px-2 py-1 rounded" %> °C
       </div>
+    </div>
+  <% end %>
 
-  <div class="flex p-4">
-    <%= submit_tag "업데이트", class: "btn bg-primary" %>
+  <div class="pt-4">
+    <%= submit_tag "업데이트", class: "bg-blue-500 text-white px-4 py-2 rounded" %>
   </div>
 <% end %>

config/initializers/modbus_service.rb

@@ -1,3 +1,5 @@
+$modbus_polling_threads ||= []
+
 Rails.application.config.after_initialize do
   Modbus::PollingService.start
 end

config/routes.rb

@@ -14,9 +14,8 @@ Rails.application.routes.draw do
 
   resources :modbus do
     collection do
-      get "start"
-      get "stop"
-      get "status"
+      post "start"
+      post "stop"
       get "schedule_edit"
       post "schedule_edit_update"
     end

db/seeds.rb

@@ -8,6 +8,6 @@
 #     MovieGenre.find_or_create_by!(name: genre_name)
 #   end
 
-(0..23).each_with_index do |h, index|
-  Schedule.create!(hour: h, temperature: index)
+(0..23).each do | h |
+  Schedule.create!(hour: h, temperature: 15.0)
 end