2015-11-20 32 views
1

这里是我的函数,它假定从“in1.txt”中取值,并将它们输出到“out1.txt”中,并附有说明它们是否为素数的说明。如何为我的输出添加更多值?

代码:

import java.util.Scanner; 
import java.util.ArrayList; 
import java.io.File; 
import java.io.PrintWriter; 
public class task1 
{ 
public static boolean isPrime(int n) { 
    if (n <= 1) { 
    return false; 
    } 
    for (int i = 2; i < Math.sqrt(n); i++) { 
    if (n % i == 0) { 
     return false; 
    } 
} 
return true; 
} 
public static ArrayList<String> check_primes(String in_file, String out_file) 
{ 
    File temp = new File(in_file); 
    Scanner input_file; 
    String filename = "out1.txt"; 
    PrintWriter out = null; 

    try 
    { 
     input_file = new Scanner(temp); 
     out = new PrintWriter(filename); 
    } 
    catch (Exception e) 
    { 
     System.out.printf("Failed to open file %s\n",in_file); 
     return null; 
    } 
    ArrayList<String> result = new ArrayList<String>(); 
    while(input_file.hasNextLine()) 
    { 
     String line = input_file.nextLine(); 
     result.add(line); 

     for(int i = 0; i<result.size();i++) 
     { 
      String x = result.get(i); 
      int xx = Integer.parseInt(x); 
      if(isPrime(xx)) 
      { 
       out.printf(xx + " is a prime\r\n"); 
      } 
      else 
      { 
       out.printf(xx + " is not a prime\r\n"); 
      }   
     } 

     out.close(); 
    } 

    input_file.close(); 

    return result; 
} 
public static void main(String[] args) 
{ 
check_primes("in1.txt", "out1.txt"); 
System.out.printf("Exiting...\n"); 
} 
} 

它只是增加了INT1文件的第一个值,我假设它是某种循环错误的或者说我缺少在我的循环的东西。我测试了看“x”还是“xx”是否有错误,并将它们打印出来并将x打印出来形成一个金字塔,[7] [7,10] [7,10,13]等等,xx打印出来的方式与7 \ n 7 \ n 10 \ n 7 \ n 10 \ n 13 \ n 有人帮助我找出问题所在。

+4

考虑不关闭输出回路:) – zapl

+0

@zapl +1为察觉的主要问题,但是,即使在环路输出不是封闭的,你不觉得for循环每次重新启动一次新的行被添加到结果? – Raf

回答

2

我认为有一个循环,然后将每行添加到列表的意图是处理后面包含所有行的列表。如果这是真的,那么你需要在你的while循环中包含result.add(line)并关闭它。在for循环之后处理所有行

while (input_file.hasNextLine()) { 
    String line = input_file.nextLine(); 
    result.add(line); 
} //close the loop here 

假设您不关闭while循环。 For循环将只执行一次,因为您的result.size()是1,并且for循环完成后,它将关闭输出流。

+0

谢谢你,现在我很生气我没有看到那个小修正xD –

0
while(input_file.hasNextLine()) { 
     String line = input_file.nextLine(); 
     result.add(line);  
} 
for(int i = 0; i<result.size();i++) 
    { 
     String x = result.get(i); 
     int xx = Integer.parseInt(x); 
     if(isPrime(xx)) 
     { 
      out.printf(xx + " is a prime\r\n"); 
     } 
     else 
     { 
      out.printf(xx + " is not a prime\r\n"); 
     }   
    } 

    out.close();