A little file cleanup.

This commit is contained in:
GJ 2013-04-18 11:23:45 -04:00
parent 150fe730bd
commit c8c68ea7d9
4 changed files with 45 additions and 57 deletions

View File

@ -1,7 +1,9 @@
package com.gmail.nossr50.config; package com.gmail.nossr50.config;
import java.io.File; import java.io.File;
import java.io.FileNotFoundException;
import java.io.FileOutputStream; import java.io.FileOutputStream;
import java.io.IOException;
import java.io.InputStream; import java.io.InputStream;
import java.io.OutputStream; import java.io.OutputStream;
@ -51,30 +53,45 @@ public abstract class ConfigLoader {
InputStream inputStream = plugin.getResource(fileName); InputStream inputStream = plugin.getResource(fileName);
if (inputStream != null) { if (inputStream == null) {
try {
copyStreamToFile(inputStream, configFile);
}
catch (Exception e) {
e.printStackTrace();
}
}
else {
plugin.getLogger().severe("Missing resource file: '" + fileName + "' please notify the plugin authors"); plugin.getLogger().severe("Missing resource file: '" + fileName + "' please notify the plugin authors");
} return;
} }
private static void copyStreamToFile(InputStream inputStream, File file) throws Exception { OutputStream outputStream = null;
OutputStream outputStream = new FileOutputStream(file);
int read = 0; try {
outputStream = new FileOutputStream(configFile);
int read;
byte[] bytes = new byte[1024]; byte[] bytes = new byte[1024];
while ((read = inputStream.read(bytes)) != -1) { while ((read = inputStream.read(bytes)) != -1) {
outputStream.write(bytes, 0, read); outputStream.write(bytes, 0, read);
} }
}
inputStream.close(); catch (FileNotFoundException e) {
e.printStackTrace();
}
catch (IOException e) {
e.printStackTrace();
}
finally {
if (outputStream != null) {
try {
outputStream.close(); outputStream.close();
} }
catch (IOException e) {
e.printStackTrace();
}
}
try {
inputStream.close();
}
catch (IOException e) {
e.printStackTrace();
}
}
}
} }

View File

@ -626,15 +626,6 @@ public final class DatabaseManager {
} }
} }
finally { finally {
if (resultSet != null) {
try {
resultSet.close();
}
catch (SQLException e) {
// Ignore the error, we're leaving
}
}
if (statement != null) { if (statement != null) {
try { try {
statement.close(); statement.close();

View File

@ -1,6 +1,5 @@
package com.gmail.nossr50.util; package com.gmail.nossr50.util;
import java.util.HashSet;
import java.util.List; import java.util.List;
import org.bukkit.ChatColor; import org.bukkit.ChatColor;
@ -675,13 +674,7 @@ public class ItemUtils {
* @return true if the item is a miscellaneous drop, false otherwise * @return true if the item is a miscellaneous drop, false otherwise
*/ */
public static boolean isMiscDrop(ItemStack is) { public static boolean isMiscDrop(ItemStack is) {
HashSet<Material> miscItems = ItemWeightConfig.getInstance().getMiscItems(); return ItemWeightConfig.getInstance().getMiscItems().contains(is.getType());
if (miscItems.contains(is.getType())) {
return true;
} else {
return false;
}
} }
public static boolean isMcMMOItem(ItemStack is) { public static boolean isMcMMOItem(ItemStack is) {

View File

@ -6,9 +6,7 @@ import java.io.FileNotFoundException;
import java.io.FileOutputStream; import java.io.FileOutputStream;
import java.io.IOException; import java.io.IOException;
import java.io.InputStream; import java.io.InputStream;
import java.io.OutputStream;
import java.util.ArrayList; import java.util.ArrayList;
import java.util.jar.JarEntry;
import java.util.jar.JarFile; import java.util.jar.JarFile;
import org.bukkit.Material; import org.bukkit.Material;
@ -51,8 +49,7 @@ public class SpoutUtils {
* @param theFilePath The name of the file path * @param theFilePath The name of the file path
*/ */
private static void writeFile(String theFileName, String theFilePath) { private static void writeFile(String theFileName, String theFilePath) {
InputStream is = null; BufferedOutputStream os = null;
OutputStream os = null;
JarFile jar = null; JarFile jar = null;
try { try {
@ -64,8 +61,9 @@ public class SpoutUtils {
} }
jar = new JarFile(mcMMO.mcmmo); jar = new JarFile(mcMMO.mcmmo);
JarEntry entry = jar.getJarEntry("resources/" + theFileName);
is = jar.getInputStream(entry); @SuppressWarnings("resource")
InputStream is = jar.getInputStream(jar.getJarEntry("resources/" + theFileName));
byte[] buf = new byte[2048]; byte[] buf = new byte[2048];
int nbRead; int nbRead;
@ -75,8 +73,6 @@ public class SpoutUtils {
while ((nbRead = is.read(buf)) != -1) { while ((nbRead = is.read(buf)) != -1) {
os.write(buf, 0, nbRead); os.write(buf, 0, nbRead);
} }
os.flush();
} }
catch (FileNotFoundException e) { catch (FileNotFoundException e) {
e.printStackTrace(); e.printStackTrace();
@ -85,9 +81,9 @@ public class SpoutUtils {
e.printStackTrace(); e.printStackTrace();
} }
finally { finally {
if (is != null) { if (jar != null) {
try { try {
is.close(); jar.close();
} }
catch (IOException ex) { catch (IOException ex) {
ex.printStackTrace(); ex.printStackTrace();
@ -101,15 +97,6 @@ public class SpoutUtils {
ex.printStackTrace(); ex.printStackTrace();
} }
} }
if (jar != null) {
try {
jar.close();
}
catch (IOException ex) {
ex.printStackTrace();
}
}
} }
} }