Persist relation to a class (and not a class instance) in Ruby on Rails with Mongoid












1














Assume my users can subscribe to several plan_types. I want to define those plans as classes, and be able to keep, for each user, a reference to a subscription_plan and also a presubscription_plan. This is like a State pattern :



class Payment::SubscriptionPlan
def self.max_subscription_quantity
120
end
end

class Payment::ConcretePlan1 < Payment::SubscriptionPlan
def self.commitment_period
2.months
end
end

class Payment::ConcretePlan2 < Payment::SubscriptionPlan
def self.max_subscription_quantity
50
end
end


What is the best way to "persist" the relation to a class in the database ? Currently I was doing the following using the as_enum gem



class Settings
setting :subscription_plans, type: Array, default: [
Payment::ConcretePlan1,
Payment::ConcretePlan2
]
end

class User
as_enum :subscription_plan, Settings.subscription_plans, map: :string
as_enum :presubscription_plan, Settings.subscription_plans, map: :string

def subscription_plan
return nil if super.nil?
Payment.const_get(super.to_s.demodulize)
end

def presubscription_plan
return nil if super.nil?
Payment.const_get(super.to_s.demodulize)
end
end


The current implementation works but I'm not sure this is the best way to tackle the problem










share|improve this question
















bumped to the homepage by Community yesterday


This question has answers that may be good or bad; the system has marked it active so that they can be reviewed.















  • ConcretePlan1 has a subscription period where as the other plans do not. Shouldn't all plans also have a subscription period? If these classes are just placeholders for returning a particular value, you can have that same functionality?
    – BKSpurgeon
    May 19 '17 at 13:21










  • Some payment systems involve automatic payment where I need to specify a period (eg. Stripe), whereas some other subscriptions are managed from the outside. I have factories to handle the subscriptions but I need to save the type of subscription for that :D And actually no commitment_period is a bit different than the subscription period : for monthly payment, our offer stipulates that you have to pay for at least 3 months when you start a subscription. When paying yearly, no such commitment period apply.
    – Cyril Duchon-Doris
    May 19 '17 at 13:27


















1














Assume my users can subscribe to several plan_types. I want to define those plans as classes, and be able to keep, for each user, a reference to a subscription_plan and also a presubscription_plan. This is like a State pattern :



class Payment::SubscriptionPlan
def self.max_subscription_quantity
120
end
end

class Payment::ConcretePlan1 < Payment::SubscriptionPlan
def self.commitment_period
2.months
end
end

class Payment::ConcretePlan2 < Payment::SubscriptionPlan
def self.max_subscription_quantity
50
end
end


What is the best way to "persist" the relation to a class in the database ? Currently I was doing the following using the as_enum gem



class Settings
setting :subscription_plans, type: Array, default: [
Payment::ConcretePlan1,
Payment::ConcretePlan2
]
end

class User
as_enum :subscription_plan, Settings.subscription_plans, map: :string
as_enum :presubscription_plan, Settings.subscription_plans, map: :string

def subscription_plan
return nil if super.nil?
Payment.const_get(super.to_s.demodulize)
end

def presubscription_plan
return nil if super.nil?
Payment.const_get(super.to_s.demodulize)
end
end


The current implementation works but I'm not sure this is the best way to tackle the problem










share|improve this question
















bumped to the homepage by Community yesterday


This question has answers that may be good or bad; the system has marked it active so that they can be reviewed.















  • ConcretePlan1 has a subscription period where as the other plans do not. Shouldn't all plans also have a subscription period? If these classes are just placeholders for returning a particular value, you can have that same functionality?
    – BKSpurgeon
    May 19 '17 at 13:21










  • Some payment systems involve automatic payment where I need to specify a period (eg. Stripe), whereas some other subscriptions are managed from the outside. I have factories to handle the subscriptions but I need to save the type of subscription for that :D And actually no commitment_period is a bit different than the subscription period : for monthly payment, our offer stipulates that you have to pay for at least 3 months when you start a subscription. When paying yearly, no such commitment period apply.
    – Cyril Duchon-Doris
    May 19 '17 at 13:27
















1












1








1







Assume my users can subscribe to several plan_types. I want to define those plans as classes, and be able to keep, for each user, a reference to a subscription_plan and also a presubscription_plan. This is like a State pattern :



class Payment::SubscriptionPlan
def self.max_subscription_quantity
120
end
end

class Payment::ConcretePlan1 < Payment::SubscriptionPlan
def self.commitment_period
2.months
end
end

class Payment::ConcretePlan2 < Payment::SubscriptionPlan
def self.max_subscription_quantity
50
end
end


What is the best way to "persist" the relation to a class in the database ? Currently I was doing the following using the as_enum gem



class Settings
setting :subscription_plans, type: Array, default: [
Payment::ConcretePlan1,
Payment::ConcretePlan2
]
end

class User
as_enum :subscription_plan, Settings.subscription_plans, map: :string
as_enum :presubscription_plan, Settings.subscription_plans, map: :string

def subscription_plan
return nil if super.nil?
Payment.const_get(super.to_s.demodulize)
end

def presubscription_plan
return nil if super.nil?
Payment.const_get(super.to_s.demodulize)
end
end


The current implementation works but I'm not sure this is the best way to tackle the problem










share|improve this question















Assume my users can subscribe to several plan_types. I want to define those plans as classes, and be able to keep, for each user, a reference to a subscription_plan and also a presubscription_plan. This is like a State pattern :



class Payment::SubscriptionPlan
def self.max_subscription_quantity
120
end
end

class Payment::ConcretePlan1 < Payment::SubscriptionPlan
def self.commitment_period
2.months
end
end

class Payment::ConcretePlan2 < Payment::SubscriptionPlan
def self.max_subscription_quantity
50
end
end


What is the best way to "persist" the relation to a class in the database ? Currently I was doing the following using the as_enum gem



class Settings
setting :subscription_plans, type: Array, default: [
Payment::ConcretePlan1,
Payment::ConcretePlan2
]
end

class User
as_enum :subscription_plan, Settings.subscription_plans, map: :string
as_enum :presubscription_plan, Settings.subscription_plans, map: :string

def subscription_plan
return nil if super.nil?
Payment.const_get(super.to_s.demodulize)
end

def presubscription_plan
return nil if super.nil?
Payment.const_get(super.to_s.demodulize)
end
end


The current implementation works but I'm not sure this is the best way to tackle the problem







ruby ruby-on-rails mongodb






share|improve this question















share|improve this question













share|improve this question




share|improve this question








edited May 17 '17 at 10:53







Cyril Duchon-Doris

















asked May 13 '17 at 15:11









Cyril Duchon-DorisCyril Duchon-Doris

1114




1114





bumped to the homepage by Community yesterday


This question has answers that may be good or bad; the system has marked it active so that they can be reviewed.







bumped to the homepage by Community yesterday


This question has answers that may be good or bad; the system has marked it active so that they can be reviewed.














  • ConcretePlan1 has a subscription period where as the other plans do not. Shouldn't all plans also have a subscription period? If these classes are just placeholders for returning a particular value, you can have that same functionality?
    – BKSpurgeon
    May 19 '17 at 13:21










  • Some payment systems involve automatic payment where I need to specify a period (eg. Stripe), whereas some other subscriptions are managed from the outside. I have factories to handle the subscriptions but I need to save the type of subscription for that :D And actually no commitment_period is a bit different than the subscription period : for monthly payment, our offer stipulates that you have to pay for at least 3 months when you start a subscription. When paying yearly, no such commitment period apply.
    – Cyril Duchon-Doris
    May 19 '17 at 13:27




















  • ConcretePlan1 has a subscription period where as the other plans do not. Shouldn't all plans also have a subscription period? If these classes are just placeholders for returning a particular value, you can have that same functionality?
    – BKSpurgeon
    May 19 '17 at 13:21










  • Some payment systems involve automatic payment where I need to specify a period (eg. Stripe), whereas some other subscriptions are managed from the outside. I have factories to handle the subscriptions but I need to save the type of subscription for that :D And actually no commitment_period is a bit different than the subscription period : for monthly payment, our offer stipulates that you have to pay for at least 3 months when you start a subscription. When paying yearly, no such commitment period apply.
    – Cyril Duchon-Doris
    May 19 '17 at 13:27


















ConcretePlan1 has a subscription period where as the other plans do not. Shouldn't all plans also have a subscription period? If these classes are just placeholders for returning a particular value, you can have that same functionality?
– BKSpurgeon
May 19 '17 at 13:21




ConcretePlan1 has a subscription period where as the other plans do not. Shouldn't all plans also have a subscription period? If these classes are just placeholders for returning a particular value, you can have that same functionality?
– BKSpurgeon
May 19 '17 at 13:21












