В программе необходимо 2 раза использовать Printwriter. Первый работает второй нет?

Question

[pfedorov031090]

Пишу шаблонизатор. Есть большой шаблон в который надо вставить данные из файла, так нужно сделать столько раз, сколько строк в файле Data. Применяю Printwriter для записи в файл. Первый цикл, который записывает шапку шаблона отрабатывает исправно, второй ничего не пишет. Не могу разобраться почему. Пробовал flush(), создавать на второй цикл новый Printwriter. Не помогает. Может кто подскажет почему. Если вместо второго неработающего Printwriter цикла сделать вывод в консоль. данные выводятся корректно. т.е. дело именно в Printwriter.

import java.io.*;
import java.nio.file.Path;

public class Main {
    public static void main(String[] args) {
        String ShablonCurrentLine;
        String DataCurrentLine;
        String pathReadShablon = "...\\Shablon.csv";
        String pathReadData = "...\\data.csv";
        try {
            BufferedReader br = new BufferedReader(new FileReader(pathReadShablon)); //Читаем входной шаблон
            ShablonCurrentLine = br.readLine();
            String[] titleShablonLine = ShablonCurrentLine.split(";");
            BufferedReader br1 = new BufferedReader(new FileReader(pathReadData)); //Берем данные для подстановки
            while((DataCurrentLine = br1.readLine()) != null){// Построчно берем данные из Data
                String[] arrLineData = DataCurrentLine.split(","); // Разделяем строку на массив по разделителю
                String path = "C:\\..." + arrLineData[0] + ".csv";
                PrintWriter printWriter = new PrintWriter(path);
                for (int i = 0; i < titleShablonLine.length; i++) {
                    printWriter.print(titleShablonLine[i] + ";"); // Тут работает
                }
                printWriter.println();
                while((ShablonCurrentLine = br.readLine()) != null) {// Построчно берем данные из шаблона
                    String[] arrLineShablon = ShablonCurrentLine.split(";");
                    for (int i = 0; i < arrLineShablon.length; i++) {
                        arrLineShablon[i] = arrLineShablon[i].replaceAll("\\{\\{ station_code }}", arrLineData[1]);
                        arrLineShablon[i] = arrLineShablon[i].replaceAll("\\{\\{ station_ip_addr }}", arrLineData[2]);
                        arrLineShablon[i] = arrLineShablon[i].replaceAll("\\{\\{ camera_code }}", arrLineData[0]);
                        arrLineShablon[i] = arrLineShablon[i].replaceAll("\\{\\{ camera_ip_addr }}", arrLineData[3]);
                    }
                    for (int i = 0; i < arrLineShablon.length; i++) { 
                        printWriter.print(arrLineShablon[i] + ";");//Тут запись не работает
                    }
                    printWriter.println();
                }
                printWriter.close();
            }
            br.close();
            br1.close();
        } catch (Exception e) {
            e.printStackTrace();
        }
    }
}

Comments

[Евгений]

У вас внутри while в конце он закрывается.
`printWriter.close();`

[pfedorov031090]

Евгений, Все верно. Цикл в котором он закрывается создаст новый файл. Разве на следующем шаге цикла не создастся новый printwriter?
Сейчас на выходе программы получаю нужное количество файлов только с результатами работы первого printwriter, то есть заголовка.

[Dmitry Roo]

