error 033: array must be indexed (variable "-unknown-")

Очень, очень сложна

Код:

---

format(string, sizeof(string), "| Этаж № %i | %s | Цена: %i$ |\n", i++, (iApartment[i][0][pBought]) ? ("продается") : (iApartment[i][0][pNamePlayer]), iApartment[i][0][pMoney]);

---

Код:

---

error 033: array must be indexed (variable "-unknown-")

---

как было выявлено из тестов ругается на ("продается"), но почему?

если даже слово продается записать в массив и в формат написать этот массив то будет ошибка

Код:

---

error 033: array must be indexed (variable "-имя массива-")

---

сложна короче

Это ошибка компилятора. Не пиши так или используй компилятор от Zeex.

А я его компилятор и использую, стандартный так вообще вылетает

Покажи ка массив iApartment.

лишь бы что то написать

Код:

---


#define MAX_APARTMENT                        2
#define MAX_APARTMENT_PLAYER        200

enum Apartment
{
        pNamePlayer[MAX_PLAYER_NAME],
        pID,
        pIDInterior,
        pStatus,
        pMoney,
        pBack,
        pBought
};
new iApartment[MAX_APARTMENT_PLAYER][MAX_APARTMENT][Apartment];

---

Версия компилятора?

Pawn compiler 3.10.20160907 Copyright (c) 1997-2006, ITB CompuPhase

Зная костыльщину павно компилятора я бы для начала сделал проверку потом занес в переменную нужную строку, а только потом делал бы формат)

qwezert, ты об этом?

PHP код:

---

string = (iApartment[i][0][pBought]) ? ("Продается") : (iApartment[i][0][pNamePlayer]);
format(string, sizeof(string), "| Этаж № %i | %s | Цена: %i$ |\n", i++, string, iApartment[i][0][pMoney]); 


---

Цитата:

---

Сообщение от KrutoyKrosch

Код:

---

enum Apartment
{
        pNamePlayer[MAX_PLAYER_NAME],
        pID,
        pIDInterior,
        pStatus,
        pMoney,
        pBack,
        pBought
};

---

---

Придумай какую-то другую приставку для констант. pMoney скорей всего встречается и в pInfo.

Решил копнуть поглубже, упростил пример:

PHP код:

---

#include <a_samp>

main()
{
    static const str[] = "Sample text";
    printf("%s", str[0]);                        // "Sample text"
    printf("%s", random(1) ? str[0] : str[0]);    // "S"
} 


---

Как видно из примера, в первом вызове printf передаётся адрес строки, но во втором из-за тернарного выражения вместо адреса передаётся содержимое ячейки. Скорее всего, по этой же причине у ТС и возникла ошибка, т.к. компилятор ожидал на месте "Продается" не строку, а отдельную ячейку (т.е. число, символ, etc.)
Полагаю, стоит сообщить об этом Zeex.

конечно стоит, я так уже третий день какую то систему квартирок сделать не могу из за этой ошибки

Цитата:

---

Сообщение от Daniel_Cortez

Решил копнуть поглубже, упростил пример:

PHP код:

---

#include <a_samp>

main()
{
    static const str[] = "Sample text";
    printf("%s", str[0]);                        // "Sample text"
    printf("%s", random(1) ? str[0] : str[0]);    // "S"
} 


---

Как видно из примера, в первом вызове printf передаётся адрес строки, но во втором из-за тернарного выражения вместо адреса передаётся содержимое ячейки. Скорее всего, по этой же причине у ТС и возникла ошибка, т.к. компилятор ожидал на месте "Продается" не строку, а отдельную ячейку (т.е. число, символ, etc.)
Полагаю, стоит сообщить об этом Zeex.

---

Я поговнокодил и нашёл своеобразное "решение" сей проблемы:

PHP код:

---

enum fix_fix_fix
{
    _fix[MAX_PLAYER_NAME]
}

new this_is_fix[fix_fix_fix] =
{
    "продается"
};

format(string, sizeof(string), "| Этаж № %i | %s | Цена: %i$ |\n", i++, (iApartment[i][0][pBought]) ? (this_is_fix[_fix]) : (iApartment[i][0][pNamePlayer]), iApartment[i][0][pMoney]); 


---

То бишь, если в обоих случаях использовать перечисление, в котором объявить массив - всё будет норм компилироваться.

Это не решение проблемы автора (я не призываю его использовать данный метод), а просто пища для размышления.

бирем значит такие суем в iApartment слово продается и се

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

или по стандарту в NamePlayer записывать продается :lol:

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

так теперь еще вопросик

лучше один глобальный string или много локальных?

Цитата:

---

Сообщение от KrutoyKrosch

бирем значит такие суем в iApartment слово продается и се

---

Если брать пример DeimoS`a, то не нужно ничего пихать в энуминатор iApartment.

PHP код:

---

new for_sale[] = "продается";
format(string, sizeof(string)
,
    "| Этаж № %i | %s | Цена: %i$ |\n"
,
    i++,
    (iApartment[i][0][pBought]) ? (for_sale) : (iApartment[i][0][pNamePlayer]),
    iApartment[i][0][pMoney]
); 


---

Цитата:

---

Сообщение от KrutoyKrosch

лучше один глобальный string или много локальных?

---

Лучше использовать локальные, и подсчитывать количество символов нежели создавать один с 1+к ячеек.

зачем 1к ячеек?

Цитата:

---

Сообщение от KrutoyKrosch

зачем 1к ячеек?

---

Обычно те кто создают глобальные массивы для строк, так и делают.
Или Вы хотите под каждою строку создавать глобальный массив и подсчитывать кол-во символов?
Лучше использовать локальные переменные с массивом для строк, также подсчет символов.
Как известно, локальные переменные после завершения функция очищаются из памяти.

подсчет символов это геморрой только лишний

и ты мне так и не привел ни один пруф почему нельзя создать глобальный стринг

Цитата:

---

Сообщение от KrutoyKrosch

и ты мне так и не привел ни один пруф почему нельзя создать глобальный стринг

---

Ну давайте начнем с того, что я написал уже выше.
Что локальные переменные очищаются из памяти, а глобальные будет инициализированные постоянным выражением.
Т.е. после первого форматирования строки Ваш глобальный массив будет уже проинициализирован.
На веду вам пример.
Допустим Вы форматируете строку для вывода информации игроку.

PHP код:

---

format(string, sizeof(string),"Глобальный массив"); 


---

Все отлично, Вы форматировали строку, вывели ее, все как нужно.
Теперь Вам опять нужно форматировать строку.

PHP код:

---

format(string, sizeof(string),"Какая разница?"); 


---

И теперь если Вывести информацию с массива, то в место текста "Какая разница?" у Вас будет писать "Глобальный массивКакая разница?"
Почему так? потому, что Ваш массив до, этого уже был проинициализирован.
Решение конечно есть.

PHP код:

---

string[0] = EOS; 


---

Т.е. Вы инициализируете первую ячейку массива завершающим символом "\0" но, остальной массив по прежнему будет проинициализирован.
Но для чего, это нубство?

Цитата:

---

Сообщение от KrutoyKrosch

подсчет символов это геморрой только лишний

---

Ну вот, Вы выше писали, зачем 1к ячеек, Вы же не будите создавать 100500 массивов для строк с разной длиной.

Цитата:

---

Сообщение от KrutoyKrosch

почему нельзя создать глобальный стринг

---

Я написал, что лучше, а не почему нельзя.

strdel есть если что, можно просто в конец каждого паблика это писать и все

опять же ты создаешь 10 стрингов в каждом паблике по 256 ячеек, вместо того что бы создать один стринг в 256 ячеек

тем более глобальный стринг не будет жрать стек и не забываем что самп это однозадачность

Цитата:

---

Сообщение от KrutoyKrosch

strdel есть если что, можно просто в конец каждого паблика это писать и все

---

Ну давайте в конце каждой функции будем вызывать нативную функцию, что бы удалить часть строки.
Даже придется вызывать не одну нативную функцию, а две (strlen), потому, что функция strdel удаляет часть строки и по этому Вам нужно будет узнать позицию последнего символа.

и да, format начинает запись с 0 ячейки и поэтому такого не будет "Глобальный массивКакая разница?"

Цитата:

---

Сообщение от Saibot

Если брать пример DeimoS`a, то не нужно ничего пихать в энуминатор iApartment.

PHP код:

---

new for_sale[] = "продается";
format(string, sizeof(string)
,
    "| Этаж № %i | %s | Цена: %i$ |\n"
,
    i++,
    (iApartment[i][0][pBought]) ? (for_sale) : (iApartment[i][0][pNamePlayer]),
    iApartment[i][0][pMoney]
); 


---

---

Ты код этот проверял?

Цитата:

---

Сообщение от KrutoyKrosch

лучше один глобальный string или много локальных?

---

Если в плане скорости, то разницы особо нет (спасибо Pawn за особенности устройства стэка и кучи).
Если в плане удобства, то с глобальным массивом может быть лишний геморрой, когда ты не обнулил массив/обнулил не те данные или же когда ты забыл размер этого массива и тебе придётся выискивать его, дабы перепроверить.
С локальными же ты можешь контролировать размеры в каждом конкретном случае и не бояться, что в нём останутся какие-либо левые данные. Больше контроля, крч.

Я предпочитаю локальные

просто мне щас нужно выводить данные о 100 квартир, это занимает 2к ячеек, то есть дохренище стека будет задействовано (если использовать локальные) и я хз что делать

а про удобство повторюсь

PHP код:

---

strdel(string, 0, sizeof(string)); 


---

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

нет не проверял тот код , я уже решил проблему записав по стандарту в NamePlayer "продается"

Цитата:

---

Сообщение от KrutoyKrosch

просто мне щас нужно выводить данные о 100 квартир, это занимает 2к ячеек, то есть дохренище стека будет задействовано (если использовать локальные) и я хз что делать

---

Решение проще некуда

PHP код:

---

static 


---

Но, опять-же, такие массивы нужно будет обнулять, если используются функции записи в этот массив без его предварительной очистки. Можно сказать, такие же "глобальные" переменные, но доступные только локально.

Цитата:

---

Сообщение от KrutoyKrosch

PHP код:

---

strdel(string, 0, sizeof(string)); 


---

---

Массив в любом случае занимает некое место. Какая нам разница, чем оно будет заполнено? Будет достаточно обнуления нулевой ячейки.

проще тогда создать создать глобальный

Цитата:

---

Сообщение от KrutoyKrosch

проще тогда создать создать глобальный

---

static и так будет работать как глобальный. Только он будет всегда под рукой, когда ты будешь изменять код своей системы.
Хот дело твоё. Тут вопрос вкуса уже