Thread: what could be wrong with this loop!?

many thanks bro.
i meant if one of them is wrong ! just exit!
i should make those && ! yes?

Originally Posted by R.Stiltskin

Your indentation is confusing. Do you really intend that the second "if" is inside the first "if" block, and the "else" refers to the first "if"?

well the second if is inside the first if , and the else refers to the second if !

Code:

               if ( temp != "not needed" )
                {
                        argument = Atoi(offsetptr);
                        offsetptr = Oct2Bin( argument );


                        if ( (temp != "INX") && (temp != "LAX" ) && (temp != "LDX") && (temp != "DEX") && (temp != "CLX" ) )
                        {
                                CommandsBinaryString = BinaryDetector( commandnameptr );
                                CommandsBinaryString.append(mixstateptr);
                                CommandsBinaryString.append(offsetptr);

                        }

///Error .Inx kar nemikonan.

                        else
                        {
                                CommandsBinaryString = commandnameptr;
                        }
                }

by the way ! the first error i used to get is gone! by changing || to && . but it seems now the second else is not functioning ! how am i supposed to attach that if with this else !?

Originally Posted by laserlight

The problem is that in your original code the else is associated with the first if.

tanx .

Originally Posted by R.Stiltskin

... and if your revised code is what you intended, you can replace

Code:

else if ( (temp == "INX") || (temp == "LAX" ) || (temp == "LDX")|| (temp == "DEX") || (temp == "CLX" ) )
{

with

Code:

else
{

tanx , i did tha , is it supposed to connect the second if and the else ? !
( i did it . but im still getting none sense here . )
snapshot.JPG
========================
i have two kind of commands :
1 plain commands
2 commands stating a specific addressing mode.(complex commands)
========================
i the following loop, i search for both commands, there is a difference the complex commands dont need the offset and M.I.X (which is an addressing mode indicator)
========================
so , when the command is recognized! the BinaryDetector() function takes the command name , and analyzes it , and then retrieve the commands binary representation!. if it is aplain command , it would be merged with mix and offset.
if not it should be either (INX, or DEX, or LDX, or LAX or ,CLX) . and its binary representaion should be retrieved!
========================

well i just dont know why it doesnt do it right!
========================
note : that first if is to eliminate declaration instrcutions, i mean because each time one string representing one instruction is taken under analysis , this is to avoid further error! because declaration are not executive commands, they just represents offsets! so the must not get into this session and accidantaly get into the memory!
each declaration when recognized in tokenizer() ( a function before this) would contain "not needed" string as its command name and also its offset value!
so by using this trick i can recognize it from other actual commands.
thats the reson its up there !

hope this intruductory helps you understand what i meant by doing such a thing

thank you all for you time and answers.