การอ่าน Android จากสตรีมอินพุตอย่างมีประสิทธิภาพ

ฉันกำลังสร้าง HTTP รับคำขอไปยังเว็บไซต์สำหรับแอปพลิเคชัน Android ที่ฉันกำลังทำ

ฉันใช้ DefaultHttpClient และใช้ HttpGet เพื่อออกคำขอ ฉันได้รับการตอบสนองเอนทิตีและจากนี้รับวัตถุ InputStream สำหรับรับ html ของหน้า

จากนั้นฉันจะวนรอบการตอบกลับโดยทำดังนี้:

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

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

อย่างไรก็ตามเรื่องนี้ช้าลงอย่างน่ากลัว

มันไม่มีประสิทธิภาพไหม? ฉันไม่ได้โหลดหน้าเว็บขนาดใหญ่ - www.cokezone.co.ukดังนั้นขนาดไฟล์ไม่ใหญ่ มีวิธีที่ดีกว่าในการทำเช่นนี้?

ขอบคุณ

แอนดี้

ปัญหาในรหัสของคุณคือมันสร้างStringวัตถุจำนวนมากคัดลอกเนื้อหาและดำเนินการกับพวกเขา แต่คุณควรใช้StringBuilderเพื่อหลีกเลี่ยงการสร้างStringวัตถุใหม่ในแต่ละผนวกและเพื่อหลีกเลี่ยงการคัดลอกอาร์เรย์ถ่าน การดำเนินการสำหรับกรณีของคุณจะเป็นดังนี้:

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

ตอนนี้คุณสามารถใช้totalโดยไม่แปลงเป็นStringแต่ถ้าคุณต้องการผลลัพธ์ที่เป็นStringเพียงแค่เพิ่ม:

ผลสตริง = total.toString ();

ฉันจะพยายามอธิบายให้ดีขึ้น ...

• a += b(หรือa = a + b) ที่ไหนaและbเป็นสตริงคัดลอกเนื้อหาของทั้งสอง a และ bไปยังวัตถุใหม่ (โปรดทราบว่าคุณกำลังคัดลอกaซึ่งมีของสะสม String ) และคุณกำลังทำสำเนาเหล่านั้นในการทำซ้ำแต่ละครั้ง
• a.append(b)ตรงไหนaa StringBuilder, ต่อท้ายbเนื้อหาโดยตรงaดังนั้นคุณไม่ต้องคัดลอกสตริงที่สะสมในแต่ละรอบซ้ำ

คุณได้ลองวิธีในตัวเพื่อแปลงกระแสข้อมูลให้เป็นสตริงหรือไม่? มันเป็นส่วนหนึ่งของห้องสมุด Apache Commons (org.apache.commons.io.IOUtils)

จากนั้นรหัสของคุณจะเป็นหนึ่งบรรทัดนี้:

String total = IOUtils.toString(inputStream);

เอกสารสำหรับมันสามารถพบได้ที่นี่: http://commons.apache.org/io/api-1.4/org/apache/commons/io/IOUtils.html#toString%28java.io.InputStream%29

สามารถดาวน์โหลดไลบรารี Apache Commons IO ได้จากที่นี่: http://commons.apache.org/io/download_io.cgi

ความเป็นไปได้อีกครั้งกับ Guava:

พึ่งพา: compile 'com.google.guava:guava:11.0.2'

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

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

ฉันเชื่อว่ามันมีประสิทธิภาพเพียงพอ ... ในการรับ String จาก InputStream ฉันจะเรียกวิธีการต่อไปนี้:

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();
}

ฉันใช้ UTF-8 เสมอ แน่นอนคุณสามารถตั้งค่าชุดอักขระเป็นอาร์กิวเมนต์นอกเหนือจาก InputStream

เกี่ยวกับสิ่งนี้ ดูเหมือนว่าจะให้ประสิทธิภาพที่ดีขึ้น

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));
}

แก้ไข: จริง ๆ แล้วการเรียงลำดับนี้ครอบคลุมทั้งทั้งเหล็กและมอริซเพอร์รี่

อาจจะเร็วกว่าคำตอบของ Jaime Soriano และไม่มีปัญหาการเข้ารหัสแบบหลายไบต์ของคำตอบของ Adrian ฉันแนะนำ:

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();
}

อาจจะค่อนข้างอ่าน 'ทีละบรรทัด' และเข้าร่วมสตริงลอง 'อ่านทั้งหมดที่มีอยู่' เพื่อหลีกเลี่ยงการสแกนจุดสิ้นสุดของบรรทัดและเพื่อหลีกเลี่ยงการรวมสตริง

เช่นInputStream.available()และInputStream.read(byte[] b), int offset, int length)

การอ่านข้อความทีละบรรทัดและการต่อท้ายบรรทัดที่กล่าวถึงสตริงนั้นเป็นการใช้เวลานานทั้งในการแยกแต่ละบรรทัดและโอเวอร์เฮดของการเรียกใช้เมธอดจำนวนมาก

ฉันสามารถรับประสิทธิภาพที่ดีขึ้นโดยการจัดสรรอาร์เรย์ขนาดที่เหมาะสมเพื่อเก็บข้อมูลสตรีมและแทนที่ด้วยอาร์เรย์ขนาดใหญ่กว่าเมื่อจำเป็นและพยายามอ่านให้มากที่สุดเท่าที่อาร์เรย์สามารถเก็บได้

ด้วยเหตุผลบางอย่าง Android ล้มเหลวในการดาวน์โหลดไฟล์ทั้งหมดซ้ำ ๆ เมื่อรหัสที่ใช้ InputStream ที่ส่งคืนโดย HTTPUrlConnection ดังนั้นฉันจึงต้องใช้ทั้ง BufferedReader และกลไกการหมดเวลาด้วยมือเพื่อให้แน่ใจว่าฉันจะได้รับทั้งไฟล์หรือยกเลิก การถ่ายโอน

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();
    }
}

แก้ไข:ปรากฎว่าถ้าคุณไม่จำเป็นต้องมีการเข้ารหัสเนื้อหาอีกครั้ง (เช่นคุณต้องการเนื้อหาตามที่เป็น ) คุณไม่ควรใช้คลาสย่อย Reader ใด ๆ เพียงใช้คลาสย่อย Stream ที่เหมาะสม

แทนที่จุดเริ่มต้นของวิธีการก่อนหน้านี้ที่มีเส้นที่สอดคล้องกันดังต่อไปนี้เพื่อความเร็วขึ้นเป็นพิเศษ 2 ถึง 3 ครั้ง

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];

หากไฟล์มีความยาวคุณสามารถเพิ่มประสิทธิภาพรหัสของคุณโดยผนวกเข้ากับ StringBuilder แทนการใช้การเรียงต่อกันสตริงสำหรับแต่ละบรรทัด

    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);

ในการแปลง InputStream เป็น String เราใช้ เมธอดBufferedReader.readLine () เราทำซ้ำจนกระทั่งBufferedReaderคืนค่าว่างซึ่งหมายความว่าไม่มีข้อมูลให้อ่านอีกต่อไป แต่ละบรรทัดจะผนวกเข้ากับStringBuilderและส่งกลับเป็นสตริง

 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();
    }
}`

และในที่สุดจากคลาสใด ๆ ที่คุณต้องการแปลงการเรียกใช้ฟังก์ชัน

String dataString = Utils.convertStreamToString(in);

สมบูรณ์

ฉันใช้อ่านข้อมูลเต็ม:

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