Datoalgoritme til to ting - hurtig hjælp behøves

re 1)

SimpleDateFormat parse og catch exception

Min kode for at kunne tilføje et arrangement, hvor datovalideringen også skal foregå, ser indtil videre sådan ud...


      if(e.getSource() == addEvent || e.getSource() == addBtn)
      {
          String eventname = eventnameField.getText();
          String title = titleField.getText();
          String artist = artistField.getText();
          String date = dateField.getText();
          String genre = genreField.getText();
          String time = timeField.getText();
          String location = locationField.getText();
          String price = priceField.getText();
         
          Book book = new Book(eventname, title, artist, date, genre, time, location, price);
       
          if (eventname.length()==0 || title.length()==0 || artist.length()==0 || genre.length()==0 ||
              date.length()==0 || time.length()==0 || location.length()==0 || price.length()==0)
          {
            infoLabel.setText("Tjek om alle felter er udfyldt og at datoen er i DD-MM-ÅÅ format!");
          }
          else
          {
            try
            {
              archive.addBook(book);
              displayArea.setText("Arrangement tilføjet med succes!");
              clearFields();
              infoLabel.setText(" ");
            }
            catch (InvalidISBNException ex)
            {
              infoLabel.setText(ex.getMessage());
              displayArea.setText(" ");
            }
          }
        }

Hjælp mig lige ud engang, please :)

private static DateFormat df = new SimpleDateFormat("dd-MM-yy");

og

      try {
        df.parse(datefield);
        // ok
      } catch (ParseException e) {
        // error
      }

Hmm, jeg tilføjer private static DateFormat df = new SimpleDateFormat("dd-MM-yy"); sammen med mine andre variabeldeklarationer.

Når jeg så tilføjer df.parse(dateField); i try-catch blokken i addEvent/addBtn metoden jeg har listet her, laver den vrøvl om at parse ikke er en metode for et JTextField.

df.parse(datefield.getText());

1) Har Arne svaret på. Den kan udvides til at laves som en metode:

public boolean validateDate(String date)
{
  DateFormat df = new SimpleDateFormat("dd-MM-yy");
  try
  {
      df.parse(datefield);
      return true;
  }
  catch (ParseException e)
  {
      return false;
  }
}

Og du kalder den så bare sådan her:

if (!validateDate (datefield.getText())
{
  JOptionPane.showMessage ("Fejl i dato");
  return;
}


2)

Kunne være noget a la:

public Book[] findBooksFromDate (String date)
{
  if (!validateDate (datefield.getText())
  {
      JOptionPane.showMessage ("Fejl i dato");
      return new Book[0];
  }
 
  Book[] books = archive.getBooks(); // Du må have en eller anden metode hvor du kan hente alle Book's ud?
  ArrayList foundBooks = new ArrayList();
 
  for (int i = 0; i < books.length; i++)
  {
      if (books[i].getDate().equals (date))
        foundBooks.add (books[i]);
  }
 
  return (Book[])foundBooks.toArray (new Book[0]);
}

3)

Måske:

3a)

public Book[] findBooksFromMonth (int month) throws ParseException
{
  if (month < 0 || month > 12)
  {
      JOptionPane.showMessage ("Fejl i måned");
      return new Book[0];
  }
 
  Book[] books = archive.getBooks();
  ArrayList foundBooks = new ArrayList();
 
  for (int i = 0; i < books.length; i++)
  {
      String month = books.getDate().split("-")[1];
      int intMonth = Integer.parseInt (month);
      if (intMonth == month)
        foundBooks.add (books[i]);
  }
}

3b)

public int convertMonth (String month)
{
  if (month.equalsIgnoreCase("januar"))
      return 0;
  if (month.equalsIgnoreCase("februar"))
      return 1;
  if (month.equalsIgnoreCase("marts"))
      return 2;
  if (month.equalsIgnoreCase("april"))
      return 3;
  if (month.equalsIgnoreCase("maj"))
      return 4;
  if (month.equalsIgnoreCase("juni"))
      return 5;
  if (month.equalsIgnoreCase("juli"))
      return 6;
  if (month.equalsIgnoreCase("august"))
      return 7;
  if (month.equalsIgnoreCase("setptember"))
      return 8;     
  if (month.equalsIgnoreCase("oktober"))
      return 9;     
  if (month.equalsIgnoreCase("november"))
      return 10;   
  if (month.equalsIgnoreCase("december"))
      return 11;     
}

