Форум программистов [Powered by Invision Power Board]

Версия для печати темы
Нажмите сюда для просмотра этой темы в оригинальном формате

Форум программистов > C++ Builder > запись значений StringGrid в динамические массивы

Автор: wolver17 22.11.2011, 23:53

Такая вот проблемка: необходимо считать в два динамических массива значения столбцов компонента StringGrid2, состоящего из 2-х столбцов произвольной длины. (В конце, для проверки что всё норм, вывожу эти же массивы в другой - StringGrid1).
На форме такие компоненты: StringGrid1, StringGrid2, Edit1, CheckBox1, BitBtn1.

Код


//---------------------------------------------------------------------------

#include <vcl.h>
#pragma hdrstop
#include<iostream.h>
#include<conio.h>
#include<math.h>
#include<SysUtils.hpp>
#include "Unit1.h"
//---------------------------------------------------------------------------
#pragma package(smart_init)
#pragma resource "*.dfm"
         int n;
         float *x=new float[n];
         float *y=new float[n];
TForm1 *Form1;
//---------------------------------------------------------------------------
__fastcall TForm1::TForm1(TComponent* Owner)
        : TForm(Owner)
{
}
//---------------------------------------------------------------------------

void __fastcall TForm1::BitBtn1Click(TObject *Sender)
{
int j=0;
float temp, temp1;
for(int i=0;i<n;i++)
    for(j=0;j<2;j++)
    {
         if (j==0)
         {
                 temp=StrToFloatDef(Form1->StringGrid2->Cells[j][i],0);
                 x[i]=temp;
         }
         else
         {
                 temp1=StrToFloatDef(Form1->StringGrid2->Cells[j][i],0);
                 y[i]=temp1;
         }
    }
Form1->StringGrid2->Enabled=false;
//====================================
for(int i=0;i<n;i++)
    for(j=0;j<2;j++)
    if (j==0)
    {
        StringGrid1->Cells[j][i]=FloatToStr(x[i]);
    } 
    else
    {
        StringGrid1->Cells[j][i] = FloatToStr(y[i]);
    }
Form1->StringGrid1->Enabled=false;
delete []x;
delete []y;
}
//---------------------------------------------------------------------------

void __fastcall TForm1::CheckBox1Click(TObject *Sender)
{
n=StrToInt(Form1->Edit1->Text);
StringGrid2->RowCount = n;
StringGrid1->RowCount = n;
StringGrid2->ColCount = 2;
StringGrid2->Enabled=true;
}
//---------------------------------------------------------------------------

код работает, и 1-й массив прекрасно записывается/считуется, а вот 2-й... в общем вот что на скрине:

http://ipicture.ru/Gallery/View/6975512.html

подскажите, пожалуйста, что не так?

Автор: MAXIQ 24.11.2011, 16:57

Вроде работает...

Код


void __fastcall TForm1::Button1Click(TObject *Sender)
{
int n=4;
float *x=new float[n];
float *y=new float[n];
float temp, temp1;
for(int i=0;i<n;i++)
{
    for(int j=0;j<2;j++)
    {
         if (j==0)
         {
                 temp=StrToFloatDef(Form1->StringGrid1->Cells[j][i],0);
                 x[i]=temp;
         }
         else
         {
                 temp1=StrToFloatDef(Form1->StringGrid1->Cells[j][i],0);
                 y[i]=temp1;
         }
    }
}
//====================================
for(int i=0;i<n;i++)
{
    for(int j=0;j<2;j++)
    if (j==0)
    {
        StringGrid2->Cells[j][i]=FloatToStr(x[i]);
    }
    else
    {
        StringGrid2->Cells[j][i] = FloatToStr(y[i]);
    }
}
delete []x;
delete []y;

Автор: XPyCT 25.11.2011, 08:30

Для чего вы создаете динамический массив если

Код


int n=4;
float *x=new float[n];
float *y=new float[n];

n - вы явно присваиваете значение 4 (размер массива);

не лучше уж сделать так

Код


int n= StringGrid2->RowCount-1; //если есть фиксированный ряд
int n= StringGrid2->RowCount; //если без фиксированного ряда
//Выберите что вам нужно с фик рядом или без

а далее уже как у вас написано

Код



float *x=new float[n];
float *y=new float[n];

Динамический массив по ходу для того и создают когда не знаешь сколько записей в таблице

Автор: wolver17 29.11.2011, 00:22

Цитата(XPyCT @ 25.11.2011, 08:30

)

Для чего вы создаете динамический массив если

код C++ Builder
1:
2:
3:
int n=4;
float *x=new float[n];
float *y=new float[n];

n - вы явно присваиваете значение 4 (размер массива);

простите, протупил smile

то я забыл удалить, когда проверял через простые массивы, там просто
int n, а не int n=4.

Автор: wolver17 29.11.2011, 00:38

я вроде разобрался с проблемой, только решение создаёт другую проблему)
создавая массивы функцией new, прога ж не знает размер массив т.к. они динамические, и получает их после ввода значения n уже при работе проги. И я проверял в обычном Borland C++, если
cin>>n;
делать после объявления массива, а не до - то подобные глюки с потерей значений в массиве происходят, а если делать ввод а потом объявление, т.е. как надо - то всё норм. Так вот, проблемка в том что мне мои массивы нужны потом для расчётов в моих ф-циях, и я не могу их описать локально. Как же тогда исправить глюк или заставить элементы записываться правильно?
(или я чего-то не знаю, и можно передавать пользовательские переменные в стандартные ф-ции, где параметр только скажем TObject *Sender?)

Автор: MAXIQ 5.12.2011, 09:56

Мда.. или я совсем туплю или Вы уважаемый чего-то не договариваете по поводу cin>>n...
Так у тебя проект в BCB и ты используешь формы и стандартные элементы управления? Или ты чего-то консольное творишь? Тестовую переменную с инициализацией я специально оставил для наглядности int n=4...; Указатель на массив нужно объявить глобально. new и delete вызывать только один раз.. на создании (Create) и (Destroy) соответственно. А дальше, если это нужно, можешь изменять его размерность, записывать, перезаписывать, в общем глумиться как хочешь написав свою функцию обработки в которую можно передавать в качестве аргумента тот самый глобальный указатель на массив, а ее уже вызывать из различных событий элементов управления "где параметр только скажем TObject *Sender")))))). Таким макаром у тебя и данные из массива будут всегда под рукой, и код короче.... до поры до времени))). А лучше возьми книжку в руки и еще разок динамическое выделение памяти почитай.. и желательно не ту что уже читал, а другую.. вопросы сами отвалятся.

Автор: wolver17 6.12.2011, 20:49

Цитата(XPyCT @ 25.11.2011, 08:30

)

сорри, только сейчас увидел, что это было не мне адресовано, а Вам от ХруСТ'а smile

Цитата(MAXIQ @ 5.12.2011, 09:56

)

Указатель на массив нужно объявить глобально.

Вооот, как я и думал, что или я где-то туплю или хз smile

. Я посмотрел одну из старых прог с дин. массивами, и там как Вы описали, указатель был глобальный, а сам массив создаётся new уже внутри ф-ции. Спс, что натолкнули на мыслю, MAXIQ, всё работает как надо)

сделал глобально:

Код


            int n;
            typedef float *x,*y;

потом внутри ф-ции BitBtn1Click объявил массив:

Код


            float *x=new float[n];
            float *y=new float[n];

Осталось только свою прогу переделать, раз примерчик работает, думаю и там проблем не будет)

Автор: XPyCT 19.12.2011, 21:05

Цитата(MAXIQ @ 5.12.2011, 09:56)

тогда сорри!

тогда действительно лучше глобально обьявить.
Но если будут планы реализовывать и на других формах сие чудо тогда уже делайте класс N пихайте в закрытый раздел и всего 1н или два метода