0

**这是我的第一篇文章,所以不知道我是否做对了。但是我必须阅读一个文件,然后按人口从小到大的顺序排列列表。我得到的只是阿拉巴马州,而且只出现过一次。我认为我的问题来自“for”语句,但我不确定。它也可能来自“return”语句。文件是这样设置的

阿拉巴马州,4779736
阿拉斯加州,7102313**

import java.io.File;
import java.io.IOException;
import java.io.PrintWriter;     
import java.util.Scanner;

public class Inorder {

    /**
     * @param args
     * @throws IOException 
     */
    public static void main(String[] args) throws IOException {
        // TODO Auto-generated method stub
         PrintWriter prw = new PrintWriter("outfile.txt");
                        File f = new File("census2010.txt");
                        if(!f.exists()) {
                            System.out.println( "f does not exist ");
                        }
                        Scanner infile = new Scanner(f);
                        infile.useDelimiter ("[\t|,|\n|\r]+");
                        final int MAX = 50;
                        int [] myarray = new int [MAX];
                        String[] statearray = new String[MAX];
                        int fillsize;


                        fillsize = fillarray (myarray, statearray, infile);

                        printarray (myarray, fillsize, prw);
                        sortarray(myarray, statearray, fillsize);

                    }

                    public static int fillarray (int[] num, String[] states, Scanner infile){


                        for( int count = 0; count < 50; count++){
                              int retcnt = 0;
                                int pop;
                                String state;
                                state = infile.next();
                                pop = infile.nextInt();
                            System.out.println(state + " " + pop + " ");
                            states[retcnt] = state;
                            num[retcnt] = pop;
                            retcnt++;


                        return (retcnt); }

                    }


                    public static void printarray (int[] num, int fillsize, PrintWriter prw){
                        for (int counts = 0; counts < fillsize ; counts++){
                            System.out.println("For the position ["+counts+"] the value is " + num[counts]);
                            prw.println("For the position ["+counts+"] the value is " + num[counts]);
                        }
                        return;
                    }

                    public static void  sortarray(int[] poparray, String[] statearray, int fillsize){

                        for( int fill = 0; fill < fillsize -1; fill = fill+1){
                            for ( int compare = fill+1; compare < fillsize; compare++){
                                if( poparray[compare] < poparray[fill]){

                                    int poptemp = poparray[fill];  
                                    poparray[fill] = poparray[compare]; 
                                    poparray[compare]  = poptemp;
                                // do I need something here?    
                                    String statetemp = statearray[fill];  
                                    statearray[fill] = statearray[compare]; 
                                    statearray[compare]  = statetemp;
                                }
                            }
                        }
                    }
}
4

1 回答 1

2

看起来您只需要将您的 return 语句移到 for 循环之外。

public static int fillarray (int[] num, String[] states, Scanner infile){
    for( int count = 0; count < 50; count++){
        // ...
    } // Finish *all* iterations of the loop, *then* return
    return (retcnt); 
 }

通过return在你的循环中,你只执行第一次迭代并且方法返回(阻止所有其他 49 次迭代)。你在你的printarray方法中正确地做到了这一点。

编辑:

正如您所提到的,将 return 语句移到循环之外使其retcnt不再可访问。这是因为您在循环retcnt 内声明;如果您retcnt在循环之前声明,您将没有问题。

int retcnt = 0;
for (//...) {
    //...
}
// The variable retcnt is now accessible for the entire method scope, 
// instead of just the loop block
return retcnt;
于 2013-04-19T21:52:16.257 回答