C# – Add all natural numbers that are multiples of 3 and 5 : What is the bug in the following code

I know that this can be easily done by using

if(i%5 == 0 OR i%3 ==0) sum+=i;

But what is wrong in the following C#code:

    int sum = 0;
    for(int i = 0, j = 0; i < 1000; i+=3, j+=5)
    {
        Console.WriteLine("i = " + i);
        Console.WriteLine("j = " + j);

        sum += i;

        Console.WriteLine("Sum after adding i  = " + sum);

        if(j < 995 && j % 3 != 0)
        {
            sum += j;
        }

        Console.WriteLine("Sum after adding j  = " + sum);

    }

Best Answer

The statement j < 995 should probably be j <= 995, otherwise you are not going to add 995 to your sum.

Related Solutions

C# – the difference between String and string in C#

string is an alias in C# for System.String.
So technically, there is no difference. It's like int vs. System.Int32.

As far as guidelines, it's generally recommended to use string any time you're referring to an object.

e.g.

string place = "world";

Likewise, I think it's generally recommended to use String if you need to refer specifically to the class.

e.g.

string greet = String.Format("Hello {0}!", place);

This is the style that Microsoft tends to use in their examples.

It appears that the guidance in this area may have changed, as StyleCop now enforces the use of the C# specific aliases.

C# – Catch multiple exceptions at once

Catch System.Exception and switch on the types

catch (Exception ex)            
{                
    if (ex is FormatException || ex is OverflowException)
    {
        WebId = Guid.Empty;
        return;
    }
    
    throw;
}

Best Answer

Related Solutions

C# – the difference between String and string in C#

This is the style that Microsoft tends to use in their examples.

C# – Catch multiple exceptions at once

Related Topic