Originally Posted by Mysthik (Post 34709560)

Aus dem Beispiel werden einige Dinge nicht ganz klar.

Die paintComponent wird unteranderem dann aufgerufen, wenn:

1. Die GUI initialisiert UND angezeigt wird
2. ein update der Oberfläche aufgerufen wird
3. die parent Komponente aktuallisiert wird
4. Bei bestimmten events (resize usw.)

Wenn du aber jedesmal new Game().str aufrufst wird jedes mal ein neues Game-Object erzeugt. Bei dem erzeugen Object wird die paintComponent aber niemals aufgerufen also wird der String auch nicht gesetzt.

Originally Posted by .Scy (Post 34710323)

(was in deinem stück code auch fehlt).

Originally Posted by exceble (Post 34708726)

Um es mal simpel zu veranschaulichen (unwichtigen Code gelöscht)

Originally Posted by .Scy (Post 34712018)

wie schon gesagt, ist dein painComponent Schwachsinn. zu der zeit wo du das ding initialisierst hat es keine Größe. die Größe bekommt es erst vom frame, sobald es visible ist, dann zeichnest du zwar, aber nur ein mal und danach nie wieder außer du erzwingst einen repaint, was du in dem gezeigten code nicht tust. Zudem hast du eine Bedingung drinnen, welche man mit dem code nichtmal nachvollziehen kann.

natürlich falsch:
du kannst nicht einfach beliebig häufig new Game().str rufen. du hast deinem Str fest den wert "boo" gegeben und du rufst eben kein paintComponent auf, da du ja nichtmal weißt, wann java es für nötig hält paintComponent aufzurufen. zudem finde ich es mehr als suboptimal abhängig von einer position ein wort auszuwählen.

Originally Posted by exceble (Post 34709634)

Ja auch wenn ich vorher ein Game-Objekt erstelle, macht das kein Unterschied

Originally Posted by .Scy (Post 34713176)

du verstehst deinen code ja nichtmal, du erzeugst dir in deinem listener, was total schwachsinnig ist und was ich auch nicht gezeigt habe, weiterhin ein Game Objekt. Dieses zeichnet sich NIE! du hast es zwar, aber es liegt auf keinem frame.

bevor du hier weiter machst solltest du dir erstmal swing beibringen. du wirst immer boo heraus bekommen wenn du dein jpanel nicht in ein frame, welches visible ist, legst.

ich weiß nicht wo du dein frame erzeugst, aber dort MUSST du dein Game erzeugen UND Game deinem frame hinzufügen. so wie du es im moment zeigst ruft sich paintComponent nie auf, weil JPanel sich nur malt, wenn es vom frame dazu den Aufruf bekommt, oder du eben game.PaintComponent aufrufst, aber da du keinen gültigen Graphics hast wird das nicht klappen.

---

package typewords.ui;

import typewords.data.DataException;
import typewords.data.WordInterface;
import typewords.data.WordManager;

import javax.swing.*;
import java.awt.*;
import java.io.IOException;
import java.util.concurrent.ThreadLocalRandom;

public class Game extends JPanel{

    JFrame game = new JFrame("Typing-Words");

    public int points = 0, lives = 10;

    JPanel subPanel = new JPanel();
    JTextField jf = new JTextField("Hello", 42);
    JLabel jPoints = new JLabel("Points: " + points);
    JLabel jLives = new JLabel("Lives: " + lives);

    public String  str = "boo";

    private WordInterface wordProvider;
    int xPos = -50, yPos = 75, ms = 4;

    public Game() {
        wordProvider = new WordManager();
        game.setDefaultCloseOperation(WindowConstants.EXIT_ON_CLOSE);
        game.setPreferredSize(new Dimension(600,300));
    }

    public void startGame() throws IOException {
        game.add(new Game());

        jPoints.setBackground(Color.GREEN);
        jLives.setBackground(Color.RED);
        jPoints.setOpaque(true);
        jLives.setOpaque(true);

        subPanel.add(jf, 0);
        subPanel.add(jPoints, 1);
        subPanel.add(jLives, 2);
        subPanel.setBorder(BorderFactory.createLineBorder(Color.black));
        game.getContentPane().add(BorderLayout.PAGE_END, subPanel);

        game.pack();
        game.setVisible(true);

        System.out.println(jf.getText());
        System.out.println(jPoints.getText());

        jf.addKeyListener(new KeyEventListener());
    }

      [MENTION=295804]Override[/MENTION]
    public void paintComponent(Graphics g) {
        super.paintComponent(g);
        Graphics2D g2 = (Graphics2D) g;

        g2.setFont(new Font("Times Roman", Font.PLAIN, 18));

        try { Thread.sleep(ms); } catch (InterruptedException e) { e.printStackTrace(); }
        xPos += 1;

        if (xPos >= this.getWidth()) {
            lives-=1;
            jLives.setText("Lives: " + lives);
            System.out.println(jLives.getText());

            xPos = -50;
            yPos = ThreadLocalRandom.current().nextInt(25, 240);
            try {
                str = wordProvider.declareWord();
            } catch (DataException e) {
                e.printStackTrace();
            }
        }

        g2.drawString(str, xPos, yPos);
        repaint();
    }
}

---

---

package typewords.ui;

import java.awt.event.KeyEvent;
import java.awt.event.KeyListener;

public class KeyEventListener implements KeyListener {
    String word;
    private Game game = new Game();

      [MENTION=295804]Override[/MENTION]
    public void keyTyped(KeyEvent e) { }
      [MENTION=295804]Override[/MENTION]
    public void keyPressed(KeyEvent e) { }


      [MENTION=295804]Override[/MENTION]
    public void keyReleased(KeyEvent e) {
        getKey(e);
    }


    private void getKey(KeyEvent released) {
        char k = released.getKeyChar();
        System.out.println(released.getKeyChar());
        word = game.str;

        System.out.println(word);

        if(word.charAt(0) == k) {System.out.println("First char released");}
        if(word.charAt(1) == k) {System.out.println("Second char released");}
    }
}

---

Originally Posted by .Scy (Post 34713288)

das erste problem ist, dass du bei startGame
machst, dadurch hast du zwar ein game Objekt aber darauf hast du keinen zugriff, weil es anonym ist.

den keyListener wirst du nicht auf diese weise häufiger verwenden, also baut man sich einen anonymen keyListener, in diesem kannst du dann auf something zugreifen.

listener würde man so oder so entweder als innere class oder direkt als anonyme class bauen, damit sie eben zugriff auf alles in der klasse haben. dein problem mit der extra class für den KeyListener ist, dass du dein Objekt von Game public setzen musst, dies kann man durch innere oder anonyme klassen umgehen.

du hast genau 1 Game-Objekt welches gezeichnet wird, dies liegt in deiner startGame, deswegen musst du auch auf das Objekt zugreifen und kein neues erstellen, weil die anderen nicht gezeichnet werden. hoffe soweit verständlich