Issue using HttpParams












1















Angular Version: 4.3




I have a screen with 3 search fields: id, name and category. I do the research as the fields are filled out.



I would like a better suggestion for this code, because if I had several fields of research, it would be an impracticable implementation for the amount of combinations. The fact that the HttpParams object is immutable is that it is leaving me trapped in the field combinations, since I can not concatenate as I check if the field has been filled.



See the code below:



pesquisaRestaurantes(id ? : string, name ? : string, category ? : string): Observable < Restaurant > {
let params: HttpParams = undefined;
if (id) {
params = new HttpParams().append('id', id)
} else {
if (name && category) {
params = new HttpParams().append('name', name).append('category', category)
} else if (name) {
params = new HttpParams().append('name', name)
} else if (category) {
params = new HttpParams().append('category', category)
}
}
return this.http.get < Restaurant > (`${MEAT_API}/restaurants`, {
params: params
})
}


I look forward to a better implementation. Thank you.










share|improve this question





























    1















    Angular Version: 4.3




    I have a screen with 3 search fields: id, name and category. I do the research as the fields are filled out.



    I would like a better suggestion for this code, because if I had several fields of research, it would be an impracticable implementation for the amount of combinations. The fact that the HttpParams object is immutable is that it is leaving me trapped in the field combinations, since I can not concatenate as I check if the field has been filled.



    See the code below:



    pesquisaRestaurantes(id ? : string, name ? : string, category ? : string): Observable < Restaurant > {
    let params: HttpParams = undefined;
    if (id) {
    params = new HttpParams().append('id', id)
    } else {
    if (name && category) {
    params = new HttpParams().append('name', name).append('category', category)
    } else if (name) {
    params = new HttpParams().append('name', name)
    } else if (category) {
    params = new HttpParams().append('category', category)
    }
    }
    return this.http.get < Restaurant > (`${MEAT_API}/restaurants`, {
    params: params
    })
    }


    I look forward to a better implementation. Thank you.










    share|improve this question



























      1












      1








      1








      Angular Version: 4.3




      I have a screen with 3 search fields: id, name and category. I do the research as the fields are filled out.



      I would like a better suggestion for this code, because if I had several fields of research, it would be an impracticable implementation for the amount of combinations. The fact that the HttpParams object is immutable is that it is leaving me trapped in the field combinations, since I can not concatenate as I check if the field has been filled.



      See the code below:



      pesquisaRestaurantes(id ? : string, name ? : string, category ? : string): Observable < Restaurant > {
      let params: HttpParams = undefined;
      if (id) {
      params = new HttpParams().append('id', id)
      } else {
      if (name && category) {
      params = new HttpParams().append('name', name).append('category', category)
      } else if (name) {
      params = new HttpParams().append('name', name)
      } else if (category) {
      params = new HttpParams().append('category', category)
      }
      }
      return this.http.get < Restaurant > (`${MEAT_API}/restaurants`, {
      params: params
      })
      }


      I look forward to a better implementation. Thank you.










      share|improve this question
















      Angular Version: 4.3




      I have a screen with 3 search fields: id, name and category. I do the research as the fields are filled out.



      I would like a better suggestion for this code, because if I had several fields of research, it would be an impracticable implementation for the amount of combinations. The fact that the HttpParams object is immutable is that it is leaving me trapped in the field combinations, since I can not concatenate as I check if the field has been filled.



      See the code below:



      pesquisaRestaurantes(id ? : string, name ? : string, category ? : string): Observable < Restaurant > {
      let params: HttpParams = undefined;
      if (id) {
      params = new HttpParams().append('id', id)
      } else {
      if (name && category) {
      params = new HttpParams().append('name', name).append('category', category)
      } else if (name) {
      params = new HttpParams().append('name', name)
      } else if (category) {
      params = new HttpParams().append('category', category)
      }
      }
      return this.http.get < Restaurant > (`${MEAT_API}/restaurants`, {
      params: params
      })
      }


      I look forward to a better implementation. Thank you.







      angular






      share|improve this question















      share|improve this question













      share|improve this question




      share|improve this question








      edited Nov 23 '18 at 7:15









      barbsan

      2,21811122




      2,21811122










      asked Nov 23 '18 at 1:38









      Robson Brito

      111




      111
























          1 Answer
          1






          active

          oldest

          votes


















          1














          Should work:



          ...



           params = new HttpParams();

          If(name) params = params.append('name', name);

          If(category) params = params.append('category', category);


          ...






          share|improve this answer





















          • Já tentei esta alternativa, mas não funciona porque o objeto params é imutável. A partir do momento em que chamo pelo método .append ou .set, o objeto é clonado e devolvido esta nova instância somente com este último parâmetro informado.
            – Robson Brito
            Nov 23 '18 at 12:30













          Your Answer






          StackExchange.ifUsing("editor", function () {
          StackExchange.using("externalEditor", function () {
          StackExchange.using("snippets", function () {
          StackExchange.snippets.init();
          });
          });
          }, "code-snippets");

          StackExchange.ready(function() {
          var channelOptions = {
          tags: "".split(" "),
          id: "1"
          };
          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',
          autoActivateHeartbeat: false,
          convertImagesToLinks: true,
          noModals: true,
          showLowRepImageUploadWarning: true,
          reputationToPostImages: 10,
          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
          });


          }
          });














          draft saved

          draft discarded


















          StackExchange.ready(
          function () {
          StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53439719%2fissue-using-httpparams%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









          1














          Should work:



          ...



           params = new HttpParams();

          If(name) params = params.append('name', name);

          If(category) params = params.append('category', category);


          ...






          share|improve this answer





















          • Já tentei esta alternativa, mas não funciona porque o objeto params é imutável. A partir do momento em que chamo pelo método .append ou .set, o objeto é clonado e devolvido esta nova instância somente com este último parâmetro informado.
            – Robson Brito
            Nov 23 '18 at 12:30


















          1














          Should work:



          ...



           params = new HttpParams();

          If(name) params = params.append('name', name);

          If(category) params = params.append('category', category);


          ...






          share|improve this answer





















          • Já tentei esta alternativa, mas não funciona porque o objeto params é imutável. A partir do momento em que chamo pelo método .append ou .set, o objeto é clonado e devolvido esta nova instância somente com este último parâmetro informado.
            – Robson Brito
            Nov 23 '18 at 12:30
















          1












          1








          1






          Should work:



          ...



           params = new HttpParams();

          If(name) params = params.append('name', name);

          If(category) params = params.append('category', category);


          ...






          share|improve this answer












          Should work:



          ...



           params = new HttpParams();

          If(name) params = params.append('name', name);

          If(category) params = params.append('category', category);


          ...







          share|improve this answer












          share|improve this answer



          share|improve this answer










          answered Nov 23 '18 at 4:50









          Alexander Poshtaruk

          1,292510




          1,292510












          • Já tentei esta alternativa, mas não funciona porque o objeto params é imutável. A partir do momento em que chamo pelo método .append ou .set, o objeto é clonado e devolvido esta nova instância somente com este último parâmetro informado.
            – Robson Brito
            Nov 23 '18 at 12:30




















          • Já tentei esta alternativa, mas não funciona porque o objeto params é imutável. A partir do momento em que chamo pelo método .append ou .set, o objeto é clonado e devolvido esta nova instância somente com este último parâmetro informado.
            – Robson Brito
            Nov 23 '18 at 12:30


















          Já tentei esta alternativa, mas não funciona porque o objeto params é imutável. A partir do momento em que chamo pelo método .append ou .set, o objeto é clonado e devolvido esta nova instância somente com este último parâmetro informado.
          – Robson Brito
          Nov 23 '18 at 12:30






          Já tentei esta alternativa, mas não funciona porque o objeto params é imutável. A partir do momento em que chamo pelo método .append ou .set, o objeto é clonado e devolvido esta nova instância somente com este último parâmetro informado.
          – Robson Brito
          Nov 23 '18 at 12:30




















          draft saved

          draft discarded




















































          Thanks for contributing an answer to Stack Overflow!


          • 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%2fstackoverflow.com%2fquestions%2f53439719%2fissue-using-httpparams%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

          Berounka

          Sphinx de Gizeh

          Different font size/position of beamer's navigation symbols template's content depending on regular/plain...