[RogueDrifter]

You want comments? stop bumping this shit and let people decide on their own.

It's a joke, first of all DINI? (I know right i'm the one to talk LOL) BUT I IMPROVED.

Overall, my point is DINI is SLOW, many other ways could've been used other than that and come on i mean 2018 and dini in a whole system isn't just decent, the system itself looks like one of those custom-made ones and hardly fits anywhere if it even works.

This:

PHP код:

new perkpoints[MAX_PLAYERS];
new perks[MAX_PLAYERS][10][5];
new perkcd[MAX_PLAYERS][10];
new perkregentimer[MAX_PLAYERS];
new Float:pmaxhealth[MAX_PLAYERS];
new Float:pmaxarmour[MAX_PLAYERS];
new Float:phealth[MAX_PLAYERS]; 


Why not group all of that into an enum to make the code look more decent if you're going to use so many player variables?

And the names don't make sense one bit.

Код:

#define accounts "accounts"

Creativity at it's finest lmao.


The code also isn't fully indented and you're leaving unused functions around which is very lazy and annoying.

The whole use of this:

Код:

 dini_Int(FileDirectory(accounts, playername(playerid)), "maxhealth");

Made my eyes bleed a bit, why not create a global variable for the file, open the file then use the variables for gods sake this is destroying the readability even more.

Actually i'm starting to think this is a compilation of shit copy-pasta went wrong, the use of

Код:

phealth[playerid]

...the variable doesn't make any sense at all, scroll through your own code you'll shit yourself and give yourself a few kicks maybe if you even wrote it.

So you created a function to grab a player's name, then you do this:

Код:

                 new pname[MAX_PLAYER_NAME];
                 GetPlayerName(playerid, pname, MAX_PLAYER_NAME);

*facepalm*


And this thing:

PHP код:

                 SendClientMessage(playerid, COLOR_ORANGE, "You Are Now Immune To Almost Everything in this world");
                 SetPlayerHealth(playerid, 99999); 


Sorry son you're not immune to everything in the world, actually you're not immune to anything lol that's not how 'god mode' works at all.

And these arrays in the variables:

Код:

new perkcd[MAX_PLAYERS][10];
new perks[MAX_PLAYERS][10][5];

Don't make any sense either, you didn't even use them all and i'm not sure why it was needed in the first place.

This at OnDialogResponse:

Код:

       return 0;
    }
    return 1;
}

You're supposed to return 0 at the end of OnDialogResponse to pass it on to the next script and not limit it to this certain one.

Congrats on your spaghetti code keep bumping it more maybe it'll get better.
I didn't really want to get all pissy but I don't understand why bump a release if you're not announcing an update? Doesn't make any sense.

*Looks at Pottus* Not decent enough to be a release, 1-star from me. ^-^