0

私はクラス用の小さな API を作成しています。先生は、Runnable を実装する単純な Web サーバーを提供するチュートリアルへのリンクを提供してくれました。

引数を解析する (または少なくともリクエスト文字列を取得する) コードと、いくつかの単純な xml を返すコードを既に記述しています。

ただし、ファビコンのような特定のリクエストが送信されていると思います。コードが台無しになっていると思います。それをif elseでラップしましたが、機能していないようです。

=====基本的な文字列比較エラーのため編集済み=====

package server;

import java.io.IOException;


import java.io.InputStream;
import java.io.OutputStream;
import java.net.Socket;
import java.util.*;
import java.io.*; 
import java.net.*;
import parkinglots.*;

/**
import our API 
*/
public class WorkerRunnable implements Runnable{

protected Socket clientSocket = null;
protected String serverText   = null;

public WorkerRunnable(Socket clientSocket, String serverText) {
    this.clientSocket = clientSocket;
    this.serverText   = serverText;
}

public Boolean authenticateAPI(String key){
    //Authenticate Key against Stored Keys

    //TODO: Create Stored Keys and Compare
    return true;
}

public void run() {
    try {
        InputStream input = clientSocket.getInputStream();
        OutputStream output = clientSocket.getOutputStream();
        long time = System.currentTimeMillis();


        //TODO: Parse args and output different formats and Authentication
        //Parse URL Arguments
        BufferedReader in = new BufferedReader( 
                new InputStreamReader(clientSocket.getInputStream(), "8859_1"));

        String request = in.readLine();

        //Server gets Favicon Request so skip that and goto args
        System.out.println(request);
        if ( !"GET /favicon.ico HTTP/1.1".equals(request) && !"GET / HTTP/1.1".equals(request) && !"".equals(request) ){

            String format = "", apikey ="";

            System.out.println("I am Here");

            String request_location = request.split(" ")[1];
            String request_args = request_location.replace("/","");
            request_args = request_args.replace("?","");
            String[] queries = request_args.split("&");

            System.out.println(queries[0]);

            for ( int i = 0; i < queries.length; i++ ){
                if( queries[i].equals("format") ){
                    format = queries[i].split("=")[1];
                }
                else if( queries[i].equals("apikey") ){
                    apikey = queries[i].split("=")[1];
                }
            }

            if( apikey.equals("") ){
                apikey = "None";
            }

            if( format.equals("") ){
                format = "xml";
            }

            System.out.println("format: "+format);

            Boolean auth = authenticateAPI(apikey);

            if ( auth ){
                if ( format.equals("xml")){
                    // Retrieve XML Document
                    String xml = LotFromDB.getParkingLotXML();
                    output.write((xml).getBytes());
                }else{
                    //Retrieve JSON
                    String json = LotFromDB.getParkingLotJSON();
                    output.write((json).getBytes());
                }
            }else{
                output.write(("Access Denied - User is Not Authenticated").getBytes());
            }
        }else{
            output.write(("Access Denied Must Pass API Key").getBytes());
        }

        output.close();
        input.close();
        System.out.println("Request processed: " + time);
    } catch (IOException e) {
        //report exceptions
        e.printStackTrace();
    }
}
}

私が得るコンソール出力

GET /?format=json HTTP/1.1
I am Here
format=json
json
format: xml

常に XML も返します。

これは、Web サーバーを作成し、Java でネットワークを処理することに初めて触れたものであり、一般的に私を非常に苛立たせています。

他のみんなから少し助けを得た後、ここでこの部分であることが判明しました。文字列の間違った部分を比較していたので、一致しませんでした

                if( queries[i].split("=")[0].equals("format") ){
                    format = queries[i].split("=")[1];
                }
                else if( queries[i].split("=")[0].equals("apikey") ){
                    apikey = queries[i].split("=")[1];
                }
4

3 に答える 3

5

equalsすべての文字列比較では、代わりに 使用する必要があります==

例:if( format.equals("xml"))条件ブロックでrequestは、null かどうかも確認します

if (request != null && !"GET /favicon.ico HTTP/1.1".equals(request) 
  && !"GET / HTTP/1.1".equals(request) &&  !"".equals(request) )
于 2012-04-04T05:35:46.077 に答える
3

しかし、あなたはJavaを使用していますか?Javaでは、この方法で文字列を比較することはできません

request != "GET /favicon.ico HTTP/1.1"

あなたは書く必要があります

!"GET /favicon.ico HTTP/1.1".equals(request)
于 2012-04-04T05:40:32.643 に答える
1

サーバーがメッセージを送信するまでループで待機する必要があります。ソケットから読み取るときにこれを試してください:

    String request;
     while((request=in.readLine())==null);
     System.out.println(request);
于 2012-04-04T05:43:13.083 に答える