dovigod / V-Gesture

Other
2 stars 0 forks source link

[Enhancement] - rearrange class dependency #27

Closed dovigod closed 5 months ago

dovigod commented 5 months ago

Start Date

2024/05/17

Summary

Current class depenedency looks as below.


     VGesture - GestureManager -Gestures
                     |- Detector -Camera(with canvas)

It seems awkward that detector has camera as child hiearchy

Therefore, I think its better to hoist every crucial calculations up to VGesture to make each classes weaker coupling.

Basic Example

No response

Drawbacks

Its just relocation of codes. no extra features or revisions.

Implementation PR

28

Reference Issues

No response