Skip to content

Commit f13f406

Browse files
committed
remove common io library and add more attributes in default order
1 parent 6b83995 commit f13f406

File tree

6 files changed

+54
-90
lines changed

6 files changed

+54
-90
lines changed

.androidide/editor/openedFiles.json

Lines changed: 0 additions & 50 deletions
This file was deleted.

app/build.gradle

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@ repositories {
88

99
dependencies {
1010
implementation 'commons-cli:commons-cli:1.4'
11-
implementation 'commons-io:commons-io:2.11.0'
1211
implementation 'org.apache.commons:commons-lang3:3.12.0'
1312
implementation 'org.jdom:jdom:1.1.3'
1413
}

app/src/main/java/com/bytehamster/androidxmlformatter/Main.java

Lines changed: 16 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
package com.bytehamster.androidxmlformatter;
22

3-
import com.bytehamster.androidxmlformatter.utils.FileUtil;
3+
import com.bytehamster.androidxmlformatter.utils.FileUtils;
44
import com.bytehamster.androidxmlformatter.utils.FormatterOptions;
55
import java.io.File;
66
import java.io.FileInputStream;
@@ -40,7 +40,6 @@ private int run(String[] args) throws Exception {
4040
}
4141

4242
List<File> files = new ArrayList<>();
43-
4443
for (String arg : cmd.getArgList()) {
4544
if (arg.equals("-")) return 1;
4645

@@ -55,24 +54,28 @@ private int run(String[] args) throws Exception {
5554
return 1;
5655
}
5756

58-
int formatted = 0;
57+
int formattedCount = 0;
5958
for (File file : files) {
60-
formatted += formatFile(file, cmd) ? 1 : 0;
59+
if (formatFile(file, cmd)) {
60+
formattedCount += 1;
61+
}
6162
}
62-
System.out.println(formatted + " files formated!");
63+
System.out.println(formattedCount + " files formated!");
6364

6465
return 0;
6566
}
6667

6768
private void addFilesFromDir(List<File> formatFiles, File file) {
68-
if (file.isFile()) {
69-
if (file.getName().endsWith(".xml")) {
70-
formatFiles.add(file);
69+
if (file.isDirectory()) {
70+
for (File f : file.listFiles()) {
71+
addFilesFromDir(formatFiles, f);
7172
}
7273
return;
7374
}
7475

75-
for (File f : file.listFiles()) addFilesFromDir(formatFiles, f);
76+
if (file.getName().endsWith(".xml")) {
77+
formatFiles.add(file);
78+
}
7679
}
7780

7881
private boolean formatFile(File file, CommandLine cmd) {
@@ -86,11 +89,12 @@ private boolean formatFile(File file, CommandLine cmd) {
8689
cmd.hasOption(FormatterOptions.OPT_NAMESPACE_SORT)
8790
);
8891

89-
FileUtil.writeFile(file, outputter.outputString(new SAXBuilder().build(new FileInputStream(FileUtil.getFilePath(file)))).trim());
90-
System.out.println("Done formatting: " + FileUtil.getFilePath(file));
92+
String formatedText = outputter.outputString(new SAXBuilder().build(new FileInputStream(file))).trim();
93+
FileUtils.writeFile(file, formatedText );
94+
System.out.println("Done formatting: " + file);
9195
return true;
9296
} catch (Exception e) {
93-
System.out.println("Error formatting: " + FileUtil.getFilePath(file) + ". Exception: " + e.getMessage());
97+
System.out.println("Error formatting: " + file + ". Exception: " + e.getMessage());
9498
return false;
9599
}
96100
}

app/src/main/java/com/bytehamster/androidxmlformatter/utils/FileUtil.java

Lines changed: 0 additions & 25 deletions
This file was deleted.
Lines changed: 36 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,36 @@
1+
package com.bytehamster.androidxmlformatter.utils;
2+
3+
import java.io.File;
4+
import java.io.IOException;
5+
import java.io.FileOutputStream;
6+
import java.io.FileInputStream;
7+
8+
public class FileUtils {
9+
10+
public static void writeFile(File file, String content) {
11+
try (FileOutputStream outputStream = new FileOutputStream(file)) {
12+
13+
byte[] bytes = content.getBytes();
14+
15+
outputStream.write(bytes);
16+
} catch (IOException e) {
17+
e.printStackTrace();
18+
}
19+
}
20+
21+
public static final String readFile(File file) {
22+
try (FileInputStream inputStream = new FileInputStream(file)) {
23+
return new String(inputStream.readAllBytes());
24+
} catch (IOException ioe) {
25+
return "";
26+
}
27+
}
28+
29+
public static String getFilePath(File file) {
30+
try {
31+
return file.getCanonicalPath();
32+
} catch (Exception e) {
33+
return file.getAbsolutePath();
34+
}
35+
}
36+
}

app/src/main/java/com/bytehamster/androidxmlformatter/utils/FormatterOptions.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,8 +12,8 @@ public class FormatterOptions {
1212
public static final String OPT_NAMESPACE_SORT = "namespace-sort";
1313

1414
public static final String DEFAULT_INDENTION = "2";
15-
public static final String DEFAULT_ATTRIBUTE_ORDER = "id,layout_width,layout_height";
16-
public static final String DEFAULT_NAMESPACE_ORDER = "android,app";
15+
public static final String DEFAULT_ATTRIBUTE_ORDER = "id,style,layout_width,layout_height,layout_margin,layout_marginLeft,layout_marginStart,layout_marginTop,layout_marginRight,layout_marginEnd,layout_marginBottom";
16+
public static final String DEFAULT_NAMESPACE_ORDER = "xmlns,android,app,tools";
1717

1818
public static Options getOptions(String[] args) {
1919
Options options = new Options();

0 commit comments

Comments
 (0)