Как ускорить работу?

@Delacroic · Регистрация: 04.12.2013

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

Прога ещё не доработана, сейчас интересует именно графический режим, когда нажимается клавиша 1-4 один из 4-х квадратов должен подсвечиваться красным, так то оно всё работает, но что бы оно подсветило нужно ткнуть раз 5-6 по клавише, вот собственно код:

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
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
// sdfsdf.cpp: определяет точку входа для консольного приложения.
//
 
#include "stdafx.h"
#include <iostream>
#include <Windows.h>
#include <time.h>
#include <ctime>
#include <tchar.h>
#include <conio.h>
#include <sstream>
#include <locale>
#include <string>
#include <dos.h>
 
using namespace std;
 
class GroupMachine
 {
   private:
           int retrr;
           int vdet;
           int countTime;         //переменная для счета времени
           int countMachine;      //кол-во станков в группе
           int timeToWorked;      //время работы 
           int * timeToEndWork;   //время до конца работы станка
           int * countDetail;     //кол-во деталей обработаных станком
           int * modeMachine;     //режим работы станка
           int * retr;
   public:                                          
          GroupMachine ()                              
          {
 
 
              cout << "BBeDuTe KoJl-BO CTAHKOB: "; cin >> countMachine;
              cout << "BBeDuTe BpeM9l Pab0Tld: ";  cin >> timeToWorked;
              
              modeMachine = new int [countMachine];
              timeToEndWork = new int [countMachine];             
              countDetail = new int [countMachine];
              retr = new int [countMachine];
              
              for(int i = 0; i < countMachine; i++)
                  {
                      timeToEndWork[i] = timeToWorked;
                      countDetail[i] = 0;
                      modeMachine[i] = 0;
                      retr[i] = 0;
                  }
          }
          ~GroupMachine()
          {
              delete [] modeMachine; modeMachine = NULL;
              delete [] timeToEndWork; timeToEndWork = NULL;
              delete [] countDetail; countDetail = NULL;
              delete [] retr; retr = NULL;
          }
        void proc(int queueMachine)   
            { 
                vdet=0;
                retrr = 0;
                for(int i=0;i<countMachine;i++)
                retr[i] = 0;
 
                for(int i=0;i<countMachine;i++)
                 {
                     if((modeMachine[i] == 0) && (queueMachine > 0)) 
                         {
                             modeMachine[i] = 1;
                             queueMachine--;
                             retr[i]++;
                         }
                     if((timeToEndWork[i] >= 0) && (modeMachine[i] == 1))
                         timeToEndWork[i]--;
                     if(timeToEndWork[i] == 0)
                     {
                         if(modeMachine[i] == 1)
                            vdet++;
                         timeToEndWork[i] = timeToWorked;
                         countDetail[i]++;
                         modeMachine[i]=0;                      
                     }
 
                      cout << endl << "Vrem'a do konca robotu= " << 
                      timeToEndWork[i] << endl << "rejim rabotu - " << 
                      modeMachine[i] << endl << "Kol-vo obrabotanuh detalei = " << 
                      countDetail[i] << endl << queueMachine << endl<<vdet;
                     
                 }
                for(int i=0;i<countMachine;i++)
                    retrr+=retr[i];
            }
        int getretrr()
            {
                return retrr;
            }
        int getvdet()
            {
                return vdet;
            }
        void graph(int rejim)
            {
                HDC hDC = GetDC( GetConsoleWindow( ) );
                HPEN WPen = CreatePen(PS_SOLID,2, RGB(255,255,255));
                HPEN RPen = CreatePen(PS_SOLID,2, RGB(255,0,0));
                HBRUSH Red = CreateSolidBrush(RGB(255,0,0));
                HBRUSH Grean = CreateSolidBrush(RGB(0,255,0));
                HBRUSH Black = CreateSolidBrush(RGB(0,0,0));
                HFONT F=CreateFont(14,0,0,0,FW_NORMAL,0,0,0,ANSI_CHARSET,OUT_DEFAULT_PRECIS,CLIP_DEFAULT_PRECIS,DEFAULT_QUALITY,DEFAULT_PITCH,_T("Arial")); 
                HPEN Grey=CreatePen(PS_SOLID,1,RGB(100,100,100));
                SetBkColor(hDC,RGB(0,0,0));
                SetTextColor(hDC,RGB(255,255,255));
                TextOut(hDC,5,460,L"Выберите группу станков с помощью цифр 1-4",56);
                SelectObject(hDC,WPen);
                SelectObject( hDC, Black);
                Rectangle(hDC,50,30,250,230);
                Rectangle(hDC,360,30,560,230);
                Rectangle(hDC,50,250,250,450);
                Rectangle(hDC,360,250,560,450);
                SelectObject(hDC,RPen);
                if (rejim==1)
                    Rectangle(hDC,50,30,250,230);
                if (rejim==2)
                    Rectangle(hDC,360,30,560,230);
                if (rejim==3)
                    Rectangle(hDC,50,250,250,450);
                if (rejim==4)
                    Rectangle(hDC,360,250,560,450);
                SelectObject(hDC,WPen);
 
 
 
            }
    
 };
 
