[CloudLEL]

Hi,

I was wondering, is it possible to count how many houses their are without using mysql.

I've been doing it by getting the row count from the database, but I feel like their must be an easier way to do it.

Thanks.

[iKarim]

why don't you just use row count? it is harder to count an array. you'll have to loop through all elements until it reaches an unused cell.

[CloudLEL]

If that's what you think is best, I'll continue doing so.

[DRIFT_HUNTER]

Quote:

Originally Posted by PawnHunter why don't you just use row count? it is harder to count an array. you'll have to loop through all elements until it reaches an unused cell.

And mysql has to send request, wait for mysql server to select all rows, count them and than return result. Looping thru array is 1000x times faster (literally).


Just loop thru your houses and skip unused cells/rows. My simple trick is to check if houses entrance Z is 0 or if label/icon is invalid. Simple example:

pawn Код:

stock CountHouses()
{
    new count = 0;
    for(new i = 0; i < MAX_HOUSES; i++)
    {
        if(HouseInfo[i][EntZ] != 0.0) count++;
    }
    return count;
}

While EntZ should be float and comparing floats is discouraged feel free to do it in pawn.

[Konstantinos]

Quote:

Originally Posted by DRIFT_HUNTER Just loop thru your houses and skip unused cells/rows.

Storing the number of houses loaded from the database when the server starts in a global variable and increasing/decreasing by 1 when a house is created/deleted respectively is the best way.

[DRIFT_HUNTER]

Quote:

Originally Posted by Konstantinos Storing the number of houses loaded from the database when the server starts in a global variable and increasing/decreasing by 1 when a house is created/deleted respectively is the best way.

Or that

[Vince]

If the only purpose of a query is to get a count of something then the COUNT() aggregate function should be used. Sending over a single value is obviously much faster than sending over an entire result set which isn't going to be used.

[AlexBlack]

Or more simple, use foreach & y_iterator.