Endlosschleife keine Rückgabe

[Gelöschtes Mitglied 71416]

Hi,

ich versuche den höchsten Pixelwert im Array zu ermitteln, aber bekomme mit diesem Code keine Rückgabe und sämtliche Methoden, in denen ich max() aufrufe, funktionieren ebenfalls nicht.

  public int max() {
        Picture maximumfinden = new Picture(bild);
        Pixel[] pixelfeld = maximumfinden.getPixels();
        Pixel max;
        max = pixelfeld[0];
        for (int i = 0; i < pixelfeld.length; i++) {
            Pixel bildpunkt = pixelfeld[i];
            int grau = (bildpunkt.getRed() + bildpunkt.getGreen() + bildpunkt.getBlue()) / 3;
            bildpunkt.setRed(grau);
            bildpunkt.setGreen(grau);
            bildpunkt.setBlue(grau);
            if (bildpunkt.getRed() > max.getRed()) {
            max = pixelfeld[i];
            }
        }
        return max.getRed();
     }

 

[yfons123]

System.out.println(....max())

ich denke dass der rückgabe wert 0 ist

 

[temi]

funktionieren ebenfalls nicht

Was bedeutet das genau?

Die Fehlerbeschreibung ist leicht ungenau.

 

[Gelöschtes Mitglied 71416]

Was bedeutet das genau?

Die Fehlerbeschreibung ist leicht ungenau.

Wenn ich eine Methode ausführe, die zum Beispiel max() enthält, wird mir zwar running angezeigt, aber das wars dann und die Grafische Oberfläche reagiert nicht mehr.

 

[yfons123]

dann kommentier alles aus bis du den grund gefunden hast

wenn du den grund hast warums abschmiert, löse es ... wenn du nicht mal weißt worans liegt wie sollst du dann eine Lösung finden ?

 

[KonradN]

Wie groß ist das Array? Generell ist es schlecht, lang laufende Dinge in dem UI Thread zu machen. Das dürfte vermutlich der Fall sein, wenn Du sagst:

und die Grafische Oberfläche reagiert nicht mehr.

Hier hilft es dann auch oft, so Methoden in Unit Tests zu testen - da bekommt man dann auch die Laufzeit mit und kann ggf. Gegenmaßnahmen treffen.

 

[Gelöschtes Mitglied 71416]

Wie groß ist das Array? Generell ist es schlecht, lang laufende Dinge in dem UI Thread zu machen. Das dürfte vermutlich der Fall sein, wenn Du sagst:


Hier hilft es dann auch oft, so Methoden in Unit Tests zu testen - da bekommt man dann auch die Laufzeit mit und kann ggf. Gegenmaßnahmen treffen.

Das sollte eigentlich kein Problem sein, ich nutze das gleiche Array auch in anderen Methode ohne Rückgaben und da läuft alles flüssig. Wenn ich den Wert in der Methode probeweise ausgeben lasse, funktioniert das auch alles, also liegt es irgendwie an der Rückgabe.
Denke mittlerweile aber, dass das Problem eher bei Netbeans liegen muss und nicht beim Code selbst

 

[yfons123]

raten bei fehlern ist eher ungeeignet

 

[Jw456]

int grau = (bildpunkt.getRed() + bildpunkt.getGreen() + bildpunkt.getBlue()) / 3;

Ist aber nicht die Umrechnung in Grauwerte so wie es das menschliche Auge sieht.

Grauwert = 0,299 × Rotanteil + 0,587 × Grünanteil + 0,114 × Blauanteil





Was genau ist denn das Problem?
Bleibt deine Methode hängen?
Wird der falsche wert zurückgeben?

 

[Gelöschtes Mitglied 71416]

int grau = (bildpunkt.getRed() + bildpunkt.getGreen() + bildpunkt.getBlue()) / 3;

Ist aber nicht die Umrechnung in Grauwerte so wie es das menschliche Auge sieht.

