Printable View
Can someone please tell me why I am getting an 'else' without 'if' error message when I try to compile the following code:
I can't figure out what I am doing wrong. It states that the problem is at the else lineCode:
public class Example3 {
public static void main(String [] args){
int nums[] = new int[50];
for(int i = 0; i < 50; i++){
if(i % 2 == 0){
nums[i] = 1;
else{
nums[i] = 0;
System.out.println(nums[i]);
}
}
}
}
}
Count your curly brackets and b.t.w. fix your indendation because as it is now it stinks.Quote:
Originally Posted by jim01
kind regards,
Jos
I have an open and closed curly brace for my class, an open and closed curly brace for my method, an open and closed curly brace for my for loop, and one each for my if statement and my else statement. Going back over my notes, each one of these should have open and closed curly braces. Am I mistaken on that? The book only shows three open and closed curly braces for most of its examples, one for the class, one for the method and one for the for loop. It leaves off the curly braces for the if and else statements, but I though they done that because you are not required top have them if you only have one. However, a couple of people on the forum in the past have told me that you should always use curly braces even if you are only going to use an if/even statement once or else you will likely forget to put one in when you need it.Quote:
Originally Posted by JosAH
Sorry, it moved everything when I cut and pasted it. This is how it should have appeared. Is that better?Code:
public class Example3 {
public static void main(String [] args){
int nums[] = new int[50];
for(int i = 0; i < 50; i++){
if(i % 2 == 0){
nums[i] = 1;
else{
nums[i] = 0;
System.out.println(nums[i]);
}
}
}
}
}
You forgot the second part of my remark: you indentation stinks; an if-else statement is supposed to look like this:Quote:
Originally Posted by jim01
Fix your indentation and most likely you'll see your mistake. b.t.w. my parts between < and > are just paceholders.Code:if (<condition>) {
<statement>
}
else {
<statement>
}
kind regards,
Jos
I see where I was making my mistake. I was using the curly brackets for the if/else statements as you would a loop statement.Quote:
Originally Posted by JosAH
Thank you for the help and correcting my on my indentation.
Your welcome of course but hold on for a second: a for-loop doesn't have different curly brackets from, say, an if-statement; have a look:Quote:
Originally Posted by jim01
They're all the same, see?Code:for (<for-loop-header>) {
<statement>
}
if (<condition>) {
<statement>
}
kind regards,
Jos
Yes, but from my understanding you incorporate the if statements into the loop by adding the if statement before the loops closing curly brace. For example:Quote:
Originally Posted by JosAH
I was trying to do the same thing with the if and else statements, which of course was wrong.Code:
public class PracticeExample2A {
public static void main(String[] args){
//declare variable sum
int evenSum = 0;
int oddSum = 0;
//perform for loop
for(int i = 1; i <= 1000; i++){
if(i % 2 == 0){
evenSum += i;
}
else{
oddSum += i;
}
}
//output
System.out.println("The sum of all even numbers is " + evenSum);
System.out.println("The sum of all odd numbers is " + oddSum);
}
}