전남대 android_신혜서 1주차 과제 (Step2) #20

tlsgptj · 2024-06-28T06:48:14Z

[이전의 오류는 해결했습니다]

MainActivity.xml으로 연락처를 입력하면 collection.xml에 저장이 이루어지도록 하였습니다.
Room을 이용하여 데이터베이스를 구현하였고,
RecyclerView를 이용하여 연락처를 나타낼 수 있도록 하였습니다. itemview를 클릭하면 activity_whoami로 이동해 자세한 정보가 표시될 수 있도록 하였습니다.
하지만 collection.kt에서 오류가 발생합니다.
putExtra와 122줄에 있는 contacts, Contact.ContactData에서 오류가 발생합니다.
contact.kt로 분리해보기도 했습니다. 하지만 오류가 해결되지 않았습니다. 어떻게 하면 좋을까요?
그러고 안드로이드 에뮬레이터 실행도 되지 않는 문제도 발생했습니다. 어떻게 해결하면 좋을까여?

This reverts commit 8b9b2f4.

결국 Sharedpre로 갔습니다...

tlsgptj · 2024-07-03T06:52:48Z

[수정 후 기능 변화]

우선 RoomDB에서 오류가 많이 나서 일단 삭제하고 SharedPreference로 구현을 했습니다.
collection_xml의 내용을 멘토님의 조언을 받아 FrameLayout에서 ConstraintLayout으로 변경하여
RecyclerView로 구현을 해보았습니다.
collection.kt의 오류를 수정하였습니다.
recyclerView와 ItemView가 collection.xml에 표시되게끔 만들었습니다.

bigstark

리뷰가 늦어 죄송합니다! 전반적으로 함수의 분리와, 사용하지 않는 코드를 제거하는 부분에 대해 조금 더 신경써주셔서 작성해주시면 좋을 것 같아요!

일단 SharedPreferences 를 사용하고 있는 부분은 존재하고 있지 않은 것 같아요. 추가를 하더라도 리스트에 표현이 되고 있지 않은데, 어떻게 처리하면 좋을지 고민해보시면 좋을 것 같아요 :)

bigstark · 2024-07-07T05:55:51Z

app/src/main/java/campus/tech/kakao/contacts/CollectionActivity.kt

+        messageTextView = findViewById(R.id.message)
+        recyclerView = findViewById(R.id.recyclerView)


ViewBinding 을 통해서 findViewById 를 사용하지 않을 수 있습니다!

bigstark · 2024-07-07T05:56:37Z

app/src/main/java/campus/tech/kakao/contacts/CollectionActivity.kt

+        addButton.setOnClickListener {
+            val intent = Intent(this, MainActivity::class.java)
+            startActivity(intent)
+        }


setOnClickListener 나 layoutMangager 를 설정하는 부분은 초기화의 영역으로 보여요. 초기화를 하는 메소드로 따로 처리해두는건 어떨까요?

bigstark · 2024-07-07T05:57:48Z

app/src/main/java/campus/tech/kakao/contacts/CollectionActivity.kt

+        val contacts = getContacts()
+        contactAdapter = ContactAdapter(contacts)
+        recyclerView.adapter = contactAdapter


loadContacts 내부에서 adapter 를 매번 생성하고 할당하는 것보다는, Adapter 에 contacts set / add / remove 할 수 있는 메소드를 추가하는 편이 조금 더 좋은 방향으로 보여요.

bigstark · 2024-07-07T06:01:04Z

app/src/main/java/campus/tech/kakao/contacts/MainActivity.kt

+        private val etName: EditText by lazy { findViewById(R.id.name) }
+        private val etPhone: EditText by lazy { findViewById(R.id.phone) }
+        private val etmail: EditText by lazy { findViewById(R.id.email) }
+        private val etmessage: EditText by lazy { findViewById(R.id.message) }
+        val btnbirthday: Button by lazy { findViewById(R.id.birthday) }
+        private val rgfemale: RadioButton by lazy { findViewById(R.id.female) }
+        private val rgmale: RadioButton by lazy { findViewById(R.id.male) }