Grauwert = 0,299 × Rotanteil + 0,587 × Grünanteil + 0,114 × Blauanteil





Was genau ist denn das Problem?
Bleibt deine Methode hängen?
Wird der falsche wert zurückgeben?

Okay, aber ist bei dem Algorithmus nicht das Problem, dass ich zwischen int und double hin und her springe?


Das Problem ist, dass ich gar keinen Output bekomme. Die Methode funktioniert und ermittelt auch den richtigen Wert, aber gibt nichts zurück, sondern das Programm hängt sich dann einfach auf.

 

[yfons123]

dann kommentier alles aus bis du den grund gefunden hast

wenn du den grund hast warums abschmiert, löse es ... wenn du nicht mal weißt worans liegt wie sollst du dann eine Lösung finden ?

Das Problem ist, dass ich gar keinen Output bekomme. Die Methode funktioniert und ermittelt auch den richtigen Wert, aber gibt nichts zurück, sondern das Programm hängt sich dann einfach auf.

 

[KonradN]

Also generell glaube ich nicht, dass es an Netbeans liegen kann. Eine Entwicklungsumgebung startet ja auch einfach nur einen Java Prozess. Das ist also nichts anderes, als wenn Du es ohne Entwicklungsumgebung startest. (Maximal wäre denkbar, dass Du es im Debug Modus startest und irgendwelche Breakpoints gesetzt hast. Aber dann würdest Du in NetBeans ja sehen, dass er irgendwo angehalten hat!)

Und diese Aussage macht so keinen Sinn:

Das Problem ist, dass ich gar keinen Output bekomme. Die Methode funktioniert und ermittelt auch den richtigen Wert, aber gibt nichts zurück, sondern das Programm hängt sich dann einfach auf.

Entweder die Methode funktioniert und gibt dann auch den Wert zurück oder sie funktioniert eben nicht. Und ein Programm hängt sich nicht einfach auf.

Da wir aber nicht den ganzen Code von Dir kennen, können wir diesbezüglich nichts sagen. Du kannst natürlich Ausgaben in deinen Code aufnehmen - dann siehst Du relativ schnell, ob der Code an gewisse Stellen kommt oder eben nicht.
Des Weiteren solltest Du genau prüfen: Hast Du evtl. try / catch Konstrukte wo Du im catch nichts ausgibst oder so?

Okay, aber ist bei dem Algorithmus nicht das Problem, dass ich zwischen int und double hin und her springe?

Wo springst Du hin und her? Was für ein Typ gibt denn getGreen zurück?

 

[Jw456]

dann caste es in einen int

int grau = (int) (bildpunkt.getRed() + bildpunkt.getGreen() + bildpunkt.getBlue()) / 3;

 

[Jw456]

aber seit wann sind Farbwerte Kommazahlen.

PS. aus welcher Lib nimmst du Picture u. Pixel?
Oder sind das Deine Klassen?

 

[KonradN]

Er wird mit der Aussage vermutlich gemeint haben, dass er dann Gleitkommazahlen hat, wenn er Deine Berechnung nutzt. Die Werte selbst müssen int Werte sein, weil es sonst nicht übersetzbar wäre ohne Expliziten Cast.

Daher: Wenn Du mit der genauen Rechnung:
Grauwert = 0,299 × Rotanteil + 0,587 × Grünanteil + 0,114 × Blauanteil
arbeiten willst, dann wäre ein cast notwendig, also sowas wie:
int grau = (int) (0.299 * bildpunkt.getRed() + 0.587 * bildpunkt.getGreen() + 0.114 * bildpunkt.getBlue());

Aber das sind unabhängige Probleme! Nicht zu viel auf einmal vermischen! Mach einen Punkt nach dem anderen! Also sollte die Frage ausschließlich sein, wo dein Programm bisher nicht richtig funktioniert. Ob die Grauwerte nun stimmig sind oder nicht ist dabei erst einmal nebensächlich.

 

