ein if else problem

[a.pst]

salve,
das programm soll (irgendwann mal) die entropie eines textes berechnen....
aber ich bekomme die einzelnen werte nicht aus dem "if else"...

so hätte ichs gerne gehabt, aber so geht das leider irgendwie nicht :

float entropie= ea+eb+ec....ex+ey+ez;

wie bekomme ich ea...bis ez aus dem if else block raus zum rechnen?

gemeint ist ea bis ez:

                 if (a<0){
                 float wa = a/anzahlbuchstaben;
                 float la= (float) (Math.log10(1 / wa) / log2);
                 float ea= wa*la;
                 }

mein ganzer code:

public static void main(String[] args) {

        {
            BufferedReader ff;
            int cc;
            float a=0, b=0, c=0, d=0, e=0, f=0, g=0, h=0, i=0, j=0, k=0, l=0,
                    m=0, n=0, o=0, p=0, q=0, r=0, s=0, t=0, u=0, v=0, w=0,
                    x=0, y=0, z = 0, ß=0;
            float log2=(float) 0.30103;

            try {
                ff = new BufferedReader(new FileReader("text"));
                while ((cc = ff.read()) != -1) {
                    
                    //zählt die einzelnen buchstaben und summiert sie auf
                    char ccc = (char) cc;
                    if (ccc == 'a') {a++;}
                    else if(ccc=='b'){b++;}
                    else if(ccc=='c'){c++;}
                    else if(ccc=='d'){d++;}
                    else if(ccc=='e'){e++;}
                    else if(ccc=='d'){d++;}
                    else if(ccc=='f'){f++;}
                    else if(ccc=='g'){g++;}
                    else if(ccc=='h'){h++;}
                    else if(ccc=='i'){i++;}
                    else if(ccc=='j'){j++;}
                    else if(ccc=='k'){k++;}
                    else if(ccc=='l'){l++;}
                    else if(ccc=='m'){m++;}
                    else if(ccc=='n'){n++;}
                    else if(ccc=='o'){o++;}
                    else if(ccc=='p'){p++;}
                    else if(ccc=='q'){q++;}
                    else if(ccc=='r'){r++;}
                    else if(ccc=='s'){s++;}
                    else if(ccc=='ß'){ß++;}
                    else if(ccc=='t'){t++;}
                    else if(ccc=='u'){u++;}
                    else if(ccc=='v'){v++;}
                    else if(ccc=='w'){w++;}
                    else if(ccc=='x'){x++;}
                    else if(ccc=='y'){y++;}
                    else if(ccc=='z'){z++;}

                    else if(ccc=='A'){a++;}
                    else if(ccc=='B'){b++;}
                    else if(ccc=='C'){c++;}
                    else if(ccc=='E'){e++;}
                    else if(ccc=='F'){f++;}
                    else if(ccc=='G'){g++;}
                    else if(ccc=='H'){h++;}
                    else if(ccc=='I'){i++;}
                    else if(ccc=='J'){j++;}
                    else if(ccc=='K'){k++;}
                    else if(ccc=='L'){l++;}
                    else if(ccc=='M'){m++;}
                    else if(ccc=='N'){n++;}
                    else if(ccc=='O'){o++;}
                    else if(ccc=='P'){p++;}
                    else if(ccc=='Q'){q++;}
                    else if(ccc=='R'){r++;}
                    else if(ccc=='S'){s++;}
                    else if(ccc=='T'){t++;}
                    else if(ccc=='U'){u++;}
                    else if(ccc=='V'){v++;}
                    else if(ccc=='W'){w++;}
                    else if(ccc=='X'){x++;}
                    else if(ccc=='Y'){y++;}
                    else if(ccc=='Z'){z++;}
                }
                ff.close();
                
                 float anzahlbuchstaben=a+b+c+d+e+f+g+h+i+j+k+l+m+n+o+p+q+r+s+ß+t+u+v+w+x+y+z;
            
                 //berechnet wort auftritswahrscheinlichkeit, wort länge und einen teil der entropie
                 if (a<0){
                 float wa = a/anzahlbuchstaben;
                 float la= (float) (Math.log10(1 / wa) / log2);
                 float ea= wa*la;
                 }else if (b<0){
                 float wb=b/anzahlbuchstaben;
                 float lb= (float) (Math.log10(1 / wb) / log2);
                 float eb=wb*lb;
                 }else if(c<0){
                 float wc=c/anzahlbuchstaben;
                 float lc= (float) (Math.log10(1 / wc) / log2);
                 float ec=wc*lc;
                 }else if(d<0){
                 float wd=d/anzahlbuchstaben;
                 float ld= (float) (Math.log10(1 / wd) / log2);
                 float ed=wd*ld;
                 }else if(e<0){
                 float we=e/anzahlbuchstaben;
                 float le= (float) (Math.log10(1 / we) / log2);
                 float ee=we*le;
                 }else if(f<0){
                 float wf=f/anzahlbuchstaben;
                 float lf= (float) (Math.log10(1 / wf) / log2);
                 float ef=wf*lf;
                 }else if(g<0){
                 float wg=g/anzahlbuchstaben;
                 float lg= (float) (Math.log10(1 / wg) / log2);
                 float eg=wg*lg;
                 }else if(h<0){
                 float wh=h/anzahlbuchstaben;
                 float lh= (float) (Math.log10(1 / wh) / log2);
                 float eh=wh*lh;
                 }else if(i<0){
                 float wi=i/anzahlbuchstaben;
                 float li= (float) (Math.log10(1 / wi) / log2);
                 float ei=wi*li;
                 }else if(j<0){
                 float wj=j/anzahlbuchstaben;
                 float lj= (float) (Math.log10(1 / wj) / log2);
                 float ej=wj*lj;
                 }else if(k<0){
                 float wk=k/anzahlbuchstaben;
                 float lk= (float) (Math.log10(1 / wk) / log2);
                 float ek=wk*lk;
                 }else if(l<0){
                 float wl=l/anzahlbuchstaben;
                 float ll= (float) (Math.log10(1 / wl) / log2);
                 float el=wl*ll;
                 }else if(m<0){
                 float wm=m/anzahlbuchstaben;
                 float lm= (float) (Math.log10(1 / wm) / log2);
                 float em=wm*lm;
                 }else if(n<0){
                 float wn=n/anzahlbuchstaben;
                 float ln= (float) (Math.log10(1 / wn) / log2);
                 float en=wn*ln;
                 }else if(o<0){
                 float wo=0/anzahlbuchstaben;
                 float lo= (float) (Math.log10(1 / wo) / log2);
                 float eo=wo*lo;
                 }else if(p<0){
                 float wp=p/anzahlbuchstaben;
                 float lp= (float) (Math.log10(1 / wp) / log2);
                 float ep=wp*lp;
                 }else if(q<0){
                 float wq=q/anzahlbuchstaben;
                 float lq= (float) (Math.log10(1 / wq) / log2);
                 float eq=wq*lq;
                 }else if(r<0){
                 float wr=r/anzahlbuchstaben;
                 float lr= (float) (Math.log10(1 / wr) / log2);
                 float er=wr*lr;
                 }else if(s<0){
                 float ws=s/anzahlbuchstaben;
                 float ls= (float) (Math.log10(1 / ws) / log2);
                 float es=ws*ls;
                 }else if(ß<0){
                 float wß=ß/anzahlbuchstaben;
                 float lß= (float) (Math.log10(1 / wß) / log2);
                 float eß=wß*lß;
                 }else if(t<0){
                 float wt=t/anzahlbuchstaben;
                 float lt= (float) (Math.log10(1 / wt) / log2);
                 float et=wt*lt;
                 }else if(u<0){
                 float wu=u/anzahlbuchstaben;
                 float lu= (float) (Math.log10(1 / wu) / log2);
                 float eu=wu*lu;
                 }else if(w<0){
                 float wv=v/anzahlbuchstaben;
                 float lv= (float) (Math.log10(1 / wv) / log2);
                 float ev=wv*lv;
                 }else if(w<0){
                 float ww=w/anzahlbuchstaben;
                 float lw= (float) (Math.log10(1 / ww) / log2);
                 float ew=ww*lw;
                 }else if(x<0){
                 float wx=x/anzahlbuchstaben;
                 float lx= (float) (Math.log10(1 / wx) / log2);
                 float ex=wx*lx;
                 }else if(y<0){
                 float wy=y/anzahlbuchstaben;
                 float ly= (float) (Math.log10(1 / wy) / log2);
                 float ey=wy*wy;
                 }else if(z<0){
                 float wz=z/anzahlbuchstaben;                                                                                                                                                                               
                 float lz= (float) (Math.log10(1 / wz) / log2);
                 float ez=wz*lz;
                 }
            } 
            catch (IOException ee) {
                System.out.println("Fehler beim Lesen der Datei");
            }
        }
    }

 

