VPF::Проблема в VC++ 6 - Форум программистов

Гость_Unknown

Дата 21.1.2004, 18:47 (ссылка)

(голосов: 0)

Загрузка ...

Unregistered

Мне нужно написать небольшую программу-диалог.
Для хранения и изменения некоторых данных я добавил в класс своего диалога нек. переменные.
При попытке изменить их программа вылетает!

Приведу фрагмент кода:

Код


	unsigned int index;
	for(int i = 0; i < MAX_SITES; i++)
  if(m_Sites[i].is_used == 0)
  {
  	index = i;
  	break;
  }
	m_Sites[index].is_used = 1;

Причем если изменять строго определенный элемент массива, все работает нормально!(например: m_Sites[23].is_used = 1 )

Если кто-то сталкивался с подобным, пожалуйста, напишите!

Alex101

Дата 21.1.2004, 19:43 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник Клуба
Сообщений: 891
Регистрация: 8.4.2002
Где: Москва

Репутация: нет
Всего: 10

Так ты может в другую область памяти лезешь, ведь в C у тебя значение index после цикла будет MAX_SITES+1 (если брык не сработает)
Или дело не в этом?

Это сообщение отредактировал(а) Alex101 - 21.1.2004, 19:57

--------------------

С уважением, А. Фролов.

Dov

Дата 21.1.2004, 19:46 (ссылка)

(нет голосов)

Загрузка ...

аСинизатор

Профиль
Группа: Завсегдатай
Сообщений: 1721
Регистрация: 10.5.2003
Где: Эрец-Исраэль

Репутация: 15
Всего: 88

Обрати внимание на эту строку:

Код

if(m_Sites[i].is_used == 0)

и подумай о том, что будет хранится в index в том случае, если твой if не сработает.

--------------------

Тут вечности запах томительный,
И свежие фрукты дешевые,
А климат у нас – изумительный,
И только соседи – #уевые.
Игорь Губерман.

mr.DUDA

Дата 21.1.2004, 20:16 (ссылка)

(нет голосов)

Загрузка ...

3D-маньяк

Профиль
Группа: Экс. модератор
Сообщений: 8244
Регистрация: 27.7.2003
Где: город-герой Минск

Репутация: 25
Всего: 232

Цитата

подумай о том, что будет хранится в index в том случае, если твой if не сработает

правильно, т.е. тут действует ситуация, когда "index" так и останется непроинициализированным, а обращение по индексу идёт:

Цитата

unsigned int index; // а можно так: unsigned int index = 0xffffffff;
for(int i = 0; i < MAX_SITES; i++)
if(m_Sites[i].is_used == 0)
{
index = i;
break;
}

// надо добавить:
if(index != 0xffffffff)
m_Sites[index].is_used = 1;

--------------------

Alex101

Дата 21.1.2004, 20:31 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник Клуба
Сообщений: 891
Регистрация: 8.4.2002
Где: Москва

Репутация: нет
Всего: 10

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

Не, надо так написать:

Код

if(index<MAX_SITES) m_Sites[index].is_used=1;

--------------------

С уважением, А. Фролов.

Гость_switch

Дата 22.1.2004, 16:04 (ссылка)

(голосов: 0)

Загрузка ...

Unregistered

У тебя плохо структурирован код:
попробуй так:

for(int i = 0; i < MAX_SITES; i++)
if(m_Sites[i].is_used == 0) {
m_Sites[i].is_used = 1;
break;
}

или так

for(int i = 0; i < MAX_SITES; i++)
if(m_Sites[i].is_used == 0) break;
m_Sites[i].is_used = 1;

1 Пользователей читают эту тему (1 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема \| C/C++: Общие вопросы \| Следующая тема »