-1

Main1とdate（自分で定義した日付クラス）の2つのクラスがあります。日、月、年のユーザー入力を取得しようとしています。次に、日付クラスを使用して、これが有効な日付であるかどうかを確認しようとしています。私は本当に混乱していて、これが正しい方法かどうかわかりません。

Main1クラスのコードは次のとおりです。

public class Main1 {
public static void main(String[] args){     
    int myDay = 0, myMonth = 0, myYear = 0;
    boolean dataCorrect = false;
    date travelDate; // travelDate will hold the date entered
    do {
        try {
            do {
                myDay = Integer.parseInt(JOptionPane.showInputDialog(null, "Enter the day: "));
                myMonth = Integer.parseInt(JOptionPane.showInputDialog(null, "Enter the month: "));
                myYear = Integer.parseInt(JOptionPane.showInputDialog(null, "Enter the year: "));                   
                travelDate = new date(myDay,myMonth,myYear);                    
            } while (travelDate.checkInputs(travelDate.formDate()) == false); // keeps looping until the date entered is correct                    
            dataCorrect = true;                 
        }
        catch(Exception e) {
            JOptionPane.showMessageDialog(null, "Error! Enter a correct date");
        }
    } while (dataCorrect == false); 
    JOptionPane.showMessageDialog(null, myDay + "-" + myMonth + "-" + myYear);
}

}

これは、私自身が定義した日付クラスのコードです。

import java.text.SimpleDateFormat;
import java.text.ParseException;

public class date {
private int day;
private int month;
private int year;
private String userTravel;

// Constructor 
public date(int d, int m, int y) {
    setDate(d,m,y);     
}

// Methods
public void setDate(int d, int m, int y) {
    setDay(d);
    setMonth(m);
    setYear(y);
}

public String formDate() {
    userTravel = day + "-" + month + "-" + year;
    return userTravel;
}

public boolean checkInputs(String inDate) {
    if (inDate == null)
      return false;
    //set the format to use as a constructor argument
    SimpleDateFormat dateFormat = new SimpleDateFormat("dd-MM-yyyy");

    if (inDate.trim().length() != dateFormat.toPattern().length())
      return false;

    dateFormat.setLenient(false);

    try {
      //parse the inDate parameter
      dateFormat.parse(inDate.trim());
    }
    catch (ParseException pe) {
      return false;
    }
    return true;
}

public void setDay(int d) {
    day = d;
}

public void setMonth(int m) {
    month = m; 
}

public void setYear(int y) {
    year = y; 
}

また、checkInputsメソッドのコードが正しいかどうかもわかりません。

編集：以下の回答で述べられているように、IFステートメントを削除しました。ありがとうございます。とにかく、ユーザーに4桁の年YYYYを常に入力するように強制できるかどうかを尋ねたかっただけです。そうしないと、これが発生するまでループし続けます。

4

2 に答える 2

1

String入力された日付の長さを日付形式のパターンと照合する必要はありません。

if (inDate.trim().length() != dateFormat.toPattern().length())
   return false;

たとえば、の入力日を持つことができます2-3-2013が、パターンはdd-MM-yyyyこれら2つの長さが一致しないため、ループにとどまることからStrings戻ります。もちろん、などの日付は無限ループを引き起こしません。falsecheckInputs10-12-2013

を使用して解析するだけSimpleDateFormatで、必要な検証を行うことができます。

于 2013-03-27T12:48:26.730 に答える
1

この部分では、checkInputsメソッドが失敗します。

if (inDate.trim().length() != dateFormat.toPattern().length())
  return false;

日または月の整数が2桁でない場合（1,2,3 ..、9など）、inDateの長さはパターンの長さと等しくなりません。たとえば、日付1-1-2013は8文字ですが、パターンは10文字を想定しています。

この条件を削除すると、コードは期待どおりに機能すると思います。

于 2013-03-27T12:55:55.740 に答える