Existe algo errado com o ano SimpleDateFormat e o fuso horário

Question

Estou à altura do meu juízo acabar com esse problema irritante. Basicamente, não conseguiu consertar isso por um longo tempo.

     java.util.Calendar calendar_now = java.util.Calendar.getInstance();
     java.util.Calendar calendar_entry = java.util.Calendar.getInstance();
     java.util.Date dt = new Date();
     java.text.SimpleDateFormat formatter;
  try{
     // if(this.Time.length() == 0) {this.Time = "00:00";}
     //this.Time = "00:00";
     // System.out.println("*" + this.Time + "*");
     if((this.Time.substring(this.Time.length() - 2) == "am")||(this.Time.substring(this.Time.length() - 2) == "pm"))
     {
      formatter = new SimpleDateFormat("yyyy E MMM d H:mmaa z",Locale.US);
       dt = (Date)formatter.parse(calendar_now.get(java.util.Calendar.YEAR) + " " + this.Date + " " + this.Time + " EST");
      //calendar_entry.setTimeZone(java.util.TimeZone.getTimeZone("America/New_York"));
     }
      calendar_entry.setTime(dt);
      if (calendar_entry.get(java.util.Calendar.MONTH) < calendar_now.get(java.util.Calendar.MONTH)){
       calendar_entry.set(java.util.Calendar.YEAR,calendar_now.get(java.util.Calendar.YEAR) + 1);
      }
    else{
       calendar_entry.set(java.util.Calendar.YEAR,calendar_now.get(java.util.Calendar.YEAR));
    }
//    calendar_entry.setTimeZone(java.util.TimeZone.getTimeZone("GMT"));
      System.out.println(calendar_now.get(java.util.Calendar.YEAR) + " " + this.Date + " " + this.Time + " EST");
      System.out.println(dt.toString());
      System.out.println(calendar_entry);

Isso produz:

2010 Fri Oct 1 10:00am EST
Thu Oct 01 10:00:00 SGT 1970
java.util.GregorianCalendar[time=?,areFieldsSet=false,areAllFieldsSet=true,lenient=true,zone=sun.uti
l.calendar.ZoneInfo[id="GMT",offset=0,dstSavings=0,useDaylight=false,transitions=0,lastRule=null],fi
rstDayOfWeek=1,minimalDaysInFirstWeek=1,ERA=1,YEAR=2010,MONTH=9,WEEK_OF_YEAR=40,WEEK_OF_MONTH=1,DAY_
OF_MONTH=1,DAY_OF_YEAR=274,DAY_OF_WEEK=5,DAY_OF_WEEK_IN_MONTH=1,AM_PM=0,HOUR=2,HOUR_OF_DAY=2,MINUTE=
30,SECOND=0,MILLISECOND=0,ZONE_OFFSET=0,DST_OFFSET=0]

A pergunta é ... por quê?

Answer 1

Não tenho ideia do que você tem e do que gostaria de acabar, mas vejo pelo menos um grande erro no código. Nesta linha

if((this.Time.substring(this.Time.length() - 2) == "am")||(this.Time.substring(this.Time.length() - 2) == "pm"))

você está comparando String por referência, não por valor. Você deveria estar usando Object#equals() Sempre que você quiser comparar dois objetos diferentes. UMA String ou seja, um objeto, não um primitivo como int, boolean, etc, para o qual o == funcionaria como pretendido e esperado.

if((this.Time.substring(this.Time.length() - 2).equals("am"))||(this.Time.substring(this.Time.length() - 2).equals("pm")))

No entanto, eu sugiro usar o String#endsWith() método em vez disso. Isso não é apenas mais conciso, mas também torna o código mais auto-explicante:

if (this.Time.endsWith("am") || this.Time.endsWith("pm")) 

Dito isto,, sem dúvida, é uma maneira mais elegante para o que você deseja alcançar, mas como os requisitos funcionais não são claros, é impossível dar um exemplo disso.

Answer 2

Passando pelo seu código, parece que as seguintes linhas de código (que você espera para a formatação) nunca são executadas.

 formatter = new SimpleDateFormat("yyyy E MMM d H:mmaa z",Locale.US);
 dt = (Date)formatter.parse(calendar_now.get(java.util.Calendar.YEAR) + " " + this.Date + " " + this.Time + " EST");

Como apontado corretamente por Balusc, isso se deve a uma declaração condicional inválida para a comaprisão de string. ou seja, você está usando o método "==" em vez de ".equals". Seu problema seria resolvido corrigindo o mesmo que sugerido por Balusc usando:

if (this.Time.endsWith("am") || this.Time.endsWith("pm")) 

ao invés de:

if((this.Time.substring(this.Time.length() - 2) == "am")||(this.Time.substring(this.Time.length() - 2) == "pm"))