Форум C++

Вот написал прогу. Вроде работает, но через раз. Помогите пожалуйста.
Суть: кинотеатр. При запуске выводит зал. Мышью отмечаем нужные места, щелкаем на серый прямоугольник и они покупаются - становятся красными. Однако если долго щелкать по залу, то он начнет рисовать белые квадраты вместо зеленых и при масштабировании окна, вообще зарисовывать его заголовок. Почему-то если отметить 61 место, то будет нормально, зато на 62-е начнет глючить. В чем фишка??? Помогите!

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

PS Проверил в C++ Builder - тоже самое, т.е. дело не в среде программирования, пока даже не представляю в чём дело, возможно зря используется NULL во втором параметре функции

InvalidateRect(hwnd,NULL,TRUE)

Оно и без NULL не работало... NULL по идее обозначает, что перерисовывается весь экран. Там просто если вообще убрать кусок кода, отвечающий за серую кнопку, только оставить

if(<Проверка мыши на нахождение в области серого прямоугольника>) { MessageBeep(0);//кстати, не знаете ли, как другие звуки делать? }

то все равно глючить будет. Интересно, почему? Сначала вроде все нормально идет, потом - фигня.

Мне кажется, я понял, где ошибка, но не понял какая и не понял, почему.
Я решил написать прогу заново и причем очень часто тестировал. Так вот у нас ошибка при вызове функции myrectangle, причем даже если написать со статическим РЕКТ-ом:

void myrectangle(HDC hdc,int x1,int y1,int x2,int y2,COLORREF rgb) { HBRUSH hbrush; hbrush=CreateSolidBrush(rgb); RECT rect; rect.left=  x1; rect.bottom=y1; rect.right= x2; rect.top=   y2; FillRect(hdc,&rect,hbrush); }

то все равно не будет работать. И как с этим бороться, ума не приложу!

АААААААААААААААААААААААААа....................... я вроде понял! Если много раз вызывать createsolidbrush, то будет глючить. Надо типа так:

HBRUSH redbrush=CreateSolidBrush(RGB(200,0,0)); HBRUSH cyanbrush=CreateSolidBrush(RGB(0,200,200));

И не менять в процессе программы эти кисти - тока юзать.
Вот так оно работает!

ЗЫ: черте что в этом си плюсплюсе((((

А ну да, у вас просто лимит кистей исчерпывается, кисть лучше в WinMain определить один раз...

А.... по-понятней можно???

Звиняюсь за офтоп)
Был вопрос про MessageBeep
Если ей вместо 0 передать одно из следующих значений:

MB_ICONASTERISK
MB_ICONEXCLAMATION
MB_ICONHAND
MB_ICONQUESTION
MB_OK

то она проиграет звуки, установленные для MessageBox'ов с соответствующими стилями.

Еще можно юзать

BOOL Beep(DWORD dwFreq, DWORD dwDuration)

Она воспроизводит звук частоты dwFreq герц в течение dwDuration миллисекунд

>А.... по-понятней можно???
Дело в том, что под объекты GDI (перья, кисти, шрифты, области, растровые изображения, палитры) выделяется память, которую нужно удалять при помощи функции DeleteObject(). Если этого не делать, операционная система выделит определённо количество объектов и больше не будет ничего выдавать, пока старые объекты не будут удалены. Обычно большое количество кистей (читай цветов) не требуется - выделите несколько в главной функции WinMain() и используйте их на протяжении всей программы. Даже можно не удалять - по завершению программы, в оперативную систему будет возвращена вся оперативная память, которая использовалась программой... правда если программа выполняется долго, это может приводить к утечкам памяти, т.е. программа резервирует память под объекты, а обратно не отдаёт (особенно не приятна утечка в программа выполняющихся долгое время без перезагрузки, например в серверах).

А вот вы не знаете, как проигрывать стандартные виндоус звуки, кроме тех, которые проигрывает MessageBeep?

В MessageBeep() передавайте соответствующие параметры

MB_ICONASTERISK MB_ICONEXCLAMATION MB_ICONHAND MB_ICONQUESTION MB_OK

Откопайте winuser.h - в нём этих параметров будет побольше, но они не все звуки производят...

PS Под новые вопросы, лучше заводить новые темы, а то потом ссылаться на тему очень сложно.