Unwanted Null output.

[steinarsen]

Hi!

Im just getting started with java and im having some troubles.
I want to print out some grades, but im just getting null as an output and i cant seem to figure out the problem.

Thanks in advance!

import java.util.Random;
import java.util.Arrays;
import java.util.ArrayList;
import java.math.BigInteger;
import javax.swing.*;
import java.awt.*;


public class Studenter {
    private int antallStudenter = 20;//Setter antall studenter
    private int maksPoeng = 200;//Maks oppnåelig poeng
    private int[] studenter = new int [antallStudenter];//Oppretter arryaer som programmet setter data inn i
    private int[] poeng = new int[antallStudenter];
    private int[] poengKlone = new int[antallStudenter];
    private int[] prosent = new int[antallStudenter];
    private int[] bestaat = new int[antallStudenter];
    private String[] karakter = new String[antallStudenter];
    private static final Random rPoeng = new Random();//random genererer poeng.
    
        private int bestaa = 1;        
        
        
        private int A = 1;
        private int B = 1;
        private int C = 1;
        private int D = 1;
        private int E = 1;
        private int F = 1;
        private ArrayList<String> liste = new ArrayList<String>();
        private boolean brukt = false; 

    
             
		
		
    
    public void DelUt()//"Deler" ut all infoen
    {
        for(int i = 0; i<antallStudenter; i++)
        {
            studenter[i] = studentNummer(i);
            poeng[i] = tildelPoeng(i);
            prosent[i] = tildelProsent(i);
            poengKlone[i] = poeng[i];
            bestaat[i] = bestodStudent(i);
            
        }
    
    }
    
    public int studentNummer(int i)//Opretter studentnummer
            {
                int studentNummer = 1 + i;
                return studentNummer;
            }
                
    public int tildelPoeng(int i)//Oppretter poeng
    {
        int studentPoeng = rPoeng.nextInt(maksPoeng + 1);
        return studentPoeng;
    }
    
    public int tildelProsent(int i)//Regener ut prosent
    {
        int tildelProsent = ((poeng[i]*100)/maksPoeng);
        return tildelProsent;
    }
    
           public int bestodStudent(int i)
    {
        int studentNummer;
        if(poeng[i]>30)
        {
        studentNummer = studenter[i]; 
        bestaa++;
        }else{
            studentNummer = 0;
        }
        return studentNummer;
    }
           
           public void karStudenter()
    {
        int a;
        Arrays.sort(poengKlone); //Sorterer klonen av poeng så vi slipper å bytte om på karakterene til studentene
        Arrays.sort(bestaat); //Sorterer adressene til hvem som har passert
        double counter = 0.0;
        for(int i = 0; i<antallStudenter; i++)
        {
            a = searchClone(i);  //Finner riktig plassering i "points" for denne studenten så riktig student beholder poengsummen og karakteren den først fikk
            if(bestaat[i]>=30)
            {
                 counter++;
                if((counter<=(bestaa*10)/100))
                {
                    karakter[a] = "E";
                    E++;
                }else if(counter<=((bestaa*35)/100) && counter >((bestaa*10)/100))
                {
                    karakter[a] = "D";
                    D++;
                }else if(counter<=((bestaa*65)/100) && counter>((bestaa*35)/100))
                {
                    karakter[a] = "C";
                    C++;
                }else if(counter<=((bestaa*90)/100) && counter >((bestaa*65)/100))
                {
                    karakter[a] = "B";
                    B++;
                }else if(counter <=bestaa && counter >(bestaa*90)/100)
                {
                    karakter[a] = "A";
                    A++;
                }
            }else if(bestaat[i]<30||bestaat[i]==-1){
                karakter[a] = "F";
                F++;
            }
        }
    }

           public void addToList(BigInteger z) //Legger til adressen som er blitt brukt
    {
        String s = z.toString();
        liste.add(s);
    }

        public void checkList(BigInteger z) //Sjekker lista for å finne hvem som er blitt brukt og hvem som skal brukes neste
    {
        String s = z.toString();
        if(liste.contains(s))
        {
            brukt = true;
        }else{
            brukt = false;
        }
    }
        
    public int searchClone(int i) //Finner riktig plassering i "points" for denne studenten så riktig student beholder poengsummen og karakteren den først fikk
    {
        int x = poengKlone[i];
        int location = 0;
            for(int j = 0; j<antallStudenter; j++)
            {
                checkList(BigInteger.valueOf(j));
                    if(poeng[j]==x&&brukt==false)
                {
                    location=j; //Sender tilbake hvilken adresse neste adressen som ikke er brukt for å ungå karakteren "null"
                } 
            }
        addToList(BigInteger.valueOf(location)); //Legger til at denne adressen er brukt og ikke skal bli brukt igjen
        return location;
    }
       
    public void utSkrift()//Skriver ut hele greia:)
    {
        System.out.printf("%s%9s%9s%9s\n", "StudentNr", "Poeng", "Prosent", "Karakter");
        for(int counter = 0; counter<antallStudenter; counter++)
        {
         System.out.printf("\t%d%9d%9d%9s\n", studenter[counter], poeng[counter], prosent[counter], karakter[counter]);   
        }
    }   
}

[steinarsen]

And please tell me what the Java BB code is :P

And tell me what you need to know if it makes finding the problem easier:D

[Tolls]

What output do you actually see?
How is the above code called?

You need to sort out some of your indenting, but you found the code tags at least...which is something of a rarity around here.
:)

[steinarsen]

This is displayed in the output:


Here is how i call the code.

package obligatorisk.innlevering.pkg2;


public class ObligatoriskInnlevering2 {


    public static void main(String[] args) {
        Studenter Stud = new Studenter();
        Stud.DelUt();
        Stud.utSkrift();
        Stud.karStudenter();
    }
}

[Tolls]

Your calls are in the wrong order.
The 'karakter' array is not populated before the utSkrift() call.
You have it populated afterwards.

[steinarsen]

ahh! I changed the order and atleast i get an output now!

Only problem is that everyone gets an F...

[steinarsen]

can anyone see why i only get F`s? I cant figure this shit out and its killing me slowly...

[Tolls]

You'll need to do some debugging.
Put some println() statements in there printing out what various values are, specifically around the values being put into the bestaat array, since that's what defines what grade is used.

[steinarsen]

You'll need to do some debugging.
Put some println() statements in there printing out what various values are, specifically around the values being put into the bestaat array, since that's what defines what grade is used.

Hey! I got it working yesterday already!

Turns out i had to change

public int studentNummer(int i)//Studentnummer
            {
                int studentNummer = 1 + i;//Her er det viktig at studentnummeret starter på over 30. Om vi starter på for ekesempel 1, får alle studentene F som karakter. Dette er en svakhet i koden.
                return studentNummer;
            }

to this

public int studentNummer(int i)//Studentnummer
            {
                int studentNummer = 30 + i;//Her er det viktig at studentnummeret starter på over 30. Om vi starter på for ekesempel 1, får alle studentene F som karakter. Dette er en svakhet i koden.
                return studentNummer;
            }

The studenNummer has to be over 30 for it to work.
This is a weakness in the code, but i see no other way of making it work.