I Need help with solveing a java problem :)

[angryboy]

still no code from OP.........?

[Fubarable]

I hope to god you aren't holding your breath waiting...

[javaNoob2]

I have it done im just occupied with finishing a website , anwho havent been in college to get it its rag week here ...if anyone knows what that is ...pure drunknness all week ...btw im irish lol :)

[javaNoob2]

ok i have my code off my school server :)

here it is i would like some help if possible to help me improve this code , its fairly basic tho.

 *
 * ProgrammingAssignment application
 *
 * @author 
 * @version 1.02009/3/6
 */
 
public class ProgrammingAssignment {

public static void main(String[] args) {
        //Input all integers involved here
        int music=0;
        
        int Rap=0;
        int Rock=0;
        int Alternative=0;
        int Techno=0;
        int Dance=0;
        int j;
        
        for(j = 1; j < 21; j = j + 1 ){
                
                //Insert Screen Output Messages
                Screen.newline();
                Screen.message(Favourite Music);
                Screen.newline();
                Screen.message(1.Rap,2.Rock,3.Alternative,4.Techno,5.Dance);
                Screen.newline();
                Screen.message(Insert Favourite Type of Music Here);
                music = Keyboard.readInt();
                
                //Checking user input and increasing the appropiate music type by 1
                if (music==1)
                        Rap=Rap + 1;
                        else if (music==2)
                                Rock=Rock + 1;
                        else if (music==3)
                                Alternative=Alternative + 1;
                        else if (music==4)
                                Techno=Techno + 1;
                        else if (music==5)
                                Dance=Dance + 1;
                                
                        else
                                Screen.message(Invalid Number);
                        
                        //Input messages which describe what students like
                        
                        Screen.newline();
                        Screen.message(Number Of Students Who like Rap Are);
                        Screen.writeint(Rap);
                        Screen.newline();
                        Screen.message(Number Of Students Who like Rock);
                        Screen.writeint(Rock);
                        Screen.newline();
                        Screen.message(Number Of Students Who Like Alternative);
                        Screen.writeint(Alternative);
                        Screen.newline();
                        Screen.message(Number Of Students Who Like Techno);
                        Screen.writeint(Techno);
                        Screen.newline();
                        Screen.message(Number Of Students Who Like Dance);
                        Screen.writeint(Dance);
                        Screen.newline();
                        
                        //An if function which says whether people like or dislike the music type
                        
                        if (Rap > 10)
                                         
                                Screen.message(people who love rap);
                        else 
                        
                                Screen.message(people who dont like rap);
                                
                        Screen.newline();        
                                
                        if (Rock > 10)
                                
                                Screen.message(people who love rock);
                        else 
                        
                                Screen.message(people who dont like rock);
                                
                                        Screen.newline();
                                
                        if (Alternative > 10)
                        
                                Screen.message(people who love alternative);
                        else 
                        
                                Screen.message(people who dont like alternative);
                                
                                        Screen.newline();
                                        
                                        
                        if (Techno > 10)
                                         
                                Screen.message(people who love techno);
                        else 
                        
                                Screen.message(people who dont like techno);
                                
                                Screen.newline();
                                        
                        if (Dance > 10)
                                         
                                Screen.message(people who love dance);
                        else 
                        
                                Screen.message(people who dont like dance);        
                                
                                Screen.newline();
                                        
                
                
        }
        
        
}
}

[Eranga]

When you post a code segment again in a forum, please use code tags. Un-formatted codes are hard to read.

[javaNoob2]

When you post a code segment again in a forum, please use code tags. Un-formatted codes are hard to read.

Right sorry about that fixed it just there :D

[Eranga]

NP, that's much better. Now let see what can we do on this code.

[emceenugget]

can you say what you would like to "improve"?

personally, i'd remove as much from the main method as possible.

also, what is this Screen class you use? i've never seen that before...

[javaNoob2]

Sorry if i dint make it clear lol . I kinda want to know about different ways of doing this assignment maybe changing my code a bit etc. Im still new to java only have a few months experience with it. The class file i used is EasyI0.