충남대 Android_김재민 2주차 1단계 과제 제출

[Kjamm]

코드 작성하면서 어려웠던 점

• SQL을 공부하기는 했지만, 아직 문법이 어색해 원하는 흐름대로 바로바로 만들지 못했습니다.
• 처음에 데이터가 삽입되는 과정을 그냥 로그로 찍어서 로그가 나오면 데이터가 삽입되었을 것 같아 바로 넘어갔었는데, 최종 테스트 과정에서 App inspection을 확인해보니 db가 closed로 되어있어 이 문제를 해결하는데도 오래걸렸습니다. 이렇게 테스트를 어떤 방식으로 진행할지 정하는 것도 어려운 것 같습니다.

중점적으로 리뷰해주셨으면 하는 부분

• 로그 확인 결과 데이터들을 삽입하는 과정에서 로그는 찍히는데 App inspection에서 확인해보려고 하면, db에 데이터가 삽입된 것은 같으나, 같은 데이터가 여러번 반복해서 뜨는 것 같은 문제가 발생하는 것처럼 보이는데, 잘 진행된 것인지 궁금합니다.

실행화면

[InyoungAum]

SELECT 쿼리를 배우셨는지 궁금하네요
데이터가 잘 들어갔는지 확인하기 위해서 데이터 저장후 SELECT 쿼리등을 이용해서 확인을 해도 좋을 것 같습니다.

[InyoungAum]

Helper에 onCreate에서 이미 생성해주고 있어서 Create는 여기서 필요 없을 것 같네요

[Kjamm]

넵. 삭제하도록 하겠습니다. 감사합니다!

[InyoungAum]

이렇게 되면 같은데이터가 10 개씩 들어가고 앱이 실행할떄마다 또 똑같은데이터가 10개씩 들어 갈 것 같은데요
이거 외에도 중복문제가 있나요?

[Kjamm]

아니요. 해당 원인 때문에 똑같은 데이터가 또 들어간 것 같습니다. 감사합니다!

[InyoungAum]

repository 클래스를 만들고 거기서 dbHelper와

"INSERT INTO ${DatabaseHelper.TABLE_NAME} (${DatabaseHelper.COLUMN_NAME}, ${DatabaseHelper.COLUMN_ADDRESS}) VALUES ('cafe$i', '대전 유성구 봉명동 $i')"

이러한 SQL동작들도 함수로 따로 뺴서 사용하면 훨씬 좋을 것 같네요
같은 insert기능인데 중복인 코드가 많이 보입니다.

[Kjamm]

넵 따로 만들도록 하겠습니다. 감사합니다!

[InyoungAum]

그리고 아직 mvvm은 적용 안하신거죠?

[Kjamm]

그리고 아직 mvvm은 적용 안하신거죠?

넵 처음 들어보는 개념이라 아직 공부중이기도 하고 신경쓰지 못했습니다.

[InyoungAum]

LGTM