Speeding up object name check by record ID
.everyoneloves__top-leaderboard:empty,.everyoneloves__mid-leaderboard:empty{ margin-bottom:0;
}
up vote
2
down vote
favorite
I have been able to piece together some code that determines object name based on record ID without any hard-coding or SOQL. It works with both custom and standard objects but it is very slow, I'm guessing due to loading so much schema data and a lot of looping.
I am looking for a way to speed this function up, potentially by limiting the object names searched or the amount of data being sorted through.
Here is the entire (working, but slow) function:
// Get sObject type by record ID
public static String getObjName(String recordId) {
// Get record prefix
String recordPrefix = recordId.substring(0, 3);
// Get global "describe"
Map<String, Schema.SObjectType> gd = Schema.getGlobalDescribe();
// Loop through all objects to find match for record ID prefix
for (Schema.SObjectType otype : gd.values()) {
Schema.DescribeSObjectResult r = otype.getDescribe();
String prefix = r.getKeyPrefix();
// Found match!
if (prefix != null && prefix.equals(recordPrefix)) return r.getName();
}
return '';
}
Any ideas?
apex
New contributor
add a comment |
up vote
2
down vote
favorite
I have been able to piece together some code that determines object name based on record ID without any hard-coding or SOQL. It works with both custom and standard objects but it is very slow, I'm guessing due to loading so much schema data and a lot of looping.
I am looking for a way to speed this function up, potentially by limiting the object names searched or the amount of data being sorted through.
Here is the entire (working, but slow) function:
// Get sObject type by record ID
public static String getObjName(String recordId) {
// Get record prefix
String recordPrefix = recordId.substring(0, 3);
// Get global "describe"
Map<String, Schema.SObjectType> gd = Schema.getGlobalDescribe();
// Loop through all objects to find match for record ID prefix
for (Schema.SObjectType otype : gd.values()) {
Schema.DescribeSObjectResult r = otype.getDescribe();
String prefix = r.getKeyPrefix();
// Found match!
if (prefix != null && prefix.equals(recordPrefix)) return r.getName();
}
return '';
}
Any ideas?
apex
New contributor
add a comment |
up vote
2
down vote
favorite
up vote
2
down vote
favorite
I have been able to piece together some code that determines object name based on record ID without any hard-coding or SOQL. It works with both custom and standard objects but it is very slow, I'm guessing due to loading so much schema data and a lot of looping.
I am looking for a way to speed this function up, potentially by limiting the object names searched or the amount of data being sorted through.
Here is the entire (working, but slow) function:
// Get sObject type by record ID
public static String getObjName(String recordId) {
// Get record prefix
String recordPrefix = recordId.substring(0, 3);
// Get global "describe"
Map<String, Schema.SObjectType> gd = Schema.getGlobalDescribe();
// Loop through all objects to find match for record ID prefix
for (Schema.SObjectType otype : gd.values()) {
Schema.DescribeSObjectResult r = otype.getDescribe();
String prefix = r.getKeyPrefix();
// Found match!
if (prefix != null && prefix.equals(recordPrefix)) return r.getName();
}
return '';
}
Any ideas?
apex
New contributor
I have been able to piece together some code that determines object name based on record ID without any hard-coding or SOQL. It works with both custom and standard objects but it is very slow, I'm guessing due to loading so much schema data and a lot of looping.
I am looking for a way to speed this function up, potentially by limiting the object names searched or the amount of data being sorted through.
Here is the entire (working, but slow) function:
// Get sObject type by record ID
public static String getObjName(String recordId) {
// Get record prefix
String recordPrefix = recordId.substring(0, 3);
// Get global "describe"
Map<String, Schema.SObjectType> gd = Schema.getGlobalDescribe();
// Loop through all objects to find match for record ID prefix
for (Schema.SObjectType otype : gd.values()) {
Schema.DescribeSObjectResult r = otype.getDescribe();
String prefix = r.getKeyPrefix();
// Found match!
if (prefix != null && prefix.equals(recordPrefix)) return r.getName();
}
return '';
}
Any ideas?
apex
apex
New contributor
New contributor
New contributor
asked 7 hours ago
Ixalmida
1113
1113
New contributor
New contributor
add a comment |
add a comment |
1 Answer
1
active
oldest
votes
up vote
6
down vote
The Id
class already contains an instance method to do exactly what you wish, named getSObjectType
. The entire body of your method can be simplified to the below:
return (recordId == null) ? '' : String.valueOf(recordId.getSObjectType());
Indeed...that worked, thanks! I hope my question makes this answer easier for others to find because Google was not helping me at all.
– Ixalmida
7 hours ago
1
You should mark it as "Accepted" (The tick next to the number of votes) so it comes up properly in searches;
– Sebastian Kessel
6 hours ago
1
First...it wouldn't let me mark as accepted for 10 minutes or whatever, so I was waiting. Second...I just figured out for myself that this answer actually won't work because it mixes return types (String / Schema.SObjectType). The correct answer is actually:return (recordId == null) ? '' : recordId.getSObjectType().getDescribe().getName();
I will accept the answer when it is corrected.
– Ixalmida
6 hours ago
1
You can just useString.valueOf
, which should be much faster. Depending on your use case, it would actually be much better to just change your return type toSObjectType
. Static types are preferable toString
where they fit.
– Adrian Larson♦
6 hours ago
1
@AdrianLarson you can also justreturn recordId == null? '': ''+recordId.getSobjectType()
; the string + operator takes care of the String.valueOf for you.
– sfdcfox
2 hours ago
|
show 2 more comments
1 Answer
1
active
oldest
votes
1 Answer
1
active
oldest
votes
active
oldest
votes
active
oldest
votes
up vote
6
down vote
The Id
class already contains an instance method to do exactly what you wish, named getSObjectType
. The entire body of your method can be simplified to the below:
return (recordId == null) ? '' : String.valueOf(recordId.getSObjectType());
Indeed...that worked, thanks! I hope my question makes this answer easier for others to find because Google was not helping me at all.
– Ixalmida
7 hours ago
1
You should mark it as "Accepted" (The tick next to the number of votes) so it comes up properly in searches;
– Sebastian Kessel
6 hours ago
1
First...it wouldn't let me mark as accepted for 10 minutes or whatever, so I was waiting. Second...I just figured out for myself that this answer actually won't work because it mixes return types (String / Schema.SObjectType). The correct answer is actually:return (recordId == null) ? '' : recordId.getSObjectType().getDescribe().getName();
I will accept the answer when it is corrected.
– Ixalmida
6 hours ago
1
You can just useString.valueOf
, which should be much faster. Depending on your use case, it would actually be much better to just change your return type toSObjectType
. Static types are preferable toString
where they fit.
– Adrian Larson♦
6 hours ago
1
@AdrianLarson you can also justreturn recordId == null? '': ''+recordId.getSobjectType()
; the string + operator takes care of the String.valueOf for you.
– sfdcfox
2 hours ago
|
show 2 more comments
up vote
6
down vote
The Id
class already contains an instance method to do exactly what you wish, named getSObjectType
. The entire body of your method can be simplified to the below:
return (recordId == null) ? '' : String.valueOf(recordId.getSObjectType());
Indeed...that worked, thanks! I hope my question makes this answer easier for others to find because Google was not helping me at all.
– Ixalmida
7 hours ago
1
You should mark it as "Accepted" (The tick next to the number of votes) so it comes up properly in searches;
– Sebastian Kessel
6 hours ago
1
First...it wouldn't let me mark as accepted for 10 minutes or whatever, so I was waiting. Second...I just figured out for myself that this answer actually won't work because it mixes return types (String / Schema.SObjectType). The correct answer is actually:return (recordId == null) ? '' : recordId.getSObjectType().getDescribe().getName();
I will accept the answer when it is corrected.
– Ixalmida
6 hours ago
1
You can just useString.valueOf
, which should be much faster. Depending on your use case, it would actually be much better to just change your return type toSObjectType
. Static types are preferable toString
where they fit.
– Adrian Larson♦
6 hours ago
1
@AdrianLarson you can also justreturn recordId == null? '': ''+recordId.getSobjectType()
; the string + operator takes care of the String.valueOf for you.
– sfdcfox
2 hours ago
|
show 2 more comments
up vote
6
down vote
up vote
6
down vote
The Id
class already contains an instance method to do exactly what you wish, named getSObjectType
. The entire body of your method can be simplified to the below:
return (recordId == null) ? '' : String.valueOf(recordId.getSObjectType());
The Id
class already contains an instance method to do exactly what you wish, named getSObjectType
. The entire body of your method can be simplified to the below:
return (recordId == null) ? '' : String.valueOf(recordId.getSObjectType());
edited 6 hours ago
answered 7 hours ago
Adrian Larson♦
103k19110233
103k19110233
Indeed...that worked, thanks! I hope my question makes this answer easier for others to find because Google was not helping me at all.
– Ixalmida
7 hours ago
1
You should mark it as "Accepted" (The tick next to the number of votes) so it comes up properly in searches;
– Sebastian Kessel
6 hours ago
1
First...it wouldn't let me mark as accepted for 10 minutes or whatever, so I was waiting. Second...I just figured out for myself that this answer actually won't work because it mixes return types (String / Schema.SObjectType). The correct answer is actually:return (recordId == null) ? '' : recordId.getSObjectType().getDescribe().getName();
I will accept the answer when it is corrected.
– Ixalmida
6 hours ago
1
You can just useString.valueOf
, which should be much faster. Depending on your use case, it would actually be much better to just change your return type toSObjectType
. Static types are preferable toString
where they fit.
– Adrian Larson♦
6 hours ago
1
@AdrianLarson you can also justreturn recordId == null? '': ''+recordId.getSobjectType()
; the string + operator takes care of the String.valueOf for you.
– sfdcfox
2 hours ago
|
show 2 more comments
Indeed...that worked, thanks! I hope my question makes this answer easier for others to find because Google was not helping me at all.
– Ixalmida
7 hours ago
1
You should mark it as "Accepted" (The tick next to the number of votes) so it comes up properly in searches;
– Sebastian Kessel
6 hours ago
1
First...it wouldn't let me mark as accepted for 10 minutes or whatever, so I was waiting. Second...I just figured out for myself that this answer actually won't work because it mixes return types (String / Schema.SObjectType). The correct answer is actually:return (recordId == null) ? '' : recordId.getSObjectType().getDescribe().getName();
I will accept the answer when it is corrected.
– Ixalmida
6 hours ago
1
You can just useString.valueOf
, which should be much faster. Depending on your use case, it would actually be much better to just change your return type toSObjectType
. Static types are preferable toString
where they fit.
– Adrian Larson♦
6 hours ago
1
@AdrianLarson you can also justreturn recordId == null? '': ''+recordId.getSobjectType()
; the string + operator takes care of the String.valueOf for you.
– sfdcfox
2 hours ago
Indeed...that worked, thanks! I hope my question makes this answer easier for others to find because Google was not helping me at all.
– Ixalmida
7 hours ago
Indeed...that worked, thanks! I hope my question makes this answer easier for others to find because Google was not helping me at all.
– Ixalmida
7 hours ago
1
1
You should mark it as "Accepted" (The tick next to the number of votes) so it comes up properly in searches;
– Sebastian Kessel
6 hours ago
You should mark it as "Accepted" (The tick next to the number of votes) so it comes up properly in searches;
– Sebastian Kessel
6 hours ago
1
1
First...it wouldn't let me mark as accepted for 10 minutes or whatever, so I was waiting. Second...I just figured out for myself that this answer actually won't work because it mixes return types (String / Schema.SObjectType). The correct answer is actually:
return (recordId == null) ? '' : recordId.getSObjectType().getDescribe().getName();
I will accept the answer when it is corrected.– Ixalmida
6 hours ago
First...it wouldn't let me mark as accepted for 10 minutes or whatever, so I was waiting. Second...I just figured out for myself that this answer actually won't work because it mixes return types (String / Schema.SObjectType). The correct answer is actually:
return (recordId == null) ? '' : recordId.getSObjectType().getDescribe().getName();
I will accept the answer when it is corrected.– Ixalmida
6 hours ago
1
1
You can just use
String.valueOf
, which should be much faster. Depending on your use case, it would actually be much better to just change your return type to SObjectType
. Static types are preferable to String
where they fit.– Adrian Larson♦
6 hours ago
You can just use
String.valueOf
, which should be much faster. Depending on your use case, it would actually be much better to just change your return type to SObjectType
. Static types are preferable to String
where they fit.– Adrian Larson♦
6 hours ago
1
1
@AdrianLarson you can also just
return recordId == null? '': ''+recordId.getSobjectType()
; the string + operator takes care of the String.valueOf for you.– sfdcfox
2 hours ago
@AdrianLarson you can also just
return recordId == null? '': ''+recordId.getSobjectType()
; the string + operator takes care of the String.valueOf for you.– sfdcfox
2 hours ago
|
show 2 more comments
Ixalmida is a new contributor. Be nice, and check out our Code of Conduct.
Ixalmida is a new contributor. Be nice, and check out our Code of Conduct.
Ixalmida is a new contributor. Be nice, and check out our Code of Conduct.
Ixalmida is a new contributor. Be nice, and check out our Code of Conduct.
Thanks for contributing an answer to Salesforce Stack Exchange!
- Please be sure to answer the question. Provide details and share your research!
But avoid …
- Asking for help, clarification, or responding to other answers.
- Making statements based on opinion; back them up with references or personal experience.
To learn more, see our tips on writing great answers.
Some of your past answers have not been well-received, and you're in danger of being blocked from answering.
Please pay close attention to the following guidance:
- Please be sure to answer the question. Provide details and share your research!
But avoid …
- Asking for help, clarification, or responding to other answers.
- Making statements based on opinion; back them up with references or personal experience.
To learn more, see our tips on writing great answers.
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
StackExchange.ready(
function () {
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fsalesforce.stackexchange.com%2fquestions%2f241285%2fspeeding-up-object-name-check-by-record-id%23new-answer', 'question_page');
}
);
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown