── ( Promote | Content ) ── / 【💻】ᴘʟᴜɢɪɴꜱ
This channel is designed for any plugin devs to release the "plugins" that is on ArkServerAPI website.
In the end, any 
for loop is turned into a while loop at compile time anyway, so using for loops is actually less efficient even if that's negligible). if (loadedClasses.size() > 0)
{
auto it = loadedClasses.begin();
auto end = loadedClasses.end();
while (it != end)
{
if ((*it).first.Compare(*blueprintPath) == 0)
return (*it).second;
it++;
}
}
any reason to store the iterator and end as locals? 
lastRunCommandTime map is to limit the number of commands that a user can perform in a given time interval (FILLCROPS_INTERVAL_DELAY). When a user performs the fill crops command it will store his Steam ID and the current timestamp in that map. Then if the same user tries to do the same command again, the code will check if the time difference between current request and previous request is superior to the given time interval. Basically it does "if (current time - stored time > FILLCROPS_INTERVAL_DELAY)". So if the time interval has not been reached yet the command will be rejected.
This is not mandatory to do all that (but it's good for performance and security to limit the amount of commands that can be performed in a given time). 
1c++
static std::map<unsigned long long, long long> lastRunCommandTime = std::map<unsigned long long, long long>(); new keyword or other memory allocation functions unless it's mandatory. Instead try to declare all your variables with fixed sizes so that they are allocated on the stack. That way you will not have to free the resources with the delete keyword and you will avoid memory leakage. Try to use UE containers (TArray, TMap, ...) or STD containers (std::list, std::map) when possible because they are efficient and memory/thread safe. See here for details on TArray and how to use them: https://docs.unrealengine.com/4.27/en-US/ProgrammingAndScripting/ProgrammingWithCPP/UnrealArchitecture/TArrays/
1new keyword or other memory allocation functions unless it's mandatory. Instead try to declare all your variables with fixed sizes so that they are allocated on the stack. That way you will not have to free the resources with the delete keyword and you will avoid memory leakage. Try to use UE containers (TArray, TMap, ...) or STD containers (std::list, std::map) when possible because they are efficient and memory/thread safe. See here for details on TArray and how to use them: https://docs.unrealengine.com/4.27/en-US/ProgrammingAndScripting/ProgrammingWithCPP/UnrealArchitecture/TArrays/ new keyword or other memory allocation functions unless it's mandatory. Instead try to declare all your variables with fixed sizes so that they are allocated on the stack. That way you will not have to free the resources with the delete keyword and you will avoid memory leakage. Try to use UE containers (TArray, TMap, ...) or STD containers (std::list, std::map) when possible because they are efficient and memory/thread safe. See here for details on TArray and how to use them: https://docs.unrealengine.com/4.27/en-US/ProgrammingAndScripting/ProgrammingWithCPP/UnrealArchitecture/TArrays/ 
