teratail
回答率
85.30%
質問するログイン新規登録
トップに関する質問クラスの名前の付け方について（xxxmanagerクラスはアンチパターン？)

編集履歴

質問編集履歴

12

リンク切れを修正

2016/07/11 05:14

投稿

退会済みユーザー
title CHANGED
File without changes
body CHANGED
@@ -40,7 +40,7 @@
40
40
 
41
41
  **補足**
42
42
  サンプルコード内のSpreadsheetManagerクラスは以下のSpreadsheet Service全てを網羅するラッパークラスではありません。初期化時に引数に渡すファイル名に紐づくスプレッドシートファイルを読み込み、そのシートに対する読み書きなどの操作を、単純なインターフェイスを外部に公開し利用出来るようにするものです。
43
- https://developers.google.com/apps-script/reference/spreadsheet/
43
+ [https://developers.google.com/apps-script/reference/spreadsheet/
44
-
44
+ ](https://developers.google.com/apps-script/reference/spreadsheet/)
45
45
  ** Facadeクラスの命名に関する参考記事
46
46
  [What's a good name for a façade class?](http://stackoverflow.com/questions/6737423/whats-a-good-name-for-a-fa%C3%A7ade-class)

11

参考にならなかったリンク先を削除

2016/07/11 05:14

投稿

退会済みユーザー
title CHANGED
File without changes
body CHANGED
@@ -43,6 +43,4 @@
43
43
  https://developers.google.com/apps-script/reference/spreadsheet/
44
44
 
45
45
  ** Facadeクラスの命名に関する参考記事
46
- [What's a good name for a façade class?](http://stackoverflow.com/questions/6737423/whats-a-good-name-for-a-fa%C3%A7ade-class)
46
+ [What's a good name for a façade class?](http://stackoverflow.com/questions/6737423/whats-a-good-name-for-a-fa%C3%A7ade-class)
47
- [Design pattern name for an API wrapper class
48
- ](http://programmers.stackexchange.com/questions/258090/design-pattern-name-for-an-api-wrapper-class)

10

Facadeクラスの命名に関する参考記事を追記

2016/07/11 05:07

投稿

退会済みユーザー
title CHANGED
File without changes
body CHANGED
@@ -34,9 +34,15 @@
34
34
  [http://qiita.com/KeithYokoma/items/ee21fec6a3ebb5d1e9a8](http://qiita.com/KeithYokoma/items/ee21fec6a3ebb5d1e9a8)
35
35
  [http://gihyo.jp/dev/serial/01/code/000204](http://gihyo.jp/dev/serial/01/code/000204)
36
36
 
37
+ ## 更新
37
38
  **更新**
38
39
  頂いた回答を読んで、まず、今回の例の場合はxxxManagerというクラス名はふさわしくないことは理解致しました!!
39
40
 
40
41
  **補足**
41
42
  サンプルコード内のSpreadsheetManagerクラスは以下のSpreadsheet Service全てを網羅するラッパークラスではありません。初期化時に引数に渡すファイル名に紐づくスプレッドシートファイルを読み込み、そのシートに対する読み書きなどの操作を、単純なインターフェイスを外部に公開し利用出来るようにするものです。
42
- https://developers.google.com/apps-script/reference/spreadsheet/
43
+ https://developers.google.com/apps-script/reference/spreadsheet/
44
+
45
+ ** Facadeクラスの命名に関する参考記事
46
+ [What's a good name for a façade class?](http://stackoverflow.com/questions/6737423/whats-a-good-name-for-a-fa%C3%A7ade-class)
47
+ [Design pattern name for an API wrapper class
48
+ ](http://programmers.stackexchange.com/questions/258090/design-pattern-name-for-an-api-wrapper-class)

9

より適切な表現に変更

2016/07/11 05:02

投稿

退会済みユーザー
title CHANGED
File without changes
body CHANGED
@@ -10,8 +10,8 @@
10
10
  以下のコードではSpreadsheetManagerクラス内部でGoogle apps ScriptのSpreadsheetAppクラス、Spreadsheetクラス、Sheetクラス、また、Rangeクラスのメソッドを呼び出しています。
11
11
 
12
12
  ```TypeScript
13
- let spreadSheetManager: SpreadsheetManager = new SpreadsheetManager("シートID");
13
+ let spreadSheetManager: SpreadsheetManager = new SpreadsheetManager("スプレッドシートファイルID");
14
- spreadSheetManager.selectSheetByName("商品データ");
14
+ spreadSheetManager.selectSheetByName("シート名(タブ名)");
15
15
  spreadsheetManager.appendTableData([[1,2,3],[1,2,3],[1,2,3]);
16
16
  ```
17
17
 

8

補足情報の追加

2016/07/11 04:46

投稿

退会済みユーザー
title CHANGED
File without changes
body CHANGED
@@ -32,4 +32,11 @@
32
32
  **参考 - 命名規則に関する記事**
33
33
  [http://qiita.com/magicant/items/8134edf969f9629fa66e](http://qiita.com/magicant/items/8134edf969f9629fa66e)
34
34
  [http://qiita.com/KeithYokoma/items/ee21fec6a3ebb5d1e9a8](http://qiita.com/KeithYokoma/items/ee21fec6a3ebb5d1e9a8)
35
- [http://gihyo.jp/dev/serial/01/code/000204](http://gihyo.jp/dev/serial/01/code/000204)
35
+ [http://gihyo.jp/dev/serial/01/code/000204](http://gihyo.jp/dev/serial/01/code/000204)
36
+
37
+ **更新**
38
+ 頂いた回答を読んで、まず、今回の例の場合はxxxManagerというクラス名はふさわしくないことは理解致しました!!
39
+
40
+ **補足**
41
+ サンプルコード内のSpreadsheetManagerクラスは以下のSpreadsheet Service全てを網羅するラッパークラスではありません。初期化時に引数に渡すファイル名に紐づくスプレッドシートファイルを読み込み、そのシートに対する読み書きなどの操作を、単純なインターフェイスを外部に公開し利用出来るようにするものです。
42
+ https://developers.google.com/apps-script/reference/spreadsheet/

7

リンクされていないところを修正

2016/07/11 04:26

投稿

退会済みユーザー
title CHANGED
File without changes
body CHANGED
@@ -7,7 +7,7 @@
7
7
 
8
8
  例えば、Google Apps Scriptのスプレッドシート周りのメソッドを使いやすくするために、SpreadsheetManagerというクラスを作って、その中で以下のようなメソッドを外部に公開するといったやり方をしているのですが、これはよろしくないのでしょうか？
9
9
 
10
- 以下のコードでは内部でGoogle apps ScriptのSpreadsheetAppクラスとSheetクラス、また、Rangeクラスのメソッドを呼び出しています。
10
+ 以下のコードではSpreadsheetManagerクラス内部でGoogle apps ScriptのSpreadsheetAppクラス、Spreadsheetクラス、Sheetクラス、また、Rangeクラスのメソッドを呼び出しています。
11
11
 
12
12
  ```TypeScript
13
13
  let spreadSheetManager: SpreadsheetManager = new SpreadsheetManager("シートID");
@@ -26,8 +26,8 @@
26
26
  よろしくお願い致します。
27
27
 
28
28
  **参考 - Google Spreadsheet Service **
29
- [https://developers.google.com/apps-script/reference/spreadsheet/]
29
+ [https://developers.google.com/apps-script/reference/spreadsheet/
30
- (https://developers.google.com/apps-script/reference/spreadsheet/)
30
+ ](https://developers.google.com/apps-script/reference/spreadsheet/)
31
31
 
32
32
  **参考 - 命名規則に関する記事**
33
33
  [http://qiita.com/magicant/items/8134edf969f9629fa66e](http://qiita.com/magicant/items/8134edf969f9629fa66e)

6

2016/07/11 04:21

投稿

退会済みユーザー
title CHANGED
File without changes
body CHANGED
@@ -25,7 +25,7 @@
25
25
 
26
26
  よろしくお願い致します。
27
27
 
28
- **参考 - Google Spreadsheet Service
28
+ **参考 - Google Spreadsheet Service **
29
29
  [https://developers.google.com/apps-script/reference/spreadsheet/]
30
30
  (https://developers.google.com/apps-script/reference/spreadsheet/)
31
31
 

5

2016/07/11 02:31

投稿

退会済みユーザー
title CHANGED
File without changes
body CHANGED
@@ -7,7 +7,7 @@
7
7
 
8
8
  例えば、Google Apps Scriptのスプレッドシート周りのメソッドを使いやすくするために、SpreadsheetManagerというクラスを作って、その中で以下のようなメソッドを外部に公開するといったやり方をしているのですが、これはよろしくないのでしょうか？
9
9
 
10
- 以下のコードでは内部でGoogle Ａpps ScriptのSpreadsheetAppクラスとSheetクラス、また、Rangeクラスのメソッドを呼び出しています。
10
+ 以下のコードでは内部でGoogle apps ScriptのSpreadsheetAppクラスとSheetクラス、また、Rangeクラスのメソッドを呼び出しています。
11
11
 
12
12
  ```TypeScript
13
13
  let spreadSheetManager: SpreadsheetManager = new SpreadsheetManager("シートID");
@@ -25,8 +25,11 @@
25
25
 
26
26
  よろしくお願い致します。
27
27
 
28
+ **参考 - Google Spreadsheet Service
29
+ [https://developers.google.com/apps-script/reference/spreadsheet/]
30
+ (https://developers.google.com/apps-script/reference/spreadsheet/)
28
31
 
29
- **参考**
32
+ **参考 - 命名規則に関する記事**
30
33
  [http://qiita.com/magicant/items/8134edf969f9629fa66e](http://qiita.com/magicant/items/8134edf969f9629fa66e)
31
34
  [http://qiita.com/KeithYokoma/items/ee21fec6a3ebb5d1e9a8](http://qiita.com/KeithYokoma/items/ee21fec6a3ebb5d1e9a8)
32
35
  [http://gihyo.jp/dev/serial/01/code/000204](http://gihyo.jp/dev/serial/01/code/000204)

4

2016/07/11 02:30

投稿

退会済みユーザー
title CHANGED
File without changes
body CHANGED
@@ -10,9 +10,9 @@
10
10
  以下のコードでは内部でGoogle Ａpps ScriptのSpreadsheetAppクラスとSheetクラス、また、Rangeクラスのメソッドを呼び出しています。
11
11
 
12
12
  ```TypeScript
13
- let spreadSheet: SpreadsheetManager = new SpreadsheetManager("シートID");
13
+ let spreadSheetManager: SpreadsheetManager = new SpreadsheetManager("シートID");
14
- spreadSheet.selectSheetByName("商品データ");
14
+ spreadSheetManager.selectSheetByName("商品データ");
15
- spreadsheet.appendTableData([[1,2,3],[1,2,3],[1,2,3]);
15
+ spreadsheetManager.appendTableData([[1,2,3],[1,2,3],[1,2,3]);
16
16
  ```
17
17
 
18
18
  以下のリンク先ページを見ると、xxxManagerというよりはxxxFacadeというクラス名がこのような場合は適切ということなのでしょうか？

3

2016/07/10 09:04

投稿

退会済みユーザー
title CHANGED
File without changes
body CHANGED
@@ -10,7 +10,7 @@
10
10
  以下のコードでは内部でGoogle Ａpps ScriptのSpreadsheetAppクラスとSheetクラス、また、Rangeクラスのメソッドを呼び出しています。
11
11
 
12
12
  ```TypeScript
13
- let spreadSheet: Sheet = new SpreadsheetManager("シートID");
13
+ let spreadSheet: SpreadsheetManager = new SpreadsheetManager("シートID");
14
14
  spreadSheet.selectSheetByName("商品データ");
15
15
  spreadsheet.appendTableData([[1,2,3],[1,2,3],[1,2,3]);
16
16
  ```

2

2016/07/10 09:03

投稿

退会済みユーザー
title CHANGED
File without changes
body CHANGED
@@ -7,6 +7,8 @@
7
7
 
8
8
  例えば、Google Apps Scriptのスプレッドシート周りのメソッドを使いやすくするために、SpreadsheetManagerというクラスを作って、その中で以下のようなメソッドを外部に公開するといったやり方をしているのですが、これはよろしくないのでしょうか？
9
9
 
10
+ 以下のコードでは内部でGoogle Ａpps ScriptのSpreadsheetAppクラスとSheetクラス、また、Rangeクラスのメソッドを呼び出しています。
11
+
10
12
  ```TypeScript
11
13
  let spreadSheet: Sheet = new SpreadsheetManager("シートID");
12
14
  spreadSheet.selectSheetByName("商品データ");

1

x

2016/07/10 05:18

投稿

退会済みユーザー
title CHANGED
File without changes
body CHANGED
@@ -7,9 +7,11 @@
7
7
 
8
8
  例えば、Google Apps Scriptのスプレッドシート周りのメソッドを使いやすくするために、SpreadsheetManagerというクラスを作って、その中で以下のようなメソッドを外部に公開するといったやり方をしているのですが、これはよろしくないのでしょうか？
9
9
 
10
+ ```TypeScript
10
11
  let spreadSheet: Sheet = new SpreadsheetManager("シートID");
11
12
  spreadSheet.selectSheetByName("商品データ");
12
13
  spreadsheet.appendTableData([[1,2,3],[1,2,3],[1,2,3]);
14
+ ```
13
15
 
14
16
  以下のリンク先ページを見ると、xxxManagerというよりはxxxFacadeというクラス名がこのような場合は適切ということなのでしょうか？
15
17