How to make the cursor global if it has parameters that would be passed through calling environment

Question

What I am trying to do is:
1. Pass two dates "p_start_date" and "p_end_date" to a procedure "calc_percnt". Based on these two dates the procedure creates a collection that has ticket info between the date range provided.
2. Iterate through this collection to find out the number of tickets that took less than 4 hrs (p_count_4), less than 8 hours (p_count_8), less than 12 hours(p_count_12) and less than 24 hours (24).
3. I want to move the logic of the  above point 2 (also mentioned between dotted lines in the below mentioned code) to a function and call that function.
4. For the above point 3 I will have to create a function which will accept a collection variable something like this:
create or replace cal_perc (collection_var total_tckt_colcn) .... 
I cannot do this because total_tckt_colcn needs to be declared
5. I cannot make the cursor and collection type as global by putting them in package specification because of the condition in cursor:
WHERE created_date >= p_start_date AND created_date < (p_end_date + interval '1' DAY)
it gives the error as "p_start_date"  and "p_end_date" needs to be declared.
What is the best way to do this????
create or replace
PROCEDURE calc_percnt(
  p_start_date IN N01.cc_ticket_info.LAST_CHANGED%type ,
  p_end_date   IN N01.cc_ticket_info.LAST_CHANGED%type ,
AS
  v_start_date N01.cc_ticket_notes.LAST_UPDATED_STAMP%type;
  v_end_date N01.cc_ticket_notes.LAST_UPDATED_STAMP%type;
  CURSOR cur_total_tckt
  IS
  SELECT  * from
  WHERE created_date >= p_start_date AND created_date < (p_end_date + interval '1' DAY)
type total_tckt_colcn
IS
  TABLE OF cur_total_tckt%rowtype;
  total_tckt_col total_tckt_colcn;
  total_coach_col total_tckt_colcn;
BEGIN
  total_tckt_col  := total_tckt_colcn ();
  OPEN cur_total_tckt;
  LOOP
  FETCH cur_total_tckt bulk collect INTO total_tckt_col limit 100;
END LOOP;
  EXIT
  WHEN (cur_total_tckt%NOTFOUND);
  END LOOP ;
  CLOSE cur_total_tckt;
-- ---I want to move the following code in a function which finds the time required to close the ticket and increment the counter ------
FOR i IN total_tckt_col .first..total_tckt_col .last
  LOOP
  no_of_seconds       := calc_time_diff(total_coach_col(i).created_date, total_coach_col(i).closed_date);
  IF (no_of_seconds    < 14400) THEN
  p_count_4          := p_count_4  + 1;
  p_count_8          := p_count_8  + 1;
  p_count_12         := p_count_12 + 1;
  p_count_24         := p_count_24 + 1;
  ELSIF (no_of_seconds < 28800) THEN
  p_count_8          := p_count_8  + 1;
  p_count_12         := p_count_12 + 1;
  p_count_24         := p_count_24 + 1;
  ELSIF (no_of_seconds < 43200) THEN
  p_count_12         := p_count_12 + 1;
  p_count_24         := p_count_24 + 1;
  ELSIF (no_of_seconds < 86400) THEN
  p_count_24         := p_count_24 + 1;
  END IF;
  END LOOP;
END calc_percnt;

Answer 1

I cannot add cursor definition to package because  of the following condition:
WHERE created_date >= p_start_date AND created_date < (p_end_date + interval '1' DAY)
it gives me an error: "p_start_date"  and "p_end_date" needs to be declared.
p_start_date and p_end_date are parameters that would be passed by the user.
Parameterized cursor to the rescue:
I have changed my cursor definition to:
create or replace
PACKAGE ES_REPORTS AS
CURSOR cur_total_tckt (p_start_date n01.cc_ticket_status_history.created_date%type, p_end_date n01.cc_ticket_status_history.created_date%type)
  IS
  SELECT  t.ticket_id ticket_id , t.created_date created_date, t.created_by created_by, t.ticket_status ticket_status, t.last_changed last_changed, h.closed_date
  FROM n01.cc_ticket_info t
  inner join
  (SELECT  ticket_id , MAX(created_date) closed_date
  FROM n01.cc_ticket_status_history
  WHERE ticket_status = 'CLOSED' AND created_date >= p_start_date AND created_date < (p_end_date + interval '1' DAY)
  GROUP BY ticket_id
  ) h
  on (t.ticket_id         = h.ticket_id)
  WHERE (t. ticket_status = 'NOTIFIED' OR t.ticket_status = 'CLOSED') AND t.last_changed >= p_start_date AND t.last_changed < (p_end_date + interval '1' DAY);
END ES_REPORTS;
Not sure if it is good programing practice though?