回答編集履歴

3

追記

2018/08/15 10:12

投稿

asm
asm

スコア15147

test CHANGED
@@ -31,3 +31,31 @@
31
31
  end
32
32
 
33
33
  ```
34
+
35
+
36
+
37
+ -----
38
+
39
+
40
+
41
+ ヒアドキュメントなしでなるべく読みやすくしようとすると
42
+
43
+ 処理を分割する事になりますね
44
+
45
+ ```ruby
46
+
47
+ macaddr.each do |x|
48
+
49
+ personal_info = new_data[x]
50
+
51
+ personal_msg = personal_info.reduce(""){|str, kv| str + kv.join('=>') + "\n"}
52
+
53
+ message = CiscoSpark::Message.new(text:"#{x}さんが入室しました\n #{persoanl_msg}--------------------------------")
54
+
55
+ room = CiscoSpark::Room.new(id: room_id)
56
+
57
+ room.send_message(message)
58
+
59
+ end
60
+
61
+ ```

2

訂正

2018/08/15 10:11

投稿

asm
asm

スコア15147

test CHANGED
@@ -8,7 +8,7 @@
8
8
 
9
9
  personal_info = new_data[x]
10
10
 
11
- message = CiscoSpark::Message.new(text: <<JOIN_MSG
11
+ message = CiscoSpark::Message.new(text: <<JOIN_MSG.chomp
12
12
 
13
13
  #{x}さんが入室しました
14
14
 

1

修正

2018/08/15 09:04

投稿

asm
asm

スコア15147

test CHANGED
@@ -1,4 +1,6 @@
1
1
  eachではなくmapとjoinを用いて変換・合体を行う
2
+
3
+ ついでに、ヒアドキュメントを用いて読みやすくしてみる
2
4
 
3
5
  ```ruby
4
6