Design and Architecture

venomation wrote:

It does mention that some people may find that idea "unprofessional"

I are Troll

Eddy Vluggen wrote:

I'm not going to provide a plug-in architecture for something like a ping-command

Ray Cassick wrote:

if as a developer/architect, you see an opportunity to design for expansion then why not do it?

Ray Cassick wrote:

1) Allows the user to select a command
2) Allows the user to provide basic command line args
3) Allows the user to get help explaining the user of the command
4) Allows the use of various command line options
5) ...

venomation wrote:

should be built for what the requirements currently do and not to make them for future changes unless required!

I are Troll

Eddy Vluggen wrote:

It's the command line environment that provides that functionality. It's strictly text, and as such, the ping-command does not offer any extensions for a visual plug-in to configure it.

Ray Cassick wrote:

Then perhaps we have a different definition of 'extensibility'

it says that design of software should be minimal but not entirely suitable for change.

Ray Cassick wrote:

Actually, given the way PING (as well as just about every other command line driven tool on windows) was written, you CAN wrap a visual component around it if you wanted to now.

Ray Cassick wrote:

I am not trying to say you are wrong here, and I know that we have all fallen into the 'just get it done trap' before, and I am sure at some point been bitten by it also.

I are Troll

David Skelly wrote:

what if

venomation wrote:

says that design of software should be minimal but not entirely suitable for change.

popchecker wrote:

So I am well aware of requirements.

      struct IParameter
      {
            int m_ID;
            virtual ~IParameter(){}
      };
      class IParameterSetter
      {
            public: 
            virtual ~IParameterSetter(){}
            virtual int SetParameter(IParameter *pParam) = 0; 
      };
      class IFactorySpecific
      {
            public:
            virtual void Execute()=0;                  
      };
      class ISubjectExecutor : public IFactorySpecific, public IParameterSetter
      {
            public:
            enum PARAMETERID
            {
                ID_CLSID,
                ID_Guids 
            };
            struct CLSID : public IParameter
            {
                CLSID(){m_ID = ISubjectExecutor::PARAMETERID::ID_CLSID;}                
                int a;
                ..... 
            };
            struct Guids : public IParameter
            {
                Guids(){m_ID = ISubjectExecutor::PARAMETERID::ID_Guids;}
                int x;
                ......
            };                    
      };

      class IWin32Executor: public IFactorySpecific, public IParameterSetter
      {
            public:
            enum PARAMETERID
            {
                ID_FilePath,
            };
            struct FilePath : public IParameter
            {
                FilePath(){m_ID = IWin32Executor::PARAMETERID::ID_FilePath;}
                char cszFilePath[260];
                ..... 
            };
      };
      class COMExecutor: public ISubjectExecutor
      {
            public:           
            //void setCLSID();
            //void setGuids();
            int SetParameter(IParameter *pParam)
            {
                switch(pParam->m_ID)
                {
                    case ISubjectExecutor::PARAMETERID::ID_CLSID : 
                    ISubjectExecutor::CLSID *pData = (ISubjectExecutor::CLSID *)pParam;
                    //...continue using pData, like pData->a
                    return ...;
                    break;

                    case ISubjectExecutor::PARAMETERID::ID_Guids : 
                    ISubjectExecutor::Guids *pData = (ISubjectExecutor::Guids *)pParam;
                    //...continue using pData, like pData->x
                    return ...;
                    break;
                }
            } 
          
            virtual void Execute(){.........}          

      };

      class Win32Executor : public IWin32Executor
      {
         public:                    
         //void setFilePath();
         //same can be done for  Win32Executor 
         int SetParameter(IParameter *pParam)
         {
              ......
              ......
         }
         virtual void Execute(){.........}          
      }; 


//using the above class structure
//fetch COMExecutor
IFactorySpecific *pExecutor = Factory::GetExecutor("COMExecutor");
//set CLSID
ISubjectExecutor::CLSID stParam;
stParam.a = 10;
pExecutor->SetParameter(&stParam); 

//fetch Win32Executor 
IFactorySpecific *pExecutor = Factory::GetExecutor("Win32Executor ");
//set FilePath
IWin32Executor::FilePath stParam;
strcpy(stParam.cszFilePath, "c:\\abc\\xyz.exe");
pExecutor->SetParameter(&stParam);