Sonarcloud testing Sprint Three - UQdeco2800/2022-studio-2 GitHub Wiki

Introduction

In the sprint three, we had more code smells in our code, but we had fewer bugs. Similar to the last sprint, we removed some unused imports and variables, but there were still some package naming and accessor modifier code smells that were highlighted and potentially needed to be changed. And fixed bugs that we build. Major Bugs There are we needed to change the Data Type from the following.

Solutions Fixed

Minor Issues

Some of the minor issues in the game which were fixed included: 1.

Removed unused variables
Removed old comments

Remove the temporary variable "material"

Remove the "index" field and declare it as a local variable in the relevant methods

Remove the unnecessary boolean"true"

Major Issues

This major code smells issue which has been resolved is the lack of constructors in our material factory classes. In this sprint I added an empty constructor to these classes to resolve this.

add private constructor

This major code smells issue which has been resolved is not use string format game area display class. In this sprint I use string format to rewrite this line to resolve this.

Use format specifiers

Rename Variable

Remove this useless assignment to local variable "materials"

Ignored NullPointerException

Major Bugs

Future

In the final polish phase of the game, we need to check the game for more minor errors and clean up any notes we want to keep or edit in the final sprint.