Android Membaca dari aliran input secara efisien

Saya membuat permintaan dapatkan HTTP ke situs web untuk aplikasi android yang saya buat.

Saya menggunakan DefaultHttpClient dan menggunakan HttpGet untuk mengeluarkan permintaan. Saya mendapatkan respons entitas dan dari sini mendapatkan objek InputStream untuk mendapatkan html halaman.

Saya kemudian melakukan balas dengan melakukan sebagai berikut:

BufferedReader r = new BufferedReader(new InputStreamReader(inputStream));
String x = "";
x = r.readLine();
String total = "";

while(x!= null){
total += x;
x = r.readLine();
}

Namun ini sangat lambat.

Apakah ini tidak efisien? Saya tidak memuat halaman web besar - www.cokezone.co.uk sehingga ukuran file tidak besar. Apakah ada cara yang lebih baik untuk melakukan ini?

Terima kasih

Andy

Masalah dalam kode Anda adalah membuat banyak Stringobjek berat , menyalin kontennya dan melakukan operasi padanya. Sebagai gantinya, Anda harus menggunakan StringBuilderuntuk menghindari membuat Stringobjek baru di setiap append dan untuk menghindari menyalin array char. Implementasi untuk kasus Anda akan menjadi seperti ini:

BufferedReader r = new BufferedReader(new InputStreamReader(inputStream));
StringBuilder total = new StringBuilder();
for (String line; (line = r.readLine()) != null; ) {
    total.append(line).append('\n');
}

Anda sekarang dapat menggunakan totaltanpa mengubahnya menjadi String, tetapi jika Anda membutuhkan hasilnya sebagai String, cukup tambahkan:

Hasil string = total.toString ();

Saya akan mencoba menjelaskannya dengan lebih baik ...

• a += b(atau a = a + b), di mana adan bsekarang, menyalin konten keduanya a dan b ke objek baru (perhatikan bahwa Anda juga menyalin a, yang berisi akumulasi String ), dan Anda melakukan salinan itu pada setiap iterasi.
• a.append(b), di mana aa StringBuilder, langsung menambahkan bkonten a, sehingga Anda tidak menyalin string yang terakumulasi di setiap iterasi.

Sudahkah Anda mencoba metode bawaan untuk mengonversi aliran ke string? Itu bagian dari perpustakaan Apache Commons (org.apache.commons.io.IOUtils).

Maka kode Anda adalah baris ini:

String total = IOUtils.toString(inputStream);

Dokumentasi untuk itu dapat ditemukan di sini: http://commons.apache.org/io/api-1.4/org/apache/commons/io/IOUtils.html#toString%28java.io.InputStream%29

Perpustakaan IO Apache Commons dapat diunduh dari sini: http://commons.apache.org/io/download_io.cgi

Kemungkinan lain dengan Jambu Biji:

ketergantungan: compile 'com.google.guava:guava:11.0.2'

import com.google.common.io.ByteStreams;
...

String total = new String(ByteStreams.toByteArray(inputStream ));

Saya percaya ini cukup efisien ... Untuk mendapatkan sebuah String dari InputStream, saya akan memanggil metode berikut:

public static String getStringFromInputStream(InputStream stream) throws IOException
{
    int n = 0;
    char[] buffer = new char[1024 * 4];
    InputStreamReader reader = new InputStreamReader(stream, "UTF8");
    StringWriter writer = new StringWriter();
    while (-1 != (n = reader.read(buffer))) writer.write(buffer, 0, n);
    return writer.toString();
}

Saya selalu menggunakan UTF-8. Anda tentu saja dapat menetapkan rangkaian karakter sebagai argumen, selain InputStream.

Bagaimana dengan ini. Tampaknya memberikan kinerja yang lebih baik.

byte[] bytes = new byte[1000];

StringBuilder x = new StringBuilder();

int numRead = 0;
while ((numRead = is.read(bytes)) >= 0) {
    x.append(new String(bytes, 0, numRead));
}

Sunting: Sebenarnya ini mencakup steelbytes dan Maurice Perry

Mungkin agak lebih cepat daripada jawaban Jaime Soriano, dan tanpa masalah pengkodean multi-byte dari jawaban Adrian, saya sarankan:

File file = new File("/tmp/myfile");
try {
    FileInputStream stream = new FileInputStream(file);

    int count;
    byte[] buffer = new byte[1024];
    ByteArrayOutputStream byteStream =
        new ByteArrayOutputStream(stream.available());

    while (true) {
        count = stream.read(buffer);
        if (count <= 0)
            break;
        byteStream.write(buffer, 0, count);
    }

    String string = byteStream.toString();
    System.out.format("%d bytes: \"%s\"%n", string.length(), string);
} catch (IOException e) {
    e.printStackTrace();
}

Mungkin lebih baik daripada membaca 'satu baris pada satu waktu' dan bergabung dengan string, coba 'baca semua yang tersedia' untuk menghindari pemindaian untuk akhir baris, dan juga untuk menghindari string bergabung.

