Homework_9 (View-Resources) implementation.#63
Homework_9 (View-Resources) implementation.#63Vp-Ma wants to merge 1 commit intoAndroid-Developer-Basic:masterfrom
Conversation
| android:layout_width="wrap_content" | ||
| android:layout_height="wrap_content" | ||
| android:layout_margin="8dp" | ||
| android:text="Order Total:" |
There was a problem hiding this comment.
Хоть репозиторий учебный, ресурсы текстовые и размерные всегда лучше выносить отдельно, как привычка
| android:layout_height="wrap_content" | ||
| android:layout_margin="8dp" | ||
| android:text="Order Total:" | ||
| android:textSize="28dp" |
There was a problem hiding this comment.
Для размеров текста нужно использовать sp (scale-independent pixels), а не dp. Это важно для доступности sp учитывает настройки масштаба текста пользователя в системе. Если пользователь увеличил размер шрифта в настройках, текст с sp автоматически станет больше, а с dp останется прежним
| textCartShippingAmount = findViewById<TextView>(R.id.textCartShippingAmount) | ||
| textCartTaxAmount = findViewById<TextView>(R.id.textCartTaxAmount) | ||
|
|
||
| cartItemInc = findViewById<View>(R.id.сart_item_inc_1) |
There was a problem hiding this comment.
Один и тот же код повторяется 4 раза для установки текста элементов корзины. Это нарушает принцип DRY. Вынесите это в цикл или функцию, чтобы код был более читаемым
| textCartShipping = findViewById<TextView>(R.id.textCartShipping) | ||
| textCartTax = findViewById<TextView>(R.id.textCartTax) | ||
| textCartItemCount = findViewById<TextView>(R.id.textCartItemCount) | ||
| textCartOrderTotalAmount = findViewById<TextView>(R.id.textCartOrderTotalAmount) |
There was a problem hiding this comment.
Переменные инициализируются, но значения устанавливаются напрямую в XML или через строковые ресурс, а не через эти переменные. Это лишний код, который усложняет понимание
2 participants
No description provided.