JavaEclipse setTextが上手くいかない

Question

実現したいこと

・numActionでは、１～９のボタンのどれかを押したとき、選択されているテキストフィールドに出力したい（135とおしたら135で出力）
・PmActionでテキストフィールドに入力された数字の符号を入れ替え、またテキストフィールドに出力したい

前提

JavaEclipseで電卓のプログラムを作っています。
二点上手くいかない部分がありますが原因は共通していると思われます。

発生している問題・エラーメッセージ

まず、numActionでは、123というように、順番通りに数字を押すと上手くテキストフィールドに表示されますが、157や81などバラバラに押すとテキストフィールドが勝手に空にされてしまいます。
PmActionでは、+/-ボタンを押すと、テキストフィールドに何も表示されなくなります。printlnしてみるとしっかり符号が逆になっているのでテキストフィールドに出力する部分に問題があると考えられます。
空白にしない限りエラーコードは出ません。

該当のソースコード

public class Calculator extends JFrame {

    JTextField input,input2,way, answer;
    JButton plus, minus, multiple, divide, result, clear, pm, percent, del, few, num, zero;
    double num1, num2;
   // String operator = "";
    JTextField currentFocusField;  // フォーカスされているフィールドを追跡する変数

    public static void main(String[] args) {
        JFrame w = new Calculator("Calculator");
        w.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        w.setSize(350, 400);
        w.setVisible(true);
    }

    public Calculator(String title) {
        super(title);
        JPanel pane = (JPanel) getContentPane();
        pane.setLayout(new BorderLayout());
        
        JPanel tfpanel=new JPanel();
        tfpanel.setLayout(new GridLayout(1,2));
        input = new JTextField(5);
        input.setBorder(new TitledBorder("数1"));
        input.setDocument(new NumericDocument());
        input.addFocusListener(new FocusAdapter() {
            @Override
            public void focusGained(FocusEvent e) {
                currentFocusField = input;  // inputがフォーカスされた時
            }
        });
        tfpanel.add(input);
        way = new JTextField(1);
        way.setBorder(new TitledBorder("計算方法"));
        way.setDocument(new NumericDocument2());
        tfpanel.add(way);
        input2 = new JTextField(5);
        input2.setBorder(new TitledBorder("数2"));
        input2.setDocument(new NumericDocument());
        input2.addFocusListener(new FocusAdapter() {
            @Override
            public void focusGained(FocusEvent e) {
                currentFocusField = input2;  // input2がフォーカスされた時
            }
        });
        tfpanel.add(input2);
        
        pane.add(tfpanel, BorderLayout.NORTH);//数式を入力するフィールドの設置

        answer = new JTextField(5);
        answer.setBorder(new TitledBorder("結果"));
        pane.add(answer, BorderLayout.SOUTH);//計算結果を表示するフィールドの設置

        JPanel leftpanel = new JPanel();
        leftpanel.setLayout(new GridLayout(5, 3));
        leftpanel.setPreferredSize(new Dimension(270, 400));
 
        clear = new JButton(new ClearAction());
        leftpanel.add(clear);//クリアボタン
  
        pm = new JButton(new PmAction());
        leftpanel.add(pm);//+-切り替えボタン
 
        percent = new JButton(new PercentAction());
        leftpanel.add(percent);//％表示切り替えボタン	
 
        for (int i = 1; i <= 9; i++) {
            num = new JButton(new numAction(Integer.toString(i)));
            leftpanel.add(num);
        }//1～9の番号ボタン

        del = new JButton(new DelAction());
        leftpanel.add(del);//削除ボタン

        zero=new JButton(new ZeroAction());
        leftpanel.add(zero);

        pane.add(leftpanel, BorderLayout.CENTER);

        JPanel rightpanel = new JPanel();
        rightpanel.setLayout(new GridLayout(5, 1));

        plus = new JButton(new PlusAction());
        rightpanel.add(plus);
        minus = new JButton(new MinusAction());
        rightpanel.add(minus);
        multiple = new JButton(new MultipleAction());
        rightpanel.add(multiple);
        divide = new JButton(new DivideAction());
        rightpanel.add(divide);
        result = new JButton(new ResultAction());
        rightpanel.add(result);

        pane.add(rightpanel, BorderLayout.EAST);
        input.requestFocus();  // ここでinputに初期フォーカスを設定
    }

    // Arithmetic Operations
    class PlusAction extends AbstractAction {
        PlusAction() {
            putValue(Action.NAME, "+");
            putValue( Action.SHORT_DESCRIPTION, "");
        }
        public void actionPerformed(ActionEvent e) {
        	 way.setText("+");
        }
    }