[yfons123]

Ich würde hier die if Abfrage vor dem Setzen/Ändern der Bildpunkte schreiben.

wieso hast du eine lösung für ein problem , wenn das problem noch gar nicht bekannt ist?

 

[yfons123]

Naja, weil Code gezeigt gewurde, der logisch aber nicht richtig ist, also Fehler enthält?

also folgendermaßen

if (bildpunkt.getRed() > max.getRed()) {
max = pixelfeld;
}

dieses if statement auch wenn es theoretisch in JEDER if abfrage das max setzt bzw es niemals tut hat es keinerlei auswirkung darauf dass sich der thread aufhängt der GUI da in der for schleife ( die den thread aufhängt ) nirgendwo anders benutzt wird

es KANN nicht der grund sein außer er hat bei der .getRed methode einen fehler programmiert
FALLS dies der fall sein sollte müsste sich die schleife schon bei

int grau = (bildpunkt.getRed() + bildpunkt.getGreen() + bildpunkt.getBlue()) / 3;

dem aufhängen
somit kann das if nicht der grund sein, ob das if ansich sinnlos ist weis ich nicht, kann sein dass es von der logik unsinnig ist aber das ist ja nicht wichtig ,das fällt später auf

 

[yfons123]

Wahrscheinlich ist das einfach wahnsinnig in-performant.

wenn das der fall ist

sollte man es lösen sobald es funktioniert und dann verbessern

 

[Gelöschtes Mitglied 71416]

Ich habe probeweise jetzt mal eine einfache Methode eingefügt, aber da das gleiche Problem: Das return wird nicht abgearbeitet.

public String returnTest() {
String ans = "Test";
return ans;
}

Ich werde mal probieren, das irgendwie anders zu lösen. Trotzdem danke!

 

[Jw456]

Dann zeige doch mal wo und wie du die Methode aufrufst.

 

[Gelöschtes Mitglied 71416]

Dann zeige doch mal wo und wie du die Methode aufrufst.

Naja ich hab jetzt erstmal probehalber einen Button dafür auf meiner Grafischen Oberfläche gemacht, um die Methode zu testen. Aufruf in der GUI Klasse:

private void jButton9ActionPerformed(java.awt.event.ActionEvent evt) {                                        
        bb.max();
    }

Da wo ich es dann wirklich brauche, sähe es dann so oder so ähnlich aus:

bildpunkt.setRed(255 * ((bildpunkt.getRed() - min()) / (max() - min())));

 

[Jw456]

Frage geht es den in dem listener mir der returnTest ja oder nein?

 

[Gelöschtes Mitglied 71416]

Frage geht es den in dem listener mir der returnTest ja oder nein?

Nee das geht auch nicht

 

[berndoa]

Wie gesagt wurde müsste man mal die Klassen wie Pixel und so sehen sowie deren Methoden um was sagen zu können.
Frage ist, was ist mit "hängt sich auf" gemeint?

Grundsätzlich ist immer denkbar dass da, warum auch immer, ein nicht zulässiger Wert versucht wird zurückzugeben, also bspw. null, falscher Typ oder auch ein leerer String oder so.

 

[Gelöschtes Mitglied 71416]

