Raised This Month: $7 Target: $400
 1% 

Nade Semiclip v2.7


Post New Thread Reply   
 
Thread Tools Display Modes
yRemootz
Junior Member
Join Date: Aug 2021
Location: Sao Paulo, Brazil
Old 11-06-2021 , 21:52   Re: Nade Semiclip v2.5
Reply With Quote #41

Quote:
Originally Posted by JusTGo View Post
Updated:

Code:
-V2.5
* Rewritten the code to make it clearer.
* Added a tracking cvar.
I guess it's ready for a review now.
I think you should comment out your code to explain what each line does. Just to be more explained.
But, great plugin!

Last edited by yRemootz; 11-06-2021 at 21:52.
yRemootz is offline
JusTGo
Veteran Member
Join Date: Mar 2013
Old 11-07-2021 , 02:17   Re: Nade Semiclip v2.5
Reply With Quote #42

Quote:
Originally Posted by yRemootz View Post
I think you should comment out your code to explain what each line does. Just to be more explained.
But, great plugin!
I have already commented on the parts that need to be explained, the rest can be easily figured by variable and function names, which part doesn't seem clear to you?
__________________
JusTGo is offline
iceeedr
Veteran Member
Join Date: Apr 2017
Location: Brazil
Old 11-07-2021 , 21:09   Re: Nade Semiclip v2.5
Reply With Quote #43

Do not use magic numbers "114, 96"

PHP Code:
#define getUserTeam(%1)    get_pdata_int(%1, 114)
#define isC4(%1)    (get_pdata_int(%1, 96) & (1<<8)) 
__________________


Quote:
Originally Posted by fysiks View Post
Please stop trying to help. You appear to just be posting random stuff. Wait until you actually understand more about AMX Mod X and how the game works.
https://iceeedr.com.br/
iceeedr is offline
Send a message via Skype™ to iceeedr
JusTGo
Veteran Member
Join Date: Mar 2013
Old 11-07-2021 , 22:14   Re: Nade Semiclip v2.5
Reply With Quote #44

Quote:
Originally Posted by iceeedr View Post
Do not use magic numbers "114, 96"

PHP Code:
#define getUserTeam(%1)    get_pdata_int(%1, 114)
#define isC4(%1)    (get_pdata_int(%1, 96) & (1<<8)) 
IMO it's fine if I'm using them only once, and the define name is pretty descriptive but I can change them if that's really needed.
__________________
JusTGo is offline
iceeedr
Veteran Member
Join Date: Apr 2017
Location: Brazil
Old 11-07-2021 , 23:01   Re: Nade Semiclip v2.5
Reply With Quote #45

What I told you is exactly what will be said when your plugin is analyzed, but feel free to leave it that way.
__________________


Quote:
Originally Posted by fysiks View Post
Please stop trying to help. You appear to just be posting random stuff. Wait until you actually understand more about AMX Mod X and how the game works.
https://iceeedr.com.br/
iceeedr is offline
Send a message via Skype™ to iceeedr
JusTGo
Veteran Member
Join Date: Mar 2013
Old 11-08-2021 , 03:52   Re: Nade Semiclip v2.5
Reply With Quote #46

Quote:
Originally Posted by iceeedr View Post
What I told you is exactly what will be said when your plugin is analyzed, but feel free to leave it that way.
If a reviewer asks for it then I will do it, even though I think it's fine to use it this way as I'm using these numbers only once, also in amx1.9+ there are new natives that use game data files that could be used to avoid using the numbers in the plugin but some people might be still using amx1.8.2 and it's still the official stable release.
__________________
JusTGo is offline
HamletEagle
AMX Mod X Plugin Approver
Join Date: Sep 2013
Location: Romania
Old 11-08-2021 , 04:29   Re: Nade Semiclip v2.5
Reply With Quote #47

Quote:
Originally Posted by JusTGo View Post
If a reviewer asks for it then I will do it, even though I think it's fine to use it this way as I'm using these numbers only once, also in amx1.9+ there are new natives that use game data files that could be used to avoid using the numbers in the plugin but some people might be still using amx1.8.2 and it's still the official stable release.
It's absolutely NOT fine. It doesn't matter that you are using it only once. Quick, without googling anything tell me what this line does:

