It is not, he creates a 32 cell array 4 times even though it's not really needed, since as it's already been said, IP cannot exceed 15 characters. Also, I don't see a reason why does he even create the array 4x if he uses only 3 fields of it. Just do it like this
PHP Code:
#include < amxmodx >
#include < amxmisc >
#define PLUGIN "justJoined"
#define VERSION "0.1"
public plugin_init()
register_plugin(PLUGIN, VERSION, "Blend")
public client_putinserver(id)
{
new szIp[ 15 ], szAuthid[ 34 ], szName[ 33 ]
get_user_ip( id, szIp, charsmax( szIp ), true )
get_user_authid( id, szAuthid, charsmax( szAuthid ) )
get_user_name( id, szName, charsmax( szName ) )
server_print("] Name: %s [%s]^n] SteamID: %s^n] IP: %s", szName, is_user_admin(id) ? "ADMIN" : "USER", szAuthid, szIp);
}
This is pretty much the best way of doing what you want, there's nothing more to optimize.