전역변수는 상단에 정의해주시는 편이 좋습니다.

bigstark · 2024-07-07T06:02:11Z

app/src/main/java/campus/tech/kakao/contacts/MainActivity.kt

+                rgfemale.isChecked -> ContactsContract.CommonDataKinds.StructuredName.DATA1
+                rgmale.isChecked -> ContactsContract.CommonDataKinds.StructuredName.DATA2
+                else -> ContactsContract.CommonDataKinds.StructuredName.DATA3


ContactsContract 의 사용은 의도된 것이 아닌 것으로 보입니다.

bigstark · 2024-07-07T06:03:20Z

app/src/main/java/campus/tech/kakao/contacts/MainActivity.kt

+        private fun writeContactsToXml(allContacts: List<Contact?>) {
+
+        }


사용하지 않는 함수는 제거해주시는 편이 좋을 것 같아요.

bigstark · 2024-07-07T06:04:02Z

app/src/main/java/campus/tech/kakao/contacts/MainActivity.kt

+        private fun setupPhoneNumberInput() {
+            etPhone.inputType = InputType.TYPE_CLASS_PHONE
+
+            val phoneNumberFilter = InputFilter { source, start, end, dest, dstart, dend ->
+                val phoneNumberRegex = Regex("^\\+?\\d{0,13}\$")
+                if (phoneNumberRegex.matches(source.toString())) {
+                    null
+                } else {
+                    ""
+                }
+            }
+            etPhone.filters = arrayOf(phoneNumberFilter)
+        }


사용하지 않는 메소드로 보입니다.

bigstark · 2024-07-07T06:04:22Z

app/src/main/java/campus/tech/kakao/contacts/WhoAmIActivity.kt

+        super.onCreate(savedInstanceState)
+        setContentView(R.layout.activity_who_am_iactivity)
+
+        val contact = intent.getSerializableExtra("contact") as? Contact


serializable 보다는 parcelable 로 정의해서 받는 것이 어떨까요?

bigstark · 2024-07-07T06:05:16Z

app/src/main/java/campus/tech/kakao/contacts/collection.kt

+    inner class ViewHolder(itemView: View) : RecyclerView.ViewHolder(itemView) {
+        val nameTextView: TextView = itemView.findViewById(R.id.namelist)
+
+        init {
+            itemView.setOnClickListener {
+                contacts[adapterPosition]?.let { contact ->
+                    val intent = Intent(itemView.context, WhoAmIActivity::class.java)
+                    itemView.context.startActivity(intent)
+                }
+            }
+        }
+    }


inner class 로 ViewHolder 를 선언하기 보다는 name 을 설정할 수 있는 메소드를 따로 선언하는 편이 어떨까요?

또한 ViewHolder 라는 이름보다는 조금 더 명확한 용도의 클래스 이름을 지어보는 것이 좋을 것 같아요.

bigstark · 2024-07-07T06:08:13Z

app/src/main/java/campus/tech/kakao/contacts/contact.kt

+data class ContactData(
+    val name: String,
+    val phone: String,
+    val gender: String,
+    val email: String,
+    val message: String,
+    val birthday: String
+) : Parcelable {
+    constructor(parcel: Parcel) : this(
+        parcel.readString().toString(),
+        parcel.readString().toString(),
+        parcel.readString().toString(),
+        parcel.readString().toString(),
+        parcel.readString().toString(),
+        parcel.readString().toString()
+    )
+
+    override fun writeToParcel(parcel: Parcel, flags: Int) {
+        parcel.writeString(name)
+        parcel.writeString(phone)
+        parcel.writeString(gender)
+        parcel.writeString(email)
+        parcel.writeString(message)
+        parcel.writeString(birthday)
+    }
+
+    override fun describeContents(): Int {
+        return 0
+    }
+
+    companion object CREATOR : Parcelable.Creator<ContactData> {
+        override fun createFromParcel(parcel: Parcel): ContactData {
+            return ContactData(parcel)
+        }
+
+        override fun newArray(size: Int): Array<ContactData?> {
+            return arrayOfNulls(size)
+        }
+    }
+}