PHP Code:
get_pdata_cbase(id3735
You can't, even tho I used it only once. What in the world is 373? What is this supposed to do? You don't know unless you just so happen to know that offset 373 is m_pActiveItem. Now, look at the following line:

PHP Code:
get_pdata_cbase(idm_pActiveItemXO_PLAYER
You know instantly what it does. A number doesn't say anything, but a name does.

Not having magic values in your plugin is a requirement for approval, for a very good reason. Doing otherwise is asking for unmaintainable, unreadable and crappy code.
Use 1.8.2, it's fine, in fact it's even recommended if it can be reasonably done as it is still the latest stable release. Just name things properly, it's not hard.
__________________

Last edited by HamletEagle; 11-08-2021 at 04:31.
HamletEagle is offline
JusTGo
Veteran Member
Join Date: Mar 2013
Old 11-08-2021 , 08:11   Re: Nade Semiclip v2.5
Reply With Quote #48

Quote:
Originally Posted by HamletEagle View Post
It's absolutely NOT fine. It doesn't matter that you are using it only once. Quick, without googling anything tell me what this line does:

PHP Code:
get_pdata_cbase(id3735
You can't, even tho I used it only once. What in the world is 373? What is this supposed to do? You don't know unless you just so happen to know that offset 373 is m_pActiveItem.
That's true but in my code, there is a clear context which is the defined name which is IMO pretty descriptive and straightforward.

PHP Code:
#define getUserTeam(%1)    get_pdata_int(%1, 114) 
Anyone who reads getUserTeam will immediately know that 114 is related to a user team member.

I lean towards making descriptive names in my coding style instead of writing more code/comments, but I will update it in case someone finds this confusing.

Code:
-V2.6
* Minor changes to improve readability.
__________________

Last edited by JusTGo; 11-08-2021 at 08:18.
JusTGo is offline
HamletEagle
AMX Mod X Plugin Approver
Join Date: Sep 2013
Location: Romania
Old 11-08-2021 , 10:39   Re: Nade Semiclip v2.6
Reply With Quote #49

Yes, I know it gets the users team by looking at getUserTeam. But maybe I want to understand HOW it does it. Maybe I want to check regamedll and see where offset 114 is used and how it keeps track of teams.
But I can't, because I don't know what offset 114 is.
__________________
HamletEagle is offline
JusTGo
Veteran Member
Join Date: Mar 2013
Old 11-08-2021 , 11:17   Re: Nade Semiclip v2.6
Reply With Quote #50

Quote:
Originally Posted by HamletEagle View Post
Yes, I know it gets the users team by looking at getUserTeam. But maybe I want to understand HOW it does it. Maybe I want to check regamedll and see where offset 114 is used and how it keeps track of teams.
But I can't, because I don't know what offset 114 is.
True, you are right.

A few more changes I guess can be made:

PHP Code:
#define isC4(%1)    (get_pdata_int(%1, m_bIsC4) & (1<<8)) 
to:

PHP Code:
#define isC4(%1)    get_pdata_bool(%1, m_bIsC4) 
edit: Actually get_pdata_bool is not defined in amx1.8.2... I have to drop the support for it If I do that

Also a new function for getting dmgtime as I'm using it twice:

PHP Code:
bool:aboutToExplode(ent)
{
    static 
Float:dmgtime
    pev
(entpev_dmgtimedmgtime)
    
    
// It's going to explode :D
    
return (dmgtime <= get_gametime())

__________________

Last edited by JusTGo; 11-08-2021 at 11:21.
JusTGo is offline
Reply


Thread Tools
Display Modes

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

BB code is On
Smilies are On
[IMG] code is On
HTML code is Off

Forum Jump


All times are GMT -4. The time now is 02:09.


Powered by vBulletin®
Copyright ©2000 - 2024, vBulletin Solutions, Inc.
Theme made by Freecode