Hola gente estoy haciendo un ejercicio de un libro que tengo, y nose por qué el programa no me da resultados.....

Dejo el ejercicio así saben de que se trata y por si alguien le gustaría hacerlo (no estoy pidiendo que me hagan el ejercicio)

El ejercicio dice lo siguiente:

Ok, el código que hice que no me imprime nada solo la pregunta del main es:

Main.cpp

#include <iostream>
using std::cin;
using std::cout;
 
#include "Binario.h"
 
int main()
{
	int numero;
	cout <<"Ingrese el numero binario para mostrar su equivalente decimal: ";
	cin >> numero;
	Binario Binario1(numero);
	cin.get();
	cin.get();
	return 0;
}

Binario.h

class Binario
{
public:
	Binario(int);
	void confirmarNumero(int);
 
};

Binario.cpp

#include <iostream>
using std::cout;
 
#include "Binario.h"
 
 
Binario::Binario(int numero)
{
	confirmarNumero(numero);
}
 
void Binario::confirmarNumero(int numero)
{
	int numeroDivisor = 10;
	int escala = 1;
	int numeroSacado;
 
	while ((numero / numeroDivisor) == 0)
	{
 
 
 
		numeroSacado= numero % numeroDivisor;
 
		if (escala == 1)
		{
		cout <<numeroSacado <<"*1 " ;
		}
		else
		{
			cout <<numeroSacado <<"*" <<escala <<" ";
		}
 
		numeroDivisor= numeroDivisor*10;
	}
}
 

PD: tengo que desarollar el tema de la variable escala pero eso no influye en el resultado del programa ejecutado

Hola eferion, cambie el código de la función por este:

void Binario::confirmarNumero(int numero)
{
	int numeroDivisor= 10;
	int escala = 1;
	int numeroSacado;
 
	cout <<"El equivalente decimal  del numero binario 1101 es: ";
 
	while ((numero / numero) != 0)
	{
		numeroSacado= numero % numeroDivisor;
		numero= numero / numeroDivisor;
		if (escala != 1)
		{
			cout <<numeroSacado <<"*" <<escala << " ";
			escala*= 2;
		}
		if (escala == 1)
		{
			cout <<numeroSacado <<"*" <<escala << " ";
			++escala;
		}
 
	}
}

Pero no logo entender cual es el error ahora  

PD: con respecto a lo que me dijiste sobre string no entendí muy bien a qué te refieres

Blaster, perfectamente podrías ahorrarte la variable "escala". Además, teniendo en cuenta que "numero" es una especie de "engendro" por almacenar un número en base 2 como si estuviese en base 10, el código incluso se podría optimizar un poquito más:

void Binario::confirmarNumero(int numero)
{
    int decimal = 0;
 
    while (numero > 0)
    {
        int numeroSacado = numero & 0x01;
 
        decimal = decimal * 2 + numeroSacado;
 
        numero /= 10;
    }
    cout << "El equivalente decimal es: " << decimal << endl;
}

while ((numero / numero) != 0)

¿Qué intentabas ahí exactamente?

A ver, esta parte concreta, lo que se dice funcionar... funciona, pero no es la mejor manera de hacerlo.Si tu idea es que el bucle se repita hasta que "numero sea 0" ¿por qué no usas los operadores de comparación ('==', '!=', '>', '<', '>=', '<=')??

Hay varias alternativas para expresar lo mismo y de forma mucho más clara. Que al final es de lo que se trata, hacer código legible. Te presento tres alternativas ordenadas de la más clara a la menos clara desde mi punto de vista:

* Opción 1: Comprobar si el número es distinto de 0

while ( numero != 0 )

* Opción 2: Comprobar si el número es mayor de 0

while( numero > 0 )

* Opción 3: Comprobar si el número es igual o mayor que 1

while( numero >= 1 )

El problema en sí, que hace qeu no te funcione, es que no estás utilizando en ningún momento la variable "escala"... le asignas valores, eso sí... pero no haces uso de esos valores en ningún momento.

Y luego te faltaría sacar por pantalla el número final.


101 (10 = 1100101 (2 <-- Queda claro que representar un número en base 2 ocupa más memoria que representarlo en base 10.

El número más alto que puedes almacenar en un int es el 2147483647 (2^31 - 1). Si usas este mismo int para almacenar un número en representación binaria, entonces no podrás almacenar números mayores que el 1111111111, que en decimal quedaría representado por el número 1023. Queda claro que el código de tu programa limita muchísimo el rango de números con el que es capaz de trabajar.

La idea que propongo es almacenar el número en una cadena de caracteres y recorrer esos caracteres para ir calculando el número final. Algo del tipo:

std::string cadena;
std::cout << "Introduce un número en binario" << std::endl;
std::cin >> cadena;
 
int numero = 0;
for ( unsigned int i = 0; i < cadena.length( ); i++ )
{
  numero *= 2;
  if ( cadena[ i ] == '1' )
    numero++;
}
 
std::cout << "El equivalente en decimal es: " << numero << std::endl;

Creo que el código resultante es bastante sencillo y fácil de entender.