表达式的类型必须是数组类型，但它解析为ArrayList（尝试比较两个数组中的字符串）

我试图将数组中的每个字符串或int与另一个数组进行比较，然后根据字符串是否存在于另一个数组中打印结果：下面是整个代码：当尝试使用比较两个值时，我在for循环中得到错误.equals（不确定它是否正确的方法，……我是新手）请帮忙！

public class comparer { public void compare (){ ArrayList NameofFileinDir = new ArrayList (); ArrayList Stocks = new ArrayList(); // populate array with files names in dir try { Scanner reads = new Scanner (new File("G:/Programming/StockDataDownload/NameOfFileinDir.txt")); String FileCode; while (reads.hasNext()){ FileCode = reads.nextLine(); //read next line NameofFileinDir.add(FileCode); } reads.close(); } catch (IOException e) { } // populate array with original stocks try { Scanner reads = new Scanner (new File("G:/Programming/StockDataDownload/AllSecurities.txt")); String StockCode; while (reads.hasNext()){ StockCode = reads.nextLine(); //read next line Stocks.add(StockCode); } reads.close(); for(int i = 0; i < Stocks.size(); i++) { for (int j = 0; j  0) { System.out.println("Stock:" + Stocks[i]);} else{ System.out.println("Stock not in files:" + Stocks[i]); } } } } catch (IOException e) { } }} 

Answer 1

2. ArrayList不是array 。 get the element at index i处get the element at index i的语法是不同的： list.get(i) vs arr[i] 。 阅读ArrayList ，尤其是get(int)方法。

3. 研究String.equals(Object)返回的内容以及它的含义。 浏览Object.equals(Object)也不会受到影响，因为java中的每个对象都默认获取此对象。

4. 考虑使用foreach循环而不是旧的for循环。 旧的for循环可能很有用，但这里没有意义。 看看它有多清洁：

    for (String stock : Stocks) { for (String filename : NameofFileinDir) { if (stock.equals(filename)) { System.out.println(" Stock: " + stocks); } else { System.out.println("Stock not in files: " + stocks); } } } 

5. ArrayList Stocks = new ArrayList(); 。 凉。 所以你有一个ArrayList 。 什么？！ 现在，这在技术上并不违法，但如果你的java编译器支持类型参数，你真的需要使用它们。 写而不是List stocks = new ArrayList() 。 现在，每个阅读代码的人都会立即知道列表的内容。 此外，编译器将保证您的列表只包含字符串。 如果你尝试做一些疯狂或愚蠢的事情，比如用你的字符串将一些整数和地图放入列表中，编译器就会呕吐。

6. 如果您在if..else中打印的内容是您的意思，那么您的逻辑就存在致命缺陷。 如果您碰巧比较的当前股票字符串和文件字符串不相等，则打印“库存不在文件列表中”。 但从逻辑上讲，这是否意味着股票字符串和文件字符串都不相等？

Answer 2

您不能在ArrayList上使用数组表示法。 你需要调用get(index) 。

Answer 3

将Stocks [i]更改为Stocks.get（i）和NameofFileinDir [j]更改为NameofFileinDir.get（j）

其他建议：

还要将NameofFileinDir设置为hashSet以获得更好的性能

还要遵循java camelCase命名约定，例如将NameofFileinDir更改为nameofFileinDir

Answer 4

你不能通过使用像array这样的[]运算符来获取ArrayList对象。在ArrayList上使用get()方法来获取对象。 所以改变代码

  if (Stocks[i].equals(NameofFileinDir[j])) > 0) { 

至

  if (Stocks[i].equals(NameofFileinDir.get([j]))) > 0) { 

Answer 5

你的错误在这里：

 if (Stocks[i].equals(NameofFileinDir[j])) > 0) { 

首先，你应该说Stocks.get(i).equals(NameofFileinDir.get(j) equals函数返回一个布尔值，所以你比较wether true/false > 0你可以删除> 0)

Answer 6

如果您正在使用ArrayList，那么您需要使用get（index）方法来访问该索引处的变量。 此外，.equals（object）返回一个布尔值，因此不需要> 0

 if (Stocks.get(i).equals(NameOfFileINDir.get(j)) { } 

Answer 7

如果你不想使用每个循环，那么你可以遵循这个。

 for(int i = 0; i < Stocks.size(); i++) { for (int j = 0; j < NameofFileinDir.size(); j++) { if (Stocks[i].equals(NameofFileinDir[j])) > 0) { System.out.println("Stock:" + Stocks[i]);} else{ System.out.println("Stock not in files:" + Stocks[i]); } } } 

在上面的代码中，不是有股票[i]，而是：

 for(int i = 0; i < Stocks.size(); i++) { for (int j = 0; j < NameofFileinDir.size(); j++) { if (Stocks.get(i).equals(NameofFileinDir.get(j))) > 0) { System.out.println("Stock:" + Stocks[i]);} else{ System.out.println("Stock not in files:" + Stocks.get(i)); } } }