Součet dvou binárních čísel – C / C++ – Fórum – Programujte.com

Zdravím, mám takový problém s jedním úkolem. Za úkol mám sečíst 2 binární čísla (na řádku oddělena mezerou). Na samotné sčítání jsem přišel(bitové operace), ale stále mám problém se 2 věcmi. 

1) ošetření vstupu - víceméně si myslím, že vím jak na to, ale stále mi to nefunguje, jak bych chtěl 

Pro binární čísla 
abraka1dabra0 1fuj0tajksl1

By měl vyhodit jeden Nespravny vstup.

2) po sčítání zbavit se přebytečných nul 

001 000001

Soucet: 10

Co zatím mám:

#include <iostream>
#include <string>
#include <algorithm>

using namespace std;

int main()
{
    int zbytek=0;
    int soucet=0;
    string cislo1,cislo2;
    string vysledek="";
    cout <<"Zadejte dve binarni cisla:"<<endl;
    cin >> cislo1;
    cin >> cislo2;

    for (unsigned int z=0;z<cislo1.length();z++)
    {
       if (cislo1[z]!='0' || cislo1[z]!='1')
        {
            cout <<"Nespravny vstup"<<endl;
            break;
        }
    }
    for (unsigned int y=0;y<cislo2.length();y++)
    {
       if (cislo2[y]!='0' || cislo2[y]!='1')
        {
            cout <<"Nespravny vstup"<<endl;
            break;
        }
    }

    int j=cislo2.length()-1;
    int i=cislo1.length()-1;
    while(j>=0&&i>=0){
        soucet=(cislo1[i]-48) ^ (cislo2[j]-48) ^ zbytek;
        zbytek=((cislo1[i]-48) & (cislo2[j]-48)) | ((cislo2[j]-48)&zbytek) | (zbytek& (cislo1[i]-48));
        vysledek+=(soucet+48);

    i=i-1;
    j=j-1;
    }
    while(i>=0){
    zbytek=(cislo1[i]-48)&zbytek;
    soucet=(cislo1[i]-48)^zbytek;

    vysledek+=(soucet+48);
    i=i-1;}
    while(j>=0){
    zbytek=(cislo2[j]-48)&zbytek;
    soucet=(cislo2[j]-48)^zbytek;

    vysledek+=(soucet+48);
    j=j-1;}

    if(zbytek)
        vysledek+=(zbytek+48);
        reverse(vysledek.begin(),vysledek.end());
        cout<<"Soucet: " << vysledek << endl;

    return 0;
}

Budu rád za jakoukoliv pomoc nebo radu.

Předem děkuji

1)

Ten test na '0' a '1' nebude || ale &&.

Pokud zjistíš problém, tak další kód prostě neprováděj. Můžeš ho nějak podmínit, nebo funkci předčasně ukončit returnem.

Nepiš několikrát stejný kód, vyhoď to do funkce.

2)

Prostě odmazávej počáteční nuly, dokud nenarazíš na jedničku.

----

Třída string potřebné operace obsahuje, pokud je můžeš použít, tak třeba takto:

bool IsBin(const string & s)
{
	return (s.find_first_not_of("01") == string::npos);
}

string MinBin(string s)
{
	s.erase(0, s.find_first_not_of('0'));
	if (s.length() == 0) s = '0';
	return s;
}
...
...
cin >> cislo1;
cin >> cislo2;

if (!IsBin(cislo1) || !IsBin(cislo2))
{
	cout << "Nespravny vstup" << endl;
	return 1;
}

cout << MinBin(cislo1) << " + " << MinBin(cislo2) << endl;

#2 gna
Díky moc - už to funguje ok, ale objevil se další problém a to v následujícím testu, ve kterém kontroluje delší řetězce

např. 
(0101001111) - jeden retezec

(011111000001110011100001111101110000000100111000010010110111011001101101101010000011110000110010101100001101110111110011001) - druhy retezec

zkusil jsem u té délky řetězce dát long int ale není to ok.

Netuším, co myslíš.

Mám 

dva vstupy

0101001111   - první řetězec

011111000001110011100001111101110000000100111000010010110111011001101101101010000011110000110010101100001101110111110011001
druhy retezec

vysledek má vyjít

11111000001110011100001111101110000000100111000010010110111011001101101101010000011110000110010101100001101111000011101000

a mě vychází

11111000001110011100001111101110000000100111000010010110111011001101101101010000011110000110010101100001101110000011101000

Nevíš kde může být problém v kódu?

V těch "dodatečných" cyklech prohoď ty dva řádky.

zbytek=(cislo1[i]-48)&zbytek;
soucet=(cislo1[i]-48)^zbytek;

Nebo na začátku ta čísla srovnej na stejnou délku a nebudeš je potřebovat.

A místo těch čísel 48 můžeš dát znak '0'. Bude tak jasnější o co jde.

#7 gna
OK vyzkouším - díky moc za rady

gna - fakt ti moc děkuji !!!! Vše už funguje, jak má !!. Progtest už nepíše problémy.