void main()
 {
     int nomerStanka=1;
     int test;
     int qq;                        //qq - переменная для счета времени
     int *QueueMachine;             //QueueMachine - массив с очередями деталей перед каждой группой станков
     int **temp;                    //temp - создал массив для передачи состояний станков в основную программу
     QueueMachine=new int [5];   
     temp=new int *[4];     
     GroupMachine *y=new GroupMachine[4];
     time_t tm1;
     tm1 = time (NULL);
     qq=tm1;
     for(int i=0; i<4;i++)
        QueueMachine[i]=0;
     QueueMachine[0]=999999;  // перед первой группой станков очередь бесконечная
        for(;;)
            {
                system("cls");
                cout<<nomerStanka;
                for(int i=0;i<4;i++)
                {
                    (y+i)->proc(int (QueueMachine[i]));
                    y[i].graph(int (nomerStanka));
                    QueueMachine[i]-=y[i].getretrr();
                    QueueMachine[i+1]+=y[i].getvdet();                          
                }
             do
                {
                    tm1 = time (NULL);
                    if(kbhit()) 
                    if(getch()==27)
                    {
                        delete []y;
                        return;
                    }
                    if(kbhit()) 
                    if(getch()==('1'))
                    {
                        nomerStanka=1;
                    }
                    if(kbhit()) 
                    if(getch()==('2'))
                    {
                        nomerStanka=2;
                    }
                    if(kbhit()) 
                    if(getch()==('3'))
                    {
                        nomerStanka=3;
                    }
                    if(kbhit()) 
                    if(getch()==('4'))
                    {
                        nomerStanka=4;
                    }
                }
                    while(qq==tm1);
                    qq=tm1;
            }
 }

Добавлено через 11 минут
и как можно проверку на нажатие клавиши сделать отдельным методом?

@monolit · 07.12.2013, 16:46

Сдается мне, что дело в getch... Она ж вроде блокирует действия, пока не выполнится, нет?
Используй AsyncKeyState, типа:

C++
1
2
if (AsyncKeyState('1')) ///
else if (AsyncKeyState('2')) ///

Ну, ты понял. Мог ошибиться в том, что туда передавать, но это легко в гугле найти.

ЗЫ:
это вместо

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
                    if(kbhit()) 
                    if(getch()==27)
                    {
                        delete []y;
                        return;
                    }
                    if(kbhit()) 
                    if(getch()==('1'))
                    {
                        nomerStanka=1;
                    }
                    if(kbhit()) 
                    if(getch()==('2'))
                    {
                        nomerStanka=2;
                    }
                    if(kbhit()) 
                    if(getch()==('3'))
                    {
                        nomerStanka=3;
                    }
                    if(kbhit()) 
                    if(getch()==('4'))
                    {
                        nomerStanka=4;
                    }

надо вставить)

Добавлено через 4 минуты
Черт, мое сообщение пропало...
В общем, getch() блокирует ввод пока не выполнится, поэтому и приходится жать 5 раз (пять getch в цикле у тебя), ну или больше - смотря на какой ты попасть хочешь. Используй вместо всей той конструкции GetAsyncKeyState

C++
1
2
if(GetAsyncKeyState(...)) //
else ///

ну ты понял.

@Delacroic · 07.12.2013, 16:50 **[ТС]**

Не, дело не в getch, он не тормозит, это обычная проверка на нажатие клавиши, просто она так стоит, что проверяет только в определенный момент времени, а мне нужно что бы проверяла всё время =\
И отдельный метод для проверки уже сам сделал)

@monolit · 07.12.2013, 16:53

getch() должны сработать все. т.е. должны быть нажаты 5 клавиш, чтоб перейти в следующей итерации. Причем порядок нажатия клавиш тоже важен. Используй GetAsyncKeyState, говорю ж.

@Delacroic · 07.12.2013, 16:55 **[ТС]**

если клавиши не нажимать, то программа и дальше работает, всё считает, окно обновляется, но что бы проверка сработала, нужно нажать в определенный промежуток времени =\

@monolit · 07.12.2013, 17:49

main покажи...

@Delacroic · 07.12.2013, 18:01 **[ТС]**

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
void main()
 {
     int nomerStanka=1;
     int test;
     int qq;                        //qq - переменная для счета времени
     int *QueueMachine;             //QueueMachine - массив с очередями деталей перед каждой группой станков
     int **temp;                    //temp - создал массив для передачи состояний станков в основную программу
     QueueMachine=new int [5];   
     temp=new int *[4];     
     GroupMachine *y=new GroupMachine[4];
     time_t tm1;
     tm1 = time (NULL);
     qq=tm1;
     for(int i=0; i<4;i++)
        QueueMachine[i]=0;
     QueueMachine[0]=999999;  // перед первой группой станков очередь бесконечная
        for(;;)
            {
                system("cls");
                cout<<nomerStanka;
                for(int i=0;i<4;i++)
                {
                    (y+i)->proc(int (QueueMachine[i]));
                    y[i].graph(int (nomerStanka));
                    QueueMachine[i]-=y[i].getretrr();
                    QueueMachine[i+1]+=y[i].getvdet();                          
                }
             do
                {
                    tm1 = time (NULL);
                    if(kbhit()) 
                    if(getch()==27)
                    {
                        delete []y;
                        return;
                    }
                    if(kbhit()) 
                    if(getch()==('1'))
                    {
                        nomerStanka=1;
                    }
                    if(kbhit()) 
                    if(getch()==('2'))
                    {
                        nomerStanka=2;
                    }
                    if(kbhit()) 
                    if(getch()==('3'))
                    {
                        nomerStanka=3;
                    }
                    if(kbhit()) 
                    if(getch()==('4'))
                    {
                        nomerStanka=4;
                    }
                }
                    while(qq==tm1);
                    qq=tm1;
            }
 }

@monolit · 07.12.2013, 19:09

Ну ты читать не умеешь, что ли? Я тебе 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
             do
                {
                    tm1 = time (NULL);
                    if(kbhit()) 
                    if(getch()==27)
                    {
                        delete []y;
                        return;
                    }
                    if(kbhit()) 
                    if(getch()==('1'))
                    {
                        nomerStanka=1;
                    }
                    if(kbhit()) 
                    if(getch()==('2'))
                    {
                        nomerStanka=2;
                    }
                    if(kbhit()) 
                    if(getch()==('3'))
                    {
                        nomerStanka=3;
                    }
                    if(kbhit()) 
                    if(getch()==('4'))
                    {
                        nomerStanka=4;
                    }
                }
                    while(qq==tm1);

используй getAsyncKeyState. В третий или четвертый раз уже повторяю, епт.
И убери do...while, нах он там? бесконечный цикл в бесконечном цикле, да уж...

@Max Dark · 07.12.2013, 19:21

вместо кучи иф поставте один switch

C++
1
2
3
4
5
6
7
8
9
char c;
if(kbhit())
    switch(c=getch()) {
        case 27: delete []y; return;
        case '1':
        case '2':
        case '3':
        case '4': nomerStanka=c - '0'; break;
    }

@Delacroic · 07.12.2013, 19:23 **[ТС]**

while тут не бесконечный цикл -_-, это проверка прошла ли секунда -_-

Добавлено через 32 секунды
Cra3y, спс)

@monolit · 07.12.2013, 23:41

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

это проверка прошла ли секунда

Оригинально...

Новые блоги и статьи Все статьи Все блоги /
Access VikBal 11.12.2025 Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.	Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro	Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .	От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .	Поиск "дружественных имён" СОМ портов Argus19 22.11.2025 Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .
Сколько Государство потратило денег на меня, обеспечивая инсулином. Programma_Boinc 20.11.2025 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .	Ломающие изменения в C#.NStar Alpha Etyuhibosecyu 20.11.2025 Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .	Мысли в слух kumehtar 18.11.2025 Кстати, совсем недавно имел разговор на тему медитаций с людьми. И обнаружил, что они вообще не понимают что такое медитация и зачем она нужна. Самые базовые вещи. Для них это - когда просто люди. . .	Создание Single Page Application на фреймах krapotkin 16.11.2025 Статья исключительно для начинающих. Подходы оригинальностью не блещут. В век Веб все очень привыкли к дизайну Single-Page-Application . Быстренько разберем подход "на фреймах". Мы делаем одну. . .	Фото: Daniel Greenwood kumehtar 13.11.2025

@Delacroic 0 / 0 / 0 Регистрация: 04.12.2013 Сообщений: 61
	07.12.2013, 16:50 [ТС]
	Не, дело не в getch, он не тормозит, это обычная проверка на нажатие клавиши, просто она так стоит, что проверяет только в определенный момент времени, а мне нужно что бы проверяла всё время =\ И отдельный метод для проверки уже сам сделал) 0

@monolit 188 / 187 / 46 Регистрация: 24.03.2011 Сообщений: 670
	07.12.2013, 16:53
	getch() должны сработать все. т.е. должны быть нажаты 5 клавиш, чтоб перейти в следующей итерации. Причем порядок нажатия клавиш тоже важен. Используй GetAsyncKeyState, говорю ж. 0

@Delacroic 0 / 0 / 0 Регистрация: 04.12.2013 Сообщений: 61
	07.12.2013, 16:55 [ТС]
	если клавиши не нажимать, то программа и дальше работает, всё считает, окно обновляется, но что бы проверка сработала, нужно нажать в определенный промежуток времени =\ 0

@monolit 188 / 187 / 46 Регистрация: 24.03.2011 Сообщений: 670
	07.12.2013, 17:49
	main покажи... 0

@Delacroic 0 / 0 / 0 Регистрация: 04.12.2013 Сообщений: 61
	07.12.2013, 19:23 [ТС]
	while тут не бесконечный цикл -_-, это проверка прошла ли секунда -_- Добавлено через 32 секунды Cra3y, спс) 0