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

回答編集履歴

4

追記

2018/09/23 09:10

投稿

LouiS0616
LouiS0616

スコア35678

answer CHANGED
@@ -8,6 +8,16 @@
8
8
  他の方法もあるかもしれませんが、数値に直していくのが一番自然で書きやすいかと。
9
9
  もちろん実装はいろいろ考えられますから、int[]型の変数が必須というわけではないです。
10
10
 
11
+ こんなふうにも書けますし。
12
+ ```Java
13
+ static boolean isValid(String src) {
14
+ return Arrays.stream(src.split("s"))
15
+ .mapToInt(Integer::parseInt)
16
+ .allMatch(n -> 0 <= n && n <= 1999)
17
+ ;
18
+ }
19
+ ```
20
+
11
21
  ---
12
22
  あと、変数名はもうちょっと考えて付けましょう。
13
23
  > ```Java

3

修正

2018/09/23 09:10

投稿

LouiS0616
LouiS0616

スコア35678

answer CHANGED
@@ -5,7 +5,7 @@
5
5
 
6
6
  > やはり、s_2の中の数値を表す文字をあらたなint型の配列に変換しておさめていき、その数字が0から1999の範囲内である、という調べ方しかないのでしょうか?
7
7
 
8
- 他の方法もあるかもしれませんが、方法が一番自然で書きやすいかと。
8
+ 他の方法もあるかもしれませんが、数値に直していくのが一番自然で書きやすいかと。
9
9
  もちろん実装はいろいろ考えられますから、int[]型の変数が必須というわけではないです。
10
10
 
11
11
  ---

2

追記

2018/09/23 09:00

投稿

LouiS0616
LouiS0616

スコア35678

answer CHANGED
@@ -6,6 +6,7 @@
6
6
  > やはり、s_2の中の数値を表す文字をあらたなint型の配列に変換しておさめていき、その数字が0から1999の範囲内である、という調べ方しかないのでしょうか?
7
7
 
8
8
  他の方法もあるかもしれませんが、その方法が一番自然で書きやすいかと。
9
+ もちろん実装はいろいろ考えられますから、int[]型の変数が必須というわけではないです。
9
10
 
10
11
  ---
11
12
  あと、変数名はもうちょっと考えて付けましょう。

1

追記

2018/09/23 08:58

投稿

LouiS0616
LouiS0616

スコア35678

answer CHANGED
@@ -5,4 +5,13 @@
5
5
 
6
6
  > やはり、s_2の中の数値を表す文字をあらたなint型の配列に変換しておさめていき、その数字が0から1999の範囲内である、という調べ方しかないのでしょうか?
7
7
 
8
- 他の方法もあるかもしれませんが、その方法が一番自然で書きやすいかと。
8
+ 他の方法もあるかもしれませんが、その方法が一番自然で書きやすいかと。
9
+
10
+ ---
11
+ あと、変数名はもうちょっと考えて付けましょう。
12
+ > ```Java
13
+ boolean TRUE = false;
14
+ > ```
15
+
16
+ 『TRUEの値がfalse』って、禅問答のような状況になっています。
17
+ 真偽値であることは見れば誰でも分かるんですから、『何のための』真偽値か分かるようにしましょう。