-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Styling/coding standards #3
Comments
Using PEP8 would be a good start ;). I'm sitting on a fence regarding the camelCase vs snake_notation. I'm used to snake but I also see the benefit of sticking to the original SQF function names. Deep inside me I'd prefer to stick with the original naming (less confusion) but I haven't done enough Arma modding (nor even used Intercept much) to speak authoritatively about this matter. |
Where do you guys stand on replacing |
I have some boilerplate code here: https://github.com/Tirpitz93/intercept-python/tree/python_classes. |
Intercept uses |
Which coding standards/PEP guides are you planning to adhere to.
As this is a python library aimed at python users(presumably) the PEPs would be a good place to start.
Also when implementing the classes there are a couple options regarding the OOP interface as I see it:
getPos player
becomesplayer.getPos()
. orplayer.get_pos()
@property
decorator to wrap the calls to the DLL to the user so thatgetPos player
becomesplayer.pos
, andplayer setPos [a,b,c]
becomesplayer.pos = [a,b,c]
.The text was updated successfully, but these errors were encountered: