Array Summe- wo ist der Fehler ?

[marcels87]

Hallo zusammen,
ich schreibe gerade ein Programm, dass den ASCII-Wert eines Arrays berechnen soll. Das Programm steht zu 90%, leider ist die Ausgabe nicht so, wie ich es mir vorstelle. Statt der Summe aller Felder sieht die Ausgabe so aus:
Array[0]
Array[0]+Array[1]
Array[0]+Array[1]+Array[2]
usw.

Ich habe schon versucht, die Rechnung umzustellen, aber ohne Erfolg. Schaut doch bitte mal drüber.

Besten Dank.

So sieht meine for-Schleife aus:

    public static void main(String[]args)throws IOException{
        BufferedReader in = new BufferedReader(new InputStreamReader(System.in));
        System.out.println("Bitte geben Sie den String ein.");
        String string = in.readLine();
        System.out.println("Bitte geben Sie den Wert für die Modulo-Rechnung ein.");
        //int m = Integer.parseInt(in.readLine());
       
       
           
        char[] Array1 = string.toCharArray();
        int sum=0;
        int erg=0;
         for (int i =0; i<Array1.length; i++){
             erg=     sum+=Array1[i];
            
    System.out.print(erg);
   
        
         }}}

 

[Meniskusschaden]

Wenn du das vernünftig formatierst, siehst du den Fehler wahrscheinlich sofort.

 

[Javinner]

Summe aller Felder

char[] Array1 = string.toCharArray();
int sum = 0;
int erg = 0;
for (int i = 0; i < Array1.length; i++) {
      erg += Array1[i];
}
System.out.println(erg);
/** Konsolenausgabe */
Bitte geben Sie den String ein.
string
663

Wobei ich die Umwandlung in ein char-Array hinterfrage. Es ginge doch auch mit dem String string mittels Integer.parseInt(position im String)

edit: Variable sum ist hier das fünfte Rad am Wagen

edit: charAt(Position im String) war gemeint.

 

[marcels87]

@Javinner
Mit der von dir geänderten Schleife sieht mein Ergebnis für die Eingabe "String" so aus:
115
231
345
450
560
663

Bei dieser Ausgabe ist für mich nur die letzte Zeile relevant. Wie lasse ich die anderen verschwinden?
Bzgl. des Integer.parseInt(position im String) verstehe ich ehrlich gesagt nicht, wie mir das helfen soll.

 

[Javinner]

Poste doch bitte den Code zu dem irreführendem Ergebnis

edit:
s = 115;
t = 116
..
somit musst du jedes Ergebnis der Schleife ausgeben, weil eben 115 + 116 = 231 ergibt. https://simple.wikipedia.org/wiki/ASCII

 

[marcels87]

Okay, ich habs. Ich musste die Ausgabe von der Rechnung durch eine geschweifte Klammer trennen. Vielen Dank, euch. Jetzt möchte ich die Rechnung in eine Methode packen, welche den String und den modulo-Operatror übergeben bekommt.
Ich habe gerade erst mit dem Aufbau von Methoden begonnen. Folgende habe ich jetzt gebaut:

import java.io.*;


    public class Hashwert {
       
        public static void main(String[]args)throws IOException{
            BufferedReader in = new BufferedReader(new InputStreamReader(System.in));
            System.out.println("Bitte geben Sie den String ein.");
            String s1= in.readLine();
            System.out.println("Bitte geben Sie den Wert für die Modulo-Rechnung ein.");
            int m = Integer.parseInt(in.readLine());
            Summe(s1,m);
           
            System.out.println(Summe(erg));
       
        }
       
           
            static int Summe(String s1, int m){
            char[] Array1 =s1.toCharArray();
            int sum =0;
            int erg;
            for (int i =0; i<Array1.length; i++){
            sum+=Array1[i];
            erg = sum%m;
            return erg;
           
            }
                 
 }
}

Ich möchte aus der main-methode das Ergebnis der Summe Methode ausgeben. Bei meinem jetzigen Code kommt die Fehlermeldung: erg cannot be resolved to a variable.
Ich habe diese Art Ausgabe bereits bei anderen Programmen benutzt, dort ging es aber um reine Integer-Übergaben.
Liegt es daran, dass ich public int durch etwas anderes ersetzen muss? Ich habe es mit void erfolglos versucht und mit diversen varianten von static, private und public methoden.

 

[Meniskusschaden]

Jetzt möchte ich die Rechnung in eine Methode packen, welche den String und den modulo-Operatror übergeben bekommt.

Warum übergibst du es denn dann nicht?

 

[Meniskusschaden]

Es werden die Werte übergeben, die in den Klammern stehen. Die Variablennamen spielen dabei keine Rolle.

 

[marcels87]

Ich bin mir nicht sicher, wie du dass meinst. Ich habe mich an einem anderen Programm von mir orientiert, welches Parameter aus der main übernimmt:

public class Zufallszahl {
   
    public static void main (String [] args)throws IOException {
       
        BufferedReader in = new BufferedReader(new InputStreamReader(System.in));
        System.out.println("Geben Sie nacheinander die Werte für a,b &m ein.");
        int a = Integer.parseInt(in.readLine());
        int b = Integer.parseInt(in.readLine());
        int m = Integer.parseInt(in.readLine());

        System.out.println(Methode(a,b,m));
       
    }
   
    static int Methode(int a, int b, int m){
        int z= (int)(Math.random() * Integer.MAX_VALUE);
        return (a*z+b)&m;
       
       
    }
 }

Das Programm macht was es soll und ich glaube, dass ich die Methode im neuen Programm ganau so aufbauen kann. Wo liegt mein Denkfehler?

 

[Meniskusschaden]

In diesem Beispiel haben die korrespondierenden Variablen in main und Methode dieselben Namen UND sie werden beim Methodenaufruf in Klammern in der richtigen Reihenfolge angegeben. Letzteres ist erforderlich, Ersteres nicht. In deinem Posting #6 stimmen unnötigerweise die Variablennamen überein, aber sie werden nicht übergeben, weil sie beim Methodenaufruf nicht in den Klammern stehen.

 

[Javinner]

Ich musste die Ausgabe von der Rechnung durch eine geschweifte Klammer trennen

static int Summe(String s1, int m) /** Java-Konvention beachten */
{
    char[] Array1 = s1.toCharArray();/** Java-Konvention beachten */
    int sum = 0;
    int erg;
    for (int i = 0; i < Array1.length; i++) {
        sum += Array1[i];
        erg = sum % m;
        return erg; /** ein Return in der Schleife sollte in Zusammenhang mit einer Abbruchbedingung programmiert
                                werden, sonst wird die Schleife, wie hier, gleich beim ersten Mal verlassen */
    }
} //Hier wird ein return benoetigt (NetBeans hebt diesen Fehler hervor. Tut es deine IDE nicht?!)

Ich habe Schwierigkeiten dir zu folgen, mir ist die Aufgabenstellung nicht klar. Bitte poste doch a) die Aufgabenstellung und b) die Problembeschreibung kurz und verständlich und ganz wichtig: bleib bei einem Thema.

Das Programm macht was es soll und ich glaube, dass ich die Methode im neuen Programm ganau so aufbauen kann. Wo liegt mein Denkfehler?

?

 

[marcels87]

Die Aufgabe ist folgende:
Erstellung einer Formel, welche Auf Strings beliebiger Länge angewendet werden kann. DIe ASCII-Werte der einzelnen Zeichen des Strings sollen addiert werden und dann soll eine Modulo-Operation durchgeführt werden.
Schreiben einer entsprechenden Methode, die den String als Parameter übergeben bekommt und als Rückgabe den Hash-Wert (Ascii-Summe durch modulo) liefert.
Das Schreiben eines Hauptprogrammes, das einen String von der Tastatur einliest, mit ihm die Methode aufruft und ihren Rückgabewert auf dem Bildschirm ausgibt. Das Hauptprogramm soll in einer Schleife laufen, bis der Benutzer eine Beendigung wünscht.

Mein Code sieht momentan so aus:

import java.io.*;


    public class Hashwert {
      
        public static void main(String[]args)throws IOException{
            BufferedReader in = new BufferedReader(new InputStreamReader(System.in));
            System.out.println("Bitte geben Sie den String ein.");
            String s1= in.readLine();
            System.out.println("Bitte geben Sie den Wert für die Modulo-Rechnung ein.");
            int m = Integer.parseInt(in.readLine());
          
          
            System.out.print(Summe(s1,m));
      
        }
      
          
            static int Summe(String s1, int m){
            char[] Array1 =s1.toCharArray();
            int sum =0;
            int erg=0;
            for (int i =0; i<Array1.length;){
            sum+=Array1[i];
            erg = sum%m;
            }
            return erg;
                
 }
}

Eclipse gibt mir momentan keine Fehler aus, das Programm läuft an, ich gebe meine Eingaben ein, aber es erfolgt keine Ausgabe des Ergebnisses.
In der main habe ich es auch so versucht: System.out.print(Summe(erg));

Funktioniert auch nicht.

 

[Meniskusschaden]

Deine for-Schleife ist eine Endlosschleife.

 

[Javinner]

@marcels87
Schau dir das hier an:

Spoiler: Main

public class marcels87
{

    public static void main(String[] args)
    {
        HashCode hc = new HashCode();
        Scanner scanner = new Scanner(System.in);
        boolean ende = false;
        while (!ende) {
            System.out.println("Texteingabe");
            String s = scanner.nextLine();
            if (s.trim().equalsIgnoreCase("Stop")) {
                ende = true; //Programmende
            } else {
                System.out.println(hc.getHashCode(s));
            }
        }
    }

}

Spoiler: Klasse HashCode

public class HashCode
{

    //Modulo-Wert
    private final int moduloWert = dein Modulowert;

    /**
     * liefert HashWert nach aussen;
     * @param string String, welcher zu einer Summe addiert werden soll;
     * @return {@code string} Summe durch {@code moduloWert}
     */
    protected int getHashCode(String string)
    {
        int summe = 0;
        for (int i = 0; i < string.length(); i++) {
            summe += string.charAt(i);
        }
        return summe % moduloWert;
    }
}

Zum Thema HashCode gibt es hier ein richtig guten Artikel:
http://www.angelikalanger.com/Articles/EffectiveJava/03.HashCode/03.HashCode.html