Поиск:

Ответ в темуСоздание новой темы Создание опроса
> Структура в векторе.Ошибка записи элемента вектора 
V
    Опции темы
freshAngel
Дата 10.2.2009, 16:14 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Бывалый
*


Профиль
Группа: Участник
Сообщений: 248
Регистрация: 1.10.2008

Репутация: нет
Всего: нет
Привет.
Объявляю структуру:

Код

struct s1
{
int X1;
int X2;
};

vector<s1> a;

// После эти строк, ошибка (.... Error Adress Vialation)

a[0].X1=10;
a[0].X2 = 5;


Вроде все верно.

Пробовал и другим способом.

Код

typedef struct
        {
         int X1;
         int X2;
        } s1;
s1 stroka;

vector<s1> a;

stroka.X1 = 10;
stroka.X2 = 5;

a[0] = stroka;



Опять ошибка Adress Vialation Error
В чем может быть проблема, и как правильно записать значения!?
PM MAIL   Вверх
Dem_max
Дата 10.2.2009, 16:31 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Завсегдатай
Сообщений: 1780
Регистрация: 12.4.2007

Репутация: 14
Всего: 39
Ну вообщето ты просто a сделал вектором но память под него не выделил вот и ошибка
Код

typedef struct
        {
         int X1;
         int X2;
        } s1;
s1 stroka;

vector<s1> a;

stroka.X1 = 10;
stroka.X2 = 5;
a.resize(10);
a[0] = stroka;


либо так
Код

typedef struct
        {
         int X1;
         int X2;
        } s1;
s1 stroka;

vector<s1> a;

stroka.X1 = 10;
stroka.X2 = 5;
a.push_back(stroka)



--------------------
Американские программисты долго не могли понять, почему русские при зависании Windоws всё время повторяют "Твой зайка написал" ("Yоur bunnу wrоte")
PM MAIL   Вверх
freshAngel
Дата 10.2.2009, 21:07 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Бывалый
*


Профиль
Группа: Участник
Сообщений: 248
Регистрация: 1.10.2008

Репутация: нет
Всего: нет
Цитата(Dem_max @ 10.2.2009,  16:31)
Ну вообщето ты просто a сделал вектором но память под него не выделил вот и ошибка
Код

typedef struct
        {
         int X1;
         int X2;
        } s1;
s1 stroka;

vector<s1> a;

stroka.X1 = 10;
stroka.X2 = 5;
a.resize(10);
a[0] = stroka;


либо так
Код

typedef struct
        {
         int X1;
         int X2;
        } s1;
s1 stroka;

vector<s1> a;

stroka.X1 = 10;
stroka.X2 = 5;
a.push_back(stroka)

Спасибо, все работает!
PM MAIL   Вверх
GraNit
Дата 2.5.2012, 12:27 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Шустрый
*


Профиль
Группа: Участник
Сообщений: 55
Регистрация: 20.10.2007

Репутация: нет
Всего: нет
Здравствуйте!

Хочу продолжить тему.
Пытаюсь получить список файлов с FTP сервера. Т.к. их кол-во разанее не известно, использую верктор. Вот его описание в хейдере:
Код

std::vector < std::vector <AnsiString> > RemoteFiles;

Далее подключаюсь к серверу и по NMFTP1->List() получаю список построчно и каждую строку обрабатываю вот так:
Код

void __fastcall TForm1::NMFTP1ListItem(AnsiString Listing)
{
        int EndPos, i;
        // Счетчик файлов
        FilesCount++;
        //Добавим строку в массив
        RemoteFiles.resize(FilesCount);
       // Здесь в цикле делим строку на поля(разделитель пробел): Дата, Время, Размер/Дир, Имя
        for (i=0; i<=2; i++){
                Listing = Listing.Trim();
                EndPos =  Listing.Pos(" ");
                RemoteFiles[FilesCount-1].resize(i+1);
                RemoteFiles[FilesCount-1][i] = Listing.SubString(0,EndPos);
                Listing = Listing.SubString(EndPos,Listing.Length());
        }
        // В именах тоже бавают пробелы
        RemoteFiles[FilesCount-1][i] = Listing.Trim();
}

Проблема следующая, Заполняю 24 строки массива, на 25 строке "Access violation at address..."
Честно говоря не понимаю почему.
Прошу помощи.
PM MAIL   Вверх
artsb
Дата 2.5.2012, 14:19 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
****


Профиль
Группа: Завсегдатай
Сообщений: 2280
Регистрация: 17.7.2007
Где: центр Вселенной

Репутация: 39
Всего: 64
Не видно тут 25-той строки. Предположу, что эта:
Код

// В именах тоже бавают пробелы
        RemoteFiles[FilesCount-1][i - 1] = Listing.Trim();

Во втором индексе добавьте (-1), как я написал.

А лучше Trim() перенести в цикл:
Код

        for (i=0; i<=2; i++){
                Listing = Listing.Trim();
                EndPos =  Listing.Pos(" ");
                RemoteFiles[FilesCount-1].resize(i+1);
                RemoteFiles[FilesCount-1][i] = Listing.SubString(0,EndPos).Trim();
                Listing = Listing.SubString(EndPos,Listing.Length());
        }



