Java 如何修复代码以返回正确的输出a&；而不是问号？

对于这个任务，我们应该对URL进行编码。最终，输出应该是，但我得到了www.amazon.com？id=123？author=Jim+Campbell？publisher=O%27Reilly

下面是我的代码：

package MyUrl;

import java.util.Scanner;

public class MyUrl {
    private String mUrl;

    public MyUrl(String url) {

        if (url.contains("http://"))
            mUrl = "http://" + url;
        else
            mUrl = url;
    }

    public void addArgument(String name, String value) {
        if (mUrl.indexOf(name) == '?') {
            mUrl = mUrl + '&' + urlEncode(name) + '=' + urlEncode(value);
        } else {
            mUrl = mUrl + '?' + urlEncode(name) + '=' + urlEncode(value);
        }
    }

    public void addArgument(String name, int ivalue) {
        String newValue = Integer.toString(ivalue);

        if (mUrl.indexOf(name) == '?') {
            mUrl = mUrl + '&' + urlEncode(name) + '=' + urlEncode(newValue);
        } else {
            mUrl = mUrl + '?' + urlEncode(name) + '=' + urlEncode(newValue);
        }
    }

    public void addArgument(String name, double dvalue) {
        String newValue1 = Double.toString(dvalue);
        if (mUrl.indexOf(name) == '?') {
            mUrl = mUrl + '&' + urlEncode(name) + '=' + urlEncode(newValue1);
        } else {
            mUrl = mUrl + '?' + urlEncode(name) + '=' + urlEncode(newValue1);
        }
    }

    public String toString() {
        String result = mUrl;

        return result;
    }

    public static String urlEncode(String text) {
        String newWord = "";
        for (int n = 0; n < text.length(); ++n) {

            char c = text.charAt(n);
            String hexValue = Integer.toHexString(c);


            if (c >= 'A' && c <= 'Z') {
                newWord += c;
            } else if (c >= 'a' && c <= 'z') {
                newWord += c;
            } else if (c >= '0' && c <= '9') {
                newWord += c;
            } else if (c == '_' || c == '-' || c == '.' || c == '*') {
                newWord += c;
            } else if (c == ' ') {
                newWord += '+';
            } else {
                newWord += '%' + hexValue;

            }
        }
        text = newWord;
        return text;
    }

}

    /**
     * main
     */
    public static void main(String[] args) {
        Scanner sc = new Scanner(System.in);

        // Loop, asking for a new URL to be entered.
        do {
            System.out.println();
            System.out.println("Enter URL site (or 'exit')...");
            String baseUrl = sc.nextLine();
            if (baseUrl.equalsIgnoreCase("exit"))
                break;

            // Create a new MyUrl object and call its constructor
            MyUrl u = new MyUrl(baseUrl);
            System.out.println("Url value read was: " + baseUrl);

            // Loop, asking for argument/value input
            do {
                System.out.println("Enter URL argument name (or 'done')...");
                String argName = sc.nextLine();
                if (argName.equalsIgnoreCase("done"))
                    break;
                System.out.println("Enter type of argument value (string, integer, double)...");
                String argType = sc.nextLine();

                if (argType.startsWith("s")) {
                    System.out.println("Enter a string value");
                    String s = sc.nextLine();
                    u.addArgument(argName, s);
                } else if (argType.startsWith("i")) {
                    System.out.println("Enter an integer value");
                    int i = sc.nextInt();
                    sc.nextLine();
                    u.addArgument(argName, i);
                } else if (argType.startsWith("d")) {
                    System.out.println("Enter a double value");
                    double d = sc.nextDouble();
                    sc.nextLine();
                    u.addArgument(argName, d);
                } else {
                    System.out.println("Unrecognized value type - must be (s)tring, (i)nteger, or (d)ouble");
                    continue;
                }
            } while (true);

            // Display the final url
            System.out.println("URL with appended arguments is:");
            System.out.println("  " + u.toString());

        } while (true);

        // Keep console window alive until 'enter' pressed (if needed).
        System.out.println();
        System.out.println("Done - press enter key to end program");
        String junk = sc.nextLine();
    }
}

包MyUrl；
导入java.util.Scanner；
公共类MyUrl{
私人字符串mUrl；
公共MyUrl（字符串url）{
if（url.contains（“http://”）
mUrl=“http://”+url；
其他的
mUrl=url；
}
public void addArgument（字符串名称、字符串值）{
if（mUrl.indexOf（name）=='？'）{
mUrl=mUrl+'&'+urlEncode（名称）+'='+urlEncode（值）；
}否则{
mUrl=mUrl+'？'+urlEncode（名称）+'='+urlEncode（值）；
}
}
public void addArgument（字符串名，int-ivalue）{
字符串newValue=Integer.toString（ivalue）；
if（mUrl.indexOf（name）=='？'）{
mUrl=mUrl+'&'+urlEncode（名称）+'='+urlEncode（新值）；
}否则{
mUrl=mUrl+'？'+urlEncode（名称）+'='+urlEncode（新值）；
}
}
public void addArgument（字符串名，双D值）{
字符串newValue1=Double.toString（dvalue）；
if（mUrl.indexOf（name）=='？'）{
mUrl=mUrl+'&'+urlEncode（name）+'='+urlEncode（newValue1）；
}否则{
mUrl=mUrl+'？'+urlEncode（名称）+'='+urlEncode（新值1）；
}
}
公共字符串toString（）{
字符串结果=mUrl；
返回结果；
}
公共静态字符串urlEncode（字符串文本）{
字符串newWord=“”；
对于（int n=0；n='A'&&c='A'&&c='0'&&c我注意到了这些

在构造函数中，您忘记了一个
！
。替换

if (url.contains("http://"))

与


在所有
addArgument
方法中，替换

if(mUrl.indexOf(name)=='?')

与


解释

如果
http://
已经存在，那么您正在添加它，而不是相反。
！
不符合逻辑，它会将其反转

您试图在URL中找到参数的名称，然后将索引（或-1）与
'？'
的数值进行比较。显然，这是行不通的

contains在这里是错误的。您想要startsWith。indexOf很慢；您最好跟踪一个标志，告诉您是否添加了查询字符串。这会使mUrl成为StringBuilder。@david不值得这么麻烦-您说的是优化以节省1微秒。这是他的问题中最小的一个。重要的一点是java正在制造一个将
？
从
char
自动加宽到
int
（其ASCII值），在这种情况下没有意义。不确定您是否已经修复了它，但也存在一个问题，即当它应该是“&”时有一个“？”。是的。第二个问题导致该条件（几乎）永远不会为真，因此每次都会添加一个新的“？”。
if(mUrl.indexOf(name)=='?')

if(mUrl.indexOf('?') != -1)