You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
"menu" переименовал бы на "menuId"
"click" на "clickListener" fun processTitle(textTitle: TextView) на fun TextView.changeTitile()
var itemLayoutId = R.layout.item_linear`
if (layoutManager is GridLayoutManager) {
itemLayoutId = R.layout.item_grid
}
Я бы поменял на
val itemLayoutId = when (layoutManager) {
is GridLayoutManager -> R.layout.item_grid
else -> R.layout.item_linear
}
Ещё бы из функции fun processRecycler(recycler: RecyclerView, dialog: BottomSheetDialog) вынес
if (menu > 0){}, и саму функцию вообще убрал бы. Например, вот так:
fun show(context: Context) {
...
val recycler = root.findViewById(R.id.recycler_view) as RecyclerView
if (menuId > 0) {
if (layoutManager == null) {
layoutManager = createVerticalLinearLayoutManager()
}
val itemLayoutId = when (layoutManager) {
is GridLayoutManager -> R.layout.item_grid
else -> R.layout.item_linear
}
if (adapter == null) {
adapter = createAdapter(dialog, itemLayoutId)
}
recyclerView.adapter = adapter
recyclerView.layoutManager = layoutManager
}
...
}
protected open fun processGrid(root: View) {
if (root.findViewById<View>(R.id.text_title).visibility != View.VISIBLE) {
if (layoutManager is GridLayoutManager) {
root.marginTop(24)
}
}
}
изменил на
protected open fun GridLayoutManager.setTopMarginIfTitleIsVisible(root: View) {
if (root.findViewById<View>(R.id.text_title).visibility != View.VISIBLE) {
root.marginTop(24)
}
}
The text was updated successfully, but these errors were encountered:
А ещё из fun show(context: Context) вынес контекст и убрал бы его в конструктор.
И кстати, почему fun processClick(dialog: BottomSheetDialog) нигде не используется? Это нормально?
"menu" переименовал бы на "menuId"
"click" на "clickListener"
fun processTitle(textTitle: TextView)
наfun TextView.changeTitile()
Я бы поменял на
Ещё бы из функции
fun processRecycler(recycler: RecyclerView, dialog: BottomSheetDialog)
вынесif (menu > 0){}, и саму функцию вообще убрал бы. Например, вот так:
поменял бы на
изменил на
The text was updated successfully, but these errors were encountered: