java - 重构批量完成的文本文件解析

标签 java refactoring

我有三个独立的文件解析函数,它们将文本文件转换为对象,然后将这些值插入到 sqlite 数据库中。除了对象类别之外,它们基本上都是相同的。

流程如下:

  1. 使用 http 下载文件
  2. 计算文件中的行数以计算进度
  3. 删除目标表中之前的所有记录
  4. 使用 BufferedReader 打开文件
  5. 一次读取 2000 行并将其转换为对象
  6. 在事务中向 sqlite 中插入 2000 条记录
  7. 循环直到完成

我不知道如何使这段代码通用,以允许使用任何类来创建对象,然后决定使用哪个 DAL 函数来保存数据。 Java 不是我的第一语言,因此任何指导都会很棒。

这是我正在使用的代码:

public void downloadPendingPoleInspections() {

    int count;
    String filePath;
    Inspections inspections = Inspections.getInstance();

    filePath = Environment.getExternalStorageDirectory() + File.separator + "inspections.txt";

    try {

        downloadFile("http://localhost/api/inspectionservices.aspx?o=retrieve", "pendinginspections.txt", POST_PENDING_INSPECTIONS_PROGRESS_UPDATE);

        int totalInspections = getLineCount(filePath);

        inspections.deleteAllPendingInspections();          

        File file = new File(filePath);
        BufferedReader br = new BufferedReader(new FileReader(file));
        String line;

        int i = 0;
        int j = 0;

        List<PendingInspection> batch = new ArrayList<PendingInspection>();

        while ((line = br.readLine()) != null) {
            String[] values = line.split(" ");

            PendingInspection pending = new PendingInspection(
                    Integer.parseInt(values[0]), values[1],
                    Double.parseDouble(values[2]),
                    Double.parseDouble(values[3]));

            batch.add(pending);
            i++;
            j++;

            if (i >= 2000) {

                inspections.pendingInspectionsBatchInsert(batch);                   
                batch.clear();
                i = 0;                  
            }
        }

        if (i > 0) {
            inspections.pendingInspectionsBatchInsert(batch);
            batch.clear();
        }

        br.close();
        file.delete();

    } catch (Exception e) {
        Log.e("SyncActivity", e.toString());            
    }       
}

编辑:这是接口(interface)和类声明

public interface Inspectable {
    public int getId();
    public void setId(int id);

    public String getLabel();
    public void setLabel(String label);

    public double getX();
    public void setX(double x);

    public double getY();
    public void setY(double y);
}

public class RWInspection {
private String id;
private double x;
private double y;
private String inspector;
private String comments;
private String timestamp;


public RWInspection(String id, double x, double y, String inspector, String comments, String timestamp) {
        this.id = id;       
        this.x = x;
        this.y = y;
        this.inspector = inspector;
        this.comments = comments;
        this.timestamp = timestamp;
}

snip.... getter 和 setter 实现

public class PInspection implements Inspectable{
    private int id;
    private String number;
    private double x;
    private double y;

public PInspection(int id, String poleNumber, double x, double y) {
    this.id = id;
    this.number = number ;
    this.x = x;
    this.y = y;
}

最佳答案

我将其分为一个抽象基类和一些实现。基类可能如下所示:

public abstract class Downloader {
    protected abstract void processLine(String[] line);
    protected abstract void save();
    protected abstract String file();
public void downloadPendingPoleInspections() {

    int count;
    String filePath;

    filePath = Environment.getExternalStorageDirectory() + File.separator + file();

    try {

        downloadFile("http://localhost/api/inspectionservices.aspx?o=retrieve", "pending" + file(), POST_PENDING_INSPECTIONS_PROGRESS_UPDATE);

        int totalInspections = getLineCount(filePath);

        File file = new File(filePath);
        BufferedReader br = new BufferedReader(new FileReader(file));
        String line;

        int i = 0;
        int j = 0;


        while ((line = br.readLine()) != null) {
            processLine(line.split(" "));
            i++;
            j++;

            if (i >= 2000) {
                save()
                i = 0;                  
            }
        }

        if (i > 0) {
            save()
        }

        br.close();
        file.delete();

    } catch (Exception e) {
        Log.e("SyncActivity", e.toString());            
    }       
}

对于你想要处理的每种类型,你创建一个像这样的小实现:

public class InspectionDownloader extends DownLoader {
    Inspections inspections = Inspections.getInstance();
    List<PendingInspection> batch = new ArrayList<PendingInspection>();

    public InspectionDownloader() {
        inspections.deleteAllPendingInspections();
    }

    protected void processLine(String[] values) {
        PendingInspection pending = new PendingInspection(
            Integer.parseInt(values[0]), values[1],
            Double.parseDouble(values[2]),
            Double.parseDouble(values[3]));
        batch.add(pending);
    }

    protected void save() {
        inspections.pendingInspectionsBatchInsert(batch);
        batch.clear();
    }
    protected String file() { 
        return "inspections.txt";
    }
}

通过这种方式,人们可以将可重用逻辑集中在基类中,并将特殊逻辑转移到小型且集中的专门类中。这种模式称为模板化方法。您可以看到派生类非常关注其负责的类型的特殊操作。

关于java - 重构批量完成的文本文件解析,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/13162163/

相关文章:

java - 如何在java中读取对象(java.lang.Object)

java - 将类作为组件加载到java中

java - 哪个 JVM 具有完整的字节码执行跟踪?

iphone - 如何批量编辑/重构 XIB 文件的属性?

java - 解析 csv 文件并存储结果,因为 JFree 图表数据集消耗堆空间

java - ByteBuddy 附加到本地正在运行的进程

php - 有没有关于快速 PHP 重构的工具或技巧?

c# - 简化构建字典

refactoring - 哪些工具可以帮助翻译(如法语 -> 英语而不是 C++ -> java)源代码?

java - 使用 Java8 Optional<T> 重构遗留代码