--------------------
Чем отличается умный человек от мудрого?
Умный - выпутается из любой ситуации.
Мудрый - просто в неё не попадёт.
PM MAIL   Вверх
GraNit
Дата 3.5.2012, 04:03 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Шустрый
*


Профиль
Группа: Участник
Сообщений: 55
Регистрация: 20.10.2007

Репутация: нет
Всего: нет
Действительно, отработала без ошибок, но пропало поле "Размер/Директория", т.е. мы в нее записали имя.
Увеличил счетчик цикла на один и во втором индексе добавил -1, теперь все в норме. 
Правда у меня есть вопрос, в цикле объявляется на 1 столбец больше, значит вектор занимает больше памяти чем ему требуется?
А на счет 25 строки, я имел в виду 24 элемента массива заполняются, а на 25-ом появляется ошибка.
PM MAIL   Вверх
Dem_max
Дата 3.5.2012, 06:14 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Завсегдатай
Сообщений: 1780
Регистрация: 12.4.2007

Репутация: 14
Всего: 39
вообще пользуйся через
RemoteFiles.push_back();


--------------------
Американские программисты долго не могли понять, почему русские при зависании Windоws всё время повторяют "Твой зайка написал" ("Yоur bunnу wrоte")
PM MAIL   Вверх
GraNit
Дата 3.5.2012, 07:08 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Шустрый
*


Профиль
Группа: Участник
Сообщений: 55
Регистрация: 20.10.2007

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

вообще пользуйся через
RemoteFiles.push_back(); 

Может быть как вы говорите и правильнее, но я думаю разницы нет. Хотя я не профессиональный программист на C++, вам виднее.
PM MAIL   Вверх
artsb
Дата 3.5.2012, 08:29 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
****


Профиль
Группа: Завсегдатай
Сообщений: 2280
Регистрация: 17.7.2007
Где: центр Вселенной

Репутация: 39
Всего: 64
Цитата(GraNit @  3.5.2012,  04:03 Найти цитируемый пост)
в цикле объявляется на 1 столбец больше, значит вектор занимает больше памяти чем ему требуется?

По идее, всё должно быть нормально. Но, Dem_max дело говорит. Используйте push_back()


--------------------
Чем отличается умный человек от мудрого?
Умный - выпутается из любой ситуации.
Мудрый - просто в неё не попадёт.
PM MAIL   Вверх
GraNit
Дата 3.5.2012, 12:08 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Шустрый
*


Профиль
Группа: Участник
Сообщений: 55
Регистрация: 20.10.2007

Репутация: нет
Всего: нет
Но тогда и интераторы следует пользовать, только с ними что-то не все понятно пока, а прога должна работать уже вчера, как всегда.  smile 
По позже, когда будет следующий билд, попробую сделать все как положено в C++

Это сообщение отредактировал(а) GraNit - 3.5.2012, 12:13
PM MAIL   Вверх
artsb
Дата 3.5.2012, 12:55 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
****


Профиль
Группа: Завсегдатай
Сообщений: 2280
Регистрация: 17.7.2007
Где: центр Вселенной

Репутация: 39
Всего: 64
Зачем итераторы?
Код

        for (i=0; i<=2; i++){
                Listing = Listing.Trim();
                EndPos =  Listing.Pos(" ");
                RemoteFiles[FilesCount-1].push_back(Listing.SubString(0,EndPos).Trim());
                Listing = Listing.SubString(EndPos,Listing.Length());
        }



--------------------
Чем отличается умный человек от мудрого?
Умный - выпутается из любой ситуации.
Мудрый - просто в неё не попадёт.
PM MAIL   Вверх
GraNit
Дата 31.5.2012, 05:28 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Шустрый
*


Профиль
Группа: Участник
Сообщений: 55
Регистрация: 20.10.2007

Репутация: нет
Всего: нет
Да, действительно, все работает.
Прошу прощения, не поблагодарил за помощь! smile Всем откликнувшимся спасибо!
PM MAIL   Вверх
  
 Ответ в темуСоздание новой темы Создание опроса
Правила форума "С++ Builder"
Rrader

Запрещается!

1. Публиковать ссылки на вскрытые компоненты

2. Обсуждать взлом компонентов и делиться вскрытыми компонентами

  • Литературу по С++ Builder обсуждаем здесь
  • Действия модераторов можно обсудить здесь
  • С просьбами о написании курсовой, реферата и т.п. обращаться сюда
  • Настоятельно рекомендуем заглянуть в DRKB (Delphi Russian Knowledge Base) - крупнейший в рунете сборник материалов по Дельфи

Если Вам понравилась атмосфера форума, заходите к нам чаще! С уважением, Rrader.
 
0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема | C++ Builder | Следующая тема »

Подписаться на тему | Подписка на этот форум | Скачать/Распечатать тему


 




[ Время генерации скрипта: 0.0544 ]   [ Использовано запросов: 22 ]   [ GZIP включён ]


Реклама на сайте     Информационное спонсорство

 
 По вопросам размещения рекламы пишите на vladimir(sobaka)vingrad.ru
Отказ от ответственности     Powered by Invision Power Board(R) 1.3 © 2003  IPS, Inc.