Some payment systems involve automatic payment where I need to specify a period (eg. Stripe), whereas some other subscriptions are managed from the outside. I have factories to handle the subscriptions but I need to save the type of subscription for that :D And actually no commitment_period is a bit different than the subscription period : for monthly payment, our offer stipulates that you have to pay for at least 3 months when you start a subscription. When paying yearly, no such commitment period apply.
– Cyril Duchon-Doris
May 19 '17 at 13:27






Some payment systems involve automatic payment where I need to specify a period (eg. Stripe), whereas some other subscriptions are managed from the outside. I have factories to handle the subscriptions but I need to save the type of subscription for that :D And actually no commitment_period is a bit different than the subscription period : for monthly payment, our offer stipulates that you have to pay for at least 3 months when you start a subscription. When paying yearly, no such commitment period apply.
– Cyril Duchon-Doris
May 19 '17 at 13:27












2 Answers
2






active

oldest

votes


















0














It works that's for sure.



But enums could just as easily be defined in the database. Now that you've already got it working with enums, I would say stick with enums untill you have a good reason to change it.



anyways, here's how i probably would have done it:



e.g.




  • user has_many subscription_plans

  • subscription_plan can belong to many users.

  • Limit the ability for a user to have a max of two, one of each type. Or the user must have two at any one time etc.


enter image description here



You get the picture. You can change the subscription plan table to suit your needs.



Hope this helps.






share|improve this answer





















  • Hey thanks. I'm using a class object on my server since I need to define methods on the subscription_plan (subscription_plan.max_quantity, etc.) which is why I want to initialize some Ruby object where I can add whatever methods I want. I edited my example to include this. I'm not just using this as string enum.
    – Cyril Duchon-Doris
    May 17 '17 at 10:50












  • @CyrilDuchon-Doris: You should be able to still add methods and behavior specific to a kind of plan. ActiveRecord allows you to specify a "descriminator" column to instantiate a particular concrete class, giving you some of the benefits of polymorphism.
    – Greg Burghardt
    Jul 11 '18 at 21:48










  • Also just realized the OP has a "mongo-db" tag.
    – Greg Burghardt
    Jul 11 '18 at 21:58



















0














It sounds like you really need subscription plans to be data-driven. The methods you are defining on the concrete types are really just data. No need for concrete types. You need a better database table (and I use the term "table" loosely since you tagged the question with mongodb):



| id | name   | max_subscription_quantity | commitment_period_num | commitment_period_type |
| 1 | Plan 1 | 120 | 2 | Month |
| 2 | Plan 2 | 50 | NULL | NULL |


There really isn't a need for sub classes. Just one class ought to do it:



class Payment::SubscriptionPlan
def commitment_period
# return period based on type and number
end
end


The only thing you need to define is commitment_period which does the calculation of the commitment_period_num and commitment_period_type columns.