Ich bin jetzt erstmal zu meiner ursprünglichen Variante zurück, wo ich Minimum und Maximum direkt in der Methode ermittle... Da ist nur das Problem, dass ich bei dem Code die Fehlermeldung bekomme, dass durch 0 geteilt wird, weil am Anfang natürlich min und max gleich sind. Wenn ich aber vor die Formeln unten eine if Bedingung packe, dass das nur ausgeführt wird, wenn die Zahlen unterschiedlich sind, dann bekomme ich nur ein fast schwarzes Bild zurück und nicht das was ich will, nämlich nur einen verstärkten Kontrast. Hat jemand eine Idee, wo in dem Code der Fehler liegt?

    public void kontrast() {
        Picture neuesbild = new Picture(bild);
        Pixel[] pixelfeld = neuesbild.getPixels();
        Pixel max;
        Pixel min;
        max = pixelfeld[0];
        min = pixelfeld[0];
        for (int i = 0; i < pixelfeld.length; i++) {
            Pixel bildpunkt = pixelfeld[i];
            int grau = (int) (0.299 * bildpunkt.getRed() + 0.587 * bildpunkt.getGreen() + 0.114 * bildpunkt.getBlue());
            bildpunkt.setRed(grau);
            bildpunkt.setGreen(grau);
            bildpunkt.setBlue(grau);
            if (bildpunkt.getRed() > max.getRed()) {
            max = pixelfeld[i];
            }
            if (bildpunkt.getRed() < min.getRed()) {
            min = pixelfeld[i];
            }
            bildpunkt.setRed(255 * ((bildpunkt.getRed() - min.getRed()) / (max.getRed() - min.getRed())));
            bildpunkt.setGreen(255 * ((bildpunkt.getRed() - min.getRed()) / (max.getRed() - min.getRed())));
            bildpunkt.setBlue(255 * ((bildpunkt.getRed() - min.getRed()) / (max.getRed() - min.getRed())));
            }
        neuesbild.show();
    }

 

[MarvinsDepression]

Ich gehe davon aus, dass 'min' und 'max' Extrema des gesammten Bildes darstellen. Solltest Du dann nicht diese Werte im Vorfeld bestimmen (in einer separaten Schleife), bevor damit die Bildmanipulation durchgeführt wird.

 

[Oneixee5]

Die Identische Berechnung in 3 aufeinander folgenden Zeile muss nicht sein, man könnte das Ergebnis auch zwischenspeichern:

            bildpunkt.setRed(255 * ((bildpunkt.getRed() - min.getRed()) / (max.getRed() - min.getRed())));
            bildpunkt.setGreen(255 * ((bildpunkt.getRed() - min.getRed()) / (max.getRed() - min.getRed())));
            bildpunkt.setBlue(255 * ((bildpunkt.getRed() - min.getRed()) / (max.getRed() - min.getRed())));

Die Frage ist: Was wird denn erwartet, wenn max.getRed() - min.getRed() == 0 ergibt?

            int red = calculateRed(bildpunkt, min, max);
            bildpunkt.setRed(red);
            bildpunkt.setGreen(red);
            bildpunkt.setBlue(red);
...
    private static int calculateRed(final Pixel bildpunkt, final Pixel min, final Pixel max) {
        final int t = max.getRed() - min.getRed();
        if (t != 0) {
            return 255 * ((bildpunkt.getRed() - min.getRed()) / t);
        }
        return Integer.MAX_VALUE; // oder was auch immer dann erwartet wird
    }

 

[Jw456]

Du willst doch hier sowas wie eine Tonwertkorrektur machen Tonwert-Spreizung.

Dazu brauchst du ja den hellsten und dunkelsten Pixel im Bild. Eigentlich müsstes du das Array zweimal durch gehen einmal um die beiden Tonwerte zu finden und dann um die neuen Werte zu setzen. Ist der hellste Wert zb nur 230 kannst du alle Hellen Pixel spreizen.
Denn max ist 255, und min ist 0, als Luminanz Helligkeit

Bei den Tiefen das gleiche. Die Spreizung ist auch nicht unbedingt linear.

Eine sogenannte Gradationskurve.


Das was du hier versuchst wird sicherlich nicht den Effekt haben den du erwartest.
Auch wenn dein Code keinen Fehler mehr werfen wird.

 

[Jw456]

Du hast zb 230 als hellste Pixel, 255 ist das maximum also kannst du eine Korrektur um
255-230 = 25 Tonwerte machen.

Bei den Tiefen das gleiche umgekehrt

Ist der hellste Pixel 255 und der tiefste 0 gibt es keine automatische Spreizung.
Dann kommen die Gradationskurven zum Kontrast steigen zum Einsatz.

 

