/*
 * Copyright 2014 jmrozanec
 * Licensed under the Apache License, Version 2.0 (the "License");
 * you may not use this file except in compliance with the License.
 * You may obtain a copy of the License at
 * http://www.apache.org/licenses/LICENSE-2.0
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */
package com.cronutils.utils;

import java.time.Duration;
import java.time.ZonedDateTime;
import java.util.Collections;
import java.util.List;

public class DateUtils {

    public static int workdaysCount(ZonedDateTime startDate, int days, List<ZonedDateTime> holidays, WeekendPolicy weekendPolicy){
        return workdaysCount(startDate, startDate.plusDays(days), holidays, weekendPolicy);
    }

    public static int workdaysCount(ZonedDateTime startDate, ZonedDateTime endDate, List<ZonedDateTime> holidays, WeekendPolicy weekendPolicy){
        Collections.sort(holidays);
        holidays = holidaysInRange(startDate, endDate, holidays);
        int daysToWeekend = WeekendPolicy.daysToWeekend(weekendPolicy, startDate);
        int daysFromWeekend = WeekendPolicy.daysToWeekend(weekendPolicy, endDate);
        long daysBetween = Duration.between(startDate, endDate).toDays()+1;

        //2+ [xxx+2]/7*5
        long tmpWeekdays = (daysBetween-daysToWeekend-daysFromWeekend-2)/7*5;
        tmpWeekdays = tmpWeekdays-holidays.size();

        return new Long(tmpWeekdays+daysToWeekend+daysFromWeekend).intValue();
    }

    private static List<ZonedDateTime> holidaysInRange(ZonedDateTime startDate, ZonedDateTime endDate, List<ZonedDateTime> holidays){
        if(holidays.isEmpty()){
            return holidays;
        }
        int idxstart = findIdx(0, holidays.size()-1, startDate, holidays);
        int idxend = findIdx(0, holidays.size()-1, endDate, holidays);
        return holidays.subList(idxstart, idxend+1);
    }

    private static int findIdx(int startidx, int endidx, ZonedDateTime endDate, List<ZonedDateTime> holidays){
        if(startidx==endidx){
            return startidx;
        }
        int pivot = (endidx-startidx)/2;

        if(holidays.get(pivot).equals(endDate)){
            return pivot;
        }
        if(holidays.get(pivot).isBefore(endDate)){
            return findIdx(++pivot, endidx, endDate, holidays);
        }else{
            return findIdx(startidx, pivot, endDate, holidays);
        }
    }
}

@@ -1392,20 +1392,19 @@
         
-long
+int
  daysBet
@@ -1410,16 +1410,21 @@
 tween = 
+(int)
 Duration
@@ -1502,12 +1502,11 @@
     
-long
+int
  tmp
@@ -1638,17 +1638,8 @@
 urn 
-new Long(
 tmpW
@@ -1679,20 +1679,8 @@
 kend
-).intValue()
 ;%0A  

Fix code smell: cast long to int and operate with ints instead of declaring a final Long.intValue()