[icarus2]

Also wenn du so viele if-else hintereinander hast, dann kann ich dir mit Sicherheit sagen, dass etwas an deinem Desing überhaupt nicht stimmt. Erstens wäre sicher einmal in switch-Statement angebrachter als so viele if-else Abfragen.

Allerdings wäre es nicht schlecht die einzelnen Buchstaben in einem Array zu speichern und dann für einen Vergleich eine Schleife durchlaufen zu lassen. (Man könnte die Buchstaben natürlich sogar als Objekte behandeln, scheint mir jedch für kleinere Aufgaben mti Kanonen auf Spatzen geschossen zu sein).

Hier mal so als Idee (nicht getestet und natürlich nicht fertig). Hoffe es ist mit den Kommentaren verständlich.

[Java]
import java.io.BufferedReader;
import java.io.FileReader;
import java.io.IOException;


public class TestReader {


public static void main(String[] args) {

BufferedReader reader;

int cc;

// array where all letters are stored
final char[] letters = {'a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'j', 'k', 'l',
'm', 'n', 'o', 'p', 'q', 'r', 's', 't', 'u', 'v', 'w', 'x', 'y', 'z'};

float[] occuredLetters = new float[26]; // array to count the events when a specific latter occurs
for(int i = 0; i < occuredLetters.length; i++){
occuredLetters = 0; // set all array values to 0
}

try {
reader = new BufferedReader(new FileReader("text"));

while( (cc = reader.read() ) != -1){

char currentLetter = (char) cc; // current char to check

// loops through the letters-array
for(int i = 0; i < letters.length; i++){

if( letters == currentLetter){ // if letter from the array fits the current char
occuredLetters++; // count up the amount of occurences for the current letter
break;
}
}

}

}catch(IOException e){
e.printStackTrace();
}
}

}
[/Java]

 

