年月日を入力して年内の経過日数と残り日数を計算するコードの結果について

Question

前提・実現したいこと

年月日を入力して年内の経過日数と残り日数を計算しようとしています。

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

エラーメッセージなどは表示されないのですが、
実行し答えを加算すると、366日になることについて、
これはコードにおいてよくない点があるのか、それとも
この年内の経過日数と残り日数はそのまま足し合わせてはいけない数なのでしょうか。

年内の経過日数と残り日数を求めます、
年：2018
月：4
日：1
年内で91日目です
年内の残りは275日です
もう一度しますか（1...はい/0...いいえ):

該当のソースコード

import java.util.Scanner;

public class Main {
	//---各月の日数---//
		static int[][] mdays = {
				{31, 28, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31}, //平年
				{31, 29, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31}, //閏年
		};

		//---西暦year年は閏年か---//
		static int isLeap(int year) {
			return(year %4 == 0 && year % 100 != 0 || year%400 ==0) ? 1: 0;
			}

		//---西暦y年m月d日の年内の経過日数を求める---//
		static int dayOfYear(int y, int m, int d) {
				int days = d; //日数

				for(int i = 1; i<m; i++) { //1月〜（m-1)月の日数を加える
					days += mdays[isLeap(y)][i-1];//isLeap()は閏年であれば1,平年であれば0
				}
				return days;
			}

		//---西暦y年m月d日の年内の残り日数を求める---
		static int leftDayOfYear(int y, int m, int d) {
			int remdays = mdays[isLeap(y)][m] - d; //今月の残り日数
			for(int i = 12; i>m; i--) { //12月〜(m+1)月の日数を加える
				remdays += mdays[isLeap(y)][i-1];
			}
			return remdays;
		}

		public static void main(String[] args){
			Scanner scan = new Scanner(System.in);
			int retry; //もう一度？

			System.out.println("年内の経過日数と残り日数を求めます、");

			do {
				System.out.print("年：");
				int year = scan.nextInt(); //年
				System.out.print("月：");
				int month = scan.nextInt(); //月
				System.out.print("日：");
				int day = scan.nextInt(); //日
				System.out.printf("年内で%d日目です\n", dayOfYear(year,month,day));
				System.out.printf("年内の残りは%d日です\n", leftDayOfYear(year,month,day));
				System.out.print("もう一度しますか（1...はい/0...いいえ):");
				retry = scan.nextInt();
			}while(retry == 1);
		}

}

補足情報（FW/ツールのバージョンなど）

Eclipse Java EE IDE for Web Developers.
Version: Oxygen.2 Release (4.7.2)
Build id: 20171218-0600

Answer 1

技術的な制約事項がなければ、java.time.LocalDateとjava.time.temporal.ChronoUnitを使ってくださいな。

import java.time.LocalDate;
import java.time.temporal.ChronoUnit;
import java.util.Scanner;

public class A119866 {

	public static void main(String[] args) {
		Scanner scan = new Scanner(System.in);
		int retry; // もう一度？

		System.out.println("年内の経過日数と残り日数を求めます、");

		do {
			System.out.print("年：");
			int year = scan.nextInt(); // 年
			System.out.print("月：");
			int month = scan.nextInt(); // 月
			System.out.print("日：");
			int day = scan.nextInt(); // 日
			System.out.printf("年内で%d日目です\n", dayOfYear(year, month, day));
			System.out.printf("年内の残りは%d日です\n", leftDayOfYear(year, month, day));
			System.out.print("もう一度しますか（1...はい/0...いいえ):");
			retry = scan.nextInt();
		} while (retry == 1);
	}

	static long dayOfYear(int y, int m, int d) {
		return ChronoUnit.DAYS.between(LocalDate.of(y, 1, 1).minusDays(1), LocalDate.of(y, m, d));
	}

	static long leftDayOfYear(int y, int m, int d) {
		return ChronoUnit.DAYS.between(LocalDate.of(y, m, d), LocalDate.of(y, 12, 31));
	}
}

Answer 2

残り日数を求めるロジックが間違っていました。
そのため、足し算すると366日等のおかしな合計になってたと考えられます。

//---西暦y年m月d日の年内の残り日数を求める---
    static int leftDayOfYear(int y, int m, int d) {
        // mをm-1に訂正
        int remdays = mdays[isLeap(y)][m-1] - d; //今月の残り日数
        for(int i = 12; i>m; i--) { //12月〜(m+1)月の日数を加える
            remdays += mdays[isLeap(y)][i-1];
        }
        return remdays;
    }

想定していた値と異なる数が出力された場合は
まず境界値テストを行うと、どこの処理がおかしいのか判断しやすくなると思います。
今回の場合ですと、2018/1/1(1日目で残361日と出力)や2018/12/31(例外発生)となっているため
残日数計算がおかしいと判断できました。

Answer 3

LocalDateを使うなら、割とそのままのメソッドがあったりする
というか「年の日」という概念は旧Calendarクラスでもあったなそういえば

import java.time.LocalDate;
import java.time.temporal.ChronoUnit;
import java.util.Scanner;

public class A119866 {

    public static void main(String[] args) {
        Scanner scan = new Scanner(System.in);
        int retry; // もう一度？

        System.out.println("年内の経過日数と残り日数を求めます、");

        do {
            System.out.print("年：");
            int year = scan.nextInt(); // 年
            System.out.print("月：");
            int month = scan.nextInt(); // 月
            System.out.print("日：");
            int day = scan.nextInt(); // 日
            LocalDate target = LocalDate.of(year, month, day);
            int dayOfYear = target.getDayOfYear();
            System.out.printf("年内で%d日目です\n", dayOfYear);
            System.out.printf("年内の残りは%d日です\n", target.lengthOfYear() - dayOfYear);
            System.out.print("もう一度しますか（1...はい/0...いいえ):");
            retry = scan.nextInt();
        } while (retry == 1);
    }
}

元の回答(この回答は不適でした)

三人兄弟の上から1番目は下から3番目。
そういうことです。

Answer 4

現在の主流（LocalDate等)の 計算が出ているので　一昔前のロジックを

    public static void calendar(
            int y,
            int m,
            int d) {
        System.out.println("年内の経過日数と残り日数を求めます、");
        System.out.println("年：" + y); // 入力を固定設定に代用
        System.out.println("月：" + m);// 入力を固定設定に代用
        System.out.println("日：" + d);// 入力を固定設定に代用

        Calendar c = new GregorianCalendar(y, m - 1, d);

        int max = c.getActualMaximum(Calendar.DAY_OF_YEAR);
        int cur = c.get(Calendar.DAY_OF_YEAR);

        System.out.println("年内で" + cur + "日目です");
        System.out.println("年内の残りは" + (max - cur) + "日です");
        // System.out.println(y + "年は" + max + "日間です");
        System.out.println();
    }