Og den bruges så på følgende måde:

int month = convertMonth ("februar");
Book[] books = findBooksFromMonth(month);



Det er måske ikke den bedste søgealgoritme der findes, da den løber alle poster igennem lineært. Men det er svært at gøre den bedre uden at vide om du har database til rådighed og om du har et persistenslag hvor du cacher dine objekter og lignende ting.

Og til 3b mangler der lige en default returværdi:

public int convertMonth (String month)
{
  if (month.equalsIgnoreCase("januar"))
      return 0;
  if (month.equalsIgnoreCase("februar"))
      return 1;
  if (month.equalsIgnoreCase("marts"))
      return 2;
  if (month.equalsIgnoreCase("april"))
      return 3;
  if (month.equalsIgnoreCase("maj"))
      return 4;
  if (month.equalsIgnoreCase("juni"))
      return 5;
  if (month.equalsIgnoreCase("juli"))
      return 6;
  if (month.equalsIgnoreCase("august"))
      return 7;
  if (month.equalsIgnoreCase("setptember"))
      return 8;   
  if (month.equalsIgnoreCase("oktober"))
      return 9;   
  if (month.equalsIgnoreCase("november"))
      return 10; 
  if (month.equalsIgnoreCase("december"))
      return 11; 

  return -1; 
}

Hvis metoden returnerer -1, så ved du at der ikke er indtastet en gyldig måned.

Okay, jeg prøver at implementere det i løbet af dagen. Mange tak for dit input!

mikkelbm > Jeg har lagt hele mit projekt i en fil, som du kan hente her: http://peecee.dk/?id=40049

Der får du ALT med.

Hvis du vil forsøge at få det til at virke, får du en masse ros & points! :-)

Jeg har fundet lidt ud af, at det nok er bedst hvis alle months bliver hentet ind i f.eks. en JComboBox ved opstart, og når brugeren vælger en måned fra listen, f.eks. maj, bliver alle arr. fra den måned vist. Endvidere har jeg tænkt lidt på, om man ved visning af hele listen (listeventsBtn), kan sortere arr. efter dato?

Jeg har desværre ikke tid til at kigge på det idag .
Måske imorgen aften - men jeg kan ikke garantere det.

Helt i orden :)

Hej mikkelbm, jeg har løst alt pånær det med at sortere arrangementer efter dato, så hvis du bare kigger på det, er det fint :-)

Jeg skulle faktisk lige til at kigge på det :) Kan du fortælle mig hvor det er du vil have dine data sorteret? Altså hvilken fil og ca. linjenummer.

Hmm, ja, det er egentlig lidt svært at svare på. Det er jo sådan, at EventIO.java håndterer læsning og skrivning til harddisken (du kan selv lige se dens opbygning). Men i GUI.java, hvor jeg har min actionPerformed metode, har jeg alle søgealgoritmer. Så måske det skal være under if(e.getSource() == listevents)... ?
Det hjælper nok en del hvis du bygger projektet.

Jeg kigger lige lidt på det.

1) Tilføj følgende statiske metode i "EventComponents.java" :

public static java.util.Comparator DateComparable()
{
    final java.text.SimpleDateFormat sdf = new java.text.SimpleDateFormat("dd-MM-yy");
    java.util.Comparator com = new java.util.Comparator()
      {
          public int compare(Object o1, Object o2)
          {
              EventComponents ev1 = (EventComponents)o1;
              EventComponents ev2 = (EventComponents)o2;
             
              try
              {             
                  java.util.Date date1 = sdf.parse(ev1.getDate());
                  java.util.Date date2 = sdf.parse(ev2.getDate());
                 
                  return date1.compareTo(date2);
              }
              catch (Exception e){e.printStackTrace();}
             
              return 0;
          }
      };
      return com;
}

Tilføj følgende metode til "BookTable.java":

public EventComponents[] getAllBooks()
{
    return (EventComponents[])events.values().toArray(new EventComponents[0]);
}

Og i din "GUI.java" ændrer du din if til:

