6 asked

0

안녕하세요 윌던님!

디테일한 지적 감사합니다! 네 윌던님 말씀처럼 3번이 2번 안에 들어가는 것이 더 자연스럽습니다.

해당 내용은 강의 자료를 수정하도록 하겠습니다.

감사합니다 :)

0

안녕하세요, 인프런 AI 인턴입니다. 윌던님.

강의를 진행하며 코드에서 어색함을 느끼셨군요. 걱정하지 마세요, 코드를 한번 검토해드립니다.

코드 검토

주석과 코드 논리에 따르면, 새로운 announcementId를 목록에 추가한 후 저장하는 과정에서 prefs.setStringList()가 실행되어야 합니다. 그리고 이 동작은 announcementId가 목록에 포함되지 않았을 경우에만 정확히 실행되어야 합니다.

다음과 같이 prefs.setStringList()가 if 문 안으로 이동하는 것이 적절할 수 있습니다:

static Future<void> _setViewed(int announcementId) async {
  final prefs = await SharedPreferences.getInstance();
  // 1. 기존의 공지사항 아이디를 가져온다.
  final viewedIds = prefs.getStringList(_viewedAnnouncementsKey) ?? [];

  // 2. 가져온 아이디에 포함여부 확인.
  if (!viewedIds.contains(announcementId.toString())) {
    // 3. 포함되어 있지 않다면 새로 추가한다.
    viewedIds.add(announcementId.toString());
    await prefs.setStringList(_viewedAnnouncementsKey, viewedIds);
  }
}

이렇게 변경하면, announcementId가 이미 목록에 포함되어 있는 경우 불필요한 저장 작업을 피할 수 있습니다.

해당 코드 구조가 문제없이 작동하는지 확인해 보시고, 강사님께서 안내해주신 내용이나 코드와 충돌하지 않도록 검증해 보세요.

관련하여 다른 유사한 질문 및 보완 내용을 확인하고 싶으시면, 아래 링크를 참조해 보세요:

• 옵션 설정 변경 시 고려사항
  
• SharedPreferences 관련 정보
  

저는 질문자님의 이해를 돕고자 기존 답변을 참고하여 유사한 답변 링크를 추천해드리고 있습니다. 현재 베타 기능으로 답변이 만족스럽지 않을 수 있는 점 양해 부탁드립니다. 🙏 추가적으로 궁금한 점이 있으시면, 이어서 질문해 주세요. 곧 지식공유자께서 답변해 주실 것입니다.