Primzahlen

[Gerlon]

Guten Tag,

ich habe ein Problem mit dem Programm.
Bekomme es irgendwie nicht wirklich hin eine ordentliche Übergabe von der main zur Methode und wieder zurück.
Es soll vom Benutzer eine Zahl bis 20 eingegeben werden und diese steht dann für n-Primzahlen die ausgegeben werden sollen.
Alles in der main funktioniert ohne Probleme.
Aber hier gibt er nur Blödsinn aus, weiß auch nicht so wirklich ob meine Idee stimmt.

Bitte um Hilfe

import java.util.*;

public class Uebung4
{
   public static int[] getPrime( int n ) {
  
		int[] x = new int[20];
		int[] a = new int[20];
		a[0] = 2; 	a[1] = 3;  
		a[2] = 5;	a[3] = 7;
		a[4] = 11;	a[5] = 13;
		a[6] = 17;	a[7] = 19;
		a[8] = 23;	a[9] = 29;
		a[10] =	31;	a[11] = 37;
		a[12] =	41;	a[13] = 43;
		a[14] =	47;	a[15] = 53;
		a[16] =	59;	a[17] = 61;
		a[18] =	67;	a[19] = 71; 
	 
	 for (int i = 0; i<=n-1; i++){
			
			 a[i]= x[i];   // hier ist denke mein Hauptproblem; 
                                        //Idee war das vorhandene Array zu durchlaufen und die Elemente in ein neues zu speichern, 
                                      //eben halt wieviele man wissen möchte und dieses dann zurück an die main und ausgeben
		
			
			
			}
		return x;
    }
  
  
  
  

  public static void main( String[] args )
  {
  Scanner scan = new Scanner(System.in);

	System.out.println("Bis wohin sollen die Primzahlen ausgegeben werden? (maximal bis 20):");
		int n = scan.nextInt();
		int[] a = new int[20];
		 a = getPrime(n);
		
		System.out.println(a);
		
			
  }
}

Wenn es mit der Übergabe geht wollte ich eigentlich den Code auch etwas komplexer gestalten das die Primzahlen zum Beispiel per Siebverfahren ermittelt werden und nicht einfach feststehen.

 

[Volvagia]

x[i] = a[i];

System.out.println(Arrays.toString(a));

Das gibt dir aber nicht die Primzahlen bis n aus, sondern die ersten n Primzahlen. Und am Ende hast du natürlich 20 - n leere Felder.

 

[TryToHelp]

Dein Array ist immer 20 lang, mach mal so

int[] a = new int[n];

Deine zuweisung ist Falschrum, muss so sein

x[i] = a[i];

Mach deine Ausgabe mal so

System.out.println(Arrays.toString(a));

[EDIT]Mhh zu langsam[/EDIT]

 

[Gerlon]

Danke für die schnelle Hilfe klappt wunderbar
Doof das ich solange daran saß und mir das nicht aufgefallen ist das es x = a; sein muss ^^

 

[TryToHelp]

Ich würde auch noch sowas wie eine überprüfung einbauen, ob wirklich <20 ist, da du sonst eine Out of Bounds Exception bekommst

mport java.util.*;
 
public class Uebung4{
   public static int[] getPrime( int n ) {
        int[] x = new int[n];
        int[] a = new int[20];
        a[0] = 2;   a[1] = 3;  
        a[2] = 5;   a[3] = 7;
        a[4] = 11;  a[5] = 13;
        a[6] = 17;  a[7] = 19;
        a[8] = 23;  a[9] = 29;
        a[10] = 31; a[11] = 37;
        a[12] = 41; a[13] = 43;
        a[14] = 47; a[15] = 53;
        a[16] = 59; a[17] = 61;
        a[18] = 67; a[19] = 71; 
     
     for (int i = 0; i<=n-1; i++){
       if (i<20){ // um sicher zu gehen, das es keine outOfBoundsException gibt      
         x[i]= a[i];
       }        
    }
    return x;
  }

  public static void main( String[] args ){
    Scanner scan = new Scanner(System.in);
    System.out.println("Bis wohin sollen die Primzahlen ausgegeben werden? (maximal bis 20):");
    int n = scan.nextInt();
    int[] a = new int[n];
    a = getPrime(n);
    System.out.println(Arrays.toString(a));         
  }
}

 

[bone2]

  public static void main( String[] args ){
    Scanner scan = new Scanner(System.in);
    System.out.println("Bis wohin sollen die Primzahlen ausgegeben werden? (maximal bis 20):");
    int n = scan.nextInt();
    // int[] a = new int[n]; überflüssig, wird ja eine Zeile tiefer eh wieder überschrieben
    int[] a = getPrime(n);
    System.out.println(Arrays.toString(a));         
  }
}

 

[TryToHelp]

Stimmt du hast recht