return-Operator

[newbie]

hallo. warum funktioniert folgende Fachkonzeptklasse nicht ????

public class Teiler{
    
    //Attribute
    private int teiler;
    
    //Operation zur Teilerberechnung
    public String Teiler(int Zahl){
        for (int teiler = 1; teiler <= Zahl; teiler++){
            if (( Zahl % teiler) == 0){
                
                //teiler in eine Zeichenkette umwandeln
                Integer.toString(teiler);
                
                return teiler;
            }
        }
    }
    
}

p.s.: irgendwas stimmt mit dem return-Operator nicht :-(

 

[bygones]

Deine Methode verlangt einen String als Rückgabeobjekt, aber die Variable "teiler" ist int !!

 

[newbie]

aber ich wandel doch "teiler" in einen String um. *verzweifel
wie lautet denn die Klasse richtig ?

public class Teiler{
    
    //Attribute
    private String teiler;
    
    //Operation zur Teilerberechnung
    public Teiler(int Zahl){
        for (int teiler = 1; teiler <= Zahl; teiler++){
            if (( Zahl % teiler) == 0){
                
                //teiler in eine Zeichenkette umwandeln
                Integer.toString(teiler);
                
                return teiler;
            }
        }
    }
    
}

 

[bummerland]

du kannst im konstruktor keinen wert zurückgeben, das geht nicht. du musst dir eine extra-methode dafür schreiben. aber oben hast du noch anderen code geschrieben, seh ich grad. wie is denn nun dein code?

 

[newbie]

der erste quelltext ist eigentlich der ursprüngliche ... da gehört dann noch ein GUI applet dazu, dass den wert der zahl an die obige klasse schickt

also muss ich noch eine methode schreiben? um den int in den string umzuwandeln. k ... dann probier ich das mal ...

danke

 

[Guest]

ich hätt noch mal eine Bitte: kann sich mal jemand das anschaun, was ich da rumgewurschtelt habe ??? ich krieg das einfach nich hin .... :x :x

public class Teiler{
    
    //Attribute
    private int teiler;
    
    //Konstruktor
    public void Teiler(){
    }
    
    //Operation zur Teilerberechnung
    public String Teilerberechnung(int Zahl){
        for (int teiler = 1; teiler <= Zahl; teiler++){
            if (( Zahl % teiler) == 0){
                
                //teiler in eine Zeichenkette umwandeln
                Integer.toString(teiler);
                
                return teiler;
            }
        }
    }
    
}

import java.awt.*;
import java.applet.*;
import java.awt.event.*;

public class TeilerGUI extends Applet{

    //Attribute
    
    private Teiler teiler;
      
	//{{DECLARE_CONTROLS
	java.awt.Label Ueberschrift, Eingabetext, Ergebnistext;
	java.awt.TextField eingabeTextfeld, ergebnisTextfeld;
	java.awt.Button berechnenDruckknopf;
	//}}
	
	public void init(){
	    	    
		//{{INIT_CONTROLS 

                                   ....

                                //}}

		//{{REGISTER_LISTENERS
		AktionsAbhoerer einAktionsAbhoerer = new AktionsAbhoerer();
		berechnenDruckknopf.addActionListener(einAktionsAbhoerer);
		//}}
		
	}
	
	
    //Innere Klasse	
	class AktionsAbhoerer implements ActionListener
	{
		public void actionPerformed(ActionEvent event)
		{
		    int Zahl; //Eingabe
     //       int teiler; //Ausgabe
                        
            //Hilfsgröße
            String Merke, Ergebnis;
            Integer i; //Integer ist eine Klasse
            
            //Inhalt des Eigabefelds in eine Zahl umwandeln
            Merke = eingabeTextfeld.getText();
            i = Integer.valueOf(Merke);
            Zahl = i.intValue();      
            
            if(Zahl >= 1){ 
                
                //Zahl an Fachkonzeptklasse als Botschaft senden
                teiler = new Teiler(Zahl);
                
                //Ergebisse der Teilerberechnung ausgeben
                ergebnisTextfeld.setText(" " + teiler.Teilerberechnung);
                
            }
                                         
            else {
                Ergebnis = "Bitte eine natürliche Zahl eingeben!";                             
                ergebnisTextfeld.setText(Ergebnis);  
                }      
		    }
	}
}

 

[stev.glasow]

ich habe mir jetzt nich alles durch gelesen aber

//teiler in eine Zeichenkette umwandeln 
Integer.toString(teiler); 

return teiler;

geht nicht, falls du einen String zurückgeben möchtest.


vaiante 1:

int teiler = 19273;
return (new Integer(teiler)).toString(); // geht bestimt auch anders, aber so macht es sinn

variante 2:

Integer teiler = new Integer(18109);
return teiler.toString();

vriante 3:

TeilerKlasse teiler = new TeilerKlasse(-übergabeparamter an den konstruktor-);
return teiler.toString() // dafür würde ich eine methode toString() in die TeilerKlasse schreiben die einen formatierten string zurückliefert

klar was ich gemacht habe ?

 

[Guest]

ich danke vom ganzen herzen !!! :idea:

 

[Ebenius]

//teiler in eine Zeichenkette umwandeln 
Integer.toString(teiler); 

return teiler;

... ist schon fast richtig. Richtiger wäre:

//teiler in eine Zeichenkette umwandeln 
String ret = Integer.toString(teiler); 

return ret;

Das ist auch schon alles. Du hattest den richtigen Gedanken. Du lässt Dir nen String aus dem int machen. Du musst den String aber auch zurückliefern und nicht trotzdem den int.

Es grüßt Ebenius

 

[stev.glasow]

warum mache ich das immer so umständlich ?

 

[Ebenius]

warum mache ich das immer so umständlich ?

Weiß ich nicht. Aber ich hab schon manchmal Angst, dass Du meine Adresse herausfindest und mich erschlägst :lol:

Aber ich kenne das. Ich programmiere jetzt seit vier Jahren täglich java (beruflich) und ich ärgere mich immernoch regelmäßig, weil ich wieder mal eine Lösung für ein Problem gefunden habe, die erheblich einfacher oder kürzer ist als die die mir einfiel.

Es grüßt Ebenius

 

[stev.glasow]

nö nö, brauchst keine angst - das mit dem korregieren geht in ordung - find das sogar gut.