1. SA-MP Forums Archive
  2. SA-MP Scripting and Plugins
  3. Scripting Help

Threaded Mode
Creates the last object only
vassilis
High-roller
*****
Posts: 1,679
Threads: 232
Joined: Nov 2009
Reputation: 0
#1

28.10.2016, 10:55
( Последний раз редактировалось vassilis; 28.10.2016 в 11:35. )
So i have a loading system that works correctly(at MySQL section)
At Creating the object section though i have stuck. How would be possible to create all the objects the database is loading?

It creates only the last one if even i have one. What the correct loop would be?
PHP код:
                function LoadSprayTags(playerid)
                {
                
                    new rows, fields, Index = tagid[playerid];
                    cache_get_data(rows, fields, mysql);
                    for(new i = 0; i < rows; i++)
                    {
                        cache_get_field_content(i, "TagName", TagInfo[playerid][Index][TagName], mysql,25);
                        cache_get_field_content(i, "TagOwner", TagInfo[playerid][Index][TagOwner], mysql, 25);
                        cache_get_field_content(i, "TagFont", TagInfo[playerid][Index][TagFont], mysql, 20);
                        TagInfo[playerid][Index][TagX] = cache_get_field_content_float(i, "TagX");
                        TagInfo[playerid][Index][TagY] = cache_get_field_content_float(i, "TagY");
                        TagInfo[playerid][Index][TagZ] = cache_get_field_content_float(i, "TagZ");
                        TagInfo[playerid][Index][TagRX] = cache_get_field_content_float(i, "TagRX");
                        TagInfo[playerid][Index][TagRY] = cache_get_field_content_float(i, "TagRY");
                        TagInfo[playerid][Index][TagRZ] = cache_get_field_content_float(i, "TagRZ");
                        TagInfo[playerid][Index][TagFontSize] = cache_get_field_content_int(i, "TagFontSize");
                        TagInfo[playerid][Index][TagFontColor] = cache_get_field_content_int(i, "TagFontColor");
                        TagInfo[playerid][Index][TagBold] = cache_get_field_content_int(i, "TagBold");
                        
                        TagInfo[playerid][Index][TagObject] = CreateDynamicObject(19482, TagInfo[playerid][Index][TagX], TagInfo[playerid][Index][TagY], TagInfo[playerid][Index][TagZ], TagInfo[playerid][Index][TagRX], TagInfo[playerid][Index][TagRY], TagInfo[playerid][Index][TagRZ], -1, -1, -1, 300.0, 300.0);
                        SetDynamicObjectMaterialText(TagInfo[playerid][Index][TagObject], 0, TagInfo[playerid][Index][TagName], 70, TagInfo[playerid][Index][TagFont], TagInfo[playerid][Index][TagFontSize], 1, TagInfo[playerid][Index][TagFontColor], 0, 0);
                            
                        new str[90];
                        
                        format(str, sizeof(str)," "COL_LIGHTBLUE"Graffiti Owner:"COL_WHITE" %s \n"COL_LIGHTBLUE"Tag ID:"COL_WHITE" %i", TagInfo[playerid][Index][TagOwner], TagInfo[playerid][Index][TagID]);
                        TagInfo[playerid][Index][TagAdminInfo] = CreateDynamic3DTextLabel(str, COLOR_YELLOW, TagInfo[playerid][Index][TagX], TagInfo[playerid][Index][TagY], TagInfo[playerid][Index][TagZ], 100.0);
                            
                        printf("Name:%s Owner:%s \n Font:%s X:%f Y:%f Z:%f \n RX:%f RY:%f RZ:%f \nFontSize:%d FontColor:%d Bold: %d", TagInfo[playerid][Index][TagName], TagInfo[playerid][Index][TagOwner], TagInfo[playerid][Index][TagFont],
                        TagInfo[playerid][Index][TagX], TagInfo[playerid][Index][TagY], TagInfo[playerid][Index][TagZ], TagInfo[playerid][Index][TagRX], TagInfo[playerid][Index][TagRY],TagInfo[playerid][Index][TagRZ],
                        TagInfo[playerid][Index][TagFontSize], TagInfo[playerid][Index][TagFontColor],TagInfo[playerid][Index][TagBold]);
                    }
                    
                    return 1;
                } 
Find
Reply
iLearner
Unregistered
 
#2

28.10.2016, 10:58
As I see you're assigning all vars to taginfo, and this way you replace the next one with previous, ending up with having last xyz's.

Just a wild guess.
Reply
AndySedeyn
High-roller
*****
Posts: 1,206
Threads: 39
Joined: Feb 2014
Reputation: 0
#3

28.10.2016, 11:07
You implemented that additional array dimension as I advised you to, but you're incorrectly using it. The point of it is to put all your database information in there up till the maximum allowed tags per player (which is defined as MAX_TAGS_PER_PLAYER, right?):

PHP код:
// Get rid of the 'index' variable
for(new i, j = cache_get_row_count(); i < j && i < MAX_TAGS_PER_PLAYER; i++) {
    // Use 'i' to address the second array dimension
} 
The loop goes through all rows returned by the query only if i remains smaller than MAX_TAGS_PER_PLAYER (0 to 2 = 3 tags).
Find
Reply
vassilis
High-roller
*****
Posts: 1,679
Threads: 232
Joined: Nov 2009
Reputation: 0
#4