[Gelöschtes Mitglied 71416]

Also meint ihr mit einer zweiten Schleife in der gleichen Methode, wo ich dann nur die Veränderung der Grauwerte mit der Formel abarbeite? Das Problem ist ja eigentlich gerade, dass die Formeln erst zum Zuge kommen dürften, wenn die Schleife einmal durchs Feld ist. Wäre das denn überhaupt möglich, das in einer Methode zu machen?

 

[Jw456]

Alles in einer Methode würde ich nicht machen.

Erstmal würde ich eine Methode machen die den hellsten und tiefsten Luminanz Wert zurückgibt.
Aus einen Bitmap, entweder du macht für die Rückgabe eine Klasse oder benutzt ein Array.

Dann würde ich eine Methode zum Spreizen machen die als Übergabe das Bitmap und den Anfang der Spreizung für die Tiefen und der Lichter sowie Mitten bekommt.
In der Methode wirst du dann auch die min-max Methode aufrufen müssen.

Packe nicht alles in eine Methode. Teilbereiche sollte man ausgliedern.

Tonwertkorrektur, was ist das?

Erklärungen zum Stichwort 'Tonwertkorrektur' - im Stichwortverzeichnis des kostenlosen 'Fotolehrgang im Internet' von Tom! Striewisch

www.fotolehrgang.de

 

[berndoa]

Ich verstehe nicht ganz was das Endziel ist.

Aktuell änderst du für jedes Pixel des Bildes dessen rgb werte so ab dass rot=grün=blau=mittelwert der irsprünglichen rgb werte ist.

Ansonsten guckst du nebenher welches dieser Pixel nach der Transformation den größten rot wert hat (da du ja eh rot, grün, blau gleich setzt, ist es egal ob du hier rot, grün oder blau fürs maximumsuchen betrachtest)

Sehe grundsätzlich keinen Fehler, auch wenn mir der SInn hinter der Aktion noch nicht klar ist.

Methodenintern sehe ich direkt kein Problem, daher würde ich eher vermuten dass der übergebene input n sich bereits "fehlerhaft" ist.
Will sagen, dass die bild variable vielleicht null ist,
je nachdem wie die Klasse aussieht, ist vielleicht bild ein Attribut auf das die methode gar nicht zugreifen kann.
Oder es wird aus irgendeinem grund vll. bild=null was auch zu allerlei Fehlern führen kann.

Jedenfalls, wenn bild das ist, was es sein sollte, müsste die Methode eigentlich funktionieren.

Wie rufst du denn die Methode auf, gibt es vielleicht da Fehler?


Oder, was einr echt primitiver Fehler wäre, rufst du max(bild) auf (also ohne System.out.println(...) durm herum)
und erwartest trotzdem eine konsolenausgabe?

 

[Jw456]

hier eine einfache Lösung nur für die Lichter.

import javax.imageio.ImageIO;
import javax.swing.*;
import java.awt.*;
import java.awt.image.BufferedImage;
import java.io.File;
import java.io.IOException;

public class DisplayImage {
    public static void main(String[] args) throws IOException {
        new DisplayImage();
    }

    public DisplayImage() throws IOException {
        File file = new File("pizzdunk.jpg");
        BufferedImage bufferedImage = ImageIO.read(file);

        setAutoLichter(bufferedImage);

        ImageIcon imageIcon = new ImageIcon(bufferedImage);
        JFrame jFrame = new JFrame();

        jFrame.setLayout(new FlowLayout());

        jFrame.setSize(500, 500);
        JLabel jLabel = new JLabel();

        jLabel.setIcon(imageIcon);
        jFrame.add(jLabel);
        jFrame.setVisible(true);

        jFrame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

    }