share|improve this answer





















    Your Answer





    StackExchange.ifUsing("editor", function () {
    return StackExchange.using("mathjaxEditing", function () {
    StackExchange.MarkdownEditor.creationCallbacks.add(function (editor, postfix) {
    StackExchange.mathjaxEditing.prepareWmdForMathJax(editor, postfix, [["\$", "\$"]]);
    });
    });
    }, "mathjax-editing");

    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: "196"
    };
    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: 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
    });


    }
    });














    draft saved

    draft discarded


















    StackExchange.ready(
    function () {
    StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fcodereview.stackexchange.com%2fquestions%2f163259%2fpersist-relation-to-a-class-and-not-a-class-instance-in-ruby-on-rails-with-mon%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









    0














    It works that's for sure.



    But enums could just as easily be defined in the database. Now that you've already got it working with enums, I would say stick with enums untill you have a good reason to change it.



    anyways, here's how i probably would have done it:



    e.g.




    • user has_many subscription_plans

    • subscription_plan can belong to many users.

    • Limit the ability for a user to have a max of two, one of each type. Or the user must have two at any one time etc.


    enter image description here



    You get the picture. You can change the subscription plan table to suit your needs.



    Hope this helps.






    share|improve this answer





















    • Hey thanks. I'm using a class object on my server since I need to define methods on the subscription_plan (subscription_plan.max_quantity, etc.) which is why I want to initialize some Ruby object where I can add whatever methods I want. I edited my example to include this. I'm not just using this as string enum.
      – Cyril Duchon-Doris
      May 17 '17 at 10:50












    • @CyrilDuchon-Doris: You should be able to still add methods and behavior specific to a kind of plan. ActiveRecord allows you to specify a "descriminator" column to instantiate a particular concrete class, giving you some of the benefits of polymorphism.
      – Greg Burghardt
      Jul 11 '18 at 21:48










    • Also just realized the OP has a "mongo-db" tag.
      – Greg Burghardt
      Jul 11 '18 at 21:58
















    0














    It works that's for sure.



    But enums could just as easily be defined in the database. Now that you've already got it working with enums, I would say stick with enums untill you have a good reason to change it.



    anyways, here's how i probably would have done it:



    e.g.




    • user has_many subscription_plans

    • subscription_plan can belong to many users.

    • Limit the ability for a user to have a max of two, one of each type. Or the user must have two at any one time etc.


    enter image description here



    You get the picture. You can change the subscription plan table to suit your needs.



    Hope this helps.






    share|improve this answer





















    • Hey thanks. I'm using a class object on my server since I need to define methods on the subscription_plan (subscription_plan.max_quantity, etc.) which is why I want to initialize some Ruby object where I can add whatever methods I want. I edited my example to include this. I'm not just using this as string enum.
      – Cyril Duchon-Doris
      May 17 '17 at 10:50












    • @CyrilDuchon-Doris: You should be able to still add methods and behavior specific to a kind of plan. ActiveRecord allows you to specify a "descriminator" column to instantiate a particular concrete class, giving you some of the benefits of polymorphism.
      – Greg Burghardt
      Jul 11 '18 at 21:48










    • Also just realized the OP has a "mongo-db" tag.
      – Greg Burghardt
      Jul 11 '18 at 21:58














    0












    0








    0






    It works that's for sure.



    But enums could just as easily be defined in the database. Now that you've already got it working with enums, I would say stick with enums untill you have a good reason to change it.



    anyways, here's how i probably would have done it:



    e.g.




    • user has_many subscription_plans

    • subscription_plan can belong to many users.

    • Limit the ability for a user to have a max of two, one of each type. Or the user must have two at any one time etc.


    enter image description here



    You get the picture. You can change the subscription plan table to suit your needs.



    Hope this helps.






    share|improve this answer












    It works that's for sure.



    But enums could just as easily be defined in the database. Now that you've already got it working with enums, I would say stick with enums untill you have a good reason to change it.



    anyways, here's how i probably would have done it:



    e.g.




    • user has_many subscription_plans

    • subscription_plan can belong to many users.

    • Limit the ability for a user to have a max of two, one of each type. Or the user must have two at any one time etc.


    enter image description here



    You get the picture. You can change the subscription plan table to suit your needs.



    Hope this helps.







    share|improve this answer












    share|improve this answer



    share|improve this answer










    answered May 17 '17 at 4:10









    BKSpurgeonBKSpurgeon

    96129




    96129












    • Hey thanks. I'm using a class object on my server since I need to define methods on the subscription_plan (subscription_plan.max_quantity, etc.) which is why I want to initialize some Ruby object where I can add whatever methods I want. I edited my example to include this. I'm not just using this as string enum.
      – Cyril Duchon-Doris
      May 17 '17 at 10:50












    • @CyrilDuchon-Doris: You should be able to still add methods and behavior specific to a kind of plan. ActiveRecord allows you to specify a "descriminator" column to instantiate a particular concrete class, giving you some of the benefits of polymorphism.
      – Greg Burghardt
      Jul 11 '18 at 21:48










    • Also just realized the OP has a "mongo-db" tag.
      – Greg Burghardt
      Jul 11 '18 at 21:58


















    • Hey thanks. I'm using a class object on my server since I need to define methods on the subscription_plan (subscription_plan.max_quantity, etc.) which is why I want to initialize some Ruby object where I can add whatever methods I want. I edited my example to include this. I'm not just using this as string enum.
      – Cyril Duchon-Doris
      May 17 '17 at 10:50












    • @CyrilDuchon-Doris: You should be able to still add methods and behavior specific to a kind of plan. ActiveRecord allows you to specify a "descriminator" column to instantiate a particular concrete class, giving you some of the benefits of polymorphism.
      – Greg Burghardt
      Jul 11 '18 at 21:48










    • Also just realized the OP has a "mongo-db" tag.
      – Greg Burghardt
      Jul 11 '18 at 21:58
















    Hey thanks. I'm using a class object on my server since I need to define methods on the subscription_plan (subscription_plan.max_quantity, etc.) which is why I want to initialize some Ruby object where I can add whatever methods I want. I edited my example to include this. I'm not just using this as string enum.
    – Cyril Duchon-Doris
    May 17 '17 at 10:50






    Hey thanks. I'm using a class object on my server since I need to define methods on the subscription_plan (subscription_plan.max_quantity, etc.) which is why I want to initialize some Ruby object where I can add whatever methods I want. I edited my example to include this. I'm not just using this as string enum.
    – Cyril Duchon-Doris
    May 17 '17 at 10:50














    @CyrilDuchon-Doris: You should be able to still add methods and behavior specific to a kind of plan. ActiveRecord allows you to specify a "descriminator" column to instantiate a particular concrete class, giving you some of the benefits of polymorphism.
    – Greg Burghardt
    Jul 11 '18 at 21:48




    @CyrilDuchon-Doris: You should be able to still add methods and behavior specific to a kind of plan. ActiveRecord allows you to specify a "descriminator" column to instantiate a particular concrete class, giving you some of the benefits of polymorphism.
    – Greg Burghardt
    Jul 11 '18 at 21:48












    Also just realized the OP has a "mongo-db" tag.
    – Greg Burghardt
    Jul 11 '18 at 21:58




    Also just realized the OP has a "mongo-db" tag.
    – Greg Burghardt
    Jul 11 '18 at 21:58













    0














    It sounds like you really need subscription plans to be data-driven. The methods you are defining on the concrete types are really just data. No need for concrete types. You need a better database table (and I use the term "table" loosely since you tagged the question with mongodb):



    | id | name   | max_subscription_quantity | commitment_period_num | commitment_period_type |
    | 1 | Plan 1 | 120 | 2 | Month |
    | 2 | Plan 2 | 50 | NULL | NULL |


    There really isn't a need for sub classes. Just one class ought to do it:



    class Payment::SubscriptionPlan
    def commitment_period
    # return period based on type and number
    end
    end


    The only thing you need to define is commitment_period which does the calculation of the commitment_period_num and commitment_period_type columns.






    share|improve this answer


























      0














      It sounds like you really need subscription plans to be data-driven. The methods you are defining on the concrete types are really just data. No need for concrete types. You need a better database table (and I use the term "table" loosely since you tagged the question with mongodb):



      | id | name   | max_subscription_quantity | commitment_period_num | commitment_period_type |
      | 1 | Plan 1 | 120 | 2 | Month |
      | 2 | Plan 2 | 50 | NULL | NULL |


      There really isn't a need for sub classes. Just one class ought to do it:



      class Payment::SubscriptionPlan
      def commitment_period
      # return period based on type and number
      end
      end


      The only thing you need to define is commitment_period which does the calculation of the commitment_period_num and commitment_period_type columns.






      share|improve this answer
























        0












        0








        0






        It sounds like you really need subscription plans to be data-driven. The methods you are defining on the concrete types are really just data. No need for concrete types. You need a better database table (and I use the term "table" loosely since you tagged the question with mongodb):



        | id | name   | max_subscription_quantity | commitment_period_num | commitment_period_type |
        | 1 | Plan 1 | 120 | 2 | Month |
        | 2 | Plan 2 | 50 | NULL | NULL |


        There really isn't a need for sub classes. Just one class ought to do it:



        class Payment::SubscriptionPlan
        def commitment_period
        # return period based on type and number
        end
        end


        The only thing you need to define is commitment_period which does the calculation of the commitment_period_num and commitment_period_type columns.






        share|improve this answer












        It sounds like you really need subscription plans to be data-driven. The methods you are defining on the concrete types are really just data. No need for concrete types. You need a better database table (and I use the term "table" loosely since you tagged the question with mongodb):



        | id | name   | max_subscription_quantity | commitment_period_num | commitment_period_type |
        | 1 | Plan 1 | 120 | 2 | Month |
        | 2 | Plan 2 | 50 | NULL | NULL |


        There really isn't a need for sub classes. Just one class ought to do it:



        class Payment::SubscriptionPlan
        def commitment_period
        # return period based on type and number
        end
        end


        The only thing you need to define is commitment_period which does the calculation of the commitment_period_num and commitment_period_type columns.







        share|improve this answer












        share|improve this answer



        share|improve this answer










        answered Jul 11 '18 at 21:58









        Greg BurghardtGreg Burghardt

        4,936620




        4,936620






























            draft saved

            draft discarded




















































            Thanks for contributing an answer to Code Review 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.


            Use MathJax to format equations. MathJax reference.


            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%2fcodereview.stackexchange.com%2fquestions%2f163259%2fpersist-relation-to-a-class-and-not-a-class-instance-in-ruby-on-rails-with-mon%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

            How to make a Squid Proxy server?

            Is this a new Fibonacci Identity?

            19世紀