Primzahl oder nicht?!

[figuerro]

hey leutz..
ich hab hier meine idee wie man herausfindet ob man eine primzahl hat oder nicht programmiert nur funzt seit dem ich javaeditor hab zu hause mein checkstyle nicht und von daher kann ich nicht kontrollieren was falsch ist. und nebenbei zeigt java mir auch so einen fehler an..

import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
import javax.swing.event.*;

/**
  *
  * Beschreibung
  *
  * @version 1.0 vom 02.09.2009
  * @author
  */

public class Uebung8 extends JFrame {
  // Anfang Attribute
  private JTextField jTextField1 = new JTextField();
  private JTextField jTextField2 = new JTextField();
  private JLabel jLabel1 = new JLabel();
  private JButton jButton1 = new JButton();
  private JLabel jLabel2 = new JLabel();
  // Ende Attribute

  public Uebung8(String title) {
    // Frame-Initialisierung
    super(title);
    setDefaultCloseOperation(WindowConstants.EXIT_ON_CLOSE);
    int frameWidth = 318;
    int frameHeight = 314;
    setSize(frameWidth, frameHeight);
    Dimension d = Toolkit.getDefaultToolkit().getScreenSize();
    int x = (d.width - getSize().width) / 2;
    int y = (d.height - getSize().height) / 2;
    setLocation(x, y);
    Container cp = getContentPane();
    cp.setLayout(null);
    // Anfang Komponenten

    jTextField1.setBounds(96, 80, 97, 28);
    jTextField1.setText("Eingabe");
    cp.add(jTextField1);
    jTextField2.setBounds(24, 144, 233, 28);
    jTextField2.setText("Ausgabe");
    cp.add(jTextField2);
    jLabel1.setBounds(8, 40, 280, 20);
    jLabel1.setText("Bitte geben Sei eine natürliche Zahl ein!");
    jLabel1.setFont(new Font("MS Sans Serif", Font.PLAIN, 13));
    cp.add(jLabel1);
    jButton1.setBounds(48, 200, 201, 33);
    jButton1.setText("Prüfen!");
    jButton1.addActionListener(new ActionListener() {
      public void actionPerformed(ActionEvent evt) {
        jButton1_ActionPerformed(evt);
      }
    });
    cp.add(jButton1);
    jLabel2.setBounds(48, 8, 201, 20);
    jLabel2.setText("Primzahl oder normale Zahl?");
    jLabel2.setFont(new Font("MS Sans Serif", Font.PLAIN, 13));
    cp.add(jLabel2);
    // Ende Komponenten

    setResizable(false);
    setVisible(true);
  }

  // Anfang Methoden
  public void jButton1_ActionPerformed(ActionEvent evt) {
    int divisor = 2;
    int ergebnis;
    boolean primzahl;
    
    String zahltext = Textfeld1.getText();
    int zahl = Integer.parseInt(zahltext);
    
    while((zahl-1) >= divisor){

      ergebnis = zahl % divisor;
      If (erg == 0){
        System.out.println(zahl + " ist KEINE Primzahl");
        primzahl = false;
        break;
      }
      divisor++;
    }
    if (primzahl == true)
    System.out.println(zahl + " ist eine Primzahl");
    
  }

  // Ende Methoden

  public static void main(String[] args) {
    new Uebung8("Uebung8");
  }
}

