Объявлении хендла глобально и в классе

@Korbofos · Регистрация: 15.08.2011

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

При объявлении хендла глобально всё работает нормально. При объявлении хендла в классе компиляция проходит нормально, но при попытке открыть порт вылетает какая то ошибка.

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
class Com
{
protected:
static unsigned char rx_buf[BUFSIZE], tx_buf[BUFSIZE];
static unsigned char tx_count, rx_index, rx_count;
public:
HANDLE COMport;
void open(void);
void pclose(void);
void write(unsigned char data);
void send(void);
unsigned char read(void);
void clear_data(void);
};

В чём дело?

@Avazart · 29.03.2012, 21:18

Ну а конструктор где? Где присвоение HANLE-у его значения?

@Korbofos · 29.03.2012, 21:29 **[ТС]**

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
70
71
72
73
74
void Com::open(void)
{
String portname;        
 DCB dcb;              
 COMMTIMEOUTS timeouts; 
 
 portname = Form1->ComboBox1->Text; 
 
 
 COMport = CreateFile(portname.c_str(),GENERIC_READ | GENERIC_WRITE, 0, NULL, OPEN_EXISTING, FILE_FLAG_OVERLAPPED, NULL);
 
 
 if(COMport == INVALID_HANDLE_VALUE)           
  {
          
   Form1->Message(1);     
   return;
  }
 
 
 
 dcb.DCBlength = sizeof(DCB);   
 
 
 if(!GetCommState(COMport, &dcb))
  {
   pclose();
   Form1->Message(2);
   return;
  }
 
 
 dcb.BaudRate = 9600;     
 dcb.fBinary = TRUE;                                    
 dcb.fOutxCtsFlow = FALSE;                             
 dcb.fOutxDsrFlow = FALSE;                             
 dcb.fDtrControl = DTR_CONTROL_DISABLE;               
 dcb.fDsrSensitivity = FALSE;                           
 dcb.fNull = FALSE;                                     
 dcb.fRtsControl = RTS_CONTROL_DISABLE;                
 dcb.fAbortOnError = FALSE;                            
 dcb.ByteSize = 8;                                     
 dcb.Parity = 0;                                      
 dcb.StopBits = 0;                                     
 
 
 if(!SetCommState(COMport, &dcb))   
  {
  pclose();
   Form1->Message(3);
   return;
  }
 
 
 timeouts.ReadIntervalTimeout = 0;      
 timeouts.ReadTotalTimeoutMultiplier = 0;   
 timeouts.ReadTotalTimeoutConstant = 0;        
 timeouts.WriteTotalTimeoutMultiplier = 0;     
 timeouts.WriteTotalTimeoutConstant = 0;      
 
 
 if(!SetCommTimeouts(COMport, &timeouts))   
  {
   pclose();
   Form1->Message(4);
   return;
  }
 
 
 SetupComm(COMport,2000,2000);
 
 PurgeComm(COMport, PURGE_RXCLEAR);
 Form1->Message(5);
 }

@Avazart · 29.03.2012, 22:12

А где класс объявлен может он не видет Form1->...