[a.pst]

@icarus2
design...nun ja, an und für sich war das nur als spielerei gedacht, mit wenig überlegung...
an switch case hatte ich auch schon gedacht, aber da bin ich wohl zu blöd für

dein vorschlag wird schwierig, denn wenn ein buchstabe im text nicht vorkommt, gibts eine division durch 0, und die gibt es ja nicht..., es bleibt das problem die fehlenden buchstaben aus zu sortieren ;(

 

[Final_Striker]

wie bekomme ich ea...bis ez aus dem if else block raus zum rechnen?

            float a=0, b=0, c=0, d=0, e=0, f=0, g=0, h=0, i=0, j=0, k=0, l=0,
                    m=0, n=0, o=0, p=0, q=0, r=0, s=0, t=0, u=0, v=0, w=0,
                    x=0, y=0, z = 0, ß=0;

Genauso wie du es schon oben mit den Variablen für a bis ß gemacht hast. ;-)

 

[a.pst]

            float a=0, b=0, c=0, d=0, e=0, f=0, g=0, h=0, i=0, j=0, k=0, l=0,
                    m=0, n=0, o=0, p=0, q=0, r=0, s=0, t=0, u=0, v=0, w=0,
                    x=0, y=0, z = 0, ß=0;

Genauso wie du es schon oben mit den Variablen für a bis ß gemacht hast. ;-)

vielen dank, das ist die lösung!

warum komm ich da nicht selbst drauf? *kopfschüttel*

 

[Landei]

ungetestet:

import java.util.*;
import java.io.*;

