search

Waqas-Hirose / CustomViewPractice

This is Custom View Practice. A view is created with 4 subviews and without using constraints.
1 stars 0 forks source link

CustomViewのlayoutSubviewsのソースコードが見づらい #10

Open matsumoto777 opened 4 years ago

matsumoto777 commented 4 years ago

あなたのCustomViewのlayoutSubviewsはバグが多いというのが私の印象です。

6 や #9 が発生してしまうことが問題です。

また、それを見つけるのに多大な努力をしなければいけないというのも問題です。（私はたまたま時間を掛けてあなたのソースコードをチェックしていたため見つかりましたが、普段であればチェックにこれだけ多くの時間を使いません）

原因はソースコードが見づらいことだと私は思います。 他の３人と比較したとき、あなたのソースコードだけ明らかに複雑に見えます。

バグが見つけやすいようなシンプルなコードにしてもらうことは可能でしょうかか？

Waqas-Hirose commented 4 years ago

I will try to use simple approach.