28.10.2016, 11:35
Quote:
Originally Posted by AndySedeyn
Посмотреть сообщение
PHP код:
// Get rid of the 'index' variable
for(new i, j = cache_get_row_count(); i < j && i < MAX_TAGS_PER_PLAYER; i++) {
    // Use 'i' to address the second array dimension
} 
So like that?
PHP код:
                function LoadSprayTags(playerid)
                {
                
                    new Index = tagid[playerid];
                    cache_get_data(rows, fields, mysql);
                    for(new i, j = cache_get_row_count(); i < j && i < MAX_TAGS_PER_PLAYER; i++) 
                    { 
                        cache_get_field_content(i, "TagName", TagInfo[playerid][Index][TagName], mysql,25);
                        cache_get_field_content(i, "TagOwner", TagInfo[playerid][Index][TagOwner], mysql, 25);
                        cache_get_field_content(i, "TagFont", TagInfo[playerid][Index][TagFont], mysql, 20);
                        TagInfo[playerid][Index][TagX] = cache_get_field_content_float(i, "TagX");
                        TagInfo[playerid][Index][TagY] = cache_get_field_content_float(i, "TagY");
                        TagInfo[playerid][Index][TagZ] = cache_get_field_content_float(i, "TagZ");
                        TagInfo[playerid][Index][TagRX] = cache_get_field_content_float(i, "TagRX");
                        TagInfo[playerid][Index][TagRY] = cache_get_field_content_float(i, "TagRY");
                        TagInfo[playerid][Index][TagRZ] = cache_get_field_content_float(i, "TagRZ");
                        TagInfo[playerid][Index][TagFontSize] = cache_get_field_content_int(i, "TagFontSize");
                        TagInfo[playerid][Index][TagFontColor] = cache_get_field_content_int(i, "TagFontColor");
                        TagInfo[playerid][Index][TagBold] = cache_get_field_content_int(i, "TagBold");
                        
                        TagInfo[playerid][i][TagObject] = CreateDynamicObject(19482, TagInfo[playerid][i][TagX], TagInfo[playerid][i][TagY], TagInfo[playerid][i][TagZ], TagInfo[playerid][i][TagRX], TagInfo[playerid][i][TagRY], TagInfo[playerid][i][TagRZ], -1, -1, -1, 300.0, 300.0);
                        SetDynamicObjectMaterialText(TagInfo[playerid][i][TagObject], 0, TagInfo[playerid][i][TagName], 70, TagInfo[playerid][i][TagFont], TagInfo[playerid][i][TagFontSize], 1, TagInfo[playerid][i][TagFontColor], 0, 0);
                            
                        new str[90];
                        
                        format(str, sizeof(str)," "COL_LIGHTBLUE"Graffiti Owner:"COL_WHITE" %s \n"COL_LIGHTBLUE"Tag ID:"COL_WHITE" %i", TagInfo[playerid][i][TagOwner], TagInfo[playerid][i][TagID]);
                        TagInfo[playerid][i][TagAdminInfo] = CreateDynamic3DTextLabel(str, COLOR_YELLOW, TagInfo[playerid][i][TagX], TagInfo[playerid][i][TagY], TagInfo[playerid][i][TagZ], 100.0);
                            
                        printf("Name:%s Owner:%s \n Font:%s X:%f Y:%f Z:%f \n RX:%f RY:%f RZ:%f \nFontSize:%d FontColor:%d Bold: %d", TagInfo[playerid][i][TagName], TagInfo[playerid][i][TagOwner], TagInfo[playerid][i][TagFont],
                        TagInfo[playerid][i][TagX], TagInfo[playerid][i][TagY], TagInfo[playerid][i][TagZ], TagInfo[playerid][i][TagRX], TagInfo[playerid][i][TagRY],TagInfo[playerid][i][TagRZ],
                        TagInfo[playerid][i][TagFontSize], TagInfo[playerid][i][TagFontColor],TagInfo[playerid][i][TagBold]);
                    }  
                    return 1;
                } 
Find
Reply
AndySedeyn
High-roller
*****
Posts: 1,206
Threads: 39
Joined: Feb 2014
Reputation: 0
#5

28.10.2016, 11:39
No. You're still using new Index = tagid[playerid];. Use the loop variable i to go through the indices of the second dimension.

Secondly, by using cache_get_row_count(), cache_get_data becomes redundant.
Find
Reply
vassilis
High-roller
*****
Posts: 1,679
Threads: 232
Joined: Nov 2009
Reputation: 0
#6

28.10.2016, 12:04
Quote:
Originally Posted by AndySedeyn
Посмотреть сообщение
No. You're still using new Index = tagid[playerid];. Use the loop variable i to go through the indices of the second dimension.

Secondly, by using cache_get_row_count(), cache_get_data becomes redundant.
Works thank you Andy!!
Find
Reply
« Next Oldest | Next Newest »


Forum Jump:


Users browsing this thread: 1 Guest(s)
  1. SA-MP Forums Archive
  2. SA-MP Scripting and Plugins
  3. Scripting Help