C++
1
2
3
4
void Com::open(void)
{ 
///....
portname = Form1->ComboBox1->Text;

при попытке открыть порт вылетает какая то ошибка.

А какая конкретно?

@Korbofos · 30.03.2012, 05:52 **[ТС]**

Нет дело не в этом!
Когда объявляю хендл глобально всё работает отлично!
А вот ошибка при объявлении в классе.

@Korbofos · 30.03.2012, 20:55 **[ТС]**

Везде в примерах хендл объявлен в классе у меня так не работает

Добавлено через 9 минут
Объявил хендл как статик заработало

Мой билдер от сатаны?

@Avazart · 30.03.2012, 20:57

А метод pclose можете показать?
И что за конструкция:

C++
1
Form1->Message(2);

?

@Korbofos · 30.03.2012, 21:08 **[ТС]**

C++
1
2
3
4
5
6
7
void Com::pclose(void)
{
CloseHandle(COMport);
COMport=0;
//close(handle);
//handle=0;
}

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
void __fastcall TForm1::Message(unsigned char mes)
{
static unsigned char pre_mes=0;
if(pre_mes!=mes)
{
 switch(mes)
  {
  case 0:
  break;
 
  case 1:
  StatusBar1->SimpleText = "Error open Com port";
  break;
 
  case 2:
  StatusBar1->SimpleText  = "Error read DCB";
  break;
 
  case 3:
  StatusBar1->SimpleText  = "Error set DCB";
  break;
 
  case 4:
  StatusBar1->SimpleText  = "Error set taim-out";
  break;
 
  case 5:
  Connect->Enabled=false;
  Disconnect->Enabled=true;
  Robo_ON->Enabled=true;
  Manual_ON->Enabled=true;
  ComboBox1->Enabled=false;
  ComboBox2->Enabled=true;
  StatusBar1->SimpleText = "Com port is ready";
  break;
 
  case 6:
  StatusBar1->SimpleText  = "Error transmit data";
  break;
 
  case 7:
  StatusBar1->SimpleText  = "Please wait...";
  break;
 
  case 8:
  StatusBar1->SimpleText  = "In work...";
  break;
 
  case 9:
  StatusBar1->SimpleText  = "Ready to work...";
  break;
 
  case 10:
  StatusBar1->SimpleText = "Please select program to work";
  break;
  }
}
pre_mes=mes;
}

Добавлено через 3 минуты
Ну и как статик не совсем корректно работает, открываю закрываю порт, при повторном открытие тоже программа вылетает.

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
class Com
{
protected:
static unsigned char rx_buf[BUFSIZE], tx_buf[BUFSIZE];
static unsigned char tx_count, rx_index, rx_count;
static HANDLE COMport;
static OVERLAPPED overlappedwr;
static OVERLAPPED overlapped;
public:
void open(void);
void pclose(void);
void write(unsigned char data);
void send(void);
unsigned char read(void);
void clear_data(void);
};
 
extern Com *c;

@Avazart · 30.03.2012, 21:15

У меня вроде никаких ошибок не выбивает( Несмотря на что у меня вообще COM-порта нет)
Но подозреваю что ошибка в

C++
1
CloseHandle(COMport);

Да и конструктора не заметил у вас в котором COMport=NULL;

@Korbofos · 30.03.2012, 21:23 **[ТС]**

Вылетает и всё тут! Avazart если вас не затруднит посмотрите прогу.

@Korbofos · 30.03.2012, 21:25 **[ТС]**

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

C++
1
CloseHandle(COMport);

До этого и дойти не успевает.

@Avazart · 30.03.2012, 21:40

C++
1
2
3
4
5
void __fastcall TForm1::FormCreate(TObject *Sender)
{
c = new Com;
//....
}

И

C++
1
2
3
4
void __fastcall TForm1::FormClose(TObject *Sender, TCloseAction &Action)
{
delete c;
}

Да еще возможно стоит подумать о деструкторе COM

@Korbofos · 30.03.2012, 22:27 **[ТС]**

Спасибо так заработало нормально! А почему надо создавать дополнительно объект Сом? Я так и не понял.

@Avazart · 30.03.2012, 22:29

Ну у тебя то только указатель объявлен, а объекта нет, а значит существуют только статические члены класса.

Можно было просто объявить без динамического создания и указателя

C++
1
2
3
Com c;
//....
c.open();

@Korbofos · 31.03.2012, 11:52 **[ТС]**

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
class Com
{
protected:
static unsigned char rx_buf[BUFSIZE], tx_buf[BUFSIZE];
static unsigned char tx_count, rx_index, rx_count;
HANDLE COMport;
OVERLAPPED overlappedwr;
OVERLAPPED overlapped;
public:
Com(String portname);
~Com();
void write(unsigned char data);
void send(void);
unsigned char read(void);
void clear_data(void);
};
 
c(Form1->ComboBox1->Text) = new Com;

?
Как с делать по указателю и конструктором с аргументом?
так выдаёт ошибку.

[C++ Error] Unit1.cpp(77): E2314 Call of nonfunction
[C++ Error] Unit1.cpp(77): E2285 Could not find a match for 'Com::Com()'

@Avazart · 31.03.2012, 14:04

C++
1
c= new Com(Form1->ComboBox1->Text) ;

@Korbofos · 31.03.2012, 14:17 **[ТС]**

Спс

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
void __fastcall TForm1::ConnectClick(TObject *Sender)
{
 
c = new Com(Form1->ComboBox1->Text);
 
}
//---------------------------------------------------------------------------
 
void __fastcall TForm1::DisconnectClick(TObject *Sender)
{
delete c;
Connect->Enabled=true;
Disconnect->Enabled=false;
Robo_ON->Enabled=false;
ComboBox1->Enabled=true;
Manual_ON->Enabled=false;
Form1->Message(11);
}

C++
1
2
3
4
Com::Com(String portname)
{
//bla bla bla
}

Конструктор и деструктор в ставил в кнопки. Я больше не представляю куда конструктор "засунуть"
Чтобы при нажатии кнопки целятся к порту.
Но снова появилась проблема при открытии, работой, закрытии порта и повторном открытии порта, прога снова вылетает.

@Avazart · 31.03.2012, 15:14

Место ошибки можно обнаружить ставя брекпоинты и троссируя пошагово кнопками F7 и F8.

@Korbofos · 31.03.2012, 16:10 **[ТС]**

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
unsigned char Com::read(void)
{
unsigned char data;
if(!rx_count)
{
 
COMSTAT comstat;
DWORD btr=0, temp=0, mask=0, signal=0;
overlapped.hEvent = CreateEvent(NULL, true, true, NULL);
SetCommMask(COMport, EV_RXCHAR);
 
 do{
    WaitCommEvent(COMport, &mask, &overlapped);
    signal = WaitForSingleObject(overlapped.hEvent, INFINITE);
    if(signal == WAIT_OBJECT_0)
     {
      if(GetOverlappedResult(COMport, &overlapped, &temp, true))
       if((mask & EV_RXCHAR)!=0)
        {
         ClearCommError(COMport, &temp, &comstat);
         btr = comstat.cbInQue;
         if(btr)
         {
          ReadFile(COMport, rx_buf, btr, &temp, &overlapped);
          rx_count+=btr;
 
         }
        }
     }
 
 }while(!rx_count); //здесь ошибка
CloseHandle(overlapped.hEvent);
}
data=rx_buf[rx_index++];
if(!--rx_count)rx_index=0;
return data;
}

Намудрил я тут

Добавлено через 38 минут
Всё разобрался вынес

C++
1
2
overlapped.hEvent = CreateEvent(NULL, true, true, NULL);
 SetCommMask(COMport, EV_RXCHAR);

В конструктор

Сколько изучать приходиться для роботостроения

Новые блоги и статьи Все статьи Все блоги /
Программный контроль заполнения реквизита табличной части документа Maks 02.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "СписаниеМатериалов", разработанного в конфигурации КА2. Задача: реализовать контроль заполнения реквизита "ПричинаСписания". . .	wmic не является внутренней или внешней командой Maks 02.04.2026 Решение: DISM / Online / Add-Capability / CapabilityName:WMIC~~~~ Отсюда: https:/ / winitpro. ru/ index. php/ 2025/ 02/ 14/ komanda-wmic-ne-naydena/	Программная установка даты и запрет ее изменения Maks 02.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "СписаниеМатериалов", разработанного в конфигурации КА2. Задача: при создании документов установить период списания автоматически. . .	Вывод данных в справочнике через динамический список Maks 01.04.2026 Реализация из решения ниже выполнена на примере нетипового справочника "Спецтехника" разработанного в конфигурации КА2. Задача: вывести данные из ТЧ нетипового документа. . .
Программное заполнения текстового поля в реквизите формы документа Maks 01.04.2026 Алгоритм из решения ниже реализован на нетиповом документе "ВыдачаОборудованияНаСпецтехнику" разработанного в конфигурации КА2, в дополнении к предыдущему решению. На форме документа создается. . .	К слову об оптимизации kumehtar 01.04.2026 Вспоминаю начало 2000-х, университет, когда я писал на Delphi. Тогда среди программистов на форумах активно обсуждали аккуратную работу с памятью: нужно было следить за переменными, вовремя. . .	Идея фильтра интернета (сервер = слой+фильтр). Hrethgir 31.03.2026 Суть идеи заключается в том, чтобы запустить свой сервер, о чём я если честно мечтал давно и давно приобрёл книгу как это сделать. Но не было причин его запускать. Очумелые учёные напечатали на. . .	Модель здравосоХранения 6. ESG-повестка и устойчивое развитие; углублённый анализ кадрового бренда anaschu 31.03.2026 В прикрепленном документе раздумья о том, как можно поменять модель в будущем

@Avazart 8488 / 6155 / 615 Регистрация: 10.12.2010 Сообщений: 28,683 Записей в блоге: 30
	29.03.2012, 21:18
	Ну а конструктор где? Где присвоение HANLE-у его значения? 1

@Korbofos 11 / 11 / 2 Регистрация: 15.08.2011 Сообщений: 456
	30.03.2012, 05:52 [ТС]
	Нет дело не в этом! Когда объявляю хендл глобально всё работает отлично! А вот ошибка при объявлении в классе. Миниатюры 0

@Korbofos 11 / 11 / 2 Регистрация: 15.08.2011 Сообщений: 456
	30.03.2012, 20:55 [ТС]
	Везде в примерах хендл объявлен в классе у меня так не работает Добавлено через 9 минут Объявил хендл как статик заработало Мой билдер от сатаны? 0

@Korbofos 11 / 11 / 2 Регистрация: 15.08.2011 Сообщений: 456
	30.03.2012, 22:27 [ТС]
	Спасибо так заработало нормально! А почему надо создавать дополнительно объект Сом? Я так и не понял. 0

@Avazart 8488 / 6155 / 615 Регистрация: 10.12.2010 Сообщений: 28,683 Записей в блоге: 30
	31.03.2012, 15:14
	Место ошибки можно обнаружить ставя брекпоинты и троссируя пошагово кнопками F7 и F8. 1