【Swift3.0】カスタムビューの作成の際のベストプラクティスについて

Question

###はじめに
Objective-C->Swift3.0への移行作業をしていく中で、UIViewを用いたカスタムビューをダイアログのように表示しようとしています。
カスタムビュー用のクラス(以後、CustomDialog.swift)とそれ用のxibファイル(以後、CustomDialog.xib)を使い、ViewController(以後、ViewController.swift)でaddsubviewすることで表示する想定です。
その際の、CustomDialog.swiftにおけるインスタンス生成方法のベストプラクティスについて教えていただきたく思います。

###コード
・Objective-Cでの実装内容
CustomDialog.m

- (instancetype)initWithName:(NSString *)name {
    NSString *className = NSStringFromClass([self class]);
    self = [[[NSBundle mainBundle] loadNibNamed:className owner:nil options:0] firstObject];
    
    self.dialogName = name;
    
    return self;
}

ViewController.m

- (void)viewDidLoad {
    [super viewDidLoad];

    self.customDialog = [[CustomDialog alloc] initWithName:@"hoge"];
    self.customDialog.frame = self.view.frame;
}

- (void)showDialog {
    [self.view addSubview:self.customDialog];
}

・Swiftでの実装内容
CustomDialog.swift

required init?(coder aDecoder: NSCoder) {
    dialogName = ""
    super.init(coder: aDecoder)
}
    
override init(frame: CGRect) {
    dialogName = ""
    super.init(frame: frame)
}
    
func setUp(frame: CGRect, name: String) {
        
    let customDialog: UIView = Bundle.main.loadNibNamed("CustomDialog", owner: self, options: nil)?.first as! UIView
        
    self.frame = frame
    customDialog.frame = frame
    dialogName = name
        
    addSubview(customDialog)
}

ViewController.swift

var customDialog: CustomDialog = CustomDialog()
override func viewDidLoad() {
    super.viewDidLoad()
        
    customDialog.setUp(frame: self.view.frame, name: "hoge")
}

func showDialog() {
    self.view.addSubview(customDialog)
}

ネットで調べながら実装したものなのですが、以下の2点が気になってしまいます。

・CustomDialog.swiftのRequired Initializerは実装必須であり、UIViewのDesignered Initializerも実装必須であるため、同じ初期化処理を２箇所でしてしまっている。

・CustomDialog.swiftクラスの[setUp:name:]メソッドの通り、ViewControllerの上にViewController.viewと同じframeの「CustomDialogそのもの」と「xibから呼び出したCustomDialog」が乗っていることになってしまう。

// 追記
修正依頼、ありがとうございました。
xibファイルは下図のような単純なもので、黒い半透明のviewの上に白いviewを乗せてダイアログのようにしているイメージです。

また、修正依頼でいただいた「Swiftだから」という点に関しては、
今までObjective-Cでは[initWithName:]という指定イニシャライザと最低限の初期化処理が記載してある[init:]メソッドのみ実装していました。
しかし、SwiftではViewControllerのCustomDialog()でDesignered Initializerが呼ばれ、[setUp:]メソッドのloadnibでRequired Initializerが呼ばれており、同じ初期化処理が2度呼ばれているということが気になってしまっています(動きとしては問題ないのですが)。
これは仕方のないことなのでしょうか。
// ----

この実装方法で問題がないのか、より良い実装方法があるのか判断できかねています。
よろしければご教授いただきたいです。

Comments

[fuzzball]

.xibはどういう構成になっているのでしょうか？あと、これはSwiftだからという話ではなく、Objective-Cでも同じことだと思うのですが。「Swiftだから」ということであれば、その辺りのことも書いて下さい。

Answer 1

同じ初期化処理を２箇所でしてしまっている

共通な箇所は別関数にしましょう。今回の場合は、dialogName = ""だけなのであまり意味はありませんが、そういうクセを付けておくと良いと思います。

ちなみにdialogNameの初期化に関しては、定義しているところで初期化すれば共通化出来ますね。（他に初期化する変数などがある場合は、それらと同じ場所で初期化する方が良いかも知れませんが）

var dialogName = ""

「CustomDialogそのもの」と「xibから呼び出したCustomDialog」

参考URLでお茶を濁します。（私もこの辺はちゃんと消化出来てないので）

Custom Viewクラスの初期化処理で、xibのViewをオブジェクト化して、それをaddSubviewする

Comments

[negi0205]

>共通な箇所は別関数にしましょう。
ご指摘ありがとうございます。
質問を追加編集したのですが、今回のようなxibを読み込んでカスタムビューを作る場合には初期化処理が2度呼ばれてしまうこと自体は仕方のないことなのでしょうか。

>参考URL
ありがとうございます。
Objective-Cではselfにview情報を代入することが出来ますが、Swiftではselfに代入することが出来ません。その理由(Objective-Cとの違い)ともし代入する方法があるのならば方法について、教えていただけますでしょうか(検索キーワードとなるような言葉で構いません)。

[negi0205]

連投すみません。
Swiftのselfについての記事を見つけたため貼ります。
「Swiftのmutatingはselfを書き換えられるのか #chocoswift」
http://qiita.com/haranicle/items/572c252dda688e2949dc

[fuzzball]

selfの書き換えについては情報を持ち合わせていませんのでスルーしますｗ

回答内にも書きましたが、この辺はキチンと理解していなくて、「試してみたらこうでした」的な内容になりますが、少し書いておきます。（こちらSwift2環境ですので、もしかしたら挙動の違う箇所があるかも知れません）

・ViewのCustom Classではなく、File's OwnerにCustomDialogを登録すると余計なViewが1枚減る。Required Initializerも呼ばれない。
・addSubViewのときに、self.view.addSubview(customDialog.subviews.first!) とすればトップViewを無視できる。（まぁ、当たり前の話ですが）
・setUp()はinit()の中で実行すべきものなのではないか？

以上、何かの参考になれば。

[negi0205]

>ViewのCustom Classではなく、File's OwnerにCustomDialogを登録すると余計なViewが1枚減る。Required Initializerも呼ばれない。
こちらのように、CustomDialogそのもののクラスを作るのではなく、CustomDialogとなるUIViewを管理するクラスとしてxibに登録する。
という方法で実装してみようと思います。

色々と考える切っ掛けにもなり、勉強にもなりました。投稿して良かったです。
回答ありがとうございました。