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?










share|improve this question







New contributor




Ixalmida is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
Check out our Code of Conduct.


























    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?










    share|improve this question







    New contributor




    Ixalmida is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
    Check out our Code of Conduct.






















      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?










      share|improve this question







      New contributor




      Ixalmida is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.











      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






      share|improve this question







      New contributor




      Ixalmida is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.











      share|improve this question







      New contributor




      Ixalmida is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.









      share|improve this question




      share|improve this question






      New contributor




      Ixalmida is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.









      asked 5 hours ago









      Ixalmida

      1113




      1113




      New contributor




      Ixalmida is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.





      New contributor





      Ixalmida is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.






      Ixalmida is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.






















          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());





          share|improve this answer























          • 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
            4 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
            4 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
            4 hours ago






          • 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
            4 hours ago






          • 1




            @AdrianLarson you can also just return recordId == null? '': ''+recordId.getSobjectType(); the string + operator takes care of the String.valueOf for you.
            – sfdcfox
            19 mins ago











          Your Answer








          StackExchange.ready(function() {
          var channelOptions = {
          tags: "".split(" "),
          id: "459"
          };
          initTagRenderer("".split(" "), "".split(" "), channelOptions);

          StackExchange.using("externalEditor", function() {
          // Have to fire editor after snippets, if snippets enabled
          if (StackExchange.settings.snippets.snippetsEnabled) {
          StackExchange.using("snippets", function() {
          createEditor();
          });
          }
          else {
          createEditor();
          }
          });

          function createEditor() {
          StackExchange.prepareEditor({
          heartbeatType: 'answer',
          convertImagesToLinks: false,
          noModals: true,
          showLowRepImageUploadWarning: true,
          reputationToPostImages: null,
          bindNavPrevention: true,
          postfix: "",
          imageUploader: {
          brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
          contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
          allowUrls: true
          },
          onDemand: true,
          discardSelector: ".discard-answer"
          ,immediatelyShowMarkdownHelp:true
          });


          }
          });






          Ixalmida is a new contributor. Be nice, and check out our Code of Conduct.










          draft saved

          draft discarded


















          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

























          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());





          share|improve this answer























          • 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
            4 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
            4 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
            4 hours ago






          • 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
            4 hours ago






          • 1




            @AdrianLarson you can also just return recordId == null? '': ''+recordId.getSobjectType(); the string + operator takes care of the String.valueOf for you.
            – sfdcfox
            19 mins ago















          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());





          share|improve this answer























          • 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
            4 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
            4 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
            4 hours ago






          • 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
            4 hours ago






          • 1




            @AdrianLarson you can also just return recordId == null? '': ''+recordId.getSobjectType(); the string + operator takes care of the String.valueOf for you.
            – sfdcfox
            19 mins ago













          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());





          share|improve this answer














          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());






          share|improve this answer














          share|improve this answer



          share|improve this answer








          edited 4 hours ago

























          answered 4 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
            4 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
            4 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
            4 hours ago






          • 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
            4 hours ago






          • 1




            @AdrianLarson you can also just return recordId == null? '': ''+recordId.getSobjectType(); the string + operator takes care of the String.valueOf for you.
            – sfdcfox
            19 mins 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
            4 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
            4 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
            4 hours ago






          • 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
            4 hours ago






          • 1




            @AdrianLarson you can also just return recordId == null? '': ''+recordId.getSobjectType(); the string + operator takes care of the String.valueOf for you.
            – sfdcfox
            19 mins 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
          4 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
          4 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
          4 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
          4 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
          4 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
          4 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
          4 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
          4 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
          19 mins ago




          @AdrianLarson you can also just return recordId == null? '': ''+recordId.getSobjectType(); the string + operator takes care of the String.valueOf for you.
          – sfdcfox
          19 mins ago










          Ixalmida is a new contributor. Be nice, and check out our Code of Conduct.










          draft saved

          draft discarded


















          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.




          draft saved


          draft discarded














          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





















































          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







          Popular posts from this blog

          404 Error Contact Form 7 ajax form submitting

          How to know if a Active Directory user can login interactively

          Refactoring coordinates for Minecraft Pi buildings written in Python