我的C代码有什么问题？

Question

在z/os主机上工作时，出现以下情况。我无法记录交易。 这里是我的代码， 它是使用jcl编写的事务。 Bur有些事情出错了。

/* Include standard C header files */   
#include <ctype.h>    
#include <stdio.h>    
#include <stdlib.h>    
#include <string.h>        
/* Include WebSphere MQ API header file */  
#include <cmqc.h>        
/* Boolean constants */   
#define TRUE 1     
#define FALSE 0        
/* Function prototypes */   
void displayUsage (char *programName);   
void displayMQError (char *message, MQLONG compCode, MQLONG reason);  
double parseAmount (char *amountString);   
/* Main Program    */ 

int main (int argc, char *argv[]) 
{   
    /* Variable definitions */  
    double paymentAmount;      /* Payment amount  */ 

    MQBYTE msgBuffer[1024];     /* Buffer for messages */ 

    MQCHAR* qmgrName; /* Queue manager name */ 
    MQCHAR* requestQueueName;     /* Request queue name */ 
    MQCHAR* replyQueueName;     /* Reply queue name  */ 
    MQCHAR* userID; /* Contestant user ID */ 
    MQCHAR* paymentDescription;    /* Payment description */ 

    MQGMO getMsgOptions = {MQGMO_DEFAULT}; /* Get options   */ 

    MQHCONN hConn = MQHC_UNUSABLE_HCONN;  /* Connection handle */ 

    MQHOBJ requestHObj = MQHO_UNUSABLE_HOBJ; /* Request queue handle */ 
    MQHOBJ replyHObj = MQHO_UNUSABLE_HOBJ; /* Reply queue handle */ 

    MQLONG compCode; /* API completion code */ 
    MQLONG openOptions;      /* Open queue options */ 
    MQLONG reason; /* API reason code  */ 
    MQLONG replyMsgLength = 0;    /* Reply msg length  */ 

    MQMD  requestMsgDesc = {MQMD_DEFAULT}; /* Message descriptor */ 
    MQMD  replyMsgDesc = {MQMD_DEFAULT};  /* Message descriptor */ 

    MQOD  requestQueueDesc = {MQOD_DEFAULT}; /* Object descriptor */ 
    MQOD  replyQueueDesc = {MQOD_DEFAULT}; /* Object descriptor */ 

    MQPMO putMsgOptions = {MQPMO_DEFAULT}; /* Put options   */ 

    /*****************************************/ 
    /* Initialisation and parameter checking */ 
    /*****************************************/ 

    printf ("********************************************************\n"); 
    printf ("Credit card payment unit test tool\n"); 
    printf ("********************************************************\n"); 
    printf (" \n");     

    switch (argc) 
    {  
    case 7 : qmgrName   = argv[1];  
       requestQueueName = argv[2];  
       replyQueueName = argv[3];  
       userID   = argv[4];  

       paymentAmount = parseAmount (argv[5]);      

       if ((paymentAmount < 0.01)  
       || (paymentAmount > 9999.99)) 
       {     
       printf ("The payment amount must be a valid numeric " \ 
"within the range 0.01 to 9999.99\n");   

       displayUsage (argv[0]);   
       return (1);  
       }     

       paymentDescription = argv[6];  

       if ((strlen (paymentDescription) < 1)     
       || (strlen (paymentDescription) > 35))    
       {     
       printf ("The payment description must be 1-35 " \  
"characters\n");   

       displayUsage (argv[0]);   
       return (1);  
       }     

       break;    

    default : printf ("Incorrect usage!\n");  
       displayUsage (argv[0]);   
       return (1);   
    }  

    printf ("You have requested a payment of %.2f with description " \  
      "'%s' for contestant %s\n", paymentAmount,      
      paymentDescription, userID);   

    printf (" \n");     

你能帮我吗？ 我不知道如何编写我的Parm参数...

Answer 1

当您遇到问题时，尝试通过程序跟踪所有输出回输入。

从您的其他问题之一：

********************************************************       
Credit card payment unit test tool       
********************************************************       

Incorrect usage!                 

Program Usage                 
-------------                 

PAYMENT <queue manager> <cics request queue> <reply queue> <userid> <payment value> <payment description> 

The payment description has a maximum length of 35 characters.     

The monetary value must be within the range 0.01 - 9999.99 

的前两行是很容易解释。最后两行，并非如此。他们在那里闪烁我们，因为确切的文字出现在节目中。但是，似乎在displayUsage函数中使用完全相同的文本。

在不同的错误情况下显示相同的文本是不好的。这就是为什么我们使用消息编号 - 消息可能是相同的，但编号将确切地标识它来自哪里。

线索是，你甚至不能得到一个非常简单的PARM被接受。

另一个线索是Incorrect usage!。

何时打印该信息？当有不止七个参数时。错......但你只供应六个。

所以，如果应该有六个参数，程序是错误的。如果你应该提供七个参数，你的PARM是错误的。

您可以

// PARM='A,B,C,D,1,F,G' 

，将工作证明这一点。

顺便说一句，为什么使用switch而不是简单的if呢？使用if会使问题更加明显。