jetzt zeigt der mir das hier an:
"Uebung8.java:78:20: ';' expected
If (erg == 0){
^
1 error
"
also genau zwischen der runden klammer und der geschweifen klammer soll ich ein simikolon setzen. spinnt der?!

wäre nett wenn jemand das korrigieren (ich glaube theoretisch sollte das programm eigentlich funzen^^)
und wenns geht über checkstyle laufen würde.

daaaaankkee!!

 

[Ariol]

Schreib ein kleines "i" statt dem Großen. (beim If)

Java ist case-sensitive. D.h. HALLO!=hallo

EDIT: Die Fehlermeldung ist etwas irreführend...

 

[Landei]

Außerdem schreibst du einmal "erg", das nirgendwo definiert ist, das soll wohl "ergebnis" sein?

 

[figuerro]

ah danke.. ja anfangs hab ich erg benutzt aber dann zum verständnis ergebnis.. und jetzt hab ich alles verbessert was ihr gesagat hab aber steh schon wieder vor nem anderem problem xD

import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
import javax.swing.event.*;

/**
  *
  * Beschreibung
  *
  * @version 1.0 vom 02.09.2009
  * @author
  */

public class Uebung8 extends JFrame {
  // Anfang Attribute
  private JTextField Textfeld1 = new JTextField();
  private JTextField jTextField2 = new JTextField();
  private JLabel jLabel1 = new JLabel();
  private JButton jButton1 = new JButton();
  private JLabel jLabel2 = new JLabel();
  // Ende Attribute

  public Uebung8(String title) {
    // Frame-Initialisierung
    super(title);
    setDefaultCloseOperation(WindowConstants.EXIT_ON_CLOSE);
    int frameWidth = 318;
    int frameHeight = 314;
    setSize(frameWidth, frameHeight);
    Dimension d = Toolkit.getDefaultToolkit().getScreenSize();
    int x = (d.width - getSize().width) / 2;
    int y = (d.height - getSize().height) / 2;
    setLocation(x, y);
    Container cp = getContentPane();
    cp.setLayout(null);
    // Anfang Komponenten

    Textfeld1.setBounds(96, 88, 97, 28);
    Textfeld1.setText("Eingabe");
    cp.add(Textfeld1);
    jTextField2.setBounds(24, 144, 233, 28);
    jTextField2.setText("Ausgabe");
    cp.add(jTextField2);
    jLabel1.setBounds(8, 40, 280, 20);
    jLabel1.setText("Bitte geben Sei eine natürliche Zahl ein!");
    jLabel1.setFont(new Font("MS Sans Serif", Font.PLAIN, 13));
    cp.add(jLabel1);
    jButton1.setBounds(48, 200, 201, 33);
    jButton1.setText("Prüfen!");
    jButton1.addActionListener(new ActionListener() {
      public void actionPerformed(ActionEvent evt) {
        jButton1_ActionPerformed(evt);
      }
    });
    cp.add(jButton1);
    jLabel2.setBounds(48, 8, 201, 20);
    jLabel2.setText("Primzahl oder normale Zahl?");
    jLabel2.setFont(new Font("MS Sans Serif", Font.PLAIN, 13));
    cp.add(jLabel2);
    // Ende Komponenten

    setResizable(false);
    setVisible(true);
  }

  // Anfang Methoden
  public void jButton1_ActionPerformed(ActionEvent evt) {
    int divisor = 2;
    int ergebnis;
    boolean primzahl;
    
    String zahltext = Textfeld1.getText();
    int zahl = Integer.parseInt(zahltext);
    
    while((zahl-1) >= divisor){

      ergebnis = zahl % divisor;
      if (ergebnis == 0){
        System.out.println(zahl + " ist KEINE Primzahl");
        primzahl = false;
        break;
      }
      divisor++;
    }
    if (primzahl == true)
    System.out.println(zahl + " ist eine Primzahl");
    
  }

  // Ende Methoden

  public static void main(String[] args) {
    new Uebung8("Uebung8");
  }
}

Fehlermeldung:
"Uebung8.java:85:9: variable primzahl might not have been initialized
if (primzahl == true)
^
1 error"

das komische ist.. ich hab sie vor der while schleife initialisiert^^

was tun?^^
(sorry wenn ich euch so mit meinen anfängerfehlern penetriere... )

mfg figuerro

 

[eRaaaa]

du hast sie deklariert, aber nicht initialisiert ;D
boolean primzahl=false;

 

[Schandro]

das komische ist.. ich hab sie vor der while schleife initialisiert^^

Nein, hast du nicht. Du hast sie nur deklariert:

boolean primzahl;

Du musst sie auch initialisieren:

boolean primzahl = true;

 

[Ariol]

import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
import javax.swing.event.*;

/**
 * 
 * Beschreibung
 * 
 * @version 1.0 vom 02.09.2009
 * @author
 */

public class Uebung8 extends JFrame
{
	// Anfang Attribute
	private JTextField	inputTextfield	= new JTextField("Eingabe");
	private JTextField	outputTextfield	= new JTextField("Ausgabe");
	private JLabel		inputLabel		= new JLabel("Bitte geben Sei eine natürliche Zahl ein!");
	private JButton		submitButton	= new JButton("Prüfen!");
	private JLabel		outputLabel		= new JLabel("Primzahl oder normale Zahl?");

	// Ende Attribute

	public Uebung8()
	{
		// Frame-Initialisierung
		super("Primzahltest");
		setDefaultCloseOperation(WindowConstants.EXIT_ON_CLOSE);
		int frameWidth = 318;
		int frameHeight = 314;
		setSize(frameWidth, frameHeight);
		setLocationRelativeTo(null); // Zentriert
		setLayout(null);

		// Anfang Komponenten

		// Komponeten-Position
		inputTextfield.setBounds(96, 88, 97, 28);
		outputTextfield.setBounds(24, 144, 233, 28);
		inputLabel.setBounds(8, 40, 280, 20);
		submitButton.setBounds(48, 200, 201, 33);
		outputLabel.setBounds(48, 8, 201, 20);

		// Listener setzen
		submitButton.addActionListener(new ActionListener()
		{
			public void actionPerformed(ActionEvent evt)
			{
				jButton1_ActionPerformed(evt);
			}
		});

		// Schriftarten ändern
		inputLabel.setFont(new Font("MS Sans Serif", Font.PLAIN, 13));
		outputLabel.setFont(new Font("MS Sans Serif", Font.PLAIN, 13));

		// Weitere Eigenschaften
		outputTextfield.setEditable(false);

		// Komponenten hinzufügen
		add(inputTextfield);
		add(outputTextfield);
		add(inputLabel);
		add(submitButton);
		add(outputLabel);
		// Ende Komponenten

		setResizable(false);
		setVisible(true);
	}

	// Anfang Methoden
	public void jButton1_ActionPerformed(ActionEvent evt)
	{
		try
		{
			String zahltext = inputTextfield.getText();
			int zahl = Integer.parseInt(zahltext);
			boolean prime = isPrime(zahl);

			if (prime)
				outputTextfield.setText(zahl + " ist eine Primzahl");
			else
				outputTextfield.setText(zahl + " ist KEINE Primzahl");
		}
		catch (NumberFormatException nfe)
		{
			outputTextfield.setText("Keine gültige Zahl eingegeben!");
		}
	}

	private boolean isPrime(int number)
	{
		for (int div = 2; div <= (number - 1); div++)
		{
			if ((number % div) == 0)
			{
				return false;
			}
		}
		return true;
	}

	private boolean isPrime2(int number)
	{
		for (int div = 3; div <= (number / 2); div += 2)
		{
			if ((number % div) == 0)
			{
				return false;
			}
		}
		return true;
	}

	private boolean isPrime3(int number)
	{
		int u = 4;
		int mod = (int) (Math.pow(2, number) - 1);
		for (int i = 0; i < (number - 2); i++)
		{
			u = (int) ((Math.pow(u, 2) - 2) % mod);
		}
		return u == 0;
	}
}

Um mal etwas Ordung in deinen Code zu bringen.

EDIT: Denk mal über einen Layout-Manager nach.

 

[Quaxli]

Nur so am Rande: Anstelle von

if(primzahl==true)

würde es auch

if(primzahl)

tun.

Die Angabe in der Klammer muß ja einen boolschen Wert ergeben, d. h. etwas, daß man mit true oder false bewerten kann, was ja bei einer Variable vom Typ boolean von Haus aus gegeben ist.