From 3486d39fe7c61800d6cf33e42d881d03936d99b9 Mon Sep 17 00:00:00 2001 From: Jordan Doyle Date: Mon, 13 Apr 2015 01:27:52 +0100 Subject: [PATCH] Never ended up using setField with method chaining as I originally intended so the return isn't needed --- target/test-classes/wf/doyle/blockbuster/AppTest.class | 0 src/test/java/wf/doyle/blockbuster/AppTest.java | 5 ++--- target/classes/wf/doyle/blockbuster/file/FileReader.class | 0 src/main/java/wf/doyle/blockbuster/file/FileReader.java | 11 +++++++---- 4 files changed, 7 insertions(+), 9 deletions(-) diff --git a/target/test-classes/wf/doyle/blockbuster/AppTest.class b/target/test-classes/wf/doyle/blockbuster/AppTest.class index f01aa7b8127106fbed1a55c624b2b5c670902afb..226d3f364d845c35a3e327980cd2c4bfc6c136de 100644 Binary files a/target/test-classes/wf/doyle/blockbuster/AppTest.class and a/target/test-classes/wf/doyle/blockbuster/AppTest.class differ diff --git a/src/test/java/wf/doyle/blockbuster/AppTest.java b/src/test/java/wf/doyle/blockbuster/AppTest.java index b953dcd..130ffed 100644 --- a/src/test/java/wf/doyle/blockbuster/AppTest.java +++ a/src/test/java/wf/doyle/blockbuster/AppTest.java @@ -24,13 +24,12 @@ Assert.assertNotNull(f); - LibraryItem i = FileReader.setField(f, item, "testCase"); + FileReader.setField(f, item, "testCase"); try { - Assert.assertEquals(f.get(i), "testCase"); + Assert.assertEquals(f.get(item), "testCase"); } catch(Exception e) { Assert.fail("Getting value of String using reflection failed"); - e.printStackTrace(System.err); } } } diff --git a/target/classes/wf/doyle/blockbuster/file/FileReader.class b/target/classes/wf/doyle/blockbuster/file/FileReader.class index e9b051dc910280401f94bdd51fb0ce76098e3d00..f180e102877e32249e118b6c73cf98882aa52be6 100644 Binary files a/target/classes/wf/doyle/blockbuster/file/FileReader.class and a/target/classes/wf/doyle/blockbuster/file/FileReader.class differ diff --git a/src/main/java/wf/doyle/blockbuster/file/FileReader.java b/src/main/java/wf/doyle/blockbuster/file/FileReader.java index 7b5a0b8..f83e87f 100644 --- a/src/main/java/wf/doyle/blockbuster/file/FileReader.java +++ a/src/main/java/wf/doyle/blockbuster/file/FileReader.java @@ -203,9 +203,8 @@ * instance to set the value in * @param data * data to set the value as - * @return instance of LibraryItem */ - public static LibraryItem setField(Field f, LibraryItem instance, String data) + public static void setField(Field f, LibraryItem instance, String data) { try { @@ -218,7 +217,7 @@ f.setAccessible(true); f.set(instance, val); - return instance; + return; } } catch(NumberFormatException nfe) @@ -232,7 +231,7 @@ f.setAccessible(true); f.set(instance, b); - return instance; + return; } f.setAccessible(true); @@ -243,7 +242,7 @@ App.LOGGER.error("Couldn't set value of " + f.getName() + " in " + instance.getClass().getName(), e); } - return instance; + return; } /** @@ -292,7 +291,7 @@ Field field = getField(this.lineContains[key].trim(), itemClass); - item = setField(field, item, value); + FileReader.setField(field, item, value); } // add this storage class to our main array -- rgit 0.1.5