Разбиение программы, использующей SDL, на модули

[onewho]

зато теперь я вас понял!
ну тогда все ясно))
да. наврятли получиться без предварительной инициализации как-то использовать данные в глобальном пространстве. (ну я себе такого не могу представить)

[_Bers]

Не уверен, что понимаю о чем речь. Поэтому, может быть я зря влазию.
Но возможно проблема ТС связанна с тем, что порядок инициализации глобальных данных не определён.

Это означает, что если Second инициализируется раньше, чем First, но ему для работы нужны корректные данные от First, то последствия могут быть непредсказуемыми.

Лекарство: паттерн "одиночка", или "Singleton"

[mmx358]

Вот это печалька..... Выходит, что мне все, что обращается к конкретным цветам, пихать в главный файл?

[mmx358]

_Bers, именно для предотвращения такой ситуации мне и приходится пихать все ... Выходит тогда, что мне даже функции описывать надо в главной функции?

[_Bers]

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

_Bers, именно для предотвращения такой ситуации мне и приходится пихать все ... Выходит тогда, что мне даже функции описывать надо в главной функции?

Существует такой тезис: большое количество глобальных переменных - признак ущербной архитектуры.

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

Чем меньше функция знает об окружении, тем ею проще управлять, и она надежнее.

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

Но если нет такой возможности - хотя бы минимизировать количество глобальных переменных. И инкапсулировать их при помощи паттерна "одиночка"

Доступ к этим данным - только через одиночку. Это гарантирует, что при первом же обращении к одиночке, одиночка проинициализируется только один раз корректными данными.

[onewho]

mmx358, так а я не пойму в чём проблема сделать передачу цвета как параметра?

[mmx358]

_Bers, вы, безусловно, правы, поэтому не буду трогать указатель на рабочую поверхность, с этим тогда все в порядке.

Теперь по поводу того, почему я не могу сделать передачу цвета как параметра, как это сделано в остальных ф-ях. Дело в том, что есть функция, которая выводит на экран графический объект такого цвета, который зависит от случайного числа. Т.е. на функция возложена обязанность рисовать не каким-то цветом, который ей передается как параметр, а именно одним из конкретных цветов. Короче говоря, в ф-ции присутсвтуют идентификаторы цветов (yellow, red, ..... ), но в этом случае цвета "не работают", т.е. выводится "пустое" изображение, хотя, если вставить тело функции в главную функцию, то все работает.

Есть какие-то правильные решения? Мне пока что приходит в голову безумная идея создать массив цветов и передавать его по значению или указатель в функции. Хотя мне такой вариант совсем не нравится. Что вы можете посоветовать?

[onewho]

генерить случайное число вне функции.

[mmx358]

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

[ElectroMent]

Итак, пишим на С, есть файлы:
main.c - содержит главную функцию, инициализация и закрытие SDL систем
graph.c - содержит необходимые графические функции
algor.c - содержит алгоритм программы.


пусть будет файл graph.h в нем обьявляем глобальный указатель на обьект экрана

extern SDL_Surface *pScreen;
extern void InitScreen();

в файле graph.c определяешь

SDL_Surface *pScreen = NULL;
void InitScreen()
{
   pScreen = ...
}

во всех файлах, где требуется использовать глобальный pScreen main.c и algor.c

обьявляешь

extern SDL_Surface *pScreen;

где будет инициализация pScreen проходить надо написать

extern void InitScreen();

и не забыть вызвать функцию инициализации InitScreen()

теперь можешь использовать pScreen где надо и все будет путем.


Касательно твоих цветов. Рассмотрим на примере 1 цвета.

В файле graph.h в нем обьявляем глобальный цвет в формате RGB

extern Uint32 yellow;

extern bool InitColorTable(); //функция инициализирует цвета

в файле graph.c определяешь

Uint32 yellow = 0;

bool InitColorTable()
{
   if( !pScreen ) 
        return false;
   yellow = SDL_MapRGB( pScreen->format, 0xff, 0xff, 0x00);
   return true;
}

во всех файлах, где требуется использовать глобальный цвет

обьявляешь

extern Uint32 yellow;

где будет инициализация цветов проходить надо написать

extern bool InitColorTable();

и не забыть вызвать InitColorTable()

и дальше смело юзать цвета. Иначе они будут "черные".