    class MinusAction extends AbstractAction {
        MinusAction() {
            putValue(Action.NAME, "-");
        }
        public void actionPerformed(ActionEvent e) {
           way.setText("-");
        }
    }

    class MultipleAction extends AbstractAction {
        MultipleAction() {
            putValue(Action.NAME, "*");
        }
        public void actionPerformed(ActionEvent e) {
          way.setText("*");
        }
    }

    class DivideAction extends AbstractAction {
        DivideAction() {
            putValue(Action.NAME, "/");
        }
        public void actionPerformed(ActionEvent e) {
           way.setText("/");
        }
    }

        class ResultAction extends AbstractAction {
        ResultAction() {
            putValue(Action.NAME, "=");
        }
        public void actionPerformed(ActionEvent e) {

            }
    }

    // Clear button
    class ClearAction extends AbstractAction {
        ClearAction() {
            putValue(Action.NAME, "AC");
        }
        public void actionPerformed(ActionEvent e) {
        }
    }

    // Delete last character button
    class DelAction extends AbstractAction {
        DelAction() {
            putValue(Action.NAME, "Del");
        }
        public void actionPerformed(ActionEvent e) {
  
            }
    }

    // Plus/Minus action button
    class PmAction extends AbstractAction {
        PmAction() {
            putValue(Action.NAME, "+/-");
        }
        public void actionPerformed(ActionEvent e) {
            double num = Double.parseDouble(input.getText());
            if (currentFocusField == input) {
            	if(input!=null) {
            	String text=input.getText();
                num=Double.parseDouble(text);
                num=-num;
                String change=String.valueOf(num);
                input.setText(change);
                
            	}
            	
            } else if (currentFocusField==input2) {
            	if(input2!=null) {
            	String text=input2.getText();
            	num=Double.parseDouble(text);
            	input2.setText(String.valueOf(-num));
            }
            	else {
            		input2.setText("");
            	}
            }
        }
    }

    class PercentAction extends AbstractAction {
    	double num=0;
        PercentAction() {
            putValue(Action.NAME, "%");
        }
        public void actionPerformed(ActionEvent e) {
        	}
    }

    class numAction extends AbstractAction {
        String number;
        numAction(String number) {
            this.number = number;
            putValue(Action.NAME, number);
        }
        public void actionPerformed(ActionEvent e) {
        	String inputnum=input.getText();
        	String input2num=input2.getText();
        	if (currentFocusField == input) {
                input.setText(inputnum + number);
            } else if (currentFocusField==input2) {
                input2.setText(input2num+number);
                System.out.println(input2num+number);
            }
        }
    }    
    class ZeroAction extends AbstractAction{
    	ZeroAction() { 
    		putValue(Action.NAME, "0");
        }
    	public void actionPerformed(ActionEvent e) {
            }
        }  	
    
    class NumericDocument extends PlainDocument {
        String validValues = "0123456789.+-%";

        public void insertString(int offset, String str, AttributeSet a) {
            if (validValues.indexOf(str) == -1) {
                return;  
                }
            try {
                super.insertString(offset, str, a);
            } catch (BadLocationException e) {
                System.out.println(e);
            }
        }
    }
    
    class NumericDocument2 extends PlainDocument {
        String validValues = "+-*/%";

        public void insertString(int offset, String str, AttributeSet a) {
        	 if( validValues.indexOf( str ) == -1 ){ 
        			return;
        		    }
        		    try{
        			super.insertString( offset, str, a ); 
        		    }
        		    catch( BadLocationException e ) {
        			System.out.println( e );
        		    }
            }
       
        }

    class NumericDocument3 extends PlainDocument {
        String validValues = "0123456789.+-";

        public void insertString(int offset, String str, AttributeSet a) {
        	 if( validValues.indexOf( str ) == -1 ){ 
        			return;
        		    }
        		    try{
        			super.insertString( offset, str, a ); 
        		    }
        		    catch( BadLocationException e ) {
        			System.out.println( e );
        		    }
            }
        
    }
}

Comments

[jimbe]

