Форум программистов, компьютерный форум CyberForum.ru

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

Восстановить пароль Регистрация
 
Delacroic
0 / 0 / 0
Регистрация: 04.12.2013
Сообщений: 61
07.12.2013, 16:23     Как ускорить работу? #1
Прога ещё не доработана, сейчас интересует именно графический режим, когда нажимается клавиша 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
179 / 179 / 21
Регистрация: 24.03.2011
Сообщений: 641
Завершенные тесты: 1
07.12.2013, 16:46     Как ускорить работу? #2
Сдается мне, что дело в 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
0 / 0 / 0
Регистрация: 04.12.2013
Сообщений: 61
07.12.2013, 16:50  [ТС]     Как ускорить работу? #3
Не, дело не в getch, он не тормозит, это обычная проверка на нажатие клавиши, просто она так стоит, что проверяет только в определенный момент времени, а мне нужно что бы проверяла всё время =\
И отдельный метод для проверки уже сам сделал)
monolit
179 / 179 / 21
Регистрация: 24.03.2011
Сообщений: 641
Завершенные тесты: 1
07.12.2013, 16:53     Как ускорить работу? #4
getch() должны сработать все. т.е. должны быть нажаты 5 клавиш, чтоб перейти в следующей итерации. Причем порядок нажатия клавиш тоже важен. Используй GetAsyncKeyState, говорю ж.
Delacroic
0 / 0 / 0
Регистрация: 04.12.2013
Сообщений: 61
07.12.2013, 16:55  [ТС]     Как ускорить работу? #5
если клавиши не нажимать, то программа и дальше работает, всё считает, окно обновляется, но что бы проверка сработала, нужно нажать в определенный промежуток времени =\
monolit
179 / 179 / 21
Регистрация: 24.03.2011
Сообщений: 641
Завершенные тесты: 1
07.12.2013, 17:49     Как ускорить работу? #6
main покажи...
Delacroic
0 / 0 / 0
Регистрация: 04.12.2013
Сообщений: 61
07.12.2013, 18:01  [ТС]     Как ускорить работу? #7
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
179 / 179 / 21
Регистрация: 24.03.2011
Сообщений: 641
Завершенные тесты: 1
07.12.2013, 19:09     Как ускорить работу? #8
Ну ты читать не умеешь, что ли? Я тебе 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
В поиске работы
 Аватар для Max Dark
1546 / 1399 / 501
Регистрация: 09.10.2013
Сообщений: 3,185
Записей в блоге: 8
Завершенные тесты: 2
07.12.2013, 19:21     Как ускорить работу? #9
вместо кучи иф поставте один 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
0 / 0 / 0
Регистрация: 04.12.2013
Сообщений: 61
07.12.2013, 19:23  [ТС]     Как ускорить работу? #10
while тут не бесконечный цикл -_-, это проверка прошла ли секунда -_-

Добавлено через 32 секунды
Cra3y, спс)
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
07.12.2013, 23:41     Как ускорить работу?
Еще ссылки по теме:

C++ Можно ли как нибудь ускорить работу цикла for?
C++ Как ускорить пирамидальную сортировку?
Задача про рюкзак - ускорить работу программы C++

Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
monolit
179 / 179 / 21
Регистрация: 24.03.2011
Сообщений: 641
Завершенные тесты: 1
07.12.2013, 23:41     Как ускорить работу? #11
Цитата Сообщение от Delacroic Посмотреть сообщение
это проверка прошла ли секунда
Оригинально...
Yandex
Объявления
07.12.2013, 23:41     Как ускорить работу?
Ответ Создать тему
Опции темы

Текущее время: 03:07. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru