我正在尝试学习如何在我的日常编程中使用Java 8功能(例如lambdas和stream),因为它可以使代码更清晰。
这是我目前正在做的工作: 我从一个本地文件中获取一个字符串流,其中包含一些数据,稍后我将其转换为对象。输入文件结构如下所示:
Airport name; Country; Continent; some number;
我的代码看起来像这样:
public class AirportConsumer implements AirportAPI {
List<Airport> airports = new ArrayList<Airport>();
@Override
public Stream<Airport> getAirports() {
Stream<String> stream = null;
try {
stream = Files.lines(Paths.get("resources/planes.txt"));
stream.forEach(line -> createAirport(line));
} catch (IOException e) {
e.printStackTrace();
}
return airports.stream();
}
public void createAirport(String line) {
String airport, country, continent;
int length;
airport = line.substring(0, line.indexOf(';')).trim();
line = line.replace(airport + ";", "");
country = line.substring(0,line.indexOf(';')).trim();
line = line.replace(country + ";", "");
continent = line.substring(0,line.indexOf(';')).trim();
line = line.replace(continent + ";", "");
length = Integer.parseInt(line.substring(0,line.indexOf(';')).trim());
airports.add(new Airport(airport, country, continent, length));
}
}
在我的主类中,我遍历对象流并打印出结果:
public class Main {
public void toString(Airport t){
System.out.println(t.getName() + " " + t.getContinent());
}
public static void main(String[] args) throws IOException {
Main m = new Main();
m.whatever();
}
private void whatever() throws IOException {
AirportAPI k = new AirportConsumer();
Stream<Airport> s;
s = k.getAirports();
s.forEach(this::toString);
}
}
我的问题是:如何优化此代码,因此我不必单独解析文件中的行,而是直接从源文件创建机场流?或者这是我能做到这一点的程度?
答案 0 :(得分:12)
您需要使用map()
来转换数据。
Files.lines(Paths.get("resources/planes.txt"))
.map(line -> createAirport(line));
这将返回Stream<Airport>
- 如果您想要返回List
,那么您最后需要使用collect
方法。
此方法也是无状态的,这意味着您不需要实例级airports
值。
您需要更新createAirport方法以返回一些内容:
public Airport createAirport(String line) {
String airport = line.substring(0, line.indexOf(';')).trim();
line = line.replace(airport + ";", "");
String country = line.substring(0,line.indexOf(';')).trim();
line = line.replace(country + ";", "");
String continent = line.substring(0,line.indexOf(';')).trim();
line = line.replace(continent + ";", "");
int length = Integer.parseInt(line.substring(0,line.indexOf(';')).trim());
return new Airport(airport, country, continent, length);
}
如果您正在寻找一种更具功能性的代码方法,您可能需要考虑重写createAirport
,这样就不会改变行。建筑商也很喜欢这种东西。
public Airport createAirport(final String line) {
final String[] fields = line.split(";");
return new Airport(fields[0].trim(),
fields[1].trim(),
fields[2].trim(),
Integer.parseInt(fields[3].trim()));
}
把它们全部扔在一起,你的班级现在看起来像这样。
public class AirportConsumer implements AirportAPI {
@Override
public Stream<Airport> getAirports() {
Stream<String> stream = null;
try {
stream = Files.lines(Paths.get("resources/planes.txt"))
.map(line -> createAirport(line));
} catch (IOException e) {
stream = Stream.empty();
e.printStackTrace();
}
return stream;
}
private Airport createAirport(final String line) {
final String[] fields = line.split(";");
return new Airport(fields[0].trim(),
fields[1].trim(),
fields[2].trim(),
Integer.parseInt(fields[3].trim()));
}
}
答案 1 :(得分:0)
史蒂夫发布的代码看起来很棒。但仍有两个地方可以改进: 1,如何拆分字符串。 2,如果人们忘记或不知道关闭通过调用getAirports()方法创建的流,则可能会导致问题。因此,最好完成任务(toList()或其他)。 这是AbacusUtil
的代码try(Reader reader = IOUtil.createBufferedReader(file)) {
List<Airport> airportList = Stream.of(reader).map(line -> {
String[] strs = Splitter.with(";").trim(true).splitToArray(line);
return Airport(strs[0], strs[1], strs[2], Integer.valueOf(strs[3]));
}).toList();
} catch (IOException e) {
throw new RuntimeException(e);
}
//或Try:
List<Airport> airportList = Try.stream(file).call(s -> s.map(line -> {
String[] strs = Splitter.with(";").trim(true).splitToArray(line);
return Airport(strs[0], strs[1], strs[2], Integer.valueOf(strs[3]));
}).toList())
披露:我是AbacusUtil的开发者。