Hjälp med Java

import java.util.Scanner;

public class primeCombo{
    // Testa om inmatat tal är mindre än två eller ojämnt. Skapa variabler
    public static int hittaKombinationer(int number){
        if(number <= 2 || number%2 == 1){
            System.out.println("Felinmatning, try again");
            return 0;
        }
       
       
        int halfnumber = number/2;
        int AntalKombinationer = 0;
       
        // Anropar från metoden primtal. Om i är ett primtal och number - i är ett primtal så skapar det en kombination.
        //Testar upp till hälften av talet för att undvika dubbla kombinationer.
       
        for(int i=2;i<=halfnumber;i++){
            if(primtal(i) && primtal(number-i)){
			
                AntalKombinationer++;
                System.out.println("Komination funnen: "+i+"+"+(number-i));
            }
        }
        // Returnerar Antalet funna kombinationer till metoden.
        return AntalKombinationer;
    }

    public static boolean primtal(int number){
        // Letar efter primtal i. Hittas detta returneras true till metoden ovan.
        // Om i inte är ett primtal returneras false och nästa värde på i testas.
        for (int i=2; i<number; i++) {
            if (number%i==0) {
            return false;
        }
    }
   
    return true;
    }

    public static void main(String[] args){
        // Inmatning användare
        Scanner in = new Scanner(System.in);
        int indata = 1;
       
        // Loopar tills användaren skriver in 0
        while (indata != 0){
       
            System.out.print("Mata in ett jämnt tal >2: ");
            indata = in.nextInt();
            System.out.println( "Antal olika kombinationer :  " + hittaKombinationer(indata) + "\n" );
       
       
           
        }
    }
}

public static int hittaKombinationer(int number)

public static int hittaKombinationer(int number)

Låt bli och skriv ut den då? Jag har tyvärr aldrig hållt på med java, så kände mig lite vilsen när jag läste koden. Lycka till!

När du skapar metoden hittaKombinationer så kan du istället för att returnera en int skriva void vilket betyder att den inte returnerar någonting. Alltså "public static void hittaKombinationer". Du måste sedan ta bort raden "return AntalKombinationer;" då den inte kan returnera något då den är en void. Sen kan du ju ta bort allt som har med "AntalKombinationer" att göra då dem inte verkar behövas.

I main-metoden får du då byta ut denna rad, "System.out.println( "Antal olika kombinationer : " + hittaKombinationer(indata) + "\n" );" mot bara "hittaKombinationer(indata)" som kommer att köra koden i den metoden.

Jag hoppas att det var det du ville veta?

Nu var det ett tag sen jag höll på med java så antagligen efterblivet att jag ens försöker med risk för att göra bort mig men i alla fall. Om du kör void hittaKombinationer istället, vad är det print skriver ut då? Kommer inte riktigt ihåg. En inre referens av något slag? Du borde väl kunna skriva om toString i så fall, men finns säkert en mindre omständig lösning på det hela.

Känns fint att den första som svarar inte håller på med java och den andra som svarar inte använt sig av java på ett bra tag... sorry dude.

Du borde nysta upp det där lite. Det ser inte bra ut att hålla på och skriva ut relaterad information i olika metoder (main och hittaKombinationer).

Jag skulle låtit main (i ett sådant här litet program) ansvara för att skriva ut kombinationerna som hittaKombinationer returnerar, i form av en lista. Vad listan ska innehålla kan man ju fundera på men som första steg kan du köra med strängar. Alltså List<String> som returtyp. Java har ingen Pair klass, det får man bygga själv. Ofta kan det vara vettigt när man kan beskriva ett samband mellan två objekt (t.ex. Map.Entry) men här där det inte gör något alls så känns det könligt att göra sin egen klass (på rätt vis) för att föra över värdena. Du skulle också kunna fula in en array. Så typen är List<Integer[]> och sen lagra arrayer av längden 2 i varje element i listan.

Det går också att optimera första for-loopen, fast man travar nog in på kodning som inte är så läsbar här. Skriver loopens första rad så här:
for(int i=2;i<=halfnumber;i += i != 2 ? 2 : 1){

Det sista är en ternär operator, villkorsoperatorn ?:.
i += i != 2 ? 2 : 1
Betyder addera till i resultatet av Om i är skilt från 2, ge 2, annars 1.
Vad som händer är att den första gången kommer bli falsk och det blir i + 1, dvs 3. Efter det så kommer den alltid att addera 2 till i, så serien blir 2,3,5,7,9,11,13,15 osv.
Jag skulle nog faktiskt själv inte skriva detta men det är bra att känna till lite sådant.