Slime wrote:
>>     for (unsigned int i=0; i<c.size(); i++)
>>       if (c[i]) out.append("1"); else out.append("0");
> 
> This isn't the cause of your crash, but I recommend not squishing your code 
> together like that. Keep each statement on a separate line and use brackets 
> around anything that's not a single simple statement:
> 
>     for ( unsigned int i = 0; i < c.size(); i++ )
>     {
>         if ( c[i] )
>             out.append( "1" );
>         else
>             out.append( "0" );
>     }

If he wants brevity he could also code it like this:

  for (int i = 0; i < c.size(); i++)
    out.append(c[i] ? "1" : "0");

which is just as valid and avoids running the if/else together.

-- Chris

Post a reply to this message