Change RadarValues internal implementation to make sense

Why provide subscripting through the float* operator instead of the [] operator?
Why have a second list that every new RadarCategory must be added to?