What exactly are you trying to do?
Also use a switch statement instead of the if. It should be more efficient since it can use look up tables and then if you want to do more than just 0 and 1 for your digits you can change them all. Just do something like this:
Code:
switch(cInputOld[i])
{
case '0': case '1':
iBinary[j] = cInputOld[i] - 48;
j++;
break;
}