Issues

• Missing const correctness in applyRules

  ColePoppleton/BoidsService#9 · Filter by author ColePoppleton opened on Mar 25, 2025
  

• Tests are linked to the BoidsService library as well as Qt6::Gui, which is redundant.

  ColePoppleton/BoidsService#8 · Filter by author ColePoppleton opened on Mar 25, 2025
  

• Hardcoded Qt6_DIR path.

  ColePoppleton/BoidsService#7 · Filter by author ColePoppleton opened on Mar 25, 2025
  

• Boundary Handling test case is flawed

  ColePoppleton/BoidsService#6 · Filter by author ColePoppleton opened on Mar 25, 2025
  

• Insufficient tolerance in compareVectors.

  ColePoppleton/BoidsService#5 · Filter by author ColePoppleton opened on Mar 25, 2025
  

• Potential division by zero in applyRules if separationCount is zero.

  ColePoppleton/BoidsService#4 · Filter by author ColePoppleton opened on Mar 25, 2025
  

• Boid velocity normalisation in applyRules and update leads to performance overhead.

  ColePoppleton/BoidsService#3 · Filter by author ColePoppleton opened on Mar 25, 2025
  

• No copy constructor or assignment operator in BoidsSystem - Potential for shallow copy issues.

  ColePoppleton/BoidsService#2 · Filter by author ColePoppleton opened on Mar 25, 2025
  

• srand(time(0)) called every time a BoidsSystem is created - Potential for Repeated Initialisation

  ColePoppleton/BoidsService#1 · Filter by author ColePoppleton opened on Mar 25, 2025