yaitu, InputStream.available()danInputStream.read(byte[] b), int offset, int length)

Membaca satu baris teks pada satu waktu, dan menambahkan baris kata ke string secara individual memakan waktu baik dalam mengekstraksi setiap baris dan overhead dari begitu banyak doa metode.

Saya bisa mendapatkan kinerja yang lebih baik dengan mengalokasikan byte array berukuran layak untuk menampung data stream, dan yang secara iteratif diganti dengan array yang lebih besar bila diperlukan, dan mencoba membaca sebanyak yang bisa disimpan oleh array.

Untuk beberapa alasan, Android berulang kali gagal mengunduh seluruh file ketika kode menggunakan InputStream yang dikembalikan oleh HTTPUrlConnection, jadi saya harus menggunakan BufferedReader dan mekanisme timeout linting tangan untuk memastikan saya mendapatkan seluruh file atau membatalkan transfer.

private static  final   int         kBufferExpansionSize        = 32 * 1024;
private static  final   int         kBufferInitialSize          = kBufferExpansionSize;
private static  final   int         kMillisecondsFactor         = 1000;
private static  final   int         kNetworkActionPeriod        = 12 * kMillisecondsFactor;

private String loadContentsOfReader(Reader aReader)
{
    BufferedReader  br = null;
    char[]          array = new char[kBufferInitialSize];
    int             bytesRead;
    int             totalLength = 0;
    String          resourceContent = "";
    long            stopTime;
    long            nowTime;

    try
    {
        br = new BufferedReader(aReader);

        nowTime = System.nanoTime();
        stopTime = nowTime + ((long)kNetworkActionPeriod * kMillisecondsFactor * kMillisecondsFactor);
        while(((bytesRead = br.read(array, totalLength, array.length - totalLength)) != -1)
        && (nowTime < stopTime))
        {
            totalLength += bytesRead;
            if(totalLength == array.length)
                array = Arrays.copyOf(array, array.length + kBufferExpansionSize);
            nowTime = System.nanoTime();
        }

        if(bytesRead == -1)
            resourceContent = new String(array, 0, totalLength);
    }
    catch(Exception e)
    {
        e.printStackTrace();
    }

    try
    {
        if(br != null)
            br.close();
    }
    catch(IOException e)
    {
        // TODO Auto-generated catch block
        e.printStackTrace();
    }
}

EDIT: Ternyata jika Anda tidak perlu menyandikan ulang konten (yaitu, Anda menginginkan konten SEBAGAIMANA ADANYA ), Anda tidak boleh menggunakan subkelas Pustaka apa pun. Cukup gunakan subkelas Stream yang sesuai.

Ganti awal metode sebelumnya dengan garis yang sesuai di bawah ini untuk mempercepatnya 2 hingga 3 kali lipat .

String  loadContentsFromStream(Stream aStream)
{
    BufferedInputStream br = null;
    byte[]              array;
    int                 bytesRead;
    int                 totalLength = 0;
    String              resourceContent;
    long                stopTime;
    long                nowTime;

    resourceContent = "";
    try
    {
        br = new BufferedInputStream(aStream);
        array = new byte[kBufferInitialSize];

Jika file panjang, Anda dapat mengoptimalkan kode dengan menambahkan ke StringBuilder alih-alih menggunakan penggabungan String untuk setiap baris.

    byte[] buffer = new byte[1024];  // buffer store for the stream
    int bytes; // bytes returned from read()

    // Keep listening to the InputStream until an exception occurs
    while (true) {
        try {
            // Read from the InputStream
            bytes = mmInStream.read(buffer);

            String TOKEN_ = new String(buffer, "UTF-8");

            String xx = TOKEN_.substring(0, bytes);

Untuk mengonversi InputStream ke String, kami menggunakan metode BufferedReader.readLine () . Kami beralih sampai BufferedReader mengembalikan nol yang berarti tidak ada lagi data untuk dibaca. Setiap baris akan ditambahkan ke StringBuilder dan dikembalikan sebagai String.

 public static String convertStreamToString(InputStream is) {

        BufferedReader reader = new BufferedReader(new InputStreamReader(is));
        StringBuilder sb = new StringBuilder();

        String line = null;
        try {
            while ((line = reader.readLine()) != null) {
                sb.append(line + "\n");
            }
        } catch (IOException e) {
            e.printStackTrace();
        } finally {
            try {
                is.close();
            } catch (IOException e) {
                e.printStackTrace();
            }
        }
        return sb.toString();
    }
}`

Dan akhirnya dari kelas mana pun di mana Anda ingin mengkonversi panggilan fungsi

String dataString = Utils.convertStreamToString(in);

lengkap

Saya terbiasa membaca data lengkap:

// inputStream is one instance InputStream
byte[] data = new byte[inputStream.available()];
inputStream.read(data);
String dataString = new String(data);