このコードを最適化したい:
InputStream is = rp.getEntity().getContent();
BufferedReader reader = new BufferedReader(new InputStreamReader(is));
String text = "";
String aux = "";
while ((aux = reader.readLine()) != null) {
text += aux;
}
問題は、bufferedreaderのコンテンツを読み取り、上記よりも速くStringにコピーする方法がわからないことです。できるだけ時間を費やす必要はありません。ありがとうございました
ループで文字列連結を使用するのはthe古典的なパフォーマンスキラーです(文字列は不変であるため、連結ごとにますます大きくなる文字列全体がコピーされます)。代わりにこれを行います:
StringBuilder builder = new StringBuilder();
String aux = "";
while ((aux = reader.readLine()) != null) {
builder.append(aux);
}
String text = builder.toString();
Apache IOUtils.toString を試すことができます。これは彼らがすることです:
StringWriter sw = new StringWriter();
char[] buffer = new char[1024 * 4];
int n = 0;
while (-1 != (n = input.read(buffer))) {
sw.write(buffer, 0, n);
}
String text = sw.toString();
BufferedReader
がSocketから読み取る場合、bufferedReader.ready()
を追加する必要があります。
BufferedReader br = new BufferedReader(new InputStreamReader(socket.getInputStream()));
StringBuilder sb= new StringBuilder();
String line = "";
while (br.ready() && (line = br.readLine()) != null) {
sb.append(line + "\r\n");
}
String result = sb.toString();
ワンラインソリューション:
import Java.io.*;
import static Java.lang.System.lineSeparator;
import static Java.util.stream.Collectors.joining;
BufferedReader reader = ...;
String result = reader.lines().collect(joining(lineSeparator()));
内部でIOExceptionをキャッチするStringBuilderとWhileループを使用してこれを行う簡単な関数を作成しました。
public String getString(BufferedReader bufferedReader) {
StringBuilder stringBuilder = new StringBuilder();
String line = null;
do {
try {
if ((line = bufferedReader.readLine()) != null) {
stringBuilder.append(line).append(System.lineSeparator());
}
} catch (IOException e) {
e.printStackTrace();
}
} while (line != null);
return stringBuilder.toString();
}