0 
if (ISADIGIT != atoi2(&word[i])){

なぜそれができないのですか（上記を参照）。「期待される表現」が得られますか？

#include <ctype.h>
#include <stdio.h>
#include <ctype.h>
#include <limits.h>
#include <stdlib.h>
#include <errno.h>

#define STARTSWITH0 "Starts with 0";
#define NEGATIVESIGNNOTATBEGINNING "Negative sign not at beginning";

#define ISADIGIT "Is a digit";

#define MORETHANONENEGATIVESIGN "More than one negative sing";
#define MORETHANONEDECIMALPLACE "More than one decimal place";
#define NUMBERTOOLARGEORNUMBERTOOSMALL "Number too large or number too small";
#define CONVERSIONERROR "Conversion Error";
#define CANNOTCONTAINLETTERS "Cannot contain letters";

char* atoi2(char str[]);
int main(int argc, char ** argv){

    char c, word[1000]; int count;


        printf("enter word: ");
        count = 0;

        while(1){
            c = getchar();
            if (c == EOF); 
                return 0;

            if (c == ' ') {
                continue;
            }

            if (c == '\n' || c == '\t'){
                word[count] = '\0';
                break;
            }
            word[count++] = c;
        }
        int i;


    for (i = 0; i < count; i++){

        if (ISADIGIT != atoi2(&word[i])){
            printf("%s\n", atoi2(&word[i]));
            return 1;
        }

    }

    int sum;
    for (i = 0; i < count; i++){
        sum += word[i];
    }

    printf("sum is: %d", sum);


}


char* atoi2(char * str)
{
    int i, v, d, n, errno;
    i = v = d = n = 0;

    while(str[i] != '\0'){

        /* this means the first value is a 0, therefore it should not be numeric */
        if (i == 0 && str[i] == '0' && str[i+1] != '.' && str[i+1] != '\0') return STARTSWITH0;

        if(!isdigit(str[i])) {

            if (isalpha(str[i])) return CANNOTCONTAINLETTERS; /* must make sure the digit we receive is not a letter */

            if (str[i] == '-'  && i > 0) return NEGATIVESIGNNOTATBEGINNING; /* negative sign must come first */

            if (str[i] == '-'){ ++n; ++i; continue; } /* first negative sign, pass through */

            if (str[i] == '.') { ++d; } /* periods are ok, just not too many */

        }
        ++i;
    }

    if (n > 1) return MORETHANONENEGATIVESIGN; /* can't have more than one negative sign */

    if (d > 1) return MORETHANONEDECIMALPLACE; /* this means there is more than one decimal place */

    return ISADIGIT;
}

素晴らしいご意見をありがとうございました。修正したプログラムはこちら

#include <ctype.h>
#include <stdio.h>
#include <ctype.h>
#include <limits.h>
#include <stdlib.h>
#include <errno.h>

#define STARTSWITH0 1 /*"Starts with 0"*/
#define NEGATIVESIGNNOTATBEGINNING 2 /*"Negative sign not at beginning"*/
#define MORETHANONENEGATIVESIGN 3 /*"More than one negative sing"*/
#define MORETHANONEDECIMALPLACE 4 /*"More than one decimal place"*/
#define NUMBERTOOLARGE 5
#define CANNOTCONTAINLETTERS 6 /*"Cannot contain letters"*/
#define CONTAINSAPERIOD 7 /*"Integers cannot contain decimal points"*/

int atoi2(char *);
char * geterrorstring(int);
int main(int argc, char ** argv){

    char c, nums[1000][1000];

    while(1) {
        printf("enter numbers: ");
        int digits = 0, row = 0, maxrows = 1;

        /* gets data from user, puts it into an array */
        while(1){
            c = getchar();
            if (c == EOF) return 0;
            if (c == ' ' || c == '\t') { nums[row][digits] = '\0'; row++; maxrows++; digits = 0; }
            if (c == '\n'){ nums[row][digits] = '\0'; maxrows++; digits = 0; break; }
            nums[row][digits++] = c;
        }

        int i, j;

        int error = 0;

        /* sums up data from array and prints error message if digit is invalid */
        int sum = 0, msg = 0;
        for (i = 0; i < maxrows; i++){
                msg = atoi2(nums[i]);
                if (msg > 0){
                    printf("%s\n", geterrorstring(msg));
                    error = 1;
                }
            sum += atoi(nums[i]);
        }

        if (error == 1) continue;

        printf("sum is: %d\n", sum);

    }

}

char * geterrorstring(int code)
{
    switch (code) {
        case STARTSWITH0:
            return "Starts with 0";
            break;

        case NEGATIVESIGNNOTATBEGINNING:
            return "Negative signs must be at the begging of the number";
            break;

        case MORETHANONENEGATIVESIGN:
            return "You cannot have more than one negative sign in a number";
            break;

        case MORETHANONEDECIMALPLACE:
            return "More than one decimal place";
            break;

        case NUMBERTOOLARGE:
            return "Number too large";
            break;

        case CANNOTCONTAINLETTERS:
            return "Cannot contain letters";
            break;

        case CONTAINSAPERIOD:
            return "Integers do not have decimal places";

        default:
            return "I have no idea what error you have, but you got one";
            break;
    }
}


int atoi2(char * str)
{
    int i, v, n, errno;
    i = v = n = 0;

    while(str[i] != '\0'){

        /* this means the first value is a 0, therefore it should not be numeric */
        if (i == 0 && str[i] == '0' && str[i+1] != '.' && str[i+1] != '\0') return STARTSWITH0;

        if(!isdigit(str[i])) {

            if (isalpha(str[i])) return CANNOTCONTAINLETTERS; /* must make sure the digit we receive is not a letter */

            if (sizeof(atoi(str)) > sizeof(int)) return NUMBERTOOLARGE; /* make sure number is not that large */

            if (str[i] == '-'  && i > 0) { if (n > 1) { return MORETHANONENEGATIVESIGN; } return NEGATIVESIGNNOTATBEGINNING; } /* negative sign must come first */

            if (str[i] == '-'){ ++n; ++i; continue; } /* first negative sign, pass through */

            if (str[i] == '.') { return CONTAINSAPERIOD; } /* periods are ok, just not too many */

        }
        ++i;
    }

    if (n > 1) return MORETHANONENEGATIVESIGN; /* can't have more than one negative sign */

    return 0;
}
4

2 に答える 2

3

変化する

#define ISADIGIT "Is a digit";

に

#define ISADIGIT "Is a digit"

あなたの場合、他の人が言うこととは反対に、ポインタを文字列定数と比較しても問題ありません。

于 2012-10-11T15:24:44.820 に答える
0

当面の問題の根本的な原因は、マクロ定義の末尾にあるセミコロンです (piokuc が指摘したように)。プリプロセッサ ステートメントにはセミコロンは必要ありません。現在の実装で考慮すべきもう 1 つの重要な点は次のとおりです。

はマクロであるためISADIGIT、プリプロセッサはISADIGITコード内の のすべてのインスタンスを指定された文字列に置き換えます。複数の場所で使用するISADIGITと、それぞれが文字列に置き換えられます。これらはそれぞれ、個別の独立した文字列リテラルとして扱われます。それらのコンテンツはたまたま同じですが、それらは別個のオブジェクトです。これは、atoi2関数が関数内に含まれる文字列リテラルへのポインターを返し、関数mainがそのポインターを 内に含まれる文字列リテラルへのポインターと比較していることを意味しますmain。これらは異なるオブジェクトへのポインターであるため、常に比較すると等しくありません。それらがたまたま同じ内容を含む文字列であるという事実は、コンパイラに関する限り重要ではありません。

そうは言っても、このコードは場合によっては機能しているように見えるかもしれません。一部のコンパイラには、同一の文字列リテラルを組み合わせてスペースを節約するオプションの最適化があります。その場合、ISADIGITマクロの展開は一般的な文字列リテラルへのポインターに要約され、実行している比較が機能するmain可能性があります。ただし、コードが適切に機能するために、特定のコンパイラ最適化が利用可能で有効になっていることに依存しないでください。これを行うと、コンパイラをアップグレードしたり、別のコンパイラでビルドしたり、「リリース」バージョンではなく「デバッグ」バージョンをビルドしたりすると、コードが壊れるという大きなリスクが発生します。

一般に、関数からの戻りコードとして文字列を使用しないでください。C では文字列は (他の言語とは異なり) 真のオブジェクトではありません。文字列を操作するには、多くの隠れた「落とし穴」があります。関数からステータス コードを返す従来の方法は、戻り値に整数を使用することです。関数は正常に完了するとゼロを返し、ゼロ以外の戻り値は何らかのエラーを示します。すべてのエラー コードがマクロとして抽象化されているため、#defineステートメント内の文字列を数字に置き換え、 の戻り値の型を に変更するatoi2だけintです。あなたが行う必要がある他の唯一の変更はprintf("%s")、戻り値の行にありますatoi2. 整数を文字列に変換するには、別のメソッドが必要です。これを行う一般的な 2 つの方法は、関数またはテーブルを使用することです。

// Function method:
// Use like: printf("%s", code_to_string(code));
const char* code_to_string(int code) {
    switch(code) {
        case STARTSWITH0:
            return "Starts with 0";
        case NEGATIVESIGNNOTATBEGINNING:
            return "Negative sign not at beginning";
        ...
        default:
            return "Unknown error";
    }
}

// Table method (assumes error codes are numbered 1,2,3,...
// Use like: printf("%s", error_strings[code]);
const char* error_strings[] = {
    "No error", // 0
    "Starts with 0", // 1
    "Negative sign not at beginning", // 2
    ...
};
于 2012-10-11T16:08:38.260 に答える