<div class="snippet-clipboard-content notranslate position-relative overflow-auto" data-snippet-clip

Improved ser_route <div class="snippet-clipboard-content notranslate position-rela

Imrpove BusRoute ser/de performance about lta-rs HOT 1 CLOSED

zeon256 commented on May 25, 2024

Imrpove BusRoute ser/de performance

from lta-rs.

Comments (1)

zeon256 commented on May 25, 2024

Improved ser_route

de_route                time:   [998.75 us 999.01 us 999.33 us]                      
                        change: [+9.6764% +10.537% +11.255%] (p = 0.00 < 0.05)
                        Performance has regressed.
Found 14 outliers among 100 measurements (14.00%)
  1 (1.00%) low severe
  1 (1.00%) low mild
  4 (4.00%) high mild
  8 (8.00%) high severe

ser_route               time:   [869.72 us 885.96 us 904.44 us]                      
                        change: [-33.452% -31.768% -29.834%] (p = 0.00 < 0.05)
                        Performance has improved.

        pub fn deserialize<'de, D>(deserializer: D) -> Result<Option<NaiveTime>, D::Error>
        where
            D: Deserializer<'de>,
        {

            let s: Rc<String> = Rc::new( String::deserialize(deserializer)?);
            let is_broken = &*s;
            if is_broken.eq("-") {
                return Ok(None);
            }

            let time_res = NaiveTime::parse_from_str(&s, FORMAT);

            match time_res {
                Ok(r) => Ok(Some(r)),
                Err(_) => {
                    let s2 = s.clone();
                    let chars = s.chars();
                    let chars2 = s2.chars();
                    let hr = chars.take(2).collect::<String>().parse().unwrap();
                    let min = chars2.take(4).collect::<String>().parse().unwrap();
                    let time = NaiveTime::from_hms_opt(hr,min,0);
                    Ok(time)
                }
            }
        }