요 ContactData 는 사용되지 않는 것으로 보여요. Contact 대신에 요걸 사용해보는건 어떨까요?

또한 kotlin-pareclize 를 통해 parcelabe 을 조금 더 쉽게 선언할 수 있습니다!

tlsgptj added 25 commits June 25, 2024 14:57

git commit

e0e1052

git commit

d9a0932

git commit

01d7501

git commit

55da53b

git commit

7ccd2fa

git commit

e2d6bf5

git commit

636ead4

git commit

8fe93ae

git commit

7b31203

git commit

d64c2ab

git commit

e7118f1

git commit

6800e06

git commit

8b9b2f4

Revert "git commit"

30e2ebf

This reverts commit 8b9b2f4.

git commit

42d3abd

git commit

69681aa

git commit

b3d1032

git commit

90a2d9e

git commit

37b5968

git commit

05289a5

git commit

3ac0e40

git commit

914d4de

git commit

66dd9d7

git commit

42cdb00

git commit

537fc09

tlsgptj changed the title ~~전남대 android_신혜서 Step2~~ 전남대 android_신혜서 1주차 과제 (Step2) Jun 28, 2024

tlsgptj added 4 commits June 30, 2024 15:19

git commit

510095e

git commit

6a36752

git commit

79e63a7

git commit

bcb902a

tlsgptj and others added 22 commits July 3, 2024 10:48

에뮬레이터 에러 수정

b5b07ab

roomdb에서 너무 에러가 많이 발생해

5d6db90

결국 Sharedpre로 갔습니다...

토스트메세지 수정

41f06d3

연락처 저장 후 콜렉션으로 넘어감

a52c392

연락처 저장 후 콜렉션으로 넘어감

d96b1ca

연락처 저장 후 콜렉션으로 넘어감

941e154

연락처 저장 후 콜렉션으로 넘어감

becd3e6

연락처 저장 후 콜렉션으로 넘어감

43e4a20

더미 데이터 생성

d764c36

Manifest 오류 수정

2842765

Manifest 오류 수정

9110a29

Manifest 오류 수정

62eb81b

collection.kt 오류수정

3e26fe6

Update README.md

cfb10f4

collection.kt 오류수정

3c7e0cb

collection.kt 오류 수정

88dddb2

최종수정

e3faa59

최종수정

9bfb18f

Merge remote-tracking branch 'origin/step2' into step2

c6b5a0d

토스트 메세지 수정

d01b21b

Update README.md

0ab23a6

Update README.md

59c5659

tlsgptj added 6 commits July 3, 2024 15:58

WhoAmIActivity.kt 수정

180667c

Merge remote-tracking branch 'origin/step2' into step2

ae390f3

WhoAmIActivity.kt 수정

2736700

WhoAmIActivity.kt 수정

80568ae

CollectionActivity.kt 수정

103a4d7

CollectionActivity.kt수정

023254b

bigstark requested changes Jul 7, 2024

View reviewed changes

		messageTextView = findViewById(R.id.message)
		recyclerView = findViewById(R.id.recyclerView)

		private fun writeContactsToXml(allContacts: List<Contact?>) {

		}

전남대 android_신혜서 1주차 과제 (Step2) #20

Are you sure you want to change the base?

전남대 android_신혜서 1주차 과제 (Step2) #20

Uh oh!

Conversation

tlsgptj commented Jun 28, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

tlsgptj commented Jul 3, 2024

Uh oh!

bigstark left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

tlsgptj commented Jun 28, 2024 •

edited

Loading