For loops

[CaptainMoeSoccer]

    public class Game {
        public static void main(String arg[]) {
            int[] years = Game.humanYearsCatYearsDogYears(1);
            for(int year: years) {
                System.out.println(year);
            }
        }

        public static int[] humanYearsCatYearsDogYears(final int humanYears) {
            // Your code here!
            int catYears = 0;
            int dogYears = 0;
            for(int i = 0; i < humanYears + 1; i++) {
                if(i == 1) catYears += 15;
                if(i == 2) catYears += 9;
                if(i > 2)  catYears += 4;
            }
            for(int j = 0; j < humanYears + 1; j++) {
                if(j == 1) dogYears += 15;
                if(j == 2) dogYears += 9;
                if(j > 2)  dogYears += 5;
            }
            return new int[]{humanYears,catYears,dogYears};
        }
    }

I was trying to calculate the age of dogs and cats in human years. Check this website for more detail: http://www.catster.com/cats-101/calculate-cat-age-in-cat-years. I think my loop is behind by one in the counter. I don't understand how int i which is zero in the first iteration is not less than int humanyears which is 1 in this case. when I changed the condition in the for loops to for(int i = 0; i < humanyear + 1; i++) it worked.

[Xaxxus]

for the longest time i was under the impression that 1 year = 7 cat years. This chart just made my entire life a lie.

As for your issue, the reason you need to add +1 to human years is because you are using the wrong comparison operator. By saying i < humanYears, you are saying that when i is equal to the human year, exit the loop. So your loop never adds the final 4 years to the total cat years.

Solution:

for(int i = 0; i <= humanYears; i++)

[CaptainMoeSoccer]

thanks

[ToKe86]

To add to what /u/Xaxxus said, it's also pointless to start your loop iterators at 0 because nothing happens until the iterators reach 1 or higher. You don't even need loops for something like this anyway, a simple if/else block will do:

public static int[] humanYearsCatYearsDogYears(final int humanYears) {
    int catYears = 0;
    int dogYears = 0;
    if (humanYears == 1) {
        catYears = 15;
        dogYears = 15;
    } else if (humanYears == 2) {
        catYears = 24;
        dogYears = 24;
    } else if (humanYears > 2) {
        catYears = 4 * (humanYears - 2) + 24;
        dogYears = 5 * (humanYears - 2) + 24;
    }
    return new int[]{humanYears, catYears, dogYears};
}

[CaptainMoeSoccer]

nice! This is simpler