You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Is your feature request related to a problem? Please describe.
The SekibanCommandHandlerAggregateNullException class may be better off inheriting from the ArgumentNullException class, rather than the Exception class. As a derived class of the ArgumentNullException class, it will be able to perform null checks using ThrowIfNull method of the ArgumentNullException class.
Describe the solution you'd like
Change the definition of the SekibanCommandHandlerAggregateNullExceptionclass as
public class SekibanCommandHandlerAggregateNullException : ArgumentNullException, ISekibanException;
The text was updated successfully, but these errors were encountered:
tkawae
changed the title
Inherit the SekibanCommandHandlerAggregateNullException class from the ArgumentNullException class
Inherit the SekibanCommandHandlerAggregateNullException from the ArgumentNullException
Jan 8, 2024
tkawae
changed the title
Inherit the SekibanCommandHandlerAggregateNullException from the ArgumentNullException
Inherit SekibanCommandHandlerAggregateNullException from ArgumentNullException
Jan 8, 2024
@tomohisa I realized my mistake. If I just inherit from ArgumentNullException, the ThrowIfNull method will throw ArgumentNullException. It would be better to override the ThrowIfNull method, or to stop inheriting from ArgumentNullException and implement ThrowIfNull in SekibanCommandHandlerAggregateNullException.
Is your feature request related to a problem? Please describe.
The
SekibanCommandHandlerAggregateNullException
class may be better off inheriting from theArgumentNullException
class, rather than theException
class. As a derived class of theArgumentNullException
class, it will be able to perform null checks usingThrowIfNull
method of theArgumentNullException
class.Describe the solution you'd like
Change the definition of the SekibanCommandHandlerAggregateNullExceptionclass as
The text was updated successfully, but these errors were encountered: