このGETリクエストは何が問題なのですか？

Question

私たちはAndroid Studioでこのアプリを開発しています。そこで、ウェブサイトにリクエストを出したいときに、このコードを実行すると、私は「null」というエラーを受け取り続けます。このコードの変数のうちの1つはヌルです。誰かがそれを見て、いくつかの変数が正しく使用されていないため、ヌルエラーを提供している場所を参照できますか？

public class SpotAlgo { 
String vidLink; 
int linkLoc; 
String testString = "<title>"; 
String result; 

public String gettheResult(String v) throws Exception{ 
    String sname = " "; 
    vidLink = "https://open.spotify.com/track/43PuMrRfbyyuz4QpZ3oAwN"; 

    URL obj = new URL(vidLink); 
    HttpURLConnection con = (HttpURLConnection) obj.openConnection(); 
    con.setRequestMethod("GET"); 
    BufferedReader in = new BufferedReader(new InputStreamReader(con.getInputStream())); 
    String inputLine = ""; 
    StringBuffer response = new StringBuffer(); 

    while ((inputLine = in.readLine()) != null) { 
     response.append(inputLine); 
    } 
    in.close(); 

    result = response.toString(); 


    linkLoc = result.indexOf(testString) + testString.length(); 
    for (int i = linkLoc; i < result.indexOf("on Spotify"); i++) { 
     sname += result.charAt(i) + ""; 
    } 
    return obj.toString(); 
} 

}

Answer 1

あなたは）（実際には

con.setRequestMethod("GET"); 
con.connect(); 

あなたcon.getInputStreamを接続逃しました。接続は行われず、何もconに返されないため、nullです。

Answer 2

何も問題ありません。

nullポインタ例外があると、コードの別の部分に関連していると思います。

はしかし、ここであなたがループ内でsnameの変数にテキストを追加します。このスニペットでは、それを

を改善するためのいくつかの提案です。この種の操作を行うと、StringBuilderがより効率的になります。

for (int i = linkLoc; i < result.indexOf("on Spotify"); i++) { 
    sname += result.charAt(i) + ""; 
} 
//Could be replaced b 
for (int i = linkLoc; i < result.indexOf("on Spotify"); i++) { 
    sb.append(result.charAt(i)); 
} 

その後、sb.toString（）を使用して操作の結果を取得できます。

さらに重要なことに、ネットワーク操作を実行すると、結果がどのようになるか分かりません。あなたが得る結果に影響を与える多くの変数があり、多くの場合例外がスローされます。 try {} finally {}内にネットワークコードをラップし、finallyブロックで開いたリソースを閉じておくことが重要です（finallyブロックは常に実行されます）。タイプミス

編集2

一部の人々は、私は私のマシン上でそれを実行したとき、私は1を取得していない、それは404の例外を投げていると言って、404されているのを修正

編集 con.getInputStream（）を実行するとIOExceptionがスローされるので、代わりにその代わりに