Matin, midi, soir, nuit message basé sur le Temps en java

Ce que je suis en train de faire::

Afficher un message basé sur

Bon matin (12h-12h)
Bonne après midi (12h -4h)
Bonne soirée (4h à 9h)
Bonne nuit ( de 9 heures du soir à 6 heures du matin)

CODE::

J'ai utilisé le format 24-heures pour obtenir cette logique

private void getTimeFromAndroid() {
        Date dt = new Date();
        int hours = dt.getHours();
        int min = dt.getMinutes();

        if(hours>=1 || hours<=12){
            Toast.makeText(this, "Good Morning", Toast.LENGTH_SHORT).show();
        }else if(hours>=12 || hours<=16){
            Toast.makeText(this, "Good Afternoon", Toast.LENGTH_SHORT).show();
        }else if(hours>=16 || hours<=21){
            Toast.makeText(this, "Good Evening", Toast.LENGTH_SHORT).show();
        }else if(hours>=21 || hours<=24){
            Toast.makeText(this, "Good Night", Toast.LENGTH_SHORT).show();
        }
    }

Question:

Est-ce ce qui est la meilleure façon de le faire, Si non quelle est la meilleure façon

hors-sujet la note, ne doivent pas être ET && , vous utilisez OU||, qui entraînera tous les cas (heures >= 1) chute dans Good Morning cas?
Vous devriez savoir, que les heures sont dans la gamme entre 0 et 23: getHours().
J'ai mis à jour la réponse à prendre en compte 12:59.

OriginalL'auteur Devrath | 2014-12-21

Vous devriez faire quelque chose comme:

Calendar c = Calendar.getInstance();
int timeOfDay = c.get(Calendar.HOUR_OF_DAY);

if(timeOfDay >= 0 && timeOfDay < 12){
    Toast.makeText(this, "Good Morning", Toast.LENGTH_SHORT).show();        
}else if(timeOfDay >= 12 && timeOfDay < 16){
    Toast.makeText(this, "Good Afternoon", Toast.LENGTH_SHORT).show();
}else if(timeOfDay >= 16 && timeOfDay < 21){
    Toast.makeText(this, "Good Evening", Toast.LENGTH_SHORT).show();
}else if(timeOfDay >= 21 && timeOfDay < 24){
    Toast.makeText(this, "Good Night", Toast.LENGTH_SHORT).show();
}

[+1] .... correction faite ...réponse acceptée 🙂
Je voudrais ajouter un else cas en bas pour attraper tous les imprévus de la valeur (défensive de programmation).
Exactement. Même si elle en informe votre équipe de soutien d'un étrange cas de bord. La clé est de ne pas laisser ce plantage de l'application, car il n'est pas essentiel à la mission.
getInstance(); besoin de l'api android 24.

OriginalL'auteur SMA

Je voudrais raccourcir votre if/elseif déclaration:

String greeting = null;
if(hours>=1 && hours<=12){
    greeting = "Good Morning";
} else if(hours>=12 && hours<=16){
    greeting = "Good Afternoon";
} else if(hours>=16 && hours<=21){
    greeting = "Good Evening";
} else if(hours>=21 && hours<=24){
    greeting = "Good Night";
}
Toast.makeText(this, greeting, Toast.LENGTH_SHORT).show();

[+1] pour l'optimisation

OriginalL'auteur Konstantin Yovkov

java.temps

Je vous conseille d'utiliser Java 8 LocalTime.

Peut-être créer une classe pour gérer votre temps de la journée le problème.

public class GreetingMaker { //think of a better name than this.

  private static final LocalTime MORNING = LocalTime.of(0, 0, 0);
  private static final LocalTime AFTER_NOON = LocalTime.of(12, 0, 0);
  private static final LocalTime EVENING = LocalTime.of(16, 0, 0);
  private static final LocalTime NIGHT = LocalTime.of(21, 0, 0);

  private LocalTime now;

  public GreetingMaker(LocalTime now) {
    this.now = now;
  }

  public void printTimeOfDay() { //or return String in your case
    if (between(MORNING, AFTER_NOON)) {
      System.out.println("Good Morning");
    } else if (between(AFTER_NOON, EVENING)) {
      System.out.println("Good Afternoon");
    } else if (between(EVENING, NIGHT)) {
      System.out.println("Good Evening");
    } else {
      System.out.println("Good Night");
    }
  }

  private boolean between(LocalTime start, LocalTime end) {
    return (!now.isBefore(start)) && now.isBefore(end);
  }

}

Que la première clause dans votre between test prédicat doit être ( ! now.isBefore( start ) ) pour gérer le moment où maintenant est start.
Votre totalement droit. Merci pour la nuance
Pas sûr si cela est techniquement nécessaire, mais je voudrais mettre des parenthèses autour de ce premier alinéa de l'être en sécurité et d'être claire en ce qui concerne l'effet de la NOT “!”.

OriginalL'auteur Yoshua Nahar

2

Vous de déterminer si elle est dans le premier intervalle, puis tous les autres intervalles dépend de la limite supérieure. Donc, vous pouvez la rendre encore plus courte:
```
String greeting = null;
if(hours>=1 && hours<=11){
    greeting = "Good Morning";
} else if(hours<=15){
    greeting = "Good Afternoon";
} else if(hours<=20){
    greeting = "Good Evening";
} else if(hours<=24){
    greeting = "Good Night";
}
Toast.makeText(this, greeting, Toast.LENGTH_SHORT).show();
```
Voulez-vous à 12:59 une bonne journée ou une Bonne après-midi? Vous avez besoin de tester @Devrath avant d'accepter la réponse.
almas shaikh .... pouvez-vous modifier votre réponse pour l'optimisation, comme user987339 suggéré ? ........ @ [+1] pour user987339 pour l'optimisation
almas cheikh vous avez eu raison. J'ai mis à jour la réponse pour le cas de l'12:59
Eh bien, il n'est pas bien, que je n'aurez pas de salut si hours est 0.
vous avez eu raison!

