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

回答編集履歴

5

'' -> ""

2018/08/16 05:21

投稿

退会済みユーザー
answer CHANGED
@@ -1,7 +1,7 @@
1
1
  ```
2
2
  macaddr.each do |x|
3
3
  personal_info = new_data[x].map{|k, v| v.present? ? "#{k}=>#{v}" : nil}
4
- message = CiscoSpark::Message.new(text:"#{x}さんが入室しました\n #{persoanl_info.map(&:to_s).join('\n')} --------------------------------")
4
+ message = CiscoSpark::Message.new(text:"#{x}さんが入室しました\n #{persoanl_info.map(&:to_s).join("\n")} --------------------------------")
5
5
  room = CiscoSpark::Room.new(id: room_id)
6
6
  room.send_message(message)
7
7
  end

4

修正

2018/08/16 05:21

投稿

退会済みユーザー
answer CHANGED
@@ -1,7 +1,7 @@
1
1
  ```
2
2
  macaddr.each do |x|
3
- personal_info = new_data[x].map{|k, v| v.present? ? "#{k}=>#{v}\n" : nil}
3
+ personal_info = new_data[x].map{|k, v| v.present? ? "#{k}=>#{v}" : nil}
4
- message = CiscoSpark::Message.new(text:"#{x}さんが入室しました\n #{persoanl_info.map(&:to_s).join} --------------------------------")
4
+ message = CiscoSpark::Message.new(text:"#{x}さんが入室しました\n #{persoanl_info.map(&:to_s).join('\n')} --------------------------------")
5
5
  room = CiscoSpark::Room.new(id: room_id)
6
6
  room.send_message(message)
7
7
  end

3

脱字

2018/08/16 04:01

投稿

退会済みユーザー
answer CHANGED
@@ -1,6 +1,6 @@
1
1
  ```
2
2
  macaddr.each do |x|
3
- personal_info = new_data[x].map{|k, v| v.present? ? "#{k}=>#{v}" : nil}
3
+ personal_info = new_data[x].map{|k, v| v.present? ? "#{k}=>#{v}\n" : nil}
4
4
  message = CiscoSpark::Message.new(text:"#{x}さんが入室しました\n #{persoanl_info.map(&:to_s).join} --------------------------------")
5
5
  room = CiscoSpark::Room.new(id: room_id)
6
6
  room.send_message(message)

2

編集

2018/08/16 04:00

投稿

退会済みユーザー
answer CHANGED
@@ -1,9 +1,11 @@
1
1
  ```
2
2
  macaddr.each do |x|
3
- personal_info = new_data[x].map{|k, v| "#{k}=>#{v}"}
3
+ personal_info = new_data[x].map{|k, v| v.present? ? "#{k}=>#{v}" : nil}
4
- message = CiscoSpark::Message.new(text:"#{x}さんが入室しました\n #{persoanl_info[0]}\n #{persoanl_info[1]}\n #{persoanl_info[2]}\n --------------------------------")
4
+ message = CiscoSpark::Message.new(text:"#{x}さんが入室しました\n #{persoanl_info.map(&:to_s).join} --------------------------------")
5
5
  room = CiscoSpark::Room.new(id: room_id)
6
6
  room.send_message(message)
7
7
  end
8
8
  ```
9
- みたいに`personal_info = new_data[x].map{|k, v| "#{k}=>#{v}"}`の時点で分けておいて、出力の際に一つずつ取り出すこともできます。冗長にはなりますが。
9
+ みたいに`personal_info = new_data[x].map{|k, v| "#{k}=>#{v}"}`の時点で分けておいて、出力の際に一つずつ取り出すこともできます。冗長にはなりますが。
10
+ 編集:これで値が存在しない場合にその出力はスキップされます。
11
+ 文字列の長い場合の表示を云々はヒアドキュメント使ってください。

1

修正

2018/08/16 03:11

投稿

退会済みユーザー
answer CHANGED
@@ -1,5 +1,9 @@
1
1
  ```
2
- h["a"].each do |k, v|
2
+ macaddr.each do |x|
3
- puts "#{k}--->#{v}"
3
+ personal_info = new_data[x].map{|k, v| "#{k}=>#{v}"}
4
+ message = CiscoSpark::Message.new(text:"#{x}さんが入室しました\n #{persoanl_info[0]}\n #{persoanl_info[1]}\n #{persoanl_info[2]}\n --------------------------------")
5
+ room = CiscoSpark::Room.new(id: room_id)
6
+ room.send_message(message)
4
7
  end
5
- ```
8
+ ```
9
+ みたいに`personal_info = new_data[x].map{|k, v| "#{k}=>#{v}"}`の時点で分けておいて、出力の際に一つずつ取り出すこともできます。冗長にはなりますが。