Доступ к компонентам на другой форме

Как правильно обращаться к компонентам, расположенным на Form1 из Form2? В Unit2.cpp подключил Unit1.h. При компиляции ошибок не выдаёт. Но при выполнении падает например на вызове "Form1->client->Socket->SendText(SendText)".
Второй момент: в Unit2.cpp есть также необходимость использовать переменные, объявленные в Unit1.cpp, поэтому я его заинклюдил, из-за чего была выдана куча варнингов линкера типа "Public symbol '_название_переменной' defined in both module Unit1.obj и Unit2.obj", но переменные сами вроде как работают.

Так как правильно всё-таки?

0

Сообщение от Max_92 "Form1->client->Socket->SendText(SendText)".

Сдается мне дело тут в синхронизации. Как именно объявлены компоненты? Где?

Сообщение от Max_92 Второй момент

Покажите так же где вы их объявляете. Это сразу упростит дело.

0

SatanaXIII,
AnsiString SendText, RecvText; идёт сразу после TForm1 *Form1; в "Unit1.cpp".
Компонент TClientSocket закинут на Form1. На Form2 я нажимаю по кнопке и вызывается Form1->client->Socket->SendText(SendText);

0

- Max_92, по идее, если Вы только определили глобально строки:
в модуле "Unit1", (т.е. только в файле "Unit1.cpp"), а в файле "Unit1.h" о них нет упоминания,
и в файл "Unit2.cpp" Вы включили заголовок "Unit1.h", то в модуле "Unit2" эти строки не должны быть
видны! Компилятор должен давать на них ошибки!
- Сразу-же скажу как надо определить в модуле "Unit1" глобальные строки SendText и RecvText:
1) - В файле "Unit1.cpp":
- все как и было;
2) - В файле "Unit1.h":
- Теперь эти глобальные строки будут определены один раз в данном модуле "Unit1" и видны
снаружи, всюду, куда Вы включите заголовок этого модуля "Unit1.h".
- Что касается второй проблемы, то я -бы поставил на строку - Form1->client->Socket->SendText( SendText );
точку останова отладчика, когда при выполнении программы она остановится на этой строке прошел-бы
по ней всей указателем мыши от начала и до конца и посмотрел-бы с каким компонентом этой строки
связана проблема.
- Мне кажется это одно из двух:
1) В модуле "Unit2" не правильно видится строка SendText (о чем я говорил выше).
2) У Вас название строки SendText совпадает с названием метода в злополучной строке.
Может, кстати, из за этого он и не правильно видит строку ! ...

2

Скорее всего, дело не в области видимости сокета, а в создании объекта сокета.
Вынеси сокет в отдельный модуль, можно в отдельный класс, и вызывай из любых форм как захочется.
Не забудь про создание и уничтожение сокета внутри класса.

Если с созданием класса и модуля всё сложно, вызывай форму где сокет, и сделай её инвизибл.
Все объекты внутри формы будут созданы, можно обращаться к ним из другой формы.

0

Dam33, сокеты вроде как работают и при обычном включении "Unit.h"
Staut, если определять переменные вот так: extern AnsiString SendText, RecvText; то в нужном файле потребуется их ещё раз объявить обычным способом? а то для указателя TForm * Form1 это выглядит именно так.

Сообщение от Staut в модуле "Unit1", (т.е. только в файле "Unit1.cpp"), а в файле "Unit1.h" о них нет упоминания, и в файл "Unit2.cpp" Вы включили заголовок "Unit1.h", то в модуле "Unit2" эти строки не должны быть видны! Компилятор должен давать на них ошибки!

так и есть. просто у меня помимо этого в "Unit2.cpp" есть ещё переменные с такими же названиями, как и в "Unit1.cpp" (например интовая "function"), из-за чего линкер выдаёт варнинги. конечно, в этом дублировании смысла нет наверное, так что сделаю их глобальными.
а первый раз я пробовал тупо включить "Unit1.cpp" и ничего хорошего из этого не вышло.

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

Сообщение от Staut В модуле "Unit2" не правильно видится строка SendText (о чем я говорил выше).

переменная SendText есть только в "Unit2.cpp". но я попробую сделать всё в "Unit1.cpp", как вы сказали, с extern. только непонятно, надо ли будет потом ещё раз писать и в "Unit1.cpp" и в "Unit2.cpp" обычное объявление AnsiString SendText, RecvText ??

Добавлено через 6 минут
ещё один вопрос: как можно полностью "разрушить" форму, а не закрыть методом Close() ? Дело в том, что при повторном вызове в Form1 метода Form2->Show() на второй форме остаётся всякий мусор. Кроме явного вызова деструктора на ум ничего не приходит..

0

- Max_92, что Вы, что Вы, повторю еще раз, постараюсь как можно понятнее. Приведу пример даже не
со строкой, а с простой переменной (int).
- Если Вы точно знаете, что переменная ( int first; ) должна быть определена глобально в модуле "Unit1" и не
должна быть видна ни в одном другом модуле, то просто пишете в файле "Unit1.cpp":
, и все ...Эта переменная БУДЕТ ОПРЕДЕЛЕНА ОДИН РАЗ В МОДУЛЕ "Unit1" И НЕ ВИДНА НИ В ОДНОМ ДРУГОМ МОДУЛЕ!
- Если эта переменная ( int first; ) все-же должна быть видна в других модулях проекта, то добавьте в файл
"Unit1.h" следующую запись:
- Теперь переменная ( int first; ) БУДЕТ ВИДНА ВСЮДУ, КУДА ВЫ ВКЛЮЧИТЕ ДИРЕКТИВУ ПРЕДПРОЦЕССОРА:
- Надо стараться, чтобы разные переменные одного типа по возможности не имели одинаковых имен.
Это страшный источник трудно уловимых ошибок! Ну это, конечно, не относится к рабочим переменным,
типа int i, j n; и т.д., хотя и за ними надо пристально следить. За их областью видимости!
- А все значащие для проекта переменные, строки, экземпляры классов, указатели на экземпляры классов
и т.д. надо делать с уникальными именами! Мне кажется у Вас как-раз в этом и ошибка.
Пересмотрите повнимательней имена Ваших переменных, строк, объектов ...

1

Staut, всё, спасибо. разобрался уже вчера)

0