If you have all the opening and closing braces the compiler might be complaining because you don't have a sure return. This is not normal for a compiler but I don't know what compiler your using. So you can try putting a return at the end of the function right before the last } and that should do the trick. Ken -----Original Message----- From: programmingblind-bounce@xxxxxxxxxxxxx [mailto:programmingblind-bounce@xxxxxxxxxxxxx] On Behalf Of Alex Hall Sent: Friday, February 22, 2008 8:23 PM To: programmingblind@xxxxxxxxxxxxx Subject: Re: method saying "no return statement" when there IS one Odd, I put all the braces in my original email. I opened the method, two for loops, an if statement, and an else statement and closed them all. My bn, on which I wrote that message, must have not liked it that I put them all on separate lines; it seems to do odd things with punctuation just before a new line if that punctuation is not connected to a word. Have a great day, Alex > ----- Original Message ----- >From: "Fred Rains" <fred58@xxxxxxx >To: <programmingblind@xxxxxxxxxxxxx >Date sent: Fri, 22 Feb 2008 23:13:29 -0500 >Subject: Re: method saying "no return statement" when there IS one >Hi Alex, You have several opening braces with no closing brace. Both "for" >statements have an opening brace but no closing brace. Also, your "if" and >"else" statements have opening braces with no closing brace. >Fred >----- Original Message ----- >From: "Alex Hall" <mehgcap@xxxxxxx >To: <programmingblind@xxxxxxxxxxxxx >Sent: Friday, February 22, 2008 10:46 PM >Subject: method saying "no return statement" when there IS one >> Hi all. Yet again I have what to many of you is no doubt a very simple >> question, but it is not making sense. I have a method that is checking >> collisions on a battleship grid. It is an integer method that returns 0 >> if a collision is detected and 1 if there is no problem. The compiler, >> however, keeps saying that I am missing a return statement when I clearly >> have two, one in an if statement and the second in a related else. Here >> is the method: >> public static int checkCollisions(int[][] grid){ >> int i,j; >> for(i=0;i<grid.length;i++){ >> for(j=0;j<grid[i].length;j++){ >> if(grid[i][j]<=2){ >> return 0; >> else{ >> return 1; >> Have a great day, >> Alex >> __________ >> View the list's information and change your settings at >> //www.freelists.org/list/programmingblind >__________ >View the list's information and change your settings at >//www.freelists.org/list/programmingblind __________ View the list's information and change your settings at //www.freelists.org/list/programmingblind __________ View the list's information and change your settings at //www.freelists.org/list/programmingblind