Dev - C++ - Opinión sobre código

 
Vista:
Imágen de perfil de Codej
Val: 25
Ha disminuido 1 puesto en Dev - C++ (en relación al último mes)
Gráfica de Dev - C++

Opinión sobre código

Publicado por Codej (8 intervenciones) el 20/09/2020 21:55:32
Buenas, espero que se encuentren bien.
Me gustaría recibir alguna opinión acerca de mi código, ya sea en qué debo mejorar, consejos, retroalimentación, etc.
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
#include<iostream>
#include<conio.h>
 
using namespace std;
 
void numerosDecCen(int num, char X, char L, char C, char D, char M){
	switch(num){
		case 10:
			cout<<X;
		break;
		case 20:
			cout<<X<<X;
		break;
		case 30:
			cout<<X<<X<<X;
		break;
		case 40:
			cout<<X<<L;
		break;
		case 50:
			cout<<L;
		break;
		case 60:
			cout<<L<<X;
		break;
		case 70:
			cout<<L<<X<<X;
		break;
		case 80:
			cout<<L<<X<<X<<X;
		break;
		case 90:
			cout<<X<<C;
		break;
		case 100:
			cout<<C;
		break;
	}
}
 
void numerosUnidades(int num, char I, char V, char X){ //sacar valores menores a IX (9)
	int numAux = num; //agregar valor num a la variable numAux para hacer las demas operaciones
	num -= 1; //usado para sacar los valores de I-II-III
	if(numAux <= 3){ //imprimir I-II-III
		for(int i = 0; i < (num+1); i++){
			cout<<I;
		}
	}
	if(numAux == 4){ //imprimir excepciones
		cout<<I<<V;
	}else if(numAux == 9){
		cout<<I<<X;
	}else if(numAux >= 5 && numAux < 9){
		cout<<V;
		numAux -= 5;
		for(int i = 0; i < numAux; i++){
			cout<<I;
		}
	}
}
 
int main(){
	char I = 'I', V = 'V', X = 'X', L = 'L', C = 'C', D = 'D', M = 'M';
	int num = 0, numUnid = 0, numDecCen = 0, numAux = 10;
 
	cout<<"Numero: ";
	cin>>num;
	numDecCen = num;
	if(num < 10){
		numUnid = num;
	}else{
		for(int i = 0; i < 100; i++){
			if(num >= numAux){
				numUnid = num - numAux;
				numDecCen = numAux;
			}
			numAux += 10;
		}
	}
 
	numerosDecCen(numDecCen, X, L, C, D, M);
	numerosUnidades(numUnid, I, V, X);
	cout<<"\nPresione una tecla para salir"<<endl;
 
	getch();
	return 0;
}
Valora esta pregunta
Me gusta: Está pregunta es útil y esta claraNo me gusta: Está pregunta no esta clara o no es útil
0
Responder
Imágen de perfil de Alfil
Val: 4.344
Oro
Ha mantenido su posición en Dev - C++ (en relación al último mes)
Gráfica de Dev - C++

Opinión sobre código

Publicado por Alfil (1444 intervenciones) el 20/09/2020 23:01:22
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
#include <iostream>
 
using namespace std;
 
void deDecimalARomano(int numero);
 
int main ()
{
    int maximo;
 
    cout << "Ingrese un numero: ";
    cin >> maximo;
 
    for (int i = 1; i <= maximo; i++)
    {
        deDecimalARomano(i);
        cout << "\n";
    }
 
    return 0;
 
}
 
void deDecimalARomano(int numero)
{
    int unidades, decenas, centenas, millares;
 
    unidades = numero % 10; numero /= 10;
    decenas = numero % 10; numero /= 10;
    centenas = numero % 10; numero /= 10;
    millares = numero % 10; numero /= 10;
 
    switch(millares)
    {
        case 1: cout << "M"; break;
        case 2: cout << "MM"; break;
        case 3: cout << "MMM"; break;
    }
 
    switch(centenas)
    {
        case 1: cout << "C"; break;
        case 2: cout << "CC"; break;
        case 3: cout << "CCC"; break;
        case 4: cout << "CD"; break;
        case 5: cout << "D"; break;
        case 6: cout << "DC"; break;
        case 7: cout << "DCC"; break;
        case 8: cout << "DCCC"; break;
        case 9: cout << "CM"; break;
    }
 
    switch(decenas)
    {
        case 1: cout << "X"; break;
        case 2: cout << "XX"; break;
        case 3: cout << "XXX"; break;
        case 4: cout << "XL"; break;
        case 5: cout << "L"; break;
        case 6: cout << "LX"; break;
        case 7: cout << "LXX"; break;
        case 8: cout << "LXXX"; break;
        case 9: cout << "XC"; break;
    }
 
    switch(unidades)
    {
        case 1: cout << "I"; break;
        case 2: cout << "II"; break;
        case 3: cout << "III"; break;
        case 4: cout << "IV"; break;
        case 5: cout << "V"; break;
        case 6: cout << "VI"; break;
        case 7: cout << "VII"; break;
        case 8: cout << "VIII"; break;
        case 9: cout << "IX"; break;
    }
}
Valora esta respuesta
Me gusta: Está respuesta es útil y esta claraNo me gusta: Está respuesta no esta clara o no es útil
0
Comentar