🚑 if문 논리 오류 수정

[jiminnimij]

#️⃣ 연관된 이슈

관련된 이슈 번호를 적어주세요. 예: #이슈번호

#️⃣ 작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요. (이미지 첨부 가능)

#️⃣ 테스트 결과

코드 변경에 대해 테스트를 수행한 결과를 요약해주세요. 예: 모든 테스트 통과 여부, 새로 작성한 테스트 케이스 등

#️⃣ 변경 사항 체크리스트

• 코드에 영향이 있는 모든 부분에 대한 테스트를 작성하고 실행했나요?
• 문서를 작성하거나 수정했나요? (필요한 경우)
• 코드 컨벤션에 따라 코드를 작성했나요?
• 본 PR에서 발생할 수 있는 모든 의존성 문제가 해결되었나요?

#️⃣ 스크린샷 (선택)

관련된 스크린샷이 있다면 여기에 첨부해주세요.

#️⃣ 리뷰 요구사항 (선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요.
예시: 이 부분의 코드가 잘 작동하는지 테스트해 주실 수 있나요?

📎 참고 자료 (선택)

관련 문서, 스크린샷, 또는 예시 등이 있다면 여기에 첨부해주세요

[gemini-code-assist]

Summary of Changes

Hello @jiminnimij, I'm Gemini Code Assist¹! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

이 PR은 ReviewService 내의 중요한 논리적 결함을 해결하여, 기존 리뷰가 없는 경우 발생할 수 있는 런타임 오류를 방지합니다. if 조건문을 올바르게 수정함으로써, 리뷰 번호 할당 로직이 예상대로 작동하도록 보장하여 애플리케이션의 안정성을 향상시킵니다.

Highlights

• 논리 오류 수정: ReviewService의 addReview 메서드에서 posts 리스트의 비어있음 여부를 확인하는 if 문의 논리 오류를 수정했습니다. 기존에는 posts가 비어있을 때 posts.get(0)을 호출하여 IndexOutOfBoundsException이 발생할 수 있었으나, !posts.isEmpty()로 조건을 변경하여 posts가 비어있지 않을 때만 posts.get(0)을 호출하도록 수정했습니다.

Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature	Command	Description
Code Review	/gemini review	Performs a code review for the current pull request in its current state.
Pull Request Summary	/gemini summary	Provides a summary of the current pull request in its current state.
Comment	@gemini-code-assist	Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help	/gemini help	Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution. ↩

[gemini-code-assist]

Code Review

이번 PR은 if문의 조건을 수정하여 posts 리스트가 비어있을 때 IndexOutOfBoundsException이 발생하던 버그를 해결했습니다. 올바른 수정입니다.

다만, 리뷰 과정에서 addReview 메소드에 몇 가지 추가 개선점을 발견했습니다.

1. exhibition.increaseReviewCount()가 81번과 87번 라인에서 중복으로 호출되어, 리뷰가 추가될 때마다 리뷰 수가 2씩 증가하게 됩니다. 이는 의도치 않은 동작일 수 있습니다.
2. 새로운 리뷰 번호를 생성하는 로직(77-78번 라인)은 여러 요청이 동시에 들어올 경우 동일한 번호를 부여하고 카운터가 의도치 않게 건너뛰는 동시성 문제가 발생할 수 있습니다.

이러한 문제들도 함께 해결하면 코드의 안정성이 더욱 향상될 것입니다.