• Najnowsze pytania
  • Bez odpowiedzi
  • Zadaj pytanie
  • Kategorie
  • Tagi
  • Zdobyte punkty
  • Ekipa ninja
  • IRC
  • FAQ
  • Regulamin
  • Książki warte uwagi

Recenzja kodu Kotlin

Object Storage Arubacloud
0 głosów
252 wizyt
pytanie zadane 16 lipca 2018 w Android, Swift, Symbian przez KariK-02 Mądrala (6,030 p.)

Hej! Od pewnego czasu bawię się w Kotlinie i ostatnio napisałem taki prosty kod do łączenia się z REST Api i prosiłbym o ocenę jego oraz jakieś porady co zmienić by lepiej to wyglądało i działało 

Request.kt:

class Request(
        private val url : String,
        val data : ByteArray?,
        val type : RequestType = RequestType.GET,
        private val requestFeatures: IRequestFeatures?,
        private val headers : HashMap<String, String>?
) : AsyncTask<URL, Long, String>(), IRequest {

    private constructor(builder : Builder) : this(
            builder.url,
            builder.data,
            builder.type,
            builder.requestFeatures,
            builder.headers
    )

    override fun doInBackground(vararg params: URL?): String {

        requestFeatures?.doInBackground()

        val httpConnection = URL(url).openConnection() as HttpURLConnection

        var finalResponse = ""

        try {
            when(type) {
                RequestType.POST -> httpConnection.requestMethod = "POST"
                RequestType.PATCH -> httpConnection.requestMethod = "PATCH"
                RequestType.DELETE -> httpConnection.requestMethod = "DELETE"
                else -> httpConnection.requestMethod = "GET"
            }

            httpConnection.addRequestProperty("Content-Type", "application/json")
            httpConnection.addRequestProperty("Accept", "application/json")

            headers?.forEach {
                httpConnection.addRequestProperty(it.key, it.value)
            }

            finalResponse = streamToString(httpConnection.inputStream)


        } catch (e : Exception) {
            e.printStackTrace()
        } finally {
            httpConnection.disconnect()
        }

        return finalResponse
    }

    override fun onPostExecute(result: String?) {
        super.onPostExecute(result)

        requestFeatures?.onPostExecute(result)

    }

    override fun toString(): String {
        return "Request(url='$url', data=${Arrays.toString(data)}, type=$type)"
    }

    override fun streamToString(inputStream: InputStream): String {

        val bufferReader = BufferedReader(InputStreamReader(inputStream))
        var line: String
        var result = ""

        try {
            do {
                line = bufferReader.readLine()
                if (line != null) {
                    result += line
                }
            } while (line != null)
            inputStream.close()
        } catch (ex: Exception) {

        }

        return result
    }

    class Builder {

        var url : String = ""
            private set

        var data : ByteArray? = null
            private set

        var type : RequestType = RequestType.GET
            private set

        var requestFeatures : IRequestFeatures? = null
            private set

        var headers : HashMap<String, String>? = null

        fun url(url : String) = apply { this.url = url }

        fun data(data : ByteArray) = apply { this.data = data }

        fun type(type : RequestType) = apply { this.type = type }

        fun requestFeatures(requestFeatures : IRequestFeatures) = apply { this.requestFeatures = requestFeatures }

        fun headers(headers : HashMap<String, String>) = apply { this.headers = headers }

        fun build() = Request(this)
    }

}

IRequest.kt:

interface IRequest {
    fun streamToString(inputStream: InputStream) : String
}

 

Zaloguj lub zarejestruj się, aby odpowiedzieć na to pytanie.

Podobne pytania

+1 głos
4 odpowiedzi 2,917 wizyt
pytanie zadane 31 marca 2018 w Android, Swift, Symbian przez Jasnygwint Początkujący (330 p.)
0 głosów
2 odpowiedzi 392 wizyt
pytanie zadane 12 marca 2018 w Inne języki przez Partyjny Obywatel (1,170 p.)
0 głosów
1 odpowiedź 688 wizyt
pytanie zadane 19 maja 2020 w Android, Swift, Symbian przez Oskardes Użytkownik (600 p.)

92,556 zapytań

141,404 odpowiedzi

319,561 komentarzy

61,942 pasjonatów

Motyw:

Akcja Pajacyk

Pajacyk od wielu lat dożywia dzieci. Pomóż klikając w zielony brzuszek na stronie. Dziękujemy! ♡

Oto polecana książka warta uwagi.
Pełną listę książek znajdziesz tutaj.

Akademia Sekuraka

Kolejna edycja największej imprezy hakerskiej w Polsce, czyli Mega Sekurak Hacking Party odbędzie się już 20 maja 2024r. Z tej okazji mamy dla Was kod: pasjamshp - jeżeli wpiszecie go w koszyku, to wówczas otrzymacie 40% zniżki na bilet w wersji standard!

Więcej informacji na temat imprezy znajdziecie tutaj. Dziękujemy ekipie Sekuraka za taką fajną zniżkę dla wszystkich Pasjonatów!

Akademia Sekuraka

Niedawno wystartował dodruk tej świetnej, rozchwytywanej książki (około 940 stron). Mamy dla Was kod: pasja (wpiszcie go w koszyku), dzięki któremu otrzymujemy 10% zniżki - dziękujemy zaprzyjaźnionej ekipie Sekuraka za taki bonus dla Pasjonatów! Książka to pierwszy tom z serii o ITsec, który łagodnie wprowadzi w świat bezpieczeństwa IT każdą osobę - warto, polecamy!

...