Цикл For печатает неправильно.

То, что я пытаюсь сделать, это взять java-файл, который выглядит так

public class test1 {
public static void main ( string[] args ) {
System.out.println( "This is Test 1." );
}
}

И предполагается, что выводить текстовый файл с правильными интервалами и отступами. Пока я могу получить правильный отступ для первых строк. Но у меня проблемы с моим вторым циклом for, который печатает пробелы для конечных скобок. Конечные скобки печатаются наружу, как первые 3 строки, а не внутрь. Извините, если мои переменные сбивают с толку.

Вот мой код до сих пор

public class JavaJustifier {
public static void main( String[] args ) 
           throws FileNotFoundException {

    for( int i = 1; i < 6; i++ ) {
        justifyJava( "Test" + i + ".java", 
                     "Justified" + i + ".txt", 
                     4 );
    }                
}
public static void justifyJava( String inputFileName,
                                String outputFileName,
                                int tabSize ) 
        throws FileNotFoundException {


    int counter = 0;
    int counter2 = 0;
    int blah = 0;
    File f = new File(inputFileName);
    File p = new File(outputFileName);
    if  (p.exists())
        p.delete();
    Scanner input = new Scanner (f);
    PrintStream name = new PrintStream(new File(outputFileName));

    while (input.hasNextLine()) {
        String line = input.nextLine();
        Scanner lineScan = new Scanner(line);
        if (line.contains("{") == true) {
            name.print("{\r\n");
            counter++;
            for (int i = 1; i <= counter; i++) {
                for (int j = 0; j <= tabSize; j++) {
                    name.print(" ");
                }
            }
            System.out.println(counter);
        } else if (line.contains("}") == true) {
            name.print("\r\n");
            counter--;
            for (int x = 1; x <= counter; x++) {
                for (int y = 1; y <= tabSize; y++) {
                    name.print(" ");
                }
            }
            name.print("}");
            System.out.println(counter);
        } else {
            name.print(line);
        }
    }
}

Что это дает мне

public class Test1 
{
    public static void main( String[] args ) 
    {
        System.out.println( "This is Test 1." );
}
    }

Чего я желаю, так это

public class Test1 
{
    public static void main( String[] args ) 
    {
        System.out.println( "This is Test 1." );
    }
}

person Brent    schedule 16.05.2013    source источник
comment
Что вы ожидаете печатать и как это сейчас печатается с примерами?   -  person Smit    schedule 16.05.2013
comment
Не пишите if (line.contains("{") == true) {, просто пишите if (line.contains("{")) {   -  person jlordo    schedule 16.05.2013
comment
Нам нужно увидеть ваш входной файл, текущий вывод и желаемый результат.   -  person BLuFeNiX    schedule 16.05.2013
comment
Не забудьте сделать несколько дополнительных проверок фигурных скобок внутри строковых литералов, иначе ваше форматирование станет некрасивым.   -  person mschenk74    schedule 18.05.2013
comment
Я не знаю, как вы получаете этот вывод, потому что, говоря if (line.contains("{") == true) {, вы в основном игнорируете остальную часть текста и обрабатываете только {.   -  person zEro    schedule 20.05.2013


Ответы (1)


Хорошо, я провел рефакторинг нескольких вещей (и это было необходимо), но он работает именно так, как вы ожидаете (насколько я знаю!).

Вот код:

public class JavaJustifier {
    static int counter = 0;
    static int tabSize = 4;

    public static void main(String[] args) throws FileNotFoundException {
        justifyJava("Test1.java", "Justified1.txt");
    }

    private static void processLine(PrintStream name, String line) {
        if (line.contains("{")) {
            String preText = line.substring(0, line.indexOf("{"));
            String postText = line.substring(line.indexOf("{") + 1)
                    .replaceAll("\\n", "").replaceAll("\\r\\n", "");
            printSpaces(name);
            name.println(preText);
            printSpaces(name);
            name.println("{");
            counter++;
            processLine(name, postText);

        } else if (line.contains("}")) {
            String preText = line.substring(0, line.indexOf("}"));
            String postText = line.substring(line.indexOf("}") + 1)
                    .replaceAll("\\n", "").replaceAll("\\r\\n", "");
            name.println(preText);
            counter--;
            printSpaces(name);
            name.println("}");
            processLine(name, postText);
        } else {
            if (!line.equals("\r\n") && line.length() != 0) {
                printSpaces(name);
                name.print(line);
            }
        }
    }

    private static void printSpaces(PrintStream p) {
        for (int i = 1; i <= counter; i++) {
            for (int j = 0; j <= tabSize; j++) {
                p.print(" ");
            }
        }
    }

    public static void justifyJava(String inputFileName, String outputFileName)
            throws FileNotFoundException {

        File f = new File(inputFileName);
        File p = new File(outputFileName);
        if (p.exists())
            p.delete();
        Scanner input = new Scanner(f);
        PrintStream name = new PrintStream(new File(outputFileName));

        while (input.hasNextLine()) {
            String line = input.nextLine().trim();
            processLine(name, line);
        }
    }
}
person zEro    schedule 20.05.2013
comment
Проблема в том, что это большая проблема, но я не могу изменить ни один из своих основных методов, и единственный код, который я могу добавить, — это после выбрасывания FileNotFoundException. - person Brent; 21.05.2013
comment
@BrentColeman Было бы глупо не менять код или дизайн, когда мы знаем, что это неправильно. Вам нужно запомнить изменения в counter, и единственный способ, который я могу придумать в вашей текущей настройке, - это сделать его статической переменной. Что-нибудь еще, что мы попробуем, и мы еще больше отклонимся от вашего шаблона. Измените дизайн — если нужно — чтобы все было сделано правильно!! - person zEro; 22.05.2013