Вопрос по отправке объявлений у NEWS

[Dima_Turkow]

Всем доброго времени суток, уважаемые посетители портала PRO-PAWN.RU !

Опишу суть проблемы : есть организация новостей и система редактирования объявлений.
При отправке объявления неким человеком, его должен отредактировать сотрудник NEWS и пустить в эфир.

Собственно все идеально отправляется и редактируется за исключением одного, номер телефона всегда пишется : 0.

  Открыть/закрыть

static const fmt_str_50[] = "Объявление: %s. Прислал: %s. Тел: %d";new string_50[sizeof(fmt_str_50)+MAX_PLAYER_NAME+94];format(string_50, sizeof(string_50), fmt_str_50,AdvertInfo[GetPVarInt(playerid, "AdvertID")][0][adText],AdvertInfo[GetPVarInt(playerid, "AdvertID")][0][adName],AdvertInfo[GetPVarInt(playerid, "AdvertID")][0][adPhone]);SCMTA(0x00D900C8, string_50); static const fmt_str_51[] = " Объявление опубликовал сотрудник LS News %s";new string_51[sizeof(fmt_str_51)+MAX_PLAYER_NAME-1];format(string_51, sizeof(string_51), fmt_str_51,GN(playerid));SCMTA(0x7787D8FF, string_51); for(new i = GetPVarInt(playerid, "AdvertID"); i <= TOTALADVERT[0] - 1; i++){strmid(AdvertInfo[i][0][adText], AdvertInfo[i+1][0][adText],0, strlen(AdvertInfo[i+1][0][adText]), 90);strmid(AdvertInfo[i][0][adName], AdvertInfo[i+1][0][adName],0, strlen(AdvertInfo[i+1][0][adName]), MAX_PLAYER_NAME);player[i][pnumber] = AdvertInfo[i][0][adPhone];}TOTALADVERT[0]--;AdvertTime = gettime() + 15;DeletePVar(playerid, "AdvertID");

Прошу помочь знающим людям в исправлении данного косяка .
P.S в - player[playerid][pnumber] хранится номер телефона игрока

[SteveStage]

static const fmt_str_50[] = "Объявление: %s. Прислал: %s. Тел: %d";new string_50[sizeof(fmt_str_50)+MAX_PLAYER_NAME+94];format(string_50, sizeof(string_50), fmt_str_50,AdvertInfo[GetPVarInt(playerid, "AdvertID")][0][adText],AdvertInfo[GetPVarInt(playerid, "AdvertID")][0][adName],player[GetPVarInt(playerid, "AdvertID")][pnumber]);SCMTA(0x00D900C8, string_50); static const fmt_str_51[] = " Объявление опубликовал сотрудник LS News %s";new string_51[sizeof(fmt_str_51)+MAX_PLAYER_NAME-1];format(string_51, sizeof(string_51), fmt_str_51,GN(playerid));SCMTA(0x7787D8FF, string_51); for(new i = GetPVarInt(playerid, "AdvertID"); i <= TOTALADVERT[0] - 1; i++){strmid(AdvertInfo[i][0][adText], AdvertInfo[i+1][0][adText],0, strlen(AdvertInfo[i+1][0][adText]), 90);strmid(AdvertInfo[i][0][adName], AdvertInfo[i+1][0][adName],0, strlen(AdvertInfo[i+1][0][adName]), MAX_PLAYER_NAME);player[i][pnumber] = AdvertInfo[i][0][adPhone];}TOTALADVERT[0]--;AdvertTime = gettime() + 15;DeletePVar(playerid, "AdvertID");

Попробуй так. Видимо в переменную AdvertInfo[playerid][0][adPhone] у тебя не записан номер телефона.

Хотя я бы сделал по другому:

new str_50[128+1];format(str_50, sizeof(str_50), "Объявление: %s. Прислал: %s. Тел: %d",AdvertInfo[GetPVarInt(playerid, "AdvertID")][0][adText],GN(GetPVarInt(playerid, "AdvertID")),player[GetPVarInt(playerid, "AdvertID")][pnumber]);SCMTA(0x00D900C8, str_50); format(str_50, sizeof(str_50), " Объявление опубликовал сотрудник LS News %s",GN(playerid));SCMTA(0x7787D8FF, str_50); for(new i = GetPVarInt(playerid, "AdvertID"); i <= TOTALADVERT[0] - 1; i++){strmid(AdvertInfo[i][0][adText], AdvertInfo[i+1][0][adText],0, strlen(AdvertInfo[i+1][0][adText]), 90);}TOTALADVERT[0]--;AdvertTime = gettime() + 15;DeletePVar(playerid, "AdvertID");

Вариант с static и new более выгоден, когда форматируется одна строка, а в таких случаях, как у тебя (форматируется 2 и более строки) выгоднее создавать один массив и форматировать его.

