✨ Calendar & initial impl for Matsudo #6

siketyan · 2021-03-29T16:08:59Z

Closes #2

siketyan · 2021-04-01T10:38:14Z

@tomo1560 コメント追加しました

tomo1560 · 2021-04-01T13:56:29Z

#[cfg(test)]
mod tests {
    use crate::calendar;
    use chrono::{Weekday, Local, Datelike, TimeZone};
    use crate::calendar::Calendar;

    #[test]
    fn test_in_march() {
        let calendar = calendar::matsudo::Matsudo::new(Weekday::Thu);
        let test_month_date_time = Local.ymd(2021, 3, 1).and_hms(0, 0, 0);


        let mut count = 0u32;

        loop {
            match test_month_date_time.with_day0(count) {
                None => { break; }
                Some(test_date) => {
                    let gomi = calendar.gomi_at(test_date);

                    println!(
                        "今日{} {} は {} の日です",
                        test_date,
                        test_date.weekday(),
                        gomi.iter()
                            .map(|g| g.to_string())
                            .collect::<Vec<String>>()
                            .join(", ")
                    );
                }
            }
            count += 1
        }
    }
}

今月分テストで回してみたけど3月17日から3月23日がバグってるかな
毎日が陶磁器・ガラスDay

C:/Users/tomo1560/.cargo/bin/cargo.exe test --color=always --package matsudo-gomi-hook --bin matsudo-gomi-hook tests --no-fail-fast -- -Z unstable-options --format=json --show-output
Testing started at 22:51 ...
    Finished test [unoptimized + debuginfo] target(s) in 0.03s
     Running target\debug\deps\matsudo_gomi_hook-7de719c740309d3c.exe
今日2021-03-01 00:00:00 +09:00 Mon は 燃やせるごみ の日です
今日2021-03-02 00:00:00 +09:00 Tue は その他のプラスチックなどのごみ の日です
今日2021-03-03 00:00:00 +09:00 Wed は 燃やせるごみ の日です
今日2021-03-04 00:00:00 +09:00 Thu は 資源ごみ の日です
今日2021-03-05 00:00:00 +09:00 Fri は 燃やせるごみ の日です
今日2021-03-06 00:00:00 +09:00 Sat は リサイクルするプラスチック の日です
今日2021-03-07 00:00:00 +09:00 Sun は  の日です
今日2021-03-08 00:00:00 +09:00 Mon は 燃やせるごみ の日です
今日2021-03-09 00:00:00 +09:00 Tue は その他のプラスチックなどのごみ の日です
今日2021-03-10 00:00:00 +09:00 Wed は 燃やせるごみ の日です
今日2021-03-11 00:00:00 +09:00 Thu は 資源ごみ の日です
今日2021-03-12 00:00:00 +09:00 Fri は 燃やせるごみ の日です
今日2021-03-13 00:00:00 +09:00 Sat は リサイクルするプラスチック の日です
今日2021-03-14 00:00:00 +09:00 Sun は  の日です
今日2021-03-15 00:00:00 +09:00 Mon は 燃やせるごみ の日です
今日2021-03-16 00:00:00 +09:00 Tue は その他のプラスチックなどのごみ の日です
今日2021-03-17 00:00:00 +09:00 Wed は 燃やせるごみ, 陶磁器・ガラスなどのごみ の日です
今日2021-03-18 00:00:00 +09:00 Thu は 資源ごみ, 陶磁器・ガラスなどのごみ の日です
今日2021-03-19 00:00:00 +09:00 Fri は 燃やせるごみ, 陶磁器・ガラスなどのごみ の日です
今日2021-03-20 00:00:00 +09:00 Sat は リサイクルするプラスチック, 陶磁器・ガラスなどのごみ の日です
今日2021-03-21 00:00:00 +09:00 Sun は 陶磁器・ガラスなどのごみ の日です
今日2021-03-22 00:00:00 +09:00 Mon は 燃やせるごみ, 陶磁器・ガラスなどのごみ の日です
今日2021-03-23 00:00:00 +09:00 Tue は その他のプラスチックなどのごみ, 陶磁器・ガラスなどのごみ の日です
今日2021-03-24 00:00:00 +09:00 Wed は 燃やせるごみ の日です
今日2021-03-25 00:00:00 +09:00 Thu は 資源ごみ の日です
今日2021-03-26 00:00:00 +09:00 Fri は 燃やせるごみ の日です
今日2021-03-27 00:00:00 +09:00 Sat は リサイクルするプラスチック の日です
今日2021-03-28 00:00:00 +09:00 Sun は  の日です
今日2021-03-29 00:00:00 +09:00 Mon は 燃やせるごみ の日です
今日2021-03-30 00:00:00 +09:00 Tue は その他のプラスチックなどのごみ の日です
今日2021-03-31 00:00:00 +09:00 Wed は 燃やせるごみ の日です

