Homework_Activity_3#222
Homework_Activity_3#222AnGuru-60 wants to merge 2 commits intoAndroid-Developer-Basic:homework/activity_03from
Conversation
| type = "image/*" | ||
| setPackage("org.telegram.messenger") | ||
|
|
||
| val imgUri = imageView.tag as? Uri |
There was a problem hiding this comment.
Получаете uri, но не сохраняете его в tag. Из-за этого imgUri всегда будет null, и картинка не отправится в Telegram. Нужно добавить imageView.tag = uri в метод populateImage
| findViewById<Button>(R.id.button)?.setOnClickListener { | ||
| val firstName = findViewById<EditText>(R.id.editFirstName).text.toString() | ||
| val lastName = findViewById<EditText>(R.id.editLastName).text.toString() | ||
| val age = findViewById<EditText>(R.id.editAge).text.toString().toInt() |
There was a problem hiding this comment.
Может выбросить NumberFormatException
Используйте toIntOrNull() и обработайте случай, когда значение null
| resultCode: Int, | ||
| intent: Intent? | ||
| ): UserInfo? { | ||
| if ( |
There was a problem hiding this comment.
Можем упростить: if (resultCode != RESULT_OK || intent == null) return null
| android:layout_marginTop="50dp" | ||
| android:ems="10" | ||
| android:hint="Возраст" | ||
| android:inputType="text" |
There was a problem hiding this comment.
Для поля возраста лучше использовать number или numberSigned, чтобы пользователь мог вводить только числа
|
@Lufavetskaya, большое спасибо за Ваши замечания!
Сделал через оператор Элвиса: val age = findViewById(R.id.editAge).text.toString().toIntOrNull() ?: 0 |
2 participants
No description provided.