[Pottus]

Code:

#define dcmd(%1,%2,%3) if (!strcmp((%3)[1], #%1, true, (%2)) && ((((%3)[(%2) + 1] == '\0') && (dcmd_%1(playerid, ""))) || (((%3)[(%2) + 1] == ' ') && (dcmd_%1(playerid, (%3)[(%2) + 2]))))) return 1

Come on dude use zcmd for commands.

Code:

forward ATM_TimeUpdater(); public ATM_TimeUpdater()
{
	new
		uString[156];
	for (new i = false; i <= MAX_ATM; i++){
	if(ATMInfo[i][aTime] > 0){
	ATMInfo[i][aTime]--;
	if(ATMInfo[i][aTime] <= 1){
	format(uString,156,"{ffffff}%s(%i)\n{ffffff}State: {77c97d}Active", ATMInfo[i][aName], atm_class);
	Update3DTextLabelText(ATMInfo[i][aLabel], -1, uString);
	ATMInfo[i][aDurum] = true;
	ATMInfo[i][aTime] = 0;
	}
	}
	}
	return true;
}

Not checking if an ATM is actually created.

Code:

SetPVarInt(playerid, "ATMcalinanMiktar", (random(MAX_ATM_PARA))+MIN_ATM_PARA);

You are using regular variables but mix in pvars for no reason use regular variables.

Code:

stock CreateATM(atmname[], Float:aaaX, Float:aaaY, Float:aaaZ, Float:aaaA){
atm_class++;

Code:

stock DestroyATM()
atm_class = 1;

Do I even need to explain the conniptions this will cause in your system.