OriginalL'auteur user987339

essayer ce code(obtenir des heures et obtenir des minutes de méthodes dans la classe Date sont obsolètes.)

 private void getTimeFromAndroid() {
    Date dt = new Date();
    Calendar c = Calendar.getInstance();
    c.setTime(dt);
    int hours = c.get(Calendar.HOUR_OF_DAY);
    int min = c.get(Calendar.MINUTE);

    if(hours>=1 && hours<=12){
        Toast.makeText(this, "Good Morning", Toast.LENGTH_SHORT).show();
    }else if(hours>=12 && hours<=16){
        Toast.makeText(this, "Good Afternoon", Toast.LENGTH_SHORT).show();
    }else if(hours>=16 && hours<=21){
        Toast.makeText(this, "Good Evening", Toast.LENGTH_SHORT).show();
    }else if(hours>=21 && hours<=24){
        Toast.makeText(this, "Good Night", Toast.LENGTH_SHORT).show();
    }
}

OriginalL'auteur m2k_72

À l'aide de Time4J (ou Time4A sur Android) permet aux solutions suivantes qui n'ont pas besoin de si-sinon-déclarations:

ChronoFormatter<PlainTime> parser =
    ChronoFormatter.ofTimePattern("hh:mm a", PatternType.CLDR, Locale.ENGLISH);
PlainTime time = parser.parse("10:05 AM");

Map<PlainTime, String> table = new HashMap<>();
table.put(PlainTime.of(1), "Good Morning");
table.put(PlainTime.of(12), "Good Afternoon");
table.put(PlainTime.of(16), "Good Evening");
table.put(PlainTime.of(21), "Good Night");
ChronoFormatter<PlainTime> customPrinter=
    ChronoFormatter
      .setUp(PlainTime.axis(), Locale.ENGLISH)
      .addDayPeriod(table)
      .build();
System.out.println(customPrinter.format(time)); //Good Morning

Il y a aussi un autre modèle de base de façon à laisser les paramètres régionaux de décider, de manière standard, basé sur la CLDR de données format de l'heure de l'horloge:

ChronoFormatter<PlainTime> parser =
    ChronoFormatter.ofTimePattern("hh:mm a", PatternType.CLDR, Locale.ENGLISH);
PlainTime time = parser.parse("10:05 AM");

ChronoFormatter<PlainTime> printer1 =
    ChronoFormatter.ofTimePattern("hh:mm B", PatternType.CLDR, Locale.ENGLISH);
System.out.println(printer1.format(time)); //10:05 in the morning

ChronoFormatter<PlainTime> printer2 =
    ChronoFormatter.ofTimePattern("B", PatternType.CLDR, Locale.ENGLISH)
        .with(Attributes.OUTPUT_CONTEXT, OutputContext.STANDALONE);
System.out.println(printer2.format(time)); //morning

Le seul autre bibliothèque connu pour moi qui peut le faire aussi (mais de façon un peu maladroite) est ICU4J.

OriginalL'auteur Meno Hochschild

Pour quiconque est à la recherche pour les dernières Kotlin syntaxe pour @SMA réponse, ici est la fonction d'assistance :

fun getGreetingMessage():String{
    val c = Calendar.getInstance()
    val timeOfDay = c.get(Calendar.HOUR_OF_DAY)

    return when (timeOfDay) {
           in 0..11 -> "Good Morning"
           in 12..15 -> "Good Afternoon"
           in 16..20 -> "Good Evening"
           in 21..23 -> "Good Night"
             else -> {
              "Hello"
          }
      }
    }

OriginalL'auteur Vishal Sonawane

Quand j'écris

Calendar c = Calendar.getInstance();
int timeOfDay = c.get(Calendar.HOUR_OF_DAY);

Je n'ai pas eu de sortie et qu'il n'affiche pas d'erreur. Juste le timeOfDay n'obtiendrez pas attribué aucune valeur dans le code. J'ai senti que c'était à cause d'une threading tout Calendar.getInstance() est exécutée. Mais quand je me suis effondrée les lignes, il a travaillé pour moi. Voir le code suivant:

int timeOfDay = Calendar.getInstance().get(Calendar.HOUR_OF_DAY);

if(timeOfDay >= 0 && timeOfDay < 12){
        greeting.setText("Good Morning");
}else if(timeOfDay >= 12 && timeOfDay < 16){
        greeting.setText("Good Afternoon");
}else if(timeOfDay >= 16 && timeOfDay < 21){
        greeting.setText("Good Evening");
}else if(timeOfDay >= 21 && timeOfDay < 24){
        greeting.setText("Good Morning");
}

OriginalL'auteur SREE

 private String getStringFromMilli(long millis) {

    Calendar c = Calendar.getInstance();
    c.setTimeInMillis(millis);
    int hours = c.get(Calendar.HOUR_OF_DAY);

    if(hours >= 1 && hours <= 12){
        return "MORNING";
    }else if(hours >= 12 && hours <= 16){
        return "AFTERNOON";
    }else if(hours >= 16 && hours <= 21){
        return "EVENING";
    }else if(hours >= 21 && hours <= 24){
        return "NIGHT";
    }
    return null;
}

OriginalL'auteur Sanjay Hadiya

Vous devez vous connecter pour publier un commentaire.