manuelvogel12 / sara-shield

MIT License
0 stars 1 forks source link

Initial changes to allow multiple humans #9

Closed manuelvogel12 closed 1 year ago

manuelvogel12 commented 1 year ago

I changed sara-shield to allow the presence of multiple humans. But I don't know if that was the best way to change it, and there are still some problems to be solved:

First, it requires changes to functions, that are already in use, namely the constructor and humanMeasurement

Also, I am not sure yet, whats the best way to determine the amount of HumanReach's that we want to have. I guess the options are:

  1. Explicitly add a variable to the constructor (e.g. int maxHumansInScene)
  2. Add method to increase the amount of HumanReach's e.g. void AddNewHuman()
  3. Implicitly add HumanReach, when the index of the measured human exceeds the current amount of HumansReach's

I think I would tend towards option 3, what`s your opinion?