コードは「コードのマークダウン」(ファイル毎に ``` だけの行で前後を囲む) を用いてください。普通の文章の中に書くと一部の記号がマークダウンと解釈されて消えてしまったりします。

また出来れば、一部抜粋のようなコードでは無く状況が再現できる実行可能な状態のコードをご提示ください。

[juner]

参考
https://teratail.com/help#about-markdown

[dodox86]

既にコメントいただいていることへの対応はするとして:

> JavaEclipse setTextが上手くいかない
...
> まず、numActionでは、123というように、順番通りに数字を押すと上手くテキストフィールドに表示されますが、157や81などバラバラに押すとテキストフィールドが勝手に空にされてしまいます。

せっかくEclipseを使っているのですから、まずはご自身でもデバッグしませんか。

[nonfi]

プログラミングを始めたばかりでまだよくわからないこと、慣れていないことがあり申し訳ございません。

、、、lang-java
import java.awt.BorderLayout;
import java.awt.Dimension;
import java.awt.GridLayout;
import java.awt.event.ActionEvent;
import java.awt.event.FocusAdapter;
import java.awt.event.FocusEvent;

import javax.swing.AbstractAction;
import javax.swing.Action;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.JTextField;
import javax.swing.border.TitledBorder;
import javax.swing.text.AttributeSet;
import javax.swing.text.BadLocationException;
import javax.swing.text.PlainDocument;

public class Calculator extends JFrame {

JTextField input,input2,way, answer;
JButton plus, minus, multiple, divide, result, clear, pm, percent, del, few, num, zero;
double num1, num2;
// String operator = "";
JTextField currentFocusField; // フォーカスされているフィールドを追跡する変数

public static void main(String[] args) {
JFrame w = new Calculator("Calculator");
w.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
w.setSize(350, 400);
w.setVisible(true);
}

public Calculator(String title) {
super(title);
JPanel pane = (JPanel) getContentPane();
pane.setLayout(new BorderLayout());

JPanel tfpanel=new JPanel();
tfpanel.setLayout(new GridLayout(1,2));
input = new JTextField(5);
input.setBorder(new TitledBorder("数1"));
input.setDocument(new NumericDocument());
input.addFocusListener(new FocusAdapter() {
@Override
public void focusGained(FocusEvent e) {
currentFocusField = input; // inputがフォーカスされた時
}
});
tfpanel.add(input);
way = new JTextField(1);
way.setBorder(new TitledBorder("計算方法"));
way.setDocument(new NumericDocument2());
tfpanel.add(way);
input2 = new JTextField(5);
input2.setBorder(new TitledBorder("数2"));
input2.setDocument(new NumericDocument());
input2.addFocusListener(new FocusAdapter() {
@Override
public void focusGained(FocusEvent e) {
currentFocusField = input2; // input2がフォーカスされた時
}
});
tfpanel.add(input2);

pane.add(tfpanel, BorderLayout.NORTH);//数式を入力するフィールドの設置

answer = new JTextField(5);
answer.setBorder(new TitledBorder("結果"));
pane.add(answer, BorderLayout.SOUTH);//計算結果を表示するフィールドの設置

JPanel leftpanel = new JPanel();
leftpanel.setLayout(new GridLayout(5, 3));
leftpanel.setPreferredSize(new Dimension(270, 400));

clear = new JButton(new ClearAction());
leftpanel.add(clear);//クリアボタン

pm = new JButton(new PmAction());
leftpanel.add(pm);//+-切り替えボタン

percent = new JButton(new PercentAction());
leftpanel.add(percent);//％表示切り替えボタン

for (int i = 1; i <= 9; i++) {
num = new JButton(new numAction(Integer.toString(i)));
leftpanel.add(num);
}//1～9の番号ボタン

del = new JButton(new DelAction());
leftpanel.add(del);//削除ボタン

zero=new JButton(new ZeroAction());
leftpanel.add(zero);

pane.add(leftpanel, BorderLayout.CENTER);

JPanel rightpanel = new JPanel();
rightpanel.setLayout(new GridLayout(5, 1));

plus = new JButton(new PlusAction());
rightpanel.add(plus);
minus = new JButton(new MinusAction());
rightpanel.add(minus);
multiple = new JButton(new MultipleAction());
rightpanel.add(multiple);
divide = new JButton(new DivideAction());
rightpanel.add(divide);
result = new JButton(new ResultAction());
rightpanel.add(result);

pane.add(rightpanel, BorderLayout.EAST);
input.requestFocus(); // ここでinputに初期フォーカスを設定
}

// Arithmetic Operations
class PlusAction extends AbstractAction {
PlusAction() {
putValue(Action.NAME, "+");
putValue( Action.SHORT_DESCRIPTION, "");
}
public void actionPerformed(ActionEvent e) {
way.setText("+");
}
}

class MinusAction extends AbstractAction {
MinusAction() {
putValue(Action.NAME, "-");
}
public void actionPerformed(ActionEvent e) {
way.setText("-");
}
}

class MultipleAction extends AbstractAction {
MultipleAction() {
putValue(Action.NAME, "*");
}
public void actionPerformed(ActionEvent e) {
way.setText("*");
}
}

class DivideAction extends AbstractAction {
DivideAction() {
putValue(Action.NAME, "/");
}
public void actionPerformed(ActionEvent e) {
way.setText("/");
}
}

class ResultAction extends AbstractAction {
ResultAction() {
putValue(Action.NAME, "=");
}
public void actionPerformed(ActionEvent e) {
String fourway=way.getText();
int ans=0;
if(fourway.equals("+")) {
ans = Integer.parseInt( input.getText() ) + Integer.parseInt( input2.getText() );
}
if(fourway.equals("-")) {
ans = Integer.parseInt( input.getText() ) - Integer.parseInt( input2.getText() );
}
if(fourway.equals("*")) {
ans = Integer.parseInt( input.getText() ) * Integer.parseInt( input2.getText() );
}
if(fourway.equals("/")) {
ans = Integer.parseInt( input.getText() ) / Integer.parseInt( input2.getText() );
}
answer.setText( String.valueOf( ans ) );
}

}

// Clear button
class ClearAction extends AbstractAction {
ClearAction() {
putValue(Action.NAME, "AC");
}
public void actionPerformed(ActionEvent e) {
input.setText(""); // Clear input
answer.setText(""); // Clear result

}
}

// Delete last character button
class DelAction extends AbstractAction {
DelAction() {
putValue(Action.NAME, "Del");
}
public void actionPerformed(ActionEvent e) {
String text = input.getText();
String text2 = input2.getText();
if (currentFocusField == input) {
if (text.length() > 0) {
input.setText(text.substring(0, text.length() - 1));
}

else if (currentFocusField == input2) {
if (text2.length() > 0) {
input2.setText(text2.substring(0, text2.length() - 1));
}
}
}
}
}

// Plus/Minus action button
class PmAction extends AbstractAction {
PmAction() {
putValue(Action.NAME, "+/-");
}
public void actionPerformed(ActionEvent e) {
double num = Double.parseDouble(input.getText());
if (currentFocusField == input) {
if(input!=null) {
String text=input.getText();
num=Double.parseDouble(text);
num=-num;
String change=String.valueOf(num);
input.setText(change);

}

} else if (currentFocusField==input2) {
if(input2!=null) {
String text=input2.getText();
num=Double.parseDouble(text);
input2.setText(String.valueOf(-num));
}
else {
input2.setText("");
}
}

}
}

class PercentAction extends AbstractAction {
double num=0;
PercentAction() {
putValue(Action.NAME, "%");
}
public void actionPerformed(ActionEvent e) {
way.setText("%");
if (currentFocusField == input) {
num=Double.parseDouble(input.getText());

} else if (currentFocusField==input2) {
num=Double.parseDouble(input2.getText());
}
answer.setText(String.valueOf( num/100 ));
}
}

class numAction extends AbstractAction {
String number;
numAction(String number) {
this.number = number;
putValue(Action.NAME, number);
}
public void actionPerformed(ActionEvent e) {
String inputnum=input.getText();
String input2num=input2.getText();
if (currentFocusField == input) {
input.setText(inputnum + number);
} else if (currentFocusField==input2) {
input2.setText(input2num+number);
System.out.println(input2num+number);
}
}
}

class ZeroAction extends AbstractAction{
ZeroAction() {
putValue(Action.NAME, "0");
}
public void actionPerformed(ActionEvent e) {
if (currentFocusField == input) {
input.setText(input.getText() + "0");
} else if (currentFocusField==input2) {
input2.setText(input2.getText() + "0");
}
}

}

class NumericDocument extends PlainDocument {
String validValues = "0123456789.+-%";

public void insertString(int offset, String str, AttributeSet a) {
if (validValues.indexOf(str) == -1) {
return;
}
try {
super.insertString(offset, str, a);
} catch (BadLocationException e) {
System.out.println(e);
}
}
}

[nonfi]

class NumericDocument2 extends PlainDocument {
String validValues = "+-*/%";

public void insertString(int offset, String str, AttributeSet a) {
if( validValues.indexOf( str ) == -1 ){
return;
}
try{
super.insertString( offset, str, a );
}
catch( BadLocationException e ) {
System.out.println( e );
}
}

}

class NumericDocument3 extends PlainDocument {
String validValues = "0123456789.+-";

public void insertString(int offset, String str, AttributeSet a) {
if( validValues.indexOf( str ) == -1 ){
return;
}
try{
super.insertString( offset, str, a );
}
catch( BadLocationException e ) {
System.out.println( e );
}
}

}
}
、、、

分かれてしまいすみません。
デバッグとはどうやってやるのかも分からず、もし可能であれば教えていただきたいです。
お手数おかけしますが、皆様よろしくお願いいたします。

[jimbe]

コメント欄にはマークダウンは使えません。
コードや質問の情報等は質問を編集で追加・変更等を行ってください。
なお、コードのマークダウンは「、、、」では無く「```」(キーボードの＠キーの上の段)です。

[nonfi]

ご丁寧にありがとうございます。字数制限の都合上いくつかクラスの中身を削りました。

Answer 1

まず、numActionでは、123というように、順番通りに数字を押すと上手くテキストフィールドに表示されますが、157や81などバラバラに押すとテキストフィールドが勝手に空にされてしまいます。

NumericDocumentがおかしいです。
insertStringのstrには今入力した１文字ではなく、入力されたすべての文字が入ってきます。
そのため連番でないとindexOfがfalseになり、insertStringされません（空になってしまいます）
１文字ずつvalidValuesと比較するか、正規表現で判定してください。

class NumericDocument extends PlainDocument {
    String validValues = "0123456789.+-%";

    public void insertString(int offset, String str, AttributeSet a) throws BadLocationException {
        for (char c : str.toCharArray()) { // １文字ずつ比較の場合
            if (validValues.indexOf(c) == -1) {
                return;
            }
        }
        super.insertString(offset, str, a);
    }
}

class NumericDocument2 extends PlainDocument {
    Pattern p = Pattern.compile("[+\\-*/%]*"); // []の中の - はエスケープが必要

    public void insertString(int offset, String str, AttributeSet a) throws BadLocationException {
        if (p.matcher(str).matches()) { // 正規表現で判定の場合
            super.insertString(offset, str, a);
        }
    }
}

---

訳のわからない動作に遭遇した場合は、できるだけシンプルな状態に戻ってください（「連番じゃないと入力できない」なんて意味不明すぎますよね？^^;

当初コードにはNumericDocumentはありませんでした。
つまり的はずれなところを探していたということになります。

「setDocumentが影響するなんて思いもしなかった」ということであっても、外してみる（コメント化してみる）というのは重要です。
無関係なら（自分が数字以外入れないように注意すれば）何の問題ないはずです。

（本題には不要なコードを）削る方法で原因がわからない場合は、JTextFieldとJButton１個から徐々に増やしていくような逆の手法も有効です。

import javax.swing.*;
import javax.swing.border.TitledBorder;
import javax.swing.text.*;
import java.awt.*;
import java.awt.event.*;
import java.util.regex.Pattern;

public class Calculator extends JFrame {
    public static void main(String[] args) {
        new Calculator().setVisible(true);
    }

    private final JTextField input;
    private final JTextField input2;
    private final JTextField way;
    private final JTextField answer;
    private JTextField currentFocusField;

    public Calculator() {
        super("Calculator");
        setDefaultCloseOperation(EXIT_ON_CLOSE);
        setSize(350, 400);
        setLocationRelativeTo(null);

        var tfPanel = new JPanel(new GridLayout(1, 3));

        input = new JTextField(5);
        input.setBorder(new TitledBorder("数1"));
        input.setDocument(new NumericDocument());
        input.addFocusListener(new FocusAdapter() {
            @Override public void focusGained(FocusEvent e) {
                currentFocusField = input;
            }
        });
        tfPanel.add(input);

        way = new JTextField(1);
        way.setBorder(new TitledBorder("計算方法"));
        way.setDocument(new NumericDocument2());
        tfPanel.add(way);

        input2 = new JTextField(5);
        input2.setBorder(new TitledBorder("数2"));
        input2.setDocument(new NumericDocument());
        input2.addFocusListener(new FocusAdapter() {
            @Override public void focusGained(FocusEvent e) {
                currentFocusField = input2;
            }
        });
        tfPanel.add(input2);
        add(tfPanel, BorderLayout.NORTH);


        answer = new JTextField(5);
        answer.setBorder(new TitledBorder("結果"));
        add(answer, BorderLayout.SOUTH);


        var leftPanel = new JPanel(new GridLayout(5, 3));
        leftPanel.setPreferredSize(new Dimension(270, 400));

        leftPanel.add(new JButton(new ClearAction()));
        leftPanel.add(new JButton(new PmAction()));
        leftPanel.add(new JButton(new PercentAction()));
        for (var i = 1; i <= 9; i++) {
            leftPanel.add(new JButton(new NumAction(Integer.toString(i))));
        }
        leftPanel.add(new JButton(new DelAction()));
        leftPanel.add(new JButton(new NumAction("0")));
        add(leftPanel, BorderLayout.CENTER);


        var rightPanel = new JPanel(new GridLayout(5, 1));
        rightPanel.add(new JButton(new CalcAction("+")));
        rightPanel.add(new JButton(new CalcAction("-")));
        rightPanel.add(new JButton(new CalcAction("*")));
        rightPanel.add(new JButton(new CalcAction("/")));
        rightPanel.add(new JButton(new ResultAction()));
        add(rightPanel, BorderLayout.EAST);

        input.requestFocus();
    }

    class ClearAction extends AbstractAction {
        ClearAction() {
            super("AC");
        }

        public void actionPerformed(ActionEvent e) {
            input.setText("");
            input2.setText("");
            way.setText("");
            answer.setText("");
        }
    }

    class PmAction extends AbstractAction {
        PmAction() {
            super("+/-");
        }

        public void actionPerformed(ActionEvent e) {
            var num = Integer.parseInt(currentFocusField.getText());
            currentFocusField.setText(String.valueOf(-num));
        }
    }

    class PercentAction extends AbstractAction {
        PercentAction() {
            super("%");
        }

        public void actionPerformed(ActionEvent e) {
            way.setText("%");
            var num = Integer.parseInt(currentFocusField.getText());
            answer.setText(String.valueOf(num / 100d));
        }
    }

    class NumAction extends AbstractAction {
        NumAction(String name) {
            super(name);
        }

        public void actionPerformed(ActionEvent e) {
            currentFocusField.setText(currentFocusField.getText() + getValue(Action.NAME));
        }
    }

    class DelAction extends AbstractAction {
        DelAction() {
            super("Del");
        }

        public void actionPerformed(ActionEvent e) {
            var text = currentFocusField.getText();
            if (!text.isEmpty()) {
                currentFocusField.setText(text.substring(0, text.length() - 1));
            }
        }
    }

    class CalcAction extends AbstractAction {
        CalcAction(String name) {
            super(name);
        }

        public void actionPerformed(ActionEvent e) {
            way.setText(getValue(Action.NAME).toString());
        }
    }

    class ResultAction extends AbstractAction {
        ResultAction() {
            super("=");
        }

        public void actionPerformed(ActionEvent e) {
            var ans = switch (way.getText()) {
                case "+" -> Integer.parseInt(input.getText()) + Integer.parseInt(input2.getText());
                case "-" -> Integer.parseInt(input.getText()) - Integer.parseInt(input2.getText());
                case "*" -> Integer.parseInt(input.getText()) * Integer.parseInt(input2.getText());
                case "/" -> Integer.parseInt(input.getText()) / Integer.parseInt(input2.getText());
                default -> 0;
            };
            answer.setText(String.valueOf(ans));
        }
    }

    static class NumericDocument extends PlainDocument {
        private final String validValues = "0123456789.+-%";

        public void insertString(int offset, String str, AttributeSet a) throws BadLocationException {
            for (var c : str.toCharArray()) {
                if (validValues.indexOf(c) == -1) {
                    return;
                }
            }
            super.insertString(offset, str, a);
        }
    }

    static class NumericDocument2 extends PlainDocument {
        private final Pattern p = Pattern.compile("[+\\-*/%]*");

        public void insertString(int offset, String str, AttributeSet a) throws BadLocationException {
            if (p.matcher(str).matches()) {
                super.insertString(offset, str, a);
            }
        }
    }

// こうして共通化してもｏｋ
//input.setDocument(new RegexDocument("[0123456789.+\\-%]*"));
//way.setDocument(new NumericDocument2("[+\\-*/%]*"));
//input2.setDocument(new RegexDocument("[0123456789.+\\-%]*"));

//    static class RegexDocument extends PlainDocument {
//        private final Pattern p;
//        public RegexDocument(String regex) {
//            p = Pattern.compile(regex);
//        }
//
//        public void insertString(int offset, String str, AttributeSet a) throws BadLocationException {
//            if (p.matcher(str).matches()) {
//                super.insertString(offset, str, a);
//            }
//        }
//    }
}

Answer 2

まず、numActionでは、123というように、順番通りに数字を押すと上手くテキストフィールドに表示されますが、157や81などバラバラに押すとテキストフィールドが勝手に空にされてしまいます。

怪しいのは input に設定している NumericDocument です。
insertString メソッドの先頭に
System.out.println("insertString: offset="+offset+",str="+str);
という一文を入れて実行し、 '1235' と入力してみた時のコンソールへの出力を見ます。

insertString: offset=0,str=1
insertString: offset=0,str=12
insertString: offset=0,str=123
insertString: offset=0,str=1235

insertString メソッドでは

  if (validValues.indexOf(str) == -1) {
    return;
  }

としており、 validValues は "0123456789.+-%" ですので最後の 1235 は if が成立し return してしまいます。
恐らく insertString の str には一文字ずつ入ってくるつもりでこのようなコードを書かれたのでしょうが、そうでは無かったためのバグと思われます。ちゃんと動作を確認してください。

プラスマイナスを変更した時の setText でも NumericDocument は反応しますので、原因は同じです。

---

insertString メソッドの str に(1文字ずつで無く)全文字列が入ってくるのは、(NumAction で) JTextField に対して setText しているからです。
ご存じの通り setText は既存の文字列を消して指定した文字列にするというものですので、 JTextField は AbstractDocument#remove 等で既存を全て消してから PlainDocument#insertString で新たな文字列を設定しようとします。しかし insertString を override して super.insertString せずに return してしまうと remove の効果だけが残り、「勝手に空にされ」た状態になるのです。
なおこの動作は setText に因るものですので、 JTextField に直接キーボードから順次入力した場合は insertString の str は1文字ずつになります。

全く関係ありませんが、 ボタンの処理を Action で書くのは本件ではほぼ意味がありませんので ActionListener でコンストラクタ内に書いたほうがコードは短くなると思います。
Action は、ある機能を実行する方法がメニューにあったりツールバーに(ボタンで)あったりコンテキストメニューにあったりする場合に 1 つの定義を流用することで、表示の統一や Enable/Disable を簡単にする為に用います。

以下は各ボタンの ActionListener で処理を行い、また input を editable=false とした上で KeyListener でボタン操作に変換します。数字以外のキーも( '%' 以外)テキトウに割り振りました。

import java.awt.*;
import java.awt.event.*;
import java.text.DecimalFormat;
import java.util.HashMap;
import java.util.Map;

import javax.swing.*;

public class Calculator extends JFrame {
  public static void main(String[] args) {
    new Calculator().setVisible(true);
  }
  //キー入力をボタン操作に変換
  private static class InputKeyAdapter extends KeyAdapter {
    private Map<Character,JButton> charMap = new HashMap<>();
    private Map<Integer,JButton> codeMap = new HashMap<>();

    void put(JButton button, char c) {
      charMap.put(c, button);
    }
    void put(JButton button, int code) {
      codeMap.put(code, button);
    }
    @Override
    public void keyTyped(KeyEvent e) {
      doClick(charMap.get(e.getKeyChar()));
    }
    @Override
    public void keyReleased(KeyEvent e) {
      doClick(codeMap.get(e.getKeyCode()));
    }
    private void doClick(JButton button) {
      if(button != null) button.doClick();
    }
  };

  private JTextField input1, input2, input, way, answer;
  private InputKeyAdapter keyAdapter = new InputKeyAdapter();

  private Calculator() {
    super("Calculator");
    setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

    //生成
    input1 = createInput("数1");
    way = createTitledField("計算方法", 3);
    input2 = createInput("数2");
    answer = createTitledField("結果", 5);

    ActionListener numActionListener = v -> { //lamda 式によりコード短縮
      String num = ((JButton)v.getSource()).getText();
      String text = input.getText();
      input.setText((text.equals("0")?"":text) + num);
    };
    JButton[] numButtons = new JButton[10];
    for(int i=0; i<10; i++) numButtons[i] = createButton(""+i, numActionListener);

    JButton dotButton = createButton(".", v -> {
      String text = input.getText();
      if(!text.contains(".")) input.setText(text + ".");
    });

    JButton acButton = createButton("AC", v -> {
      input1.setText("0");
      way.setText("");
      input2.setText("0");
      answer.setText("");
    }, KeyEvent.VK_ESCAPE);

    JButton delButton = createButton("Del", v -> {
      String text = input.getText();
      if(text.length() > (text.startsWith("-")?2:1)) {
        input.setText(text.substring(0, text.length()-1));
      } else {
        input.setText("0");
      }
    }, KeyEvent.VK_DELETE, KeyEvent.VK_BACK_SPACE);

    JButton pmButton = createButton("+/-", v -> {
      String text = input.getText();
      if(text.startsWith("-")) {
        input.setText(text.substring(1));
      } else if(!text.equals("0")) {
        input.setText("-" + text);
      }
    }, KeyEvent.VK_SHIFT);

    JButton percentButton = createButton("%", v -> {
      way.setText("%");
      setAnswer(Double.parseDouble(input.getText()) / 100);
    });

    JButton addButton = createButton('+', BinaryOperation.ADD);
    JButton subButton = createButton('-', BinaryOperation.SUB);
    JButton mulButton = createButton('*', BinaryOperation.MUL);
    JButton divButton = createButton('/', BinaryOperation.DIV);

    JButton eqButton = createButton("=", v -> {
      double a = Double.parseDouble(input1.getText());
      double b = Double.parseDouble(input2.getText());
      BinaryOperation op = BinaryOperation.of(way.getText());
      setAnswer(op == null ? 0 : op.calc(a,b));
    }, KeyEvent.VK_ENTER);

    //配置
    JPanel northPanel = new JPanel(new GridBagLayout());
    GridBagConstraints gbc = new GridBagConstraints();
    gbc.gridy = 0;
    gbc.fill = GridBagConstraints.HORIZONTAL;
    gbc.weightx = 1;
    northPanel.add(input1, gbc);//左側で伸縮
    gbc.weightx = 0;
    northPanel.add(way, gbc);//中央固定サイズ
    gbc.weightx = 1;
    northPanel.add(input2, gbc); //右側で伸縮
    add(northPanel, BorderLayout.NORTH);

    add(answer, BorderLayout.SOUTH);

    JPanel centerPanel = createGridPanel(5,3, acButton,pmButton,percentButton,delButton,numButtons[0],dotButton);
    for(int i : new int[]{3,2,1, 6,5,4, 9,8,7}) centerPanel.add(numButtons[i], 3); //インデックス固定の為, 逆順で挿入
    centerPanel.setPreferredSize(new Dimension(270, 300));
    add(centerPanel, BorderLayout.CENTER);

    add(createGridPanel(5,1,addButton,subButton,mulButton,divButton,eqButton), BorderLayout.EAST);

    pack();
    setMinimumSize(getSize());
    setLocationRelativeTo(null); //画面中央へ

    //初期化
    acButton.doClick();
    input1.requestFocus();
  }

  private static JPanel createGridPanel(int rows, int cols, Component... components) {
    JPanel panel = new JPanel(new GridLayout(rows, cols));
    for(Component c : components) panel.add(c);
    return panel;
  }

  private static JTextField createTitledField(String title, int columns) {
    JTextField textField = new JTextField(columns);
    textField.setEditable(false);
    textField.setFocusable(false);
    textField.setFont(textField.getFont().deriveFont(30f));
    textField.setBorder(BorderFactory.createTitledBorder(title));
    return textField;
  }

  private JTextField createInput(String title) {
    JTextField textField = createTitledField(title, 5);
    textField.setFocusable(true);
    textField.addKeyListener(keyAdapter);
    textField.addFocusListener(new FocusAdapter() {
      @Override
      public void focusGained(FocusEvent e) {
        if(input != null) input.setBackground(way.getBackground());
        input = textField;
        input.setBackground(Color.WHITE);
      }
    });
    return textField;
  }

  private JButton createButton(String text, ActionListener l, int... keyCodes) {
    return createButton(text, text.length()==1?text.charAt(0):0, l, keyCodes);
  }
  private JButton createButton(char keyChar, BinaryOperation op) {
    return createButton(op.text, keyChar, v -> way.setText(op.text));
  }
  private JButton createButton(String text, char keyChar, ActionListener l, int... keyCodes) {
    JButton button = new JButton(text);
    button.setFocusable(false);
    button.setFont(button.getFont().deriveFont(20f));
    button.addActionListener(l);
    if(keyChar != 0) keyAdapter.put(button, keyChar);
    for(int keyCode : keyCodes) keyAdapter.put(button, keyCode);
    return button;
  }

  private final DecimalFormat ANSWER_FORMAT = new DecimalFormat("0.######");
  public void setAnswer(Double v) {
    answer.setText(ANSWER_FORMAT.format(v));
  }

  //2項演算
  private enum BinaryOperation {
    ADD("+") { @Override double calc(double a, double b) { return a+b; } },
    SUB("-") { @Override double calc(double a, double b) { return a-b; } },
    MUL("×") { @Override double calc(double a, double b) { return a*b; } },
    DIV("÷") { @Override double calc(double a, double b) { return a/b; } };

    static BinaryOperation of(String text) {
      for(BinaryOperation op : values()) if(op.text.equals(text)) return op;
      return null;
    }

    final String text;
    BinaryOperation(String text) {
      this.text = text;
    }
    abstract double calc(double a, double b);
  }
}