Не выводит 2 Shape на форму, ошибка в функции - " is not valid integer value

@QAQA · Регистрация: 15.03.2014

Студворк — интернет-сервис помощи студентам

Есть функция:

C++
1
2
3
4
5
6
7
8
9
10
11
double readCoord(UnicodeString iskomoe, String coord)
{
        String whatFind = coord + "=";
        int startPos = PosEx(whatFind, iskomoe);
        int endPos = PosEx(";", iskomoe, startPos);
        String pos = iskomoe.SubString(startPos+2, endPos - startPos-2);
        double posConverted = StrToInt(pos);
        // ShowMessage(pos);       // LOG
        return posConverted;
 
}

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
void __fastcall TForm1::Start1Click(TObject *Sender)
{
 
    StatusBar1->Panels->Items[0]->Text = "Добро пожаловать";
    Memo1->Lines->LoadFromFile("1.txt");      // Вот так выгружаю файл   // Для лога
 
 
    int coutLines = Memo1->Lines->Count;
    Label9->Caption = coutLines;
 
    if (coutLines != 0)
    {       // if (coutLines <> 0)
 
 
        while (i < coutLines)
        {              // while (i < coutLines)
        UnicodeString XX = Label3->Caption;         // Вот так ПРИСВАИВАЮ данные в String
        Label3->Caption=Memo1->Lines->Strings[i];  // Вот так ВЫВОЖУ первую строку
 
        double posX = readCoord(XX, "x");
            Label4->Caption = posX;    // Для лога
        double posY = readCoord(XX, "y");
            Label5->Caption = posY;    // Для лога
        double posR = readCoord(XX, "R");
            Label6->Caption = posR;    // Для лога
 
        Shape[i] = new TShape(this);
        Shape[i]->Parent = this;
        Shape[i]->Visible = true;
        Shape[i]->Brush->Style=stCircle;
        Shape[i]->Brush->Color=clBlack;
 
        double cX = posX;
        double cY = posY;
        double cR = posR;
        Shape[i]->Top = cY;
        Shape[i]->Left = cX;
        Shape[i]->Height=cR;  // *2  NE ZABIT'
        Shape[i]->Width=cR;   // *2  NE ZABIT'
        Shape[i]->Shape = stCircle;
        Shape[i]->OnMouseEnter = MouseEnter;
        Shape[i]->OnMouseLeave = MouseLeave;
 
        Label2->Caption=i;   // Для лога
           StatusBar1->Panels->Items[0]->Text = posR;
             i++;
        }          // while (i < coutLines)
 
    }
else {
        ShowMessage("Файл пустой");
     }       // if (coutLines <> 0)
 
 
}

При первом нажатии на Start - выбивает окно с ошибкой " " is not valid integer value ". При повторном нажатии - всё работает, НО рисует только 7 из 9 кругов. Почему 7, а не 8 хотя бы - не пойму.
Вот так координаты заданы в 1.txt:
x=45; y=45; R=50;
x=0; y=0; R=50;
x=100; y=100; R=65;
x=450; y=450; R=50;
x=125; y=125; R=25;
x=10; y=10; R=50;
x=175; y=175; R=20;
x=150; y=150; R=15;
x=200; y=200; R=200;

Такая же история, даже если не использовать функцию readCoord, а пустить всё в void __fastcall TForm1::Start1Click(TObject *Sender).

@volvo · 14.04.2015, 23:33

Для начала: переменная i где инициализируется?

@QAQA · 14.04.2015, 23:35 **[ТС]**

В Unit.h:

C++
1
2
#endif
    int i = 0;

@volvo · 14.04.2015, 23:53

Прелестно. Значит, нажал я на кнопку, и у меня начиная с прошлого значения i будет заполняться еще coutLines элементов массива Shape? А их есть столько? Может, нужно при нажатии на кнопку обнулить i?

@QAQA · 15.04.2015, 02:21 **[ТС]**

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

Прелестно. Значит, нажал я на кнопку, и у меня начиная с прошлого значения i будет заполняться еще coutLines элементов массива Shape? А их есть столько? Может, нужно при нажатии на кнопку обнулить i?

Если честно, все равно не понял что от этого изменится, но всё же:

