summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorFlavioJS <FlavioJS@54d463be-8e91-2dee-dedb-b68131a5f0ec>2009-01-13 12:53:11 +0000
committerFlavioJS <FlavioJS@54d463be-8e91-2dee-dedb-b68131a5f0ec>2009-01-13 12:53:11 +0000
commit3b1dd44c88484af3585bcfea0e94994330383019 (patch)
tree00e0c8cee3d8329079e6c5e93b6337c6567ac28a /src
parent9d1b39d3facc6d9f1268e036e7b23bd410c683ec (diff)
downloadhercules-3b1dd44c88484af3585bcfea0e94994330383019.tar.gz
hercules-3b1dd44c88484af3585bcfea0e94994330383019.tar.bz2
hercules-3b1dd44c88484af3585bcfea0e94994330383019.tar.xz
hercules-3b1dd44c88484af3585bcfea0e94994330383019.zip
* Cleaned up command @users/#users:
- displays everything on the target user (self for @users) - uses a static array instead of a temporary DBMap - displays percentages with precision of 0.01% - uses safesnprintf instead of sprintf git-svn-id: https://rathena.svn.sourceforge.net/svnroot/rathena/trunk@13446 54d463be-8e91-2dee-dedb-b68131a5f0ec
Diffstat (limited to 'src')
-rw-r--r--src/common/mapindex.c2
-rw-r--r--src/common/mapindex.h2
-rw-r--r--src/map/atcommand.c53
3 files changed, 26 insertions, 31 deletions
diff --git a/src/common/mapindex.c b/src/common/mapindex.c
index b76a838be..4aef69574 100644
--- a/src/common/mapindex.c
+++ b/src/common/mapindex.c
@@ -11,8 +11,6 @@
#include <stdio.h>
#include <stdlib.h>
-#define MAX_MAPINDEX 2000
-
struct _indexes {
char name[MAP_NAME_LENGTH]; //Stores map name
} indexes[MAX_MAPINDEX];
diff --git a/src/common/mapindex.h b/src/common/mapindex.h
index 3a9ff3537..cd4c09d37 100644
--- a/src/common/mapindex.h
+++ b/src/common/mapindex.h
@@ -10,6 +10,8 @@ extern char mapindex_cfgfile[80];
//whether to enable auto-adding of maps during run. Not so secure as the map indexes will vary!
//#define MAPINDEX_AUTOADD
+#define MAX_MAPINDEX 2000
+
//Some definitions for the mayor city maps.
#define MAP_PRONTERA "prontera"
#define MAP_GEFFEN "geffen"
diff --git a/src/map/atcommand.c b/src/map/atcommand.c
index 4fa21908b..15364ec82 100644
--- a/src/map/atcommand.c
+++ b/src/map/atcommand.c
@@ -6463,53 +6463,48 @@ int atcommand_pettalk(const int fd, struct map_session_data* sd, const char* com
return 0;
}
-/*==========================================
- * @users - displays the number of players present on each map (percentage)
- *------------------------------------------*/
+/// @users - displays the number of players present on each map (and percentage)
+/// #users displays on the target user instead of self
int atcommand_users(const int fd, struct map_session_data* sd, const char* command, const char* message)
{
char buf[256];
- DBMap* users_db; // unsigned int mapindex -> int users
+ int i;
+ int users[MAX_MAPINDEX];
int users_all;
+ struct s_mapiterator* iter;
- users_db = uidb_alloc(DB_OPT_BASE);
+ memset(users, 0, sizeof(users));
users_all = 0;
// count users on each map
+ iter = mapit_getallusers();
+ while( true )
{
- struct s_mapiterator* iter;
- struct map_session_data* sd;
+ struct map_session_data* sd2 = (struct map_session_data*)mapit_next(iter);
+ if( sd2 == NULL )
+ break;// no more users
- iter = mapit_getallusers();
- for( sd = (struct map_session_data*)mapit_first(iter); mapit_exists(iter); sd = (struct map_session_data*)mapit_next(iter) )
- {
- int users = (int)uidb_get(users_db,sd->mapindex) + 1;
- uidb_put(users_db,(unsigned int)sd->mapindex,(void *)users);
- users_all++;
- }
- mapit_free(iter);
+ if( sd2->mapindex >= MAX_MAPINDEX )
+ continue;// invalid mapindex
+
+ if( users[sd2->mapindex] < INT_MAX ) ++users[sd2->mapindex];
+ if( users_all < INT_MAX ) ++users_all;
}
+ mapit_free(iter);
// display results for each map
+ for( i = 0; i < MAX_MAPINDEX; ++i )
{
- DBIterator* iter;
- DBKey index;
- int users;
+ if( users[i] == 0 )
+ continue;// empty
- iter = users_db->iterator(users_db);
- for( users = (int)iter->first(iter,&index); iter->exists(iter); users = (int)iter->next(iter,&index) )
- {
- sprintf(buf,"%s: %d (%d%%)",mapindex_id2name(index.i),users,users * 100 / users_all);
- clif_displaymessage(sd->fd,buf);
- }
- iter->destroy(iter);
+ safesnprintf(buf, sizeof(buf), "%s: %d (%.2f%%)", mapindex_id2name(i), users[i], (float)(100.0f*users[i]/users_all));
+ clif_displaymessage(sd->fd, buf);
}
// display overall count
- sprintf(buf,"all: %d",users_all);
- clif_displaymessage(fd,buf);
-
- users_db->destroy(users_db,NULL);
+ safesnprintf(buf, sizeof(buf), "all: %d", users_all);
+ clif_displaymessage(sd->fd, buf);
return 0;
}