if(e.getSource() == listevents || e.getSource() == listeventsBtn)
{
    String details = "ARKIVEREDE ARRANGEMENTER\n";
    String separateEvents = "----------------------------------------------------------";

    EventComponents[] books = archive.getAllBooks();
    Arrays.sort(books, EventComponents.DateComparable());
           
    for (int i = 0; i < books.length; i++)
    {
        EventComponents nextBook = books[i];
                       
        details = details +"\n PROJEKTNAVN: " + nextBook.getEventName();
        details = details +"\n TITEL: " + nextBook.getTitle();
        details = details +"\n KUNSTNER(E): " + nextBook.getArtist();
        details = details + "\n GENRE: " + nextBook.getGenre();
        details = details + "\n MÅNED: " + nextBook.getMonth();
        details = details + "\n DATO: " + nextBook.getDate();
        details = details + "\n TIDSPUNKT: " + nextBook.getTime();
        details = details + "\n STED: " + nextBook.getLocation();
        details = details + "\n PRIS: " + nextBook.getPrice() + " kr.";
        details = details + "\n\n" + separateEvents + "\n";
    }
    displayArea.setText(details);
    infoLabel.setText(" ");
}

Så burde de blive sortet i kronologisk rækkefølge...

Og hvis du vil have at den også inkluderer tidspunkt i sorteringen:

public static java.util.Comparator DateComparable()
{
    final java.text.SimpleDateFormat sdf = new java.text.SimpleDateFormat("dd-MM-yy HH:mm");
    java.util.Comparator com = new java.util.Comparator()
    {
        public int compare(Object o1, Object o2)
        {
            EventComponents ev1 = (EventComponents)o1;
            EventComponents ev2 = (EventComponents)o2;
           
            try
            {             
                java.util.Date date1 = sdf.parse(ev1.getDate() + " " + ev1.getTime());
                java.util.Date date2 = sdf.parse(ev2.getDate() + " " + ev2.getTime());
               
                return date1.compareTo(date2);
            }
            catch (Exception e){e.printStackTrace();}
           
            return 0;
        }
    };
    return com;
}

Tusind tak, jeg prøver lige at ændre filerne lidt senere!

Nu fik jeg tid til det. Det virker rigtig godt, så tak skal du ha' :-) Smid et svar...

arne_v, du må også gerne smide et svar :-)

Kommer her...

jeg formoder at mikkel får 90% af points ...

Håber ikke nogen er stødt over pointfordelingen? :-S

Nuvel, jeg har en sidste ting til dig mikkelbm. Mit program er sådan set færdigt, men jeg fundet ud af at når man tilføjer et arrangement, må der gerne være en try-catch der sammenligner om datoformat stemmer overens med den måned brugeren har valgt - dvs. den skal brokke sig hvis man f.eks. vælger januar måned og i datofeltet angiver 05-05-06. Jeg har i øvrigt programmeret en JComboBox, der lister et arrays indhold (dvs. alle måneder), så brugeren ikke længere kan skrive måneden...

Any idea?

Jeg har en idé, men den kræver en lille smule omstrukturering.
Hvorfor er det egentlig at du gemmer måned og dato? Du har jo måned implicit i din dato?

Så min idé går ud på at skrotte måned og bare have en dato.

Ja, jeg ved det godt. Men min underviser mener at det vil være bedste løsning - give den mest optimale brugervenlighed for dummies...

Hmm... Tjah... Det er så ikke min mening, når du alligevel skal vælge dag og år - men den er vel heller ikke interessant, når din lærer mener noget andet :)

Du kan lave noget a la:

String date = "05-05-06";
String month = "6";

boolean monthOk = checkMonthAndDate (date, month);

//....

private boolean checkMonthAndDate (String date, int month)
{
  String dateSplit = date.split ("-");
  int monthInDate = Integer.parseInt (dateSplit[1]);

  return monthInDate == month;
}

Du kan også konvertere din date-string til den rigtig dato og hive måned ud af den.

private boolean checkMonthAndDate (String date, int month) throws Exception
{
  SimpleDateFormat sdf = new SimpleDateFormat ("dd-MM-yy");
  Date realDate = sdf.parse (date);
  GregorianCalendar cal = new GregorianCalendar();
  cal.setTime (realDate);
  int monthInDate = cal.get (Calendar.MONTH);

  return monthInDate == month;
}