    void setAutoLichter(BufferedImage bi){
        LumMinMax minMax = getMinMaxLum(bi);
        int max = minMax.getMax();
        int diff = 255-max;

        for (int x = 0; x < bi.getWidth(); x++) {
            for (int y = 0; y < bi.getHeight(); y++) {
                int rgb = bi.getRGB(x,y);

                int r = (rgb & 0xff0000) >> 16 ;
                int g = (rgb & 0xff00) >> 8;
                int b = rgb & 0xff;
                r+=diff;
                g+=diff;
                b+=diff;

                rgb = r;
                rgb = rgb<<8;
                rgb+=g;
                rgb = rgb<<8;
                rgb+=b;

                bi.setRGB(x,y,rgb);

            }
        }
    }

    LumMinMax getMinMaxLum(BufferedImage bufferedImage) {
        int min = 0;
        int max = 0;
        int lum;

        for (int x = 0; x < bufferedImage.getWidth(); x++) {
            for (int y = 0; y < bufferedImage.getHeight(); y++) {
                int rgb = bufferedImage.getRGB(x, y);
                lum = getLom2Pixel(rgb);
                if (lum > max) {
                    max = lum;
                }
                if (lum < min) {
                    min = lum;
                }
            }
        }
        return new LumMinMax(min, max);
    }

    int getLom2Pixel(int rgb) {
        int r = (rgb & 0xff0000) >> 16;
        int g = (rgb & 0xff00) >> 8;
        int b = rgb & 0xff;
        return (int) (0.299 * r + 0.587 * g + 0.114 * b);
    }

}

public class LumMinMax {
    private int min;
    private int max;

    public LumMinMax(int min, int max) {
        this.min = min;
        this.max = max;
    }

    public int getMin() {
        return min;
    }

    public int getMax() {
        return max;
    }

}

 

[Gelöschtes Mitglied 71416]

Ich hab's jetzt erstmal so gelöst, dass ich ich Minimum und Maximum in anderen Methoden berechne und hier dann nur noch das hier:
Ich teste die Bearbeitung des Bildes nochmal mit Photoshop, aber erstmal funktioniert das jetzt so

 public void kontrast() {
        Picture neuesbild = new Picture(bild);
        Pixel[] pixelfeld = neuesbild.getPixels();
        int max = max();
        int min = min();
        for (int i = 0; i < pixelfeld.length; i++) {
            Pixel bildpunkt = pixelfeld[i];
            double t = max - min;
            double s = ((bildpunkt.getRed() - min) / t);
            int done = (int) (255 * s);
            bildpunkt.setRed(done);
            bildpunkt.setGreen(done);
            bildpunkt.setBlue(done);
        }
        neuesbild.show();
    }

 

[Jw456]

LumMinMax getMinMaxLum(BufferedImage bufferedImage) {
        int min = 255;
       ...

sorry das ist sinvoller sonnt kommt ja immer 0 raus.

 

[Jw456]

Ich hab's jetzt erstmal so gelöst, dass ich ich Minimum und Maximum in anderen Methoden berechne und hier dann nur noch das hier:
Ich teste die Bearbeitung des Bildes nochmal mit Photoshop, aber erstmal funktioniert das jetzt so

 public void kontrast() {
        Picture neuesbild = new Picture(bild);
        Pixel[] pixelfeld = neuesbild.getPixels();
        int max = max();
        int min = min();
        for (int i = 0; i < pixelfeld.length; i++) {
            Pixel bildpunkt = pixelfeld[i];
            double t = max - min;
            double s = ((bildpunkt.getRed() - min) / t);
            int done = (int) (255 * s);
            bildpunkt.setRed(done);
            bildpunkt.setGreen(done);
            bildpunkt.setBlue(done);
        }
        neuesbild.show();
    }

Hier kommt doch maximal ein SW Bild raus, und keine Farbe.

 

[Gelöschtes Mitglied 71416]

Hier kommt doch maximal ein SW Bild raus, und keine Farbe.

ja genau, war vielleicht nicht so deutlich, aber mein ganzes Programm geht nur um die Bearbeitung von SW Bildern.