public static void main(String[] args) {
        String ShablonCurrentLine; //Переменные в Java пишуться с маленькой буквы.
        String DataCurrentLine; // Незачем объявлять переменную до ее использования.
        String pathReadShablon = "...\\Shablon.csv"; // "Шаблон", по английски, будет "template". pathReadShablon - колхозное имя.
        String pathReadData = "...\\data.csv";
        try {
            BufferedReader br = new BufferedReader(new FileReader(pathReadShablon));  // Используйте try-with-resources
            ShablonCurrentLine = br.readLine();
            String[] titleShablonLine = ShablonCurrentLine.split(";");
            BufferedReader br1 = new BufferedReader(new FileReader(pathReadData)); //почему br и br1 ? Как понять чем они отличаются?
            while((DataCurrentLine = br1.readLine()) != null){
                String[] arrLineData = DataCurrentLine.split(",");
                String path = "C:\\..." + arrLineData[0] + ".csv";
                PrintWriter printWriter = new PrintWriter(path);
                for (int i = 0; i < titleShablonLine.length; i++) {
                    printWriter.print(titleShablonLine[i] + ";");
                }
                printWriter.println();
                while((ShablonCurrentLine = br.readLine()) != null) { // Одна и таже переменная, но используется в разных местах? О_о
                    String[] arrLineShablon = ShablonCurrentLine.split(";");  // Вот тут какая длина у массива?
                    for (int i = 0; i < arrLineShablon.length; i++) {
                        arrLineShablon[i] = arrLineShablon[i].replaceAll("\\{\\{ station_code }}", arrLineData[1]);
                        arrLineShablon[i] = arrLineShablon[i].replaceAll("\\{\\{ station_ip_addr }}", arrLineData[2]);
                        arrLineShablon[i] = arrLineShablon[i].replaceAll("\\{\\{ camera_code }}", arrLineData[0]);
                        arrLineShablon[i] = arrLineShablon[i].replaceAll("\\{\\{ camera_ip_addr }}", arrLineData[3]);
                    }
                    for (int i = 0; i < arrLineShablon.length; i++) {
                        printWriter.print(arrLineShablon[i] + ";");// Зачем второй цикл? Почему не сделать все в предыдущем?..
                    }
                    printWriter.println();
                }
                printWriter.close();
            }
            br.close();
            br1.close();
        } catch (Exception e) {
            e.printStackTrace();
        }
    }

[pfedorov031090]

Dmitry Roo, с большей частью согласен.

while((ShablonCurrentLine = br.readLine()) != null) { // Одна и таже переменная, но используется в разных местах? О_о

эта переменная используется для текущей строки шаблона, т.е. в первом случае в ней заголовок, далее все строки по порядку

String[] arrLineShablon = ShablonCurrentLine.split(";"); // Вот тут какая длина у массива?

заранее не известно. зависит от шаблона

[acwartz]

Dmitry Roo, pfedorov031090,
Что-то автор намудрил с кодом.

У форматирования строки есть же взятие аргумента по его индексу, все эти replaceAll'ы можно заменить одной строкой:
String.format(" %1$s;%2$s;%3$s;%4$s; ", station_code, station_addr, camera_code, camera_ip );

каждый %n$s соответствует аргументу по его порядку передачи, в строке форматирования переставляйте их как хотите и итог будет ожидаемо верный И не надо никаких replaceAll'ов.
Формирование шапки я бы тоже отдал на откуп коду, при том в тот же шаблон.

String template = "%1$s;%2$s;%3$s;%4$s;\n";
myStringBuilder.add(String.format(template, "\"station_code\"", "\"station_addr\"", "\"camera_code\"", "\"camera_ip\""); )
for(i ...) {
  myStringBuilder.add(String.format(template, station_code[]i, station_addr[i],  camera_code[i], camera_ip[i]  ); )
}
myStringBuilder.save(...)

и CSV готов.

[pfedorov031090]

acwartz, Спасибо большое за ответ! Я наверное неверно выразился, я только учусь разработке и основная цель не получить готовый рабочий код, а разобраться с потоками и Printwriter. Я хочу понять почему не работает именно мой (не самый качественный) код. Буду признателен, если объясните.

[Dmitry Roo]

pfedorov031090, "Вот тут какая длина у массива?" этот вопрос не просто так. У вас, возможно, массив пустой просто - поэтому и не пишет ничего ваш printWriter

[pfedorov031090]

Dmitry Roo, он не пустой, там есть проверка на ненулевую строку, плюс если вместо последнего printwriter сделать вывод на консоль, то данные на консоль выведутся корректно.

[Dmitry Roo]

pfedorov031090, ну добавьте какие-нибудь тестовые данные в вопрос тогда. Даже интересно стало...