質問編集履歴
1
質問内容に関する情報を追記しました。
title
CHANGED
|
File without changes
|
body
CHANGED
|
@@ -2,7 +2,38 @@
|
|
|
2
2
|
このコードの汚い箇所・構造、等アドバイスお願いします。
|
|
3
3
|
Goで書いたコード管理ツールです。
|
|
4
4
|
|
|
5
|
+
※追記しました。
|
|
6
|
+
# 追記
|
|
7
|
+
抽象的な質問失礼しました。
|
|
8
|
+
今までチーム開発の経験が無く、私は汚いと感じないのですが、他人から見た見やすいコードとは何だろうと思い質問しました。
|
|
9
|
+
## 気になった事
|
|
10
|
+
#### 構造体
|
|
11
|
+
構造体を使った方が分かりやすいでしょうか？
|
|
12
|
+
**例**
|
|
13
|
+
```go
|
|
14
|
+
type cmd_line struct {
|
|
15
|
+
Cmd string
|
|
16
|
+
Opition string
|
|
17
|
+
Name string
|
|
18
|
+
}
|
|
19
|
+
```
|
|
20
|
+
今は入力を全て配列に突っ込んでしまっています。
|
|
21
|
+
私は、分散すると行ったり来たりで分かりにくいと感じます。しかし、他人のコードを見ると構造体がよく使われています。
|
|
22
|
+
また、他に構造体にした方が良い箇所はありますか？
|
|
23
|
+
|
|
24
|
+
#### 同じような動作をどちらかに統一すべきか
|
|
25
|
+
`exec.Command("cat"~)`と`os.Open()`を、`os.Open()`にまとめるべきでしょうか
|
|
26
|
+
|
|
27
|
+
#### 関数の分け方
|
|
28
|
+
コマンドごとでしか関数を分けていませんが、`cat`,`ls`のコマンド呼び出し部分を一つにまとめた方が良いでしょうか？
|
|
29
|
+
2行程度なのでまとめ無くて良いかな〜と思っちゃいます。
|
|
30
|
+
|
|
31
|
+
質問として不適切かもしれませんが、Goをやっている、かつ、不特定多数に聞ける。のはここしか無いと思い質問させて頂きました。
|
|
32
|
+
Goをやっている人が周りに居ないもので...
|
|
33
|
+
|
|
34
|
+
## コード
|
|
5
35
|
**main.go**
|
|
36
|
+
|
|
6
37
|
```go
|
|
7
38
|
package main
|
|
8
39
|
|
|
@@ -160,7 +191,7 @@
|
|
|
160
191
|
}
|
|
161
192
|
```
|
|
162
193
|
**ディレクトリ構造**
|
|
163
|
-
```
|
|
194
|
+
```diff
|
|
164
195
|
atCoder/
|
|
165
196
|
├── BeginnersSelection
|
|
166
197
|
│ ├── abc081a.go
|