assigment4

docx

School

Arizona State University *

*We aren’t endorsed by this school

Course

222

Subject

Computer Science

Date

Feb 20, 2024

Type

docx

Pages

7

Uploaded by BaronGuineaPig1709

Report
Some Git things first -What do you think would happen if we (do NOT do this) merged Whitebox into master? The "GivenBlackbox.java" would also be deleted on master. GitHub Actions and Continuous Integration -Which branches fail on GitHub Actions? The review and master branch fails because it still runs the black box testing. The other branches either have it specified in the build.gradle to not run black box or it is deleted.
Task 1: Style checking using Checkstyle -How many violations did Checkstyle find in main? How many in test? 51 errors in main and 21 in test. My IDE was already set to spaces, so the violations did not change. Screenshots after fixes Main: main.html
Test: test.html
Your preview ends here
Eager to read complete document? Join bartleby learn and gain access to the full version
  • Access to all documents
  • Unlimited textbook solutions
  • 24/7 expert homework help
Task 2.1: Gradle
Task 2.2: Fixing the bugs SpotBugs Report
Task 3: Comparing What do you get now for these branches? Both branches had the same exact reports. Spotbugs report had 3 total warnings. The Checkstlye report had 55 errors in main and 8 errors in test. In the initial report for the StaticAnalysis branch, the Checkstyle report had 51 errors in main and 21 in test, and the Spotbugs had 4 errors in total and total density 31.25. It seems like my Code Review fixed some errors, but a majority of them are still present. After conducting Static Analysis and fixing the errors, the reports came back with less errors. BlackBox Review StaticAnalysis- before StaticAnalysis- after Spotbugs-total 3 3 4 0 Spotbugs-density 31.25 31.25 29.41 0 CheckStyle- main 55 55 51 1 CheckStyle- test 8 8 21 4
Your preview ends here
Eager to read complete document? Join bartleby learn and gain access to the full version
  • Access to all documents
  • Unlimited textbook solutions
  • 24/7 expert homework help
Task 4: Putting it together In your document, record how many violations were reported by Checkstyle and Spotbugs. Did it get better or worse, and can you explain why? In my Dev branch, the CheckStyle report indicates 2 errors in main and 4 in test. The Spotbugs report indicates 0 errors. The main package has one more error than in StaticAnalysis branch. Both errors indicates the line is longer than 100 characters. Do you think your code got better? In what order would you use these quality practices in the future? My code definitely got better. In the future, I would do the Static Analysis test first to fix those issues. Otherwise, the order the test were completed made sense to me. Final Code review Documentation: Added missing Javadoc comments for the class and methods where necessary to provide clear documentation. Variable Access Modifiers: Changed access modifiers of variables points, name, answer, and gameStatus to private/protected to encapsulate their access and ensure they're only modified through appropriate methods.