Skip to content

Comments

kkorolev-container#115

Open
kkorolev wants to merge 4 commits intoarhangeldim:masterfrom
kkorolev:kkorolev-container
Open

kkorolev-container#115
kkorolev wants to merge 4 commits intoarhangeldim:masterfrom
kkorolev:kkorolev-container

Conversation

@kkorolev
Copy link

@kkorolev kkorolev commented Nov 6, 2016

No description provided.

*/
public class Container {

private Map<String, Object> objByName; // <id, object[Car, Engine, Gear]>; id = name
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Слишком много мап, похоже что-то дублируется

field.setAccessible(true);
switch (prop.getType()) {
case VAL:
int value = Integer.parseInt(prop.getValue()); // !!! ONLY FOR INTEGER VALUES
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Нужно не только Integer

}
isUpdated.put(className, Boolean.TRUE);
return obj;
} catch (IllegalAccessException |
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

catch Exception

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants