неработает switch + for

[Кащей]

Такая конструкция не проходит дальше case 0: все три итерации цикла, почему?

int i, n;
for(n = 0; n < 3; n++)
{
   switch(n){
         case 0:
                n+=i;
         case 1:
                n-=i;
         case 2:
                n*=i;
   }
}

[Smogg]

i создается на стеке, а значит содержит мусор, отчего и поведение конструкции зависит от фазы луны.

[rrrFer]

при первой итерации цикла выполнятся все 3 кэйза - ты прибавишь к n мусор, потом его же отнимешь (n станет равным нулю) и умножишь ноль на мусор - n останеца равным нулю.
На следующей итерации n будет равным 1, поэтому выполняться только 2 последних кэйза.
n будет содержать [мусор в квадрате].
И дальше да, поведение зависит от фазы луны. (третьей итерации может и не быть)

[Стертор]

Бог ты мой, сколько слов, неужели нельзя сказать, что нужно обнулить i?

То, что переменная создана, это ЕЩЕ НЕ ЗНАЧИТ, что она пустая (==0).

[_Bers]

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

Бог ты мой, сколько слов, неужели нельзя сказать, что нужно обнулить i?

То, что переменная создана, это ЕЩЕ НЕ ЗНАЧИТ, что она пустая (==0).

Просто обнулить переменную i ненадостаточно.
Потому что, в итоге образуется вечный цикл:

int main()
{
    int i=0, n;  //<--- обнулили
    for(n = 0; n < 3; n++)
    {
       switch(n)
        {
             case 0: n+=i;
             case 1: n-=i;
             case 2: n*=i;
        }
    }

    //<--- сюда мы уже никогда не попадем
}

Но если вы прочитаете подробные объяснения процесса, любезно предоставленные выше, то вы сможете понять, в чем именно вы не правы.

[Стертор]

Гм, правда. Один ноль В Вашу пользу. Я не обратил внимание, что он обращается к счетчику цикла -совсем не гламурно.
Ну ладно-с, пущай. Просто интересно: укусит ли дракон хвост?

[Кащей]

Пардон, я затупил и неправильно задал вопрос. Конструкцию на самом деле я хотел написать такую:

int i, n;

for(i = 0; i < 3; i++){
	switch(i){
	
		case 0:
			n = 2 + 2;
		
		case 1:
			n = 2 + 3;
		
		case 2:
			n = 2 + 4;
	}
}

Аналог того что я пытался использовать и происходило действие case 0: каждую итерацию.

[Стертор]

Поздно, дружок:
http://govnokod.ru/13488

[Кащей]

Поздно, дружок

переживу. давай по теме.

[Кащей]

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

http://ideone.com/sK7dJc

Это я видел, конструкция сейчас другая.
Вот тебе тот аналог, рекурсивная функция для выделения совместных пикселей.
В первом случае проверка условия через if во втором через switch, в первом случае работает, во втором нет.
Вопрос - почему?

void mb_sim_pix(uchar* IN, uchar* OUT, int *WH, int *bSize, int *A, int *pSize, float *threshold){
	int pos = mb_calc_pos(A, WH, pSize);
	int nPos;
	int i, n;
	int B[2];
		
	for(n = 0; n < 4; n++){
		
		if(n == 0){
			B[0] = A[0] + 1;
			B[1] = A[1];
		}
		else if(n == 1){
			B[0] = A[0];
			B[1] = A[1] - 1;
		}
		else if(n == 2){
			B[0] = A[0];
			B[1] = A[1] + 1;
		}
		else if(n == 3){
			B[0] = A[0] - 1;
			B[1] = A[1];
		}
				
		if(	B[0] >= 0 && B[0] < WH[0] &&
			B[1] >= 0 && B[1] < WH[1]){
			nPos = mb_calc_pos(B, WH, pSize);
			if(mb_val_pix(OUT, &nPos, pSize) == 0){
				if(mb_sim_pack(IN, &pos, &nPos, pSize) >= *threshold){
					for(i = 0; i < *pSize; i++) OUT[nPos + i] = IN[nPos + i];
					mb_sim_pix(IN, OUT, WH, bSize, B, pSize, threshold);
				}
			}
		}
	}
}

uchar* mb_magic_wand(uchar* IN, int A[2], int WH[2], int pSize, float threshold){
	int bSize = WH[0] * WH[1] * pSize;
	uchar* OUT = malloc(bSize);
		if(OUT != NULL){
			memset(OUT, 0, bSize);
			mb_sim_pix(IN, OUT, WH, &bSize, A, &pSize, &threshold);
		}
	return OUT;
}

void mb_sim_pix(uchar* IN, uchar* OUT, int *WH, int *bSize, int *A, int *pSize, float *threshold){
	int pos = mb_calc_pos(A, WH, pSize);
	int nPos;
	int i, n;
	int B[2];
		
	for(n = 0; n < 4; n++){
		
		switch(n){
			case 0:
				B[0] = A[0] + 1;
				B[1] = A[1];
			
			case 1:
				B[0] = A[0];
				B[1] = A[1] - 1;
			
			case 2:
				B[0] = A[0];
				B[1] = A[1] + 1;
			
			case 3:
				B[0] = A[0] - 1;
				B[1] = A[1];
		}
		
		if(	B[0] >= 0 && B[0] < WH[0] &&
			B[1] >= 0 && B[1] < WH[1]){
			nPos = mb_calc_pos(B, WH, pSize);
			if(mb_val_pix(OUT, &nPos, pSize) == 0){
				if(mb_sim_pack(IN, &pos, &nPos, pSize) >= *threshold){
					for(i = 0; i < *pSize; i++) OUT[nPos + i] = IN[nPos + i];
					mb_sim_pix(IN, OUT, WH, bSize, B, pSize, threshold);
				}
			}
		}
	}
}