Jo jo, jeg deler samme mening, og jeg behøver heller ikke at strukturere programmets logik sådan, men jeg orker næsten ikke at sidde og rode med det mere (læs: vil være færdig i løbet af i dag :)...
jeg skal i øvrigt også lige se om ikke jeg kan gøre det sådan, at priceField kun accepterer tal (doubles). Men så skal det vel gemmes anderledes i filen?

Nej, ikke nødvendigvis... Du kan jo bare lave en:

String price = "13.25";
boolean isNumber = checkIsNumber (price);

private boolean checkIsNumber (String number)
{
  try
  {
      Double.parse (number);
      return true;
  }
  catch (NumberFormatException e)
  {
      return false;
  }
}

Jeg kan ikke lige få din sidste metode til at virke - kan du ikke vise ud fra addEvent, hvordan den skal tilføjes..

Jeg har ikke lige noget Java installeret på den her maskine, så jeg kan ikke køre din kode.

Hvad får du af fejl?

Egentlig ikke en fejl. Du har oprettet en metode (checkMonthAndDate). Men i addEvent skal jeg vel referere til metoden, så den virker - eller helt undlade at skrive den som metode? Har lige testet med at oprette noget i januar og angivet september i datoformatet... der brokker den sig ikke, men tilføjer det :(

Min addEvent ser således ud:


      if(e.getSource() == addEvent || e.getSource() == addBtn)
      {
          String eventname = eventnameField.getText();
          String title = titleField.getText();
          String artist = artistField.getText();
          String genre = genreField.getText();
          String month = monthsCombo.getSelectedItem().toString();
          String date = dateField.getText();
          String time = timesCombo.getSelectedItem().toString();
          String location = locationField.getText();
          String price = priceField.getText();

          EventComponents evcomp = new EventComponents(eventname, title, artist, genre, month, date, time, location, price);
         
          if (eventname.length()==0 || !eventname.equalsIgnoreCase(eventnameField.getText()) || title.length()==0 || artist.length()==0 ||
              genre.length()==0 || date.length()==0 || time.length()==0 || location.length()==0 || price.length()==0)
          {
              infoLabel.setText("Tjek om alle felter er udfyldt!");
              Toolkit.getDefaultToolkit().beep();
          }
          else
          {
              try {
                  // Be stringent with the date format!
                  df.setLenient(false);
                  // Parse the date text and validate it
                  df.parse(dateField.getText());
                  archive.addEvent(evcomp);
                  displayArea.setText("Arrangement tilføjet med succes!");
                  clearFields();
                  infoLabel.setText(" ");
              }
              catch (InvalidENException ex)
              {
                  infoLabel.setText(ex.getMessage());
                  displayArea.setText(" ");
                  // Utilize Windows' standard Beep-sound
                  Toolkit.getDefaultToolkit().beep();
              }
              catch (ParseException parsing)
              {
                  JOptionPane.showMessageDialog(null, "Fejl i dato");
              }
            }
        }

Sådan her måske:

else if (!checkIsNumber (month))
{
  infoLabel.setText("Måned ikke korrekt!");
  Toolkit.getDefaultToolkit().beep();
}
else if (!checkMonthAndDate (date, month))
{
  infoLabel.setText("Dato og måned stemmer ikke overens!");
  Toolkit.getDefaultToolkit().beep();
}

Når jeg har tilføjet de to else if-betingelser til min addEvent, får jeg flg. fejl: checkMonthAndDate(java.lang.String,int) in GUI cannot be applied to (java.lang.String,java.lang.String) - hvad mangler jeg lige her? :)

At parse din month til en integer. Det kan du enten gøre inden du smider den ned i metoden, eller du kan ændre metoden til at tage imod en String i stedet for, og så parse den String nede i metoden.

Hmm det er godt nok lidt svært at overskue lige nu. Det ville selvfølgelig hjælpe hvis du havde adgang til min kode & java :)

Jeg tror altså du gør det meget mere svært end det i virkeligheden er.

Prøv at ændre metoden til:

private boolean checkMonthAndDate (String date, String month)
{
  int monthInt = Integer.parseInt (month);
  String dateSplit = date.split ("-");
  int monthInDate = Integer.parseInt (dateSplit[1]);

  return monthInDate == monthInt;
}

Det er stadig helt skørt... jeg lader det lige være indtil du får tid til at lege med min kode og selv ser om du kan få det til at virke. Indtil da lader jeg det altså bare være...