Unassignable function return type when generated by a generic function generator in Typescript












1















When I run this code:



type Converter<T extends ConverterOutput> = (value: string) => T
type ConverterOutput = number | string

function generateConverter<T extends ConverterOutput>(
value: string,
type: 'number' | 'string'
): Converter<T> {
return (value: string): T => {
switch (type) {
case 'number':
return Number(value)

default:
return value
}
}
}


I get these 2 errors:




  • [ts] Type 'number' is not assignable to type 'T'. [2322]

  • [ts] Type 'string' is not assignable to type 'T'. [2322]


for the 2 switch ... case returned values. I don't get what can be wrong in this code since T extends ConverterOutput which itself is a number | string.



I have tried adding a generic catch: return <U extends T>(value: string): U => { but it doesn't solve anything other than telling me that it's not assignable to U this time.










share|improve this question



























    1















    When I run this code:



    type Converter<T extends ConverterOutput> = (value: string) => T
    type ConverterOutput = number | string

    function generateConverter<T extends ConverterOutput>(
    value: string,
    type: 'number' | 'string'
    ): Converter<T> {
    return (value: string): T => {
    switch (type) {
    case 'number':
    return Number(value)

    default:
    return value
    }
    }
    }


    I get these 2 errors:




    • [ts] Type 'number' is not assignable to type 'T'. [2322]

    • [ts] Type 'string' is not assignable to type 'T'. [2322]


    for the 2 switch ... case returned values. I don't get what can be wrong in this code since T extends ConverterOutput which itself is a number | string.



    I have tried adding a generic catch: return <U extends T>(value: string): U => { but it doesn't solve anything other than telling me that it's not assignable to U this time.










    share|improve this question

























      1












      1








      1








      When I run this code:



      type Converter<T extends ConverterOutput> = (value: string) => T
      type ConverterOutput = number | string

      function generateConverter<T extends ConverterOutput>(
      value: string,
      type: 'number' | 'string'
      ): Converter<T> {
      return (value: string): T => {
      switch (type) {
      case 'number':
      return Number(value)

      default:
      return value
      }
      }
      }


      I get these 2 errors:




      • [ts] Type 'number' is not assignable to type 'T'. [2322]

      • [ts] Type 'string' is not assignable to type 'T'. [2322]


      for the 2 switch ... case returned values. I don't get what can be wrong in this code since T extends ConverterOutput which itself is a number | string.



      I have tried adding a generic catch: return <U extends T>(value: string): U => { but it doesn't solve anything other than telling me that it's not assignable to U this time.










      share|improve this question














      When I run this code:



      type Converter<T extends ConverterOutput> = (value: string) => T
      type ConverterOutput = number | string

      function generateConverter<T extends ConverterOutput>(
      value: string,
      type: 'number' | 'string'
      ): Converter<T> {
      return (value: string): T => {
      switch (type) {
      case 'number':
      return Number(value)

      default:
      return value
      }
      }
      }


      I get these 2 errors:




      • [ts] Type 'number' is not assignable to type 'T'. [2322]

      • [ts] Type 'string' is not assignable to type 'T'. [2322]


      for the 2 switch ... case returned values. I don't get what can be wrong in this code since T extends ConverterOutput which itself is a number | string.



      I have tried adding a generic catch: return <U extends T>(value: string): U => { but it doesn't solve anything other than telling me that it's not assignable to U this time.







      typescript






      share|improve this question













      share|improve this question











      share|improve this question




      share|improve this question










      asked Nov 20 '18 at 14:51









      Edouard HienrichsEdouard Hienrichs

      38211




      38211
























          2 Answers
          2






          active

          oldest

          votes


















          2














          Typescript will not let you assign concrete values where a value of a generic type parameter is expected. The reason for this is that usually the value can't be validated against all possible derived types that could conceivable be passed for the generic type parameter. For example even in your case, T extends number| string so this mean it could be the number literal type 1 which would not be satisfied by the value Number(value).



          In your case I would recommend doing away with the generic type parameters and use multiple overloads, with the implementation signature returning Converter<number | string>. This has the added benefit that the type of the converter will only need to be specified as a string not as both string and type parameter.



          type Converter<T extends ConverterOutput> = (value: string) => T
          type ConverterOutput = number | string


          function generateConverter(value: string, type: 'number'): Converter<number>
          function generateConverter(value: string, type: 'string'): Converter<string>
          function generateConverter(value: string, type: 'number' | 'string'): Converter<number | string> {
          return (value) => {
          switch (type) {
          case 'number':
          return Number(value)

          default:
          return value
          }
          }
          }





          share|improve this answer































            1














            Because a number nor a string does not extend itself. That's why it's not working. You can fix it by giving the type directly e.g.



            function generateConverter(value: string, type: 'number' | 'string'): Converter<ConverterOutput> {
            return (value: string): ConverterOutput => {
            switch (type) {
            case 'number':
            return Number(value)

            default:
            return value
            }
            }
            }





            share|improve this answer























              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%2f53395619%2funassignable-function-return-type-when-generated-by-a-generic-function-generator%23new-answer', 'question_page');
              }
              );

              Post as a guest















              Required, but never shown

























              2 Answers
              2






              active

              oldest

              votes








              2 Answers
              2






              active

              oldest

              votes









              active

              oldest

              votes






              active

              oldest

              votes









              2














              Typescript will not let you assign concrete values where a value of a generic type parameter is expected. The reason for this is that usually the value can't be validated against all possible derived types that could conceivable be passed for the generic type parameter. For example even in your case, T extends number| string so this mean it could be the number literal type 1 which would not be satisfied by the value Number(value).



              In your case I would recommend doing away with the generic type parameters and use multiple overloads, with the implementation signature returning Converter<number | string>. This has the added benefit that the type of the converter will only need to be specified as a string not as both string and type parameter.



              type Converter<T extends ConverterOutput> = (value: string) => T
              type ConverterOutput = number | string


              function generateConverter(value: string, type: 'number'): Converter<number>
              function generateConverter(value: string, type: 'string'): Converter<string>
              function generateConverter(value: string, type: 'number' | 'string'): Converter<number | string> {
              return (value) => {
              switch (type) {
              case 'number':
              return Number(value)

              default:
              return value
              }
              }
              }





              share|improve this answer




























                2














                Typescript will not let you assign concrete values where a value of a generic type parameter is expected. The reason for this is that usually the value can't be validated against all possible derived types that could conceivable be passed for the generic type parameter. For example even in your case, T extends number| string so this mean it could be the number literal type 1 which would not be satisfied by the value Number(value).



                In your case I would recommend doing away with the generic type parameters and use multiple overloads, with the implementation signature returning Converter<number | string>. This has the added benefit that the type of the converter will only need to be specified as a string not as both string and type parameter.



                type Converter<T extends ConverterOutput> = (value: string) => T
                type ConverterOutput = number | string


                function generateConverter(value: string, type: 'number'): Converter<number>
                function generateConverter(value: string, type: 'string'): Converter<string>
                function generateConverter(value: string, type: 'number' | 'string'): Converter<number | string> {
                return (value) => {
                switch (type) {
                case 'number':
                return Number(value)

                default:
                return value
                }
                }
                }





                share|improve this answer


























                  2












                  2








                  2







                  Typescript will not let you assign concrete values where a value of a generic type parameter is expected. The reason for this is that usually the value can't be validated against all possible derived types that could conceivable be passed for the generic type parameter. For example even in your case, T extends number| string so this mean it could be the number literal type 1 which would not be satisfied by the value Number(value).



                  In your case I would recommend doing away with the generic type parameters and use multiple overloads, with the implementation signature returning Converter<number | string>. This has the added benefit that the type of the converter will only need to be specified as a string not as both string and type parameter.



                  type Converter<T extends ConverterOutput> = (value: string) => T
                  type ConverterOutput = number | string


                  function generateConverter(value: string, type: 'number'): Converter<number>
                  function generateConverter(value: string, type: 'string'): Converter<string>
                  function generateConverter(value: string, type: 'number' | 'string'): Converter<number | string> {
                  return (value) => {
                  switch (type) {
                  case 'number':
                  return Number(value)

                  default:
                  return value
                  }
                  }
                  }





                  share|improve this answer













                  Typescript will not let you assign concrete values where a value of a generic type parameter is expected. The reason for this is that usually the value can't be validated against all possible derived types that could conceivable be passed for the generic type parameter. For example even in your case, T extends number| string so this mean it could be the number literal type 1 which would not be satisfied by the value Number(value).



                  In your case I would recommend doing away with the generic type parameters and use multiple overloads, with the implementation signature returning Converter<number | string>. This has the added benefit that the type of the converter will only need to be specified as a string not as both string and type parameter.



                  type Converter<T extends ConverterOutput> = (value: string) => T
                  type ConverterOutput = number | string


                  function generateConverter(value: string, type: 'number'): Converter<number>
                  function generateConverter(value: string, type: 'string'): Converter<string>
                  function generateConverter(value: string, type: 'number' | 'string'): Converter<number | string> {
                  return (value) => {
                  switch (type) {
                  case 'number':
                  return Number(value)

                  default:
                  return value
                  }
                  }
                  }






                  share|improve this answer












                  share|improve this answer



                  share|improve this answer










                  answered Nov 20 '18 at 15:03









                  Titian Cernicova-DragomirTitian Cernicova-Dragomir

                  60.9k33755




                  60.9k33755

























                      1














                      Because a number nor a string does not extend itself. That's why it's not working. You can fix it by giving the type directly e.g.



                      function generateConverter(value: string, type: 'number' | 'string'): Converter<ConverterOutput> {
                      return (value: string): ConverterOutput => {
                      switch (type) {
                      case 'number':
                      return Number(value)

                      default:
                      return value
                      }
                      }
                      }





                      share|improve this answer




























                        1














                        Because a number nor a string does not extend itself. That's why it's not working. You can fix it by giving the type directly e.g.



                        function generateConverter(value: string, type: 'number' | 'string'): Converter<ConverterOutput> {
                        return (value: string): ConverterOutput => {
                        switch (type) {
                        case 'number':
                        return Number(value)

                        default:
                        return value
                        }
                        }
                        }





                        share|improve this answer


























                          1












                          1








                          1







                          Because a number nor a string does not extend itself. That's why it's not working. You can fix it by giving the type directly e.g.



                          function generateConverter(value: string, type: 'number' | 'string'): Converter<ConverterOutput> {
                          return (value: string): ConverterOutput => {
                          switch (type) {
                          case 'number':
                          return Number(value)

                          default:
                          return value
                          }
                          }
                          }





                          share|improve this answer













                          Because a number nor a string does not extend itself. That's why it's not working. You can fix it by giving the type directly e.g.



                          function generateConverter(value: string, type: 'number' | 'string'): Converter<ConverterOutput> {
                          return (value: string): ConverterOutput => {
                          switch (type) {
                          case 'number':
                          return Number(value)

                          default:
                          return value
                          }
                          }
                          }






                          share|improve this answer












                          share|improve this answer



                          share|improve this answer










                          answered Nov 20 '18 at 15:03









                          Murat KaragözMurat Karagöz

                          14.5k53568




                          14.5k53568






























                              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.




                              draft saved


                              draft discarded














                              StackExchange.ready(
                              function () {
                              StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53395619%2funassignable-function-return-type-when-generated-by-a-generic-function-generator%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

                              MongoDB - Not Authorized To Execute Command

                              in spring boot 2.1 many test slices are not allowed anymore due to multiple @BootstrapWith

                              How to fix TextFormField cause rebuild widget in Flutter