Want to win a PS4? Go Premium and enter to win our High-Tech Treats giveaway. Enter to Win

x
?
Solved

Mapping Suggestion needed

Posted on 2011-03-16
11
Medium Priority
?
246 Views
Last Modified: 2012-05-11
I have a table which contains 3 columns id,key,value. The data in the table looks like this :-

1)1,userId,test
2)1,firstName,foo
3)1, lastName, foo Test

Now when i query the table  :-

List<Property> values = hibernateDao.findById("xx");

I get list of values as i mentioned above. I can do this :-

User user = new User();
foreach(Property prop : values ) {
if(prop.getKey().equalsIgnoreCase("userId") {
user.setUserId(prop.getValue()):
}
if(prop.getKey().equalsIgnoreCase("firstName") {
user.setFirstName(prop.getValue());
}
.
.
}
return user;

This works fine but i am trying to find more efficient way of doing it instead of using nested if statements.
0
Comment
Question by:Micheal_Male
[X]
Welcome to Experts Exchange

Add your voice to the tech community where 5M+ people just like you are talking about what matters.

  • Help others & share knowledge
  • Earn cash & points
  • Learn & ask questions
  • 5
  • 4
  • 2
11 Comments
 
LVL 47

Expert Comment

by:for_yan
ID: 35153023
And what is Property ?
Is it a type defined by Hibernate ?
0
 

Author Comment

by:Micheal_Male
ID: 35153107
Its an entity class
0
 
LVL 47

Expert Comment

by:for_yan
ID: 35153116
But who defines it? Does it implement Comparable?
0
What does it mean to be "Always On"?

Is your cloud always on? With an Always On cloud you won't have to worry about downtime for maintenance or software application code updates, ensuring that your bottom line isn't affected.

 

Author Comment

by:Micheal_Male
ID: 35153146
It's just a class itself which holds field names id,key and value. It does not implement comparable.
0
 

Author Comment

by:Micheal_Male
ID: 35153158

@Entity
@Table(name="property")
public class Property {

@OneToMany(mappedBy="property")
@JoinColumn("id")
private String id;
@Column(name="key")
private String key;
@Column(name="value")
private String value;

Open in new window

0
 
LVL 47

Expert Comment

by:for_yan
ID: 35153164

If you defined it in your code, then you can implement Comparable
and define compareTo method
in such a way that Collections.sort(values) will produce the known order of keys,
then you would not need to check with if's but could know that fiirst element corresponds
say to userid, second to first name, etc

0
 

Author Comment

by:Micheal_Male
ID: 35153170
hmm that sounds a little dangerous if someone else adds a new key or it might break by putting the wrong key value in user pojo.
0
 
LVL 47

Expert Comment

by:for_yan
ID: 35153286
That is true.
If I were you, I would go for if's, as you mentioned in your original post; don't see any problems with those.
I just suggested, as you were looking for possible alternatives.  
0
 
LVL 5

Expert Comment

by:josephtsang
ID: 35158441
what about refactoring your code with the Visitor design pattern?
0
 
LVL 5

Accepted Solution

by:
josephtsang earned 1000 total points
ID: 35162738
Work in this way:

1. Replace all lines inside the for-loop by these 2 lines


Setter<User> setter = setterMap.get(prop.getKey());
setter.set(user, prop.getValue())

Open in new window


2. Before entering the for-loop, prepare a setter map as follows


Map<String,Setter<User>> setterMap = new HashMap<String,Setter<User>>();
setterMap.put( "userId", new Setter<User> { 
	public void set(User user, String value) { user.setUserId(value); } });
setterMap.put( "firstName", new Setter<User> { 
	public void set(User user, String value) { user.setFirstName(value); } });
setterMap.put( "lastName", new Setter<User> { 
	public void set(User user, String value) { user.setLastId(value); } });

Open in new window


3. Define a private inner interface Setter in your class


private interface Setter<T> { 
	public void set(T t, String value); 
}

Open in new window


So if you have defined any new key-value pairs for the records, you just need to put a new entry into the map in Step 2. Others just keep the same. Less code change for the enhancement.
0
 

Author Comment

by:Micheal_Male
ID: 35162747
Great. However I just found a mapping framework also. Hopefully that will suffix my needs.
0

Featured Post

Hire Technology Freelancers with Gigs

Work with freelancers specializing in everything from database administration to programming, who have proven themselves as experts in their field. Hire the best, collaborate easily, pay securely, and get projects done right.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

This was posted to the Netbeans forum a Feb, 2010 and I also sent it to Verisign. Who didn't help much in my struggles to get my application signed. ------------------------- Start The idea here is to target your cell phones with the correct…
Introduction This article is the second of three articles that explain why and how the Experts Exchange QA Team does test automation for our web site. This article covers the basic installation and configuration of the test automation tools used by…
Viewers learn about the third conditional statement “else if” and use it in an example program. Then additional information about conditional statements is provided, covering the topic thoroughly. Viewers learn about the third conditional statement …
Viewers will learn one way to get user input in Java. Introduce the Scanner object: Declare the variable that stores the user input: An example prompting the user for input: Methods you need to invoke in order to properly get  user input:
Suggested Courses

636 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question