И еще что я заметил - у тебя в pnumber записывается эта пустая переменная adPhone.

Да по сути AdvertInfo должен хранить один лишь текст объявления, а ник и номер можно получить без него.

[Dima_Turkow]

Замечательно, волшебно, чудесно. Жаль - оба варианта не работают, все равно : "Тел: 0"

[execution]

Ты бы хоть код в читабельный вид привёл и разделил разные действия блоками. Сделай отдельно подачу и отправку.

[SteveStage]

Замечательно, волшебно, чудесно. Жаль - оба варианта не работают, все равно : "Тел: 0"

Покажи загрузку телефона из бд в pnumber.

[Dima_Turkow]

Если я в формат засуну, к примеру player[playerid][pnumber] - то номер телефона выведется, только выведется того кто редактирует, а не того кто прислал объявление

- - - Добавлено - - -

player[playerid][pnumber] = cache_get_field_content_int(0, "Number");

[SteveStage]

Если я в формат засуну, к примеру player[playerid][pnumber] - то номер телефона выведется, только выведется того кто редактирует, а не того кто прислал объявление

Ну так ты введи не playerid, а pvar, хранящий id того, кто прислал объявление.

[Dima_Turkow]

Каким образом, продемонстрируй пожалуйста ;)

[SteveStage]

format(string_50, sizeof(string_50), fmt_str_50,AdvertInfo[GetPVarInt(playerid, "AdvertID")][0][adText],AdvertInfo[GetPVarInt(playerid, "AdvertID")][0][adName],AdvertInfo[GetPVarInt(playerid, "AdvertID")][0][adPhone]);

Вот, pvar AdvertID по видимому хранит id отправителя объявления.

[Dima_Turkow]

static const fmt_str_50[] = "Объявление: %s. Прислал: %s. Тел: %d";new string_50[sizeof(fmt_str_50)+MAX_PLAYER_NAME+94];format(string_50, sizeof(string_50), fmt_str_50,AdvertInfo[GetPVarInt(playerid, "AdvertID")][0][adText],AdvertInfo[GetPVarInt(playerid, "AdvertID")][0][adName],player[GetPVarInt(playerid, "AdvertID")][pnumber]);SCMTA(0x00D900C8, string_50); static const fmt_str_51[] = " Объявление опубликовал сотрудник LS News %s";new string_51[sizeof(fmt_str_51)+MAX_PLAYER_NAME-1];format(string_51, sizeof(string_51), fmt_str_51,GN(playerid));SCMTA(0x7787D8FF, string_51); for(new i = GetPVarInt(playerid, "AdvertID"); i <= TOTALADVERT[0] - 1; i++){strmid(AdvertInfo[i][0][adText], AdvertInfo[i+1][0][adText],0, strlen(AdvertInfo[i+1][0][adText]), 90);strmid(AdvertInfo[i][0][adName], AdvertInfo[i+1][0][adName],0, strlen(AdvertInfo[i+1][0][adName]), MAX_PLAYER_NAME);player[i][pnumber] = AdvertInfo[i][0][adPhone];}TOTALADVERT[0]--;AdvertTime = gettime() + 15;DeletePVar(playerid, "AdvertID");

Попробуй так. Видимо в переменную AdvertInfo[playerid][0][adPhone] у тебя не записан номер телефона.

Хотя я бы сделал по другому:

new str_50[128+1];format(str_50, sizeof(str_50), "Объявление: %s. Прислал: %s. Тел: %d",AdvertInfo[GetPVarInt(playerid, "AdvertID")][0][adText],GN(GetPVarInt(playerid, "AdvertID")),player[GetPVarInt(playerid, "AdvertID")][pnumber]);SCMTA(0x00D900C8, str_50); format(str_50, sizeof(str_50), " Объявление опубликовал сотрудник LS News %s",GN(playerid));SCMTA(0x7787D8FF, str_50); for(new i = GetPVarInt(playerid, "AdvertID"); i <= TOTALADVERT[0] - 1; i++){strmid(AdvertInfo[i][0][adText], AdvertInfo[i+1][0][adText],0, strlen(AdvertInfo[i+1][0][adText]), 90);}TOTALADVERT[0]--;AdvertTime = gettime() + 15;DeletePVar(playerid, "AdvertID");

Вариант с static и new более выгоден, когда форматируется одна строка, а в таких случаях, как у тебя (форматируется 2 и более строки) выгоднее создавать один массив и форматировать его.

И еще что я заметил - у тебя в pnumber записывается эта пустая переменная adPhone.

Да по сути AdvertInfo должен хранить один лишь текст объявления, а ник и номер можно получить без него.

Я сейчас сделал точно так, как ты описал второй вариант. Итог :Объявление отобразилось, Прислал : пустое место вывелось, Тел: 0

Я так понимаю проблема с PVAR ?