Running every frame (from what I get from your description) is not ideal.
I recommend using spatial for example: workspace:GetBoundingBox() with the characters values or you could try to use ZonePlus a module made by ForeverDev which uses spatial queries and they are ready to be used without having the effort of building your own touch connections. All documentation and videos on how to use are in the post I linked up.
Hope this helps