Simple Error ???

[darkblue24]

Hi All

I have MeetingMakerHelper class:

public class MeetingMakerHelper 
{

    public static int[] workingDays = {1,2,3,4,5,8,9,10,11,12,15,16,17,18,19,22,23,24,25,26,29,30};

and MeetingMaker class that has MeetingMaker method:

public class MeetingMaker
{
   private SortedSet<Integer> workingDaysCal;
        
          
  /**
   * Constructor for objects of class MeetingMaker
   */
  public MeetingMaker()
  {
     super();    
     
[COLOR="DarkOrange"]    [B] SortedSet<Integer> workingDaysCal = new this.MeetingMakerHelper.workingDays[int]; // this does not compile[/B][/COLOR]
  }
  
    /**
   * It turns int Arrays to sorted Sets
   */
  public Set MeetingMaker(int[] anArray)
  {
     Set<Integer> MetMakerSet = new TreeSet<Integer>();
   
           for (int i = 0; i < anArray.length; i++)
      {
         MetMakerSet.add(anArray[i]);
      } 
      return MetMakerSet;
  }
    
}

My constructor does not compile: <identifier> expected error

workingDaysCal should be initialised to a new sorted set
that holds the dates given by the array referenced by the MeetingMakerHelper class’s static variable workingDays.

Do you know why does not compile?

M

[JosAH]

public class MeetingMakerHelper 
{

    public static int[] workingDays = {1,2,3,4,5,8,9,10,11,12,15,16,17,18,19,22,23,24,25,26,29,30}; 
}
...
public class MeetingMaker
{
   private SortedSet<Integer> workingDaysCal;
        
          
  /**
   * Constructor for objects of class MeetingMaker
   */
  public MeetingMaker()
  {
     super();    
     
[COLOR="DarkOrange"]    [B] SortedSet<Integer> workingDaysCal = new this.MeetingMakerHelper.workingDays[int]; // this does not compile[/B][/COLOR]
  }

My constructor does not compile: <identifier> expected error

workingDaysCal should be initialised to a new sorted set
that holds the dates given by the array referenced by the MeetingMakerHelper class’s static variable workingDays.

Do you know why does not compile?

M

The 'new' operator needs a type, not some instance thereof; but above all why do you use an array for the numbers 1 ... 30? All you're doing with that array is try to put it in a SortedSet. There's an easier way to fill a set like that:

SortedSet<Integer> yourSet= new TreeSet<Integer>();
...
// fill the set:
for (int i= 1; i <= 30; i++)
   if (i%7 != 6 && i%7 != 0)
      yourSet.add(i);

kind regards,

Jos

[darkblue24]

Hi Jos

This array must be created in this particular way (its not numbers from 1 to 30 - but its working days in month)

Anyway.. I made some changes to the code and compiles fine but when I am inspecting created object value in atribute workingDaysCall is null.

What am I doing wrong?

public class MeetingMaker
{
   private SortedSet<Integer> workingDaysCal;
   private SortedSet<Integer> adamNotInCal; 
          
  /**
   * Constructor for objects of class MeetingMaker
   */
  public MeetingMaker()
  {
     super();    
     
[B][COLOR="Red"]     SortedSet<Integer> workingDaysCal = new TreeSet<Integer>(makeCalendar(MeetingMakerHelper.workingDays)); // ERROR IS HERE[/COLOR][/B]
    
  }
  
    /**
   * It turns int Arrays to sorted Sets
   */
  public SortedSet<Integer> makeCalendar(int[] anArray)
  {
    SortedSet<Integer> sorted = new TreeSet<Integer>();
      for (int number : anArray)
      {
          sorted.add(number);
       }
        
      return sorted;
  }

[JosAH]

Hi Jos

This array must be created in this particular way (its not numbers from 1 to 30 - but its working days in month)

Anyway.. I made some changes to the code and compiles fine but when I am inspecting created object value in atribute workingDaysCall is null.

What am I doing wrong?

public class MeetingMaker
{
   private SortedSet<Integer> workingDaysCal;
   private SortedSet<Integer> adamNotInCal; 
          
  /**
   * Constructor for objects of class MeetingMaker
   */
  public MeetingMaker()
  {
     super();    
     
[B][COLOR="Red"]     SortedSet<Integer> workingDaysCal = new TreeSet<Integer>(makeCalendar(MeetingMakerHelper.workingDays)); // ERROR IS HERE[/COLOR][/B]
    
  }
  
    /**
   * It turns int Arrays to sorted Sets
   */
  public SortedSet<Integer> makeCalendar(int[] anArray)
  {
    SortedSet<Integer> sorted = new TreeSet<Integer>();
      for (int number : anArray)
      {
          sorted.add(number);
       }
        
      return sorted;
  }

You are initializing a local variable also named workingDaysCal and not the member variable with the same name; change that line in your constructor to:

workingDaysCal = new TreeSet<Integer>(makeCalendar(MeetingMakerHelper.workingDays));

kind regards,

Jos