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

回答編集履歴

2

追記

2018/04/21 16:00

投稿

LouiS0616
LouiS0616

スコア35678

answer CHANGED
@@ -7,4 +7,6 @@
7
7
  ```Python
8
8
  def get_price(lastprice, curprice):
9
9
  return abs(lastprice - curprice) < 50, curprice
10
- ```
10
+ ```
11
+
12
+ これ以外にもいろいろ改善の余地がありますので、余力があったら考えてみてくださいな。

1

追記

2018/04/21 16:00

投稿

LouiS0616
LouiS0616

スコア35678

answer CHANGED
@@ -1,2 +1,10 @@
1
1
  curpriceを文字列として受け取って、そのまま使ってしまっています。
2
- 次のようにすれば良いでしょう。`curprice = int(input('Current Price : >'))`
2
+ 次のようにすれば良いでしょう。`curprice = int(input('Current Price : >'))`
3
+
4
+ 今更ですが
5
+ ---
6
+ 関数get_priceはかなり簡潔にリファクタできそうです。
7
+ ```Python
8
+ def get_price(lastprice, curprice):
9
+ return abs(lastprice - curprice) < 50, curprice
10
+ ```