(String) atau .toString ()?

Saya memiliki metode dengan Object oparameter.

Dalam metode ini, saya tahu persis ada Stringin "o" yang bukan null. Tidak perlu memeriksa, atau melakukan sesuatu yang lain. Saya harus memperlakukannya persis seperti sebuah Stringbenda.

Hanya ingin tahu - apa yang lebih murah - melemparkannya ke String, atau menggunakan Object.toString()? Atau sama dengan time- / cpu- / mem- price?

Pembaruan: Metode menerima Objectkarena ini adalah penerapan antarmuka. Tidak ada cara untuk mengubah tipe parameter.

Dan itu tidak mungkin nullsama sekali. Saya hanya ingin mengatakan bahwa saya tidak perlu memeriksanya untuk nol atau kosong. Dalam kasus saya, selalu ada string yang tidak kosong.

casting ke String lebih murah karena tidak memerlukan panggilan fungsi eksternal, hanya pemeriksaan tipe internal.

Saya akan menggunakan pemeran. Itu memvalidasi "pengetahuan" Anda bahwa itu adalah string. Jika karena alasan apa pun Anda berakhir dengan bug dan seseorang mengirimkan sesuatu selain string, saya pikir akan lebih baik untuk memberikan pengecualian (yang akan dilakukan oleh pemeran) daripada terus mengeksekusi dengan data yang cacat.

Menurut renungan kinerja konyol: x.toString () vs (String) x

Pada akhirnya, hasilnya sangat jelas: setidaknya dua kali lebih cepat untuk mentransmisikan Object ke String daripada memanggil Object.toString ()

Jika Anda tahu Objek o adalah String, saya akan mengatakan melemparkannya ke String dan menerapkannya seperti itu. Memanggil toString () pada objek yang Anda tahu pasti adalah String mungkin akan menambah kebingungan.

Jika Object o bisa berupa apa saja selain String, Anda harus memanggil toString ().

Saya tidak akan terlalu peduli dengan kinerjanya, jika operasi ini dilakukan bahkan hanya beberapa ribu kali per detik - tidak ada perbedaan yang nyata.

Namun, saya akan khawatir tentang "mengetahui" masukannya. Anda memiliki metode yang menerima Objectdan Anda harus memperlakukannya seperti itu, yaitu Anda tidak boleh tahu apa-apa tentang parameter, selain itu mematuhi Objectantarmuka, yang kebetulan memiliki toString()metode. Dalam hal ini, saya sangat menyarankan menggunakan metode itu daripada hanya mengasumsikan apa pun.

OTOH, jika input selalu salah satu Stringatau null, cukup ubah metode untuk menerima Strings, dan periksa secara eksplisit untuk nulls (yang harus Anda lakukan kapan pun ketika berhadapan dengan non-primitif ...)

Mengingat bahwa tipe referensi adalah Object dan semua Object memiliki toString () panggil saja object.toString (). String.toString () baru saja mengembalikan ini.

• toString () adalah lebih sedikit kode untuk diketik.
• toString () adalah lebih sedikit bytecode.
• casting adalah operasi yang mahal VS panggilan polimorfik.
• pemerannya bisa gagal.
• Gunakan String.valueOf (object) yang hanya memanggil object.toString () jika bukan null.

Jika apa yang Anda miliki di "o" adalah String maka tidak banyak perbedaan (kemungkinan cast lebih cepat, tapi itu adalah implementasi VM / Library).

Jika "o" mungkin bukan String tetapi seharusnya String maka cast adalah yang Anda inginkan (tetapi Anda harus membuat metode ini mengambil String, bukan Objek).

Jika "o" bisa berupa tipe apa saja maka Anda harus menggunakan toString - tetapi pastikan untuk memeriksa null terlebih dahulu.

void foo(final Object o)
{
    final String str;

    // without this you would get a class cast exception
    // be wary of using instanceof though - it is usually the wrong thing to do
    if(o instanceof String)
    {
        str = (String)o;
    }    
}

atau

void foo(final Object o)
{
    final String str;

    // if you are 100% sure that o is not null then you can get rid of the else
    if(o != null)
    {
        str = o.toString();
    }
}

Saya lebih suka membuat kode yang terakhir sebagai:

void foo(final Object o)
{
    final String str;

    if(o == null)
    {
        throw new IllegalArgumentException("o cannot be null");
    }

    str = o.toString();
}

Saya menemukan anehnya bahwa cast lebih lambat daripada pencarian vtable yang tersirat oleh panggilan tostring.

Tidak boleh ada 'string nol di o'. Jika o adalah nol, itu tidak mengandung string nol, itu hanya nol. Cukup periksa o untuk nol dulu. Jika Anda mentransmisikan atau memanggil ToString () pada null, Anda akan crash.