Finding a number 1- 9999

[Joshcannon]

I need my code to start at 1 and go to 9999. If the code meets the first if-statement it'll go on. If not, than it will start over and add 1 on to the previous number tried. This is what I have now.

Try not to give me answers, just show me where i'm going wrong at as this is a homework assignment. Thank you!

  #include <iostream>
using namespace std;

int main()
{
    int address = 0001, ones, tens, hundreds, thousands, sum = 29;
    
    while ( address < 9999)
        
    
    thousands = (address / 1000) % 10;
    hundreds = (address / 100) % 10;
    tens = (address / 10) % 10;
    ones = address % 10;
    
    
    if ( ones != tens && ones != hundreds && ones != thousands && tens != hundreds && tens != thousands && hundreds != thousands)
    {
        if ( address /2 % !0 )
        {
            if ( tens == (3 * thousands) )
            {
                if ( sum == ones + tens + hundreds + thousands )
                {
                    cout << "The address is: " << address << endl;
                }
                else
                {
                    address++;
                }
            }
            else
            {
                address++;
            }
        }
        else
        {
            address++;
        }
    }
    else
    {
        address++;
    }
    return 0;
    
    }

[Aceix]

So what's the problem?

Aceix.

[Joshcannon]

I need it to output the number that fits the conditions. Forgot to post those, whoops. They are :

1. All four digits are different

2. The digit in the thousands place is three times the digit in the tens place

3. The number is odd

4. The sum of the digits is 27.

[Joshcannon]

Could anyone help me out?

[Smac89]

Yes, first thing is that you need to make sure that everything below the while loop is executed at every iteration of the loop. As of now only line 11 is being executed and nothing else.

Next you need to make sure you are computing the right sum, your code and your post say 2 different things

Finally using a for-loop will make your code more compact so that the unnecessary else address++ is done only once

P.S. 0001 == 1