teratail header banner
teratail header banner
質問するログイン新規登録

回答編集履歴

3

追記

2018/08/15 10:12

投稿

asm
asm

スコア15149

answer CHANGED
@@ -14,4 +14,18 @@
14
14
  room = CiscoSpark::Room.new(id: room_id)
15
15
  room.send_message(message)
16
16
  end
17
+ ```
18
+
19
+ -----
20
+
21
+ ヒアドキュメントなしでなるべく読みやすくしようとすると
22
+ 処理を分割する事になりますね
23
+ ```ruby
24
+ macaddr.each do |x|
25
+ personal_info = new_data[x]
26
+ personal_msg = personal_info.reduce(""){|str, kv| str + kv.join('=>') + "\n"}
27
+ message = CiscoSpark::Message.new(text:"#{x}さんが入室しました\n #{persoanl_msg}--------------------------------")
28
+ room = CiscoSpark::Room.new(id: room_id)
29
+ room.send_message(message)
30
+ end
17
31
  ```

2

訂正

2018/08/15 10:11

投稿

asm
asm

スコア15149

answer CHANGED
@@ -3,7 +3,7 @@
3
3
  ```ruby
4
4
  macaddr.each do |x|
5
5
  personal_info = new_data[x]
6
- message = CiscoSpark::Message.new(text: <<JOIN_MSG
6
+ message = CiscoSpark::Message.new(text: <<JOIN_MSG.chomp
7
7
  #{x}さんが入室しました
8
8
  #{persoanl_info.map do |k, v|
9
9
  "#{k}=>#{v}"

1

修正

2018/08/15 09:04

投稿

asm
asm

スコア15149

answer CHANGED
@@ -1,4 +1,5 @@
1
1
  eachではなくmapとjoinを用いて変換・合体を行う
2
+ ついでに、ヒアドキュメントを用いて読みやすくしてみる
2
3
  ```ruby
3
4
  macaddr.each do |x|
4
5
  personal_info = new_data[x]