[C] problema esercizio semplice

[nongio]

Ciao a tutti, ho da fare un esercizio che prende in input un numero di 4 cifre, devo decriptarlo nel seguente modo:

Prendere ogni cifra e modificarla cosi : sommare la cifra a 7 e dividere per 10, utilizzare il resto della divisione come numero cifrato, e scambiare la 4a cifra con la seconda e la 3a con la prima...
io l'ho fatto cosi'

#include <stdio.h>

int main ()

{

	int num;
	int cif1;
	int cif2;
	int cif3;
	int cif4;
	int crit1;
	int crit2;
	int crit3;
	int crit4;
	

	printf ("Inserire un numero di 4 cifre:");
	scanf ("%d", &num);
	
	while (num != -1)
		
		{

			cif4 = num/1000;
			cif3 = num/100 - cif4 * 10;
			cif2 = num/10 - cif3 * 10 - cif4 * 100;
			cif1 = num/1 - cif2 * 10 - cif3 * 100 - cif4 * 1000;
			
			crit4 = (cif4 + 7) % 10;
			crit3 = (cif3 + 7) % 10;
			crit2 = (cif2 + 7) % 10;
			crit1 = (cif1 + 7) % 10;
			printf ("%d%d%d%d\n\n", crit2, crit1, crit4, crit3);
		
			printf ("Inserire un numero di 4 cifre:");
			scanf ("%d", &num);
			
		}
			

	return 0;
}

Questo funziona...
Ora devo fare il procedimento inverso...solo che ho dei problemi... non mi funziona come vorrei...
Io l'ho scritto cosi':

#include <stdio.h>

int main ()

{

	int num;
	int cif1;
	int cif2;
	int cif3;
	int cif4;
	int crit1;
	int crit2;
	int crit3;
	int crit4;
	

	printf ("Inserire un numero di 4 cifre:");
	scanf ("%d", &num);
	
	while (num != -1)
		
		{

			crit4 = num/1000;
			crit3 = num/100 - cif4 * 10;
			crit2 = num/10 - cif3 * 10 - cif4 * 100;
			crit1 = num/1 - cif2 * 10 - cif3 * 100 - cif4 * 1000;
			
			if (crit4 >= 7)
			{
				cif4 = crit4  -7;
                        }
			else
			{
				cif4 = crit4 + 10 -7;
			}
			if ( crit3 >= 7)
			{
				cif3 = crit3  -7;
			}
			else
			{
				cif3 = crit3 +10 -7;
			}
			if (crit2 >= 7)
			{
				cif2 = crit2 -7;
			}
			else
			{
				cif2 = crit2 +10-7;
			}
			if (crit1 >=7)
			{
				cif1 = crit1 -7;
			}
			else
			{
				cif1 = crit1 +10 -7;
			}
			
			printf ("%d%d%d%d\n\n", cif2, cif1, cif4, cif3);
		
			printf ("Inserire un numero di 4 cifre:");
			scanf ("%d", &num);
			
		}
			

	return 0;
}

vi posto un esempio di quel che succede

$ ./crittog
Inserire un numero di 4 cifre:1234
0189

$ ./decripter
Inserire un numero di 4 cifre:0189
203905919711311530503-91076686

non torna 1234!!!!!!!!!!!!!!!!!!!!!
mi aiutate

[yorkeiser]

Allora:

- In primo luogo, ti consiglio di studiare gli array; ti risparmieresti un sacco di codice da scrivere. Per scrivere un semplice programmino hai dovuto incollare una pagina di codice, non ti pare un po' troppo?

- L'errore è qui, ed è duplice:

crit4 = num/1000;
crit3 = num/100 - cif4 * 10;
crit2 = num/10 - cif3 * 10 - cif4 * 100;
crit1 = num/1 - cif2 * 10 - cif3 * 100 - cif4 * 1000;

1) Il primo è concettuale: hai dichiarato cif1,2,3,4 e li utilizzi nella parte destra di queste espressioni ma NON LI HAI INIZIALIZZATI; mai utilizzare una variabile in c/c++ senza prima averla inizializzata. Ad esempio:

 cif1 = cif2 = cif3 = cif4 = 0 ;

2) crit3 deve essere = num/100 - crit3*10, non cif3
Lo stesso vale per crit2 e crit1: devi sempre usare critX, e non cifX nelle espressioni a destra dell'uguale

Impara ad usare gli array!
Ciao

[nongio]

A parte che sono un pirla e tu hai ragione 'errore era banalissimo e non lo trovavo...pensa te..
Non ho utilizzato gli array perche' l'esercizio non richiedeva l'utilizzo di array... cioe' dovevo farlo solo e soltanto con l'if e il while..

grazie mille!