Skip to content

Commit fd0a4d5

Browse files
committed
refactor: ニュース表示ロジックを改善
- 絵文字判定を全文字チェックから先頭文字のみに変更 - if文を三項演算子でシンプル化 - ロジックをヘルパーメソッド format_news_title に分離
1 parent 8ad0b80 commit fd0a4d5

File tree

2 files changed

+6
-8
lines changed

2 files changed

+6
-8
lines changed

app/helpers/application_helper.rb

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -203,4 +203,9 @@ def translate_dojo_tag(tag_name)
203203
tag_translations[tag_name] || tag_name
204204
end
205205

206+
def format_news_title(news)
207+
has_emoji = news.title[0]&.match?(/[\p{Emoji}&&[^0-9#*]]/)
208+
has_emoji ? news.title : "📰 #{news.title}"
209+
end
210+
206211
end

app/views/home/show.html.erb

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -179,14 +179,7 @@
179179
<ul class="list" style="list-style: none;">
180180
<% @news_items.each do |news| %>
181181
<li>
182-
<% has_emoji = news.title.chars.any? { |char| char.match?(/[\p{Emoji}&&[^0-9#*]]/) } %>
183-
<% if has_emoji %>
184-
<% # 絵文字が含まれていれば、タイトルをそのまま表示 %>
185-
<%= link_to news.title, news.url, target: '_blank', rel: 'noopener noreferrer' %>
186-
<% else %>
187-
<% # 絵文字が含まれていなければ、「📰」をつけて表示 %>
188-
📰 <%= link_to news.title, news.url, target: '_blank', rel: 'noopener noreferrer' %>
189-
<% end %>
182+
<%= link_to format_news_title(news), news.url, target: '_blank', rel: 'noopener noreferrer' %>
190183
</li>
191184
<% end %>
192185
</ul>

0 commit comments

Comments
 (0)