Process finished with exit code 0

tomo1560

レビューしました
お時間がある時にご確認をお願いいたします

tomo1560 · 2021-04-01T23:59:17Z

src/main.rs

 fn main() {
-    println!("Hello, world!");
+    let calendar = calendar::matsudo::Matsudo::new(Weekday::Thu);
+    let today = Local.ymd(2021, 3, 17).and_hms(0, 0, 0);


日付だけ見て処理するのであればDateTimeじゃなくてDateで処理した方がいいかも

tomo1560 · 2021-04-02T00:06:39Z

src/calendar/matsudo.rs

+                        $other_plastic => Some(Gomi::new("その他のプラスチックなどのごみ")),
+                        _ => None,
+                    },
+                    if (week_num_by_weekday($current_time, $glass_week_day) == $glass_week_num) {


第何週目しか見てなくて陶磁器・ガラスなどのごみの週に全て「陶磁器・ガラスなどのゴミ」が入っちゃってそう

tomo1560 · 2021-04-02T00:09:52Z

src/calendar/matsudo.rs

+    }
+}
+
+fn week_num_by_weekday(time: DateTime<Local>, weekday: Weekday) -> u32 {


今日が月の第何週目を求めるなら、一年における経過した週から今月までに経過した週を引いた方がスッキリするかも

fn week_num_by_weekday2(time: Date<Local>) -> u32 { time.iso_week().week() - time.with_day0(0).unwrap().iso_week().week() + 1 } #[cfg(test)] mod tests { use chrono::{Local, TimeZone}; use crate::calendar::matsudo::week_num_by_weekday2; #[test] fn week_num_3_17() { let date = Local.ymd(2021,3,17); assert_eq!(week_num_by_weekday2(date), 3); } #[test] fn week_num_1_1() { let date = Local.ymd(2021,1,1); assert_eq!(week_num_by_weekday2(date), 1); } #[test] fn week_num_12_31() { let date = Local.ymd(2021,12,31); assert_eq!(week_num_by_weekday2(date), 5); } #[test] fn week_num_2020_2_29() { let date = Local.ymd(2020,2,29); assert_eq!(week_num_by_weekday2(date), 5); } }

siketyan added 4 commits March 30, 2021 01:06

➕ Add chrono

b9e0469

✨ Add Calendar trait

d3cc114

✨ Add a calendar for Matsudo

fc3999f

✨ Basic entrypoint

ef332e7

siketyan requested a review from tomo1560 March 29, 2021 16:08

siketyan self-assigned this Mar 29, 2021

siketyan added 2 commits March 30, 2021 01:13

🎨 Support a trailing comma

ff496fa

💬 Add comments

f491665

tomo1560 requested changes Apr 2, 2021

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

✨ Calendar & initial impl for Matsudo #6

✨ Calendar & initial impl for Matsudo #6

siketyan commented Mar 29, 2021

siketyan commented Apr 1, 2021

tomo1560 commented Apr 1, 2021

tomo1560 left a comment

tomo1560 Apr 1, 2021

tomo1560 Apr 2, 2021

tomo1560 Apr 2, 2021

✨ Calendar & initial impl for Matsudo #6

Are you sure you want to change the base?

✨ Calendar & initial impl for Matsudo #6

Conversation

siketyan commented Mar 29, 2021

siketyan commented Apr 1, 2021

tomo1560 commented Apr 1, 2021

tomo1560 left a comment

Choose a reason for hiding this comment

tomo1560 Apr 1, 2021

Choose a reason for hiding this comment

tomo1560 Apr 2, 2021

Choose a reason for hiding this comment

tomo1560 Apr 2, 2021

Choose a reason for hiding this comment