Вопрос с кодом, использование switch и if

[Dima-91]

Делал код по книге, используется if для движения ракетки, после компиляции возникла проблема: ракетка двигалась сама к правой части.
Изменил на switch, остальное закомментировал. Если бы требовалась оптимизация что лучше switch или if.

void __fastcall TForm1::RightLeft()
{
	//if ( rd != 0 )
   //	{
  //		if (rd = 1)
	//	{
	  //		if (rx2 < wp)
		//	{
	  switch (rd)
	  {
	  case 1:	Form1->Canvas->Pen->Color = cBack;
				Form1->Canvas->Rectangle(rx1, ry, rx1 + rdx, ry + 1);
				Form1->Canvas->Pen->Color = clRed;
				Form1->Canvas->Rectangle(rx2, ry, rx2 + rdx, ry + 1);
				rx1 += rdx;
				rx2 += rdx; return;
		//	}
   //		}
   //	}
   //	else
   //	{
	 //	if (rx1 > 1)
	   //	{
	  case 2:   Form1->Canvas->Pen->Color = cBack;
				Form1->Canvas->Rectangle(rx2, ry, rx2 - rdx, ry + 1);
				Form1->Canvas->Pen->Color = clRed;
				Form1->Canvas->Rectangle(rx1 -rdx, ry, rx1 + rdx, ry +1);
				rx1 -= rdx;
				rx2 -= rdx;      return;
 //		}
	}
}

Если убрать switch и case, return в обоих случаях, получим код с книги.

[GreenWizard]

.....лучше бы на функции код разбил))) оптимизировать тут нечего ибо 99% будет уходить на рисование

[Croessmah]

if (rd = 1)

Ух ты, как интересно ничего, что это всегда true? Или это не плюсики?

[Dima-91]

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

Ух ты, как интересно ничего, что это всегда true? Или это не плюсики?

Весь код не написан rd в начале равно 0, после значение принимает 1 или 2 в зависимости какую стрелку нажать.

[GreenWizard]

"if (rd = 1)" - записывает 1 в rd, а потом проверяет rd <> 0
"if (rd == 1)" - сравнивает 1 с rd

[Dima-91]

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

"if (rd = 1)" - записывает 1 в rd, а потом проверяет rd <> 0
"if (rd == 1)" - сравнивает 1 с rd

Реализация через switch имеет права на жизнь?
Пробовал rd==1 исправить, ракетка все равно влево улетает.

[GreenWizard]

Сообщение от Dima-91

Реализация через switch имеет права на жизнь?

да всё имеет право на жизнь......... но не будем впадать в философию)))))

Пробовал rd==1 исправить, ракетка все равно влево улетает.

это уже др. тема) нужно смотреть и думать, а изначальный вопрос был о другом)

[Dima-91]

Всем спасибо, пару скобочек нужно было отредактировать.