Оптимизировать код(C++)

Сейчас на форуме: morgot, Rio, CDK123, zds, tyns777, tihiy_grom, rmn (+4 невидимых)

Посл.ответ	Сообщение
Oott Ранг: 106.9 (ветеран), 27thx Активность: 0.08↘0 Статус: Участник	Создано: 10 октября 2008 20:16 · Личное сообщение · #1 Нужно было написать для одной игры элемент-> рандомный вывод объктов(мишеней) по числу от 1-3,также их расположение(x,y) тоже рандомное,но должно быть по полу кругу. Написать то получилось, но для этого объекиа кода слишком много вышло, чёта ни чё в голову не приходит, можно бы было сделать используя switch, но особо ни чё не уменьшится . Ну вобщем-то во код: Code: void TForm1::mishen() { int n1,n2,i; int yn1,yn2; yn1=345/2+40; //переменные координат. ml=random(3); //определяем количество мишеней, Canvas->Brush->Color=clBlue; Canvas->Pen->Color=clRed; while(1){ xm[0]=random(345)+25; n1=xm[0]; yn2=abs(yn1-xm[0])/2; ym[0]=yn2yn2/100+28; Canvas->Ellipse(xm[0]-20,ym[0]-20,xm[0]+20,ym[0]+20); if(ml==0){break;} //вычислям мс 1ой мишени. while(i<=1) {xm[1]=random(345)+25; if(abs(xm[1]-n1)>40) { yn2=abs(yn1-xm[1])/2; ym[1]=yn2yn2/100+28; Canvas->Ellipse(xm[1]-20,ym[1]-20,xm[1]+20,ym[1]+20);n2=xm[1];i=2;} } if(ml==1){break;} //вычислям мс 2ой мишени. while(i<=2) {xm[2]=random(345)+25; if(abs(xm[2]-n1)>40 && abs(xm[2]-n2)>40) {yn2=abs(yn1-xm[2])/2; ym[2]=yn2*yn2/100+28; Canvas->Ellipse(xm[2]-20,ym[2]-20,xm[2]+20,ym[2]+20);i=3;} }break;} //вычислям мс 3ой мишени. } xm[] и ym[] глобальные,345 это нечто вроди максимума мо х.(может чё забыл) Хотелось бы сделать, как можно прямее и оптимизированее)

arnix Ранг: 210.5 (наставник), 2thx Активность: 0.14↘0 Статус: Участник	Создано: 10 октября 2008 20:37 · Личное сообщение · #2 Этот участок Code: yn2 = abs(yn1 - xm[2]) / 2; ym[2] = yn2 * yn2 / 100 + 28; Canvas->Ellipse(xm[2] - 20, ym[2] - 20, xm[2] + 20, ym[2] + 20); используется в 3-х местах с разными индексами, сделай функцию. И еще, что там за извращения с i? ) Он даже не инициализирован.
Oott Ранг: 106.9 (ветеран), 27thx Активность: 0.08↘0 Статус: Участник	Создано: 10 октября 2008 20:51 · Поправил: Oott · Личное сообщение · #3 Можно сделать функцией, но в проекте, где восновном одни большие функи как-то не прикольно, вот если ченить по глобальней там зафунчить) i там используется, как флаг выхода из цикла, это чёта я оставил ещё когда у меня было всё одним цыклом и тогда действительно нужен бы этот флаг, ща надо на break; заменить) P.S. мне сам алгоритм не нравится О_о
Exaktus_ Ранг: 20.9 (новичок), 4thx Активность: 0.01↘0 Статус: Участник	Создано: 12 октября 2008 12:59 · Личное сообщение · #4 Как минимум Code: int yn2, yn1=345/2+40; ..... n1=xm[0]=random(345)+25;
Oott Ранг: 106.9 (ветеран), 27thx Активность: 0.08↘0 Статус: Участник	Создано: 12 октября 2008 19:11 · Поправил: Oott · Личное сообщение · #5 Exaktus_, спасибо за правку синтаксиса, но я хотел алгоритм изменить потому, как 4 while(1) ни в какие ворота не лезит. Ну вобщем то проблема уже решена, спасибо моему наставнику) (Player)

Для печати