...

    public static void main(String[] args) {

        BufferedReader ff;
        int cc;
        float log2 = (float) 0.30103;

        try {
            Map<Character, Integer> freq = new HashMap<Character, Integer>();
            for(char ch = 'a'; ch <= 'z'; ch++) {
                freq.put(ch,0);
            }
            freq.put('ß',0);
            freq.put('ä',0);
            freq.put('ö',0);
            freq.put('ü',0);
            
            ff = new BufferedReader(new FileReader("text"));
            
            while ((cc = ff.read()) != -1) {
                char ccc = Character.toLowerCase((char) cc);
                Integer frequency = freq.get(ccc);
                if(frequency != null) {
                    freq.put(ccc, frequency + 1);
                }
            }
            ff.close();

            float anzahlbuchstaben = 0;
            for(Integer frequency : freq.values()) {
                anzahlbuchstaben += frequency;
            }
            
            //berechnet wort auftritswahrscheinlichkeit, wort länge und einen teil der entropie
            Map<Character, double[]> data = new TreeMap<Character, double[]>();
            for(Map.Entry<Character, Integer> entry : freq.entrySet()) {
                double probability = 1D*entry.getValue()/anzahlbuchstaben;
                double len = Math.log10(1 / probability) / log2;
                double entropy = probability * len;
                data.put(entry.getKey(), new double[]{probability, len, entropy});
            }
            
            //jetzt kann man mit data weiterarbeiten
            for(Map.Entry<Character, double[]> entry : data.entrySet()) {
                char ch = entry.getKey();
                double probability = entry.getValue()[0];
                double len = entry.getValue()[1];
                double entropy = entry.getValue()[2];
                System.out.println("'" + ch + "' prob:" + probability + " length:" + len + " entropy:" + entropy);
            }
        } catch (IOException ee) {
            System.out.println("Fehler beim Lesen der Datei");
        }
    }

 

[icarus2]

design...nun ja, an und für sich war das nur als spielerei gedacht, mit wenig überlegung...
an switch case hatte ich auch schon gedacht, aber da bin ich wohl zu blöd für

So kleine Übungen sind gerade immer gut um zu lernen Programme "gut" zu schreiben, denn man kann diese Erfahrung dann später in grossen Projekten verwenden ;-)
Wo liegen denn die Probleme beim switch-Statement?


Landei hat dir da natürlich gerade die Paradelösung vorgestellt. Ist um einiges besser als meine, auch wenn sie vielleicht fürs Anfäger-Forum etwas kompliziert ist ^^

 

[a.pst]

ungetestet:

also, bis ich das wirklich versteh, wirds noch ein bisserl dauern...
peinlich, peinlich...da hab ich doch glatt die umlaute vergessen.

So kleine Übungen sind gerade immer gut um zu lernen Programme "gut" zu schreiben, denn man kann diese Erfahrung dann später in grossen Projekten verwenden ;-)
Wo liegen denn die Probleme beim switch-Statement?

ich hatte das switch statemen falsch formuliert...jetzt funktioniert es aber

switch(ccc){
                      case 'a':a++;break;
                      case 'b':b++;break;
                      ....
                      ....
                      case 'z':z++;break;
                      case 'ä':a++;break;
                      case 'ö':ö++;break;
                      case 'ü':ü++;break;

                      case 'A':a++;break;
                      case 'B':b++;break;
                      ...
                      ...
                      case 'Z':z++;break;
                      case 'Ä':a++;break;
                      case 'Ö':ö++;break;
                      case 'Ü':ü++;break;
                  }

eine frage hätte ich noch, wie kann ich den wert von la auf eine ganze zahl aufrunden, zb 2,123 auf 3 aufrunden?

float la= (float) (Math.log10(1 / wa) / log2);

tante edit sagt:
aufrunden geht mit:

Math.ceil(double)

 

[a.pst]

ungetestet:

FALSCH! es funktioniert wunderbar
jetzt muß ich den code "nur" noch verstehn....

 

[icarus2]

Was verstehst du denn konkret nicht?

 

[a.pst]

Was verstehst du denn konkret nicht?

zum beispiel das:

klar ist mir, daß da eine neue treemap entsteht, weniger klar ist, weshalb eine treemap?

/berechnet wort auftritswahrscheinlichkeit, wort länge und einen teil der entropie
            Map<Character, double[]> data = new TreeMap<Character, double[]>();
            for(Map.Entry<Character, Integer> entry : freq.entrySet()) {

da ist mir auch nur klar, daß probability, len, entropy in den string eingetragen werden

                data.put(entry.getKey(), new double[]{probability, len, entropy});
            }

 

[Landei]

Der einzige wichtige Unterschied zwischen einer HashMap und einer TreeMap ist, dass die TreeMap ihre Schlüssel sortiert ausgibt, während sie bei einer HashMap "durcheinander" geliefert werden.

 

[a.pst]

ich meinte eigentlich damit, warum nicht "nur" eine treemap, sondern auch noch die hashmap?

 

[Landei]

Das ist eigentlich egal, man kann auch nur TreeMaps nehmen. HahsMap ist etwas schneller, aber das wird hier kaum ins Gewicht fallen.