Mercurial > 510Connectbot
diff src/org/tn5250j/encoding/BuiltInCodePageFactory.java @ 112:77ac18bc1b2f
cleanup java formatting
author | Carl Byington <carl@five-ten-sg.com> |
---|---|
date | Wed, 18 Jun 2014 13:03:01 -0700 |
parents | b29b39f386a4 |
children |
line wrap: on
line diff
--- a/src/org/tn5250j/encoding/BuiltInCodePageFactory.java Wed Jun 18 13:00:19 2014 -0700 +++ b/src/org/tn5250j/encoding/BuiltInCodePageFactory.java Wed Jun 18 13:03:01 2014 -0700 @@ -60,90 +60,97 @@ */ class BuiltInCodePageFactory { private static final String TAG = "BuiltInCodePageFactory"; - private static BuiltInCodePageFactory singleton; + private static BuiltInCodePageFactory singleton; - private final List<Class<?>> clazzes = new ArrayList<Class<?>>(); + private final List<Class<?>> clazzes = new ArrayList<Class<?>>(); - private BuiltInCodePageFactory() { - register(); - } + private BuiltInCodePageFactory() { + register(); + } - public static synchronized final BuiltInCodePageFactory getInstance() { - if (singleton == null) { - singleton = new BuiltInCodePageFactory(); - } - return singleton; - } + public static synchronized final BuiltInCodePageFactory getInstance() { + if (singleton == null) { + singleton = new BuiltInCodePageFactory(); + } + + return singleton; + } - private void register() { - clazzes.add(CCSID37.class); - clazzes.add(CCSID273.class); - clazzes.add(CCSID277.class); - clazzes.add(CCSID278.class); - clazzes.add(CCSID280.class); - clazzes.add(CCSID284.class); - clazzes.add(CCSID285.class); - clazzes.add(CCSID297.class); - clazzes.add(CCSID424.class); - clazzes.add(CCSID500.class); - clazzes.add(CCSID870.class); - clazzes.add(CCSID871.class); - clazzes.add(CCSID875.class); - clazzes.add(CCSID1025.class); - clazzes.add(CCSID1026.class); - clazzes.add(CCSID1112.class); - clazzes.add(CCSID1140.class); - clazzes.add(CCSID1141.class); - clazzes.add(CCSID1147.class); - clazzes.add(CCSID1148.class); - } + private void register() { + clazzes.add(CCSID37.class); + clazzes.add(CCSID273.class); + clazzes.add(CCSID277.class); + clazzes.add(CCSID278.class); + clazzes.add(CCSID280.class); + clazzes.add(CCSID284.class); + clazzes.add(CCSID285.class); + clazzes.add(CCSID297.class); + clazzes.add(CCSID424.class); + clazzes.add(CCSID500.class); + clazzes.add(CCSID870.class); + clazzes.add(CCSID871.class); + clazzes.add(CCSID875.class); + clazzes.add(CCSID1025.class); + clazzes.add(CCSID1026.class); + clazzes.add(CCSID1112.class); + clazzes.add(CCSID1140.class); + clazzes.add(CCSID1141.class); + clazzes.add(CCSID1147.class); + clazzes.add(CCSID1148.class); + } + + /** + * @return unsorted list of available code pages + */ + public String[] getAvailableCodePages() { + HashSet<String> cpset = new HashSet<String>(); + + for (Class<?> clazz : clazzes) { + final ICodepageConverter converter = getConverterFromClassName(clazz); - /** - * @return unsorted list of available code pages - */ - public String[] getAvailableCodePages() { - HashSet<String> cpset = new HashSet<String>(); - for (Class<?> clazz : clazzes) { - final ICodepageConverter converter = getConverterFromClassName(clazz); - if (converter != null) { - cpset.add(converter.getName()); - } - } - return cpset.toArray(new String[cpset.size()]); - } + if (converter != null) { + cpset.add(converter.getName()); + } + } + + return cpset.toArray(new String[cpset.size()]); + } + + /** + * @param encoding + * @return an {@link ICodePage} object OR null, of not found + */ + public ICodePage getCodePage(String encoding) { + for (Class<?> clazz : clazzes) { + final ICodepageConverter converter = getConverterFromClassName(clazz); + + if (converter != null && converter.getName().equals(encoding)) { + return converter; + } + } - /** - * @param encoding - * @return an {@link ICodePage} object OR null, of not found - */ - public ICodePage getCodePage(String encoding) { - for (Class<?> clazz : clazzes) { - final ICodepageConverter converter = getConverterFromClassName(clazz); - if (converter != null && converter.getName().equals(encoding)) { - return converter; - } - } - return null; - } + return null; + } - /** - * Lazy loading converters takes time, - * but doesn't happen so often and saves memory. - * - * @param clazz {@link ICodepageConverter} - * @return - */ - private ICodepageConverter getConverterFromClassName(Class<?> clazz) { - try { - final Constructor<?> constructor = clazz.getConstructor(new Class[0]); - final ICodepageConverter converter = (ICodepageConverter) constructor.newInstance(); - converter.init(); - return converter; - } catch (Exception e) { - Log.e(TAG,"Couldn't load code page converter class:" + clazz.getCanonicalName(), e); - return null; - } - } + /** + * Lazy loading converters takes time, + * but doesn't happen so often and saves memory. + * + * @param clazz {@link ICodepageConverter} + * @return + */ + private ICodepageConverter getConverterFromClassName(Class<?> clazz) { + try { + final Constructor<?> constructor = clazz.getConstructor(new Class[0]); + final ICodepageConverter converter = (ICodepageConverter) constructor.newInstance(); + converter.init(); + return converter; + } + catch (Exception e) { + Log.e(TAG, "Couldn't load code page converter class:" + clazz.getCanonicalName(), e); + return null; + } + } }