Улучшение кода в методе get year с форматированной строкой в ​​качестве параметра - PullRequest
0 голосов
/ 05 июля 2019

Я перевожу проект Android в Kotlin и пытаюсь улучшить свои собственные навыки, и я уверен, что даже если этот метод работает, стиль кода мог бы быть лучше.

Поскольку у меня не так много опыта кодирования в Kotling, я должен признать, что большая часть кода напрямую преобразуется Android Studio. Вот код:

 * Formats the movie release date from the TMDB API and returns its year as an int value.
 * If the date is not available, returns the current year.
 *
 * @return int with the release year of the movie.
 */
fun getYear(releaseDate: String): Int {
    val sdf = SimpleDateFormat(
        Constants.TMDB_DATE_FORMAT,
        Locale.getDefault()
    )
    val calendar = GregorianCalendar()
    try {
        calendar.time = sdf.parse(releaseDate)
    } catch (e: ParseException) {
        e.printStackTrace()
    } catch (e: NullPointerException) {
        e.printStackTrace()
    }

    return calendar.get(Calendar.YEAR)
}

Я уверен, что уловка попытки с NullPointerException может быть улучшена.

Любая помощь будет по достоинству оценена,

Маурисио Гуэль

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...