C# A loop using while. Variable displayed only once

Question

I have a small problem. My code is this one :

int c = 0;
int i = 0;
int a = 28;
while (i < a) {
    i++;
    if (i % a == 0) {
        c += i;
        Console.WriteLine(i.ToString());
    }
}

Why does the string i is displayed only once, after the end of the while ? It should be displayed a times.

Your help will be appreciated !

You are playing with the perfect number 28. You are trying to sum up all numbers i that divide 28, but because you put the operands in the wrong order, you sum all numbers i that 28 divides. — Jeppe Stig Nielsen
– Jeppe Stig Nielsen, Commented Dec 3, 2013 at 21:35

Habib · Accepted Answer · 2013-12-03 21:10:31Z

5

Your if condition is opposite it should be:

 if (a % i == 0)

Currently you are trying to do remainder division with i % a and it will only meet the condition when i reaches 28, so you get the output once.

answered Dec 3, 2013 at 21:10

Habib

224k30 gold badges420 silver badges446 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

PaulG · Accepted Answer · 2013-12-03 21:13:20Z

0

% is for modulus division, which basically divides by the number and gives you back the remainder. When you're loop reaches 28 it divides it by 28 and the resulting remainder is 0. This only happens once "when your loop reaches 28".

answered Dec 3, 2013 at 21:13

PaulG

7,14212 gold badges58 silver badges101 bronze badges

Comments

Andrew Rump · Accepted Answer · 2013-12-03 21:14:31Z

0

It would help if you told us what was printed out. I guess it is 28 because

i % a

returns the reminder of

i / a

(i divided by a) and it is only 0 when i is equal to a, i.e., 28.

answered Dec 3, 2013 at 21:14

Andrew Rump

1421 gold badge1 silver badge10 bronze badges

Collectives™ on Stack Overflow

C# A loop using while. Variable displayed only once

3 Answers 3

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related