teratail
回答率
85.47%
質問するログイン新規登録
トップに関する質問【Swift】以下のイケていないコードをリファクタリングするならどうしますか？

編集履歴

質問編集履歴

5

補足を追加

2018/10/05 05:46

投稿

pamipani
pamipani

スコア11

test CHANGED
File without changes
test CHANGED
@@ -2,17 +2,25 @@
2
2
 
3
3
 
4
4
 
5
- 知りたい点
5
+ ## 知りたい点
6
6
 
7
7
  Swiftのswitch文でもうちょいよい書く方法を知りたい。
8
8
 
9
+ Swiftならではのswitchの使い方など。
10
+
11
+ 以下のコードだと `return 0`　や　`return 1` などDRYではないことが気にはなっています。
9
12
 
10
13
 
14
+
11
- 補足点
15
+ ## 補足点
12
16
 
13
17
  説明は便宜上わかりやすく書き直していて実際の実装はenumなどもう少しマシには書いてある状態。
14
18
 
15
19
  といった感じです。
20
+
21
+
22
+
23
+ 雑に質問投げて申し訳ないです。
16
24
 
17
25
 
18
26
 
@@ -69,3 +77,5 @@
69
77
 
70
78
 
71
79
  ```
80
+
81
+ ｓ

4

補足を追加

2018/10/05 05:46

投稿

pamipani
pamipani

スコア11

test CHANGED
File without changes
test CHANGED
@@ -1,4 +1,18 @@
1
1
  以下のコードをイケてるまで行かなくてもマシなレベルで書くとしたら良い書き方はありますか？
2
+
3
+
4
+
5
+ 知りたい点
6
+
7
+ Swiftのswitch文でもうちょいよい書く方法を知りたい。
8
+
9
+
10
+
11
+ 補足点
12
+
13
+ 説明は便宜上わかりやすく書き直していて実際の実装はenumなどもう少しマシには書いてある状態。
14
+
15
+ といった感じです。
2
16
 
3
17
 
4
18
 

3

コンパイルエラーを修正

2018/10/05 05:14

投稿

pamipani
pamipani

スコア11

test CHANGED
File without changes
test CHANGED
@@ -4,11 +4,11 @@
4
4
 
5
5
  ```Swift
6
6
 
7
- func testMethod(section: Int, tags: [String], products: [String]) -> Int {
7
+ func testMethod(number: Int, tags: [String], products: [String]) -> Int {
8
8
 
9
9
  if tags.isEmpty {
10
10
 
11
- switch section {
11
+ switch number {
12
12
 
13
13
  case 0:
14
14
 
@@ -26,7 +26,7 @@
26
26
 
27
27
  } else {
28
28
 
29
- switch section {
29
+ switch number {
30
30
 
31
31
  case 0:
32
32
 

2

productsタイポ

2018/10/05 05:01

投稿

pamipani
pamipani

スコア11

test CHANGED
File without changes
test CHANGED
@@ -34,7 +34,7 @@
34
34
 
35
35
  case 1...tags.count:
36
36
 
37
- if product.isEmpty { return 0 }
37
+ if products.isEmpty { return 0 }
38
38
 
39
39
  return 1
40
40
 
@@ -52,4 +52,6 @@
52
52
 
53
53
  }
54
54
 
55
+
56
+
55
57
  ```

1

productsを引数へ

2018/10/05 04:55

投稿

pamipani
pamipani

スコア11

test CHANGED
File without changes
test CHANGED
@@ -4,13 +4,11 @@
4
4
 
5
5
  ```Swift
6
6
 
7
- func testMethod(number: Int, tags: [String]) -> Int {
7
+ func testMethod(section: Int, tags: [String], products: [String]) -> Int {
8
-
9
- let product: [String] = []
10
8
 
11
9
  if tags.isEmpty {
12
10
 
13
- switch number {
11
+ switch section {
14
12
 
15
13
  case 0:
16
14
 
@@ -28,7 +26,7 @@
28
26
 
29
27
  } else {
30
28
 
31
- switch number {
29
+ switch section {
32
30
 
33
31
  case 0:
34
32