Scribd
Upload
3 views2 pages

Code Review: Common Java Issues

The document lists several issues found in a code snippet, including potential exceptions from invalid input, incorrect calculations, off-by-one errors, variable shadowing, improper use of print statements, and resource leaks due to an unclosed scanner. Each issue is identified with a description and the corresponding line number. The summary highlights the need for code corrections to improve functionality and readability.

Uploaded by

Nguyen Van Nam (K18 HCM)
Copyright
© All Rights Reserved
We take content rights seriously. If you suspect this is your content, claim it here.
Available Formats
Download as XLSX, PDF, TXT or read online on Scribd
3 views2 pages

Code Review: Common Java Issues

The document lists several issues found in a code snippet, including potential exceptions from invalid input, incorrect calculations, off-by-one errors, variable shadowing, improper use of print statements, and resource leaks due to an unclosed scanner. Each issue is identified with a description and the corresponding line number. The summary highlights the need for code corrections to improve functionality and readability.

Uploaded by

Nguyen Van Nam (K18 HCM)
Copyright
© All Rights Reserved
We take content rights seriously. If you suspect this is your content, claim it here.
Available Formats
Download as XLSX, PDF, TXT or read online on Scribd

Issue NoDescription Line Detail

1 Input valid 9 `[Link]()` can throw an exception for invalid input.


2 Incorrect F 20 `tempNthTerm = n1 - n2` should be `tempNthTerm = n1 + n2`.
3 Off-by-one 20 `i <= n - 1` should be `i <= n` to include all terms.
4 Variable 'n 3 Variable shadowing can cause unexpected behavior.
5 Improper us 13, 15, 17 `[Link]()` should be replaced with `[Link]()` for readability.
6 Scanner res 8 `scanner` is not explicitly closed, which may lead to resource leaks.
[Link]()` for readability.
source leaks.

You might also like