C++
1
2
3
4
5
6
7
8
9
void __fastcall TForm1::Start1Click(TObject *Sender)
{
    int i = 0;
    StatusBar1->Panels->Items[0]->Text = "Добро пожаловать";
    Memo1->Lines->LoadFromFile("1.txt");      // Вот так выгружаю файл   // Для лога
 
 
    int coutLines = Memo1->Lines->Count;
    Label9->Caption = coutLines;

и т.д.

Вот Unit.h:

C++
1
2
3
4
5
6
//---------------------------------------------------------------------------
extern PACKAGE TForm1 *Form1;
//---------------------------------------------------------------------------
#endif
 
    TShape* Shape[50];

Всё так же - срабатывает при втором клике на Start.
Если что, в итого выводит:
Label9->Caption = coutLines; - 9 , как и должно
Label2->Caption=i; - 8.
Вот не доходит до меня...

Добавлено через 22 минуты
еще:

C++
1
 while (i < coutLines)

Надо изменить на:

C++
1
 while (i <= coutLines)

Добавлено через 1 час 25 минут
Ошибка где-то здесь...

C++
1
2
UnicodeString XX = Label3->Caption;         // Вот так ПРИСВАИВАЮ данные в String
        Label3->Caption=Memo1->Lines->Strings[i];  // Вот так ВЫВОЖУ первую строку

Добавлено через 24 минуты
Методом проб и ошибок получилось так:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
double readCoord(UnicodeString iskomoe, String coord)
{
        String whatFind = coord + "=";
        int startPos = PosEx(whatFind, iskomoe);
        int endPos = PosEx(";", iskomoe, startPos);
        String pos = iskomoe.SubString(startPos+2, endPos - startPos-2);
        double posConverted = StrToInt(pos);
        // ShowMessage(pos);       // LOG
        return posConverted;
 
}
 
//---------------------------------------------------------------------------
 
 
void __fastcall TForm1::Start1Click(TObject *Sender)
{
 
    int i = 0;
    StatusBar1->Panels->Items[0]->Text = "Добро пожаловать";
    TStringList *file= new TStringList;//массив элементов где каждый элемент - строка
    file->LoadFromFile("1.txt"); // Загружаем файл
    int coutLines = file->Count;
 
    Label9->Caption = coutLines;
 
    if (coutLines != 0)
    {       // if (coutLines <> 0)
 
 
        while (i < coutLines)
        {              // while (i < coutLines)
        UnicodeString XX = file->Strings[i];    // Вот так ПРИСВАИВАЮ данные в String
        double posX = readCoord(XX, "x");
            Label4->Caption = posX;    // Для лога
        double posY = readCoord(XX, "y");
            Label5->Caption = posY;    // Для лога
        double posR = readCoord(XX, "R");
            Label6->Caption = posR;    // Для лога
 
        Shape[i] = new TShape(this);
        Shape[i]->Parent = this;
        Shape[i]->Visible = true;
        Shape[i]->Brush->Style=stCircle;
        Shape[i]->Brush->Color=clBlack;
 
        double cX = posX;
        double cY = posY;
        double cR = posR;
        Shape[i]->Top = cY;
        Shape[i]->Left = cX;
        Shape[i]->Height=cR;  // *2  NE ZABIT'
        Shape[i]->Width=cR;   // *2  NE ZABIT'
        Shape[i]->Shape = stCircle;
        Shape[i]->OnMouseEnter = MouseEnter;
        Shape[i]->OnMouseLeave = MouseLeave;
 
        Label2->Caption=i;   // Для лога
           StatusBar1->Panels->Items[0]->Text = posR;
             i++;
        }          // while (i < coutLines)
 
    }
else {
        ShowMessage("Файл пуст.");
     }       // if (coutLines <> 0)
 
 
}

Из серии "если работает - не трогай". Без ошибок, рисует - правильно.

Новые блоги и статьи Все статьи Все блоги /
Изучаю kubernetes lagorue 13.01.2026 А пригодятся-ли мне знания kubernetes в России?	Сукцессия микоризы: основная теория в виде двух уравнений. anaschu 11.01.2026 https:/ / rutube. ru/ video/ 7a537f578d808e67a3c6fd818a44a5c4/	WordPad для Windows 11 Jel 10.01.2026 WordPad для Windows 11 — это приложение, которое восстанавливает классический текстовый редактор WordPad в операционной системе Windows 11. После того как Microsoft исключила WordPad из. . .	Classic Notepad for Windows 11 Jel 10.01.2026 Old Classic Notepad for Windows 11 Приложение для Windows 11, позволяющее пользователям вернуть классическую версию текстового редактора «Блокнот» из Windows 10. Программа предоставляет более. . .
Почему дизайн решает? Neotwalker 09.01.2026 В современном мире, где конкуренция за внимание потребителя достигла пика, дизайн становится мощным инструментом для успеха бренда. Это не просто красивый внешний вид продукта или сайта — это. . .	Модель микоризы: классовый агентный подход 3 anaschu 06.01.2026 aa0a7f55b50dd51c5ec569d2d10c54f6/ O1rJuneU_ls https:/ / vkvideo. ru/ video-115721503_456239114	Owen Logic: О недопустимости использования связки «аналоговый ПИД» + RegKZR ФедосеевПавел 06.01.2026 Owen Logic: О недопустимости использования связки «аналоговый ПИД» + RegKZR ВВЕДЕНИЕ Введу сокращения: аналоговый ПИД — ПИД регулятор с управляющим выходом в виде числа в диапазоне от 0% до. . .	Модель микоризы: классовый агентный подход 2 anaschu 06.01.2026 репозиторий https:/ / github. com/ shumilovas/ fungi ветка по-частям. коммит Create переделка под биомассу. txt вход sc, но sm считается внутри мицелия. кстати, обьем тоже должен там считаться. . . .

@volvo Супер-модератор 33371 / 21497 / 8234 Регистрация: 22.10.2011 Сообщений: 36,893 Записей в блоге: 12
	14.04.2015, 23:33
	Для начала: переменная i где инициализируется? 0

@volvo Супер-модератор 33371 / 21497 / 8234 Регистрация: 22.10.2011 Сообщений: 36,893 Записей в блоге: 12
	14.04.2015, 23:53
	Прелестно. Значит, нажал я на кнопку, и у меня начиная с прошлого значения i будет заполняться еще coutLines элементов массива Shape? А их есть столько? Может, нужно при нажатии на кнопку обнулить i? 0