0

私のコードは機能しますが、正しく印刷されません。これは私の入力です：

文字は 1、int は 2、float は 3、単語は 4 で入力してください

1 abcdefghijklmnop 3 123.4 45.54 6.0 7890.09876 2 123 34 23 12345 4 aaaaa bbbbb ccccc ssssssssssssssssss

本来のように cccccc の後に停止させる方法がわかりません。ランダムなものを入力して、スペースを埋める前に入力する必要があります。mallocを使用していないことに関係があると確信しています。

次に、これが出力です。

タイプ 1: abcdefghijklmnop タイプ 3: 123.400002/ 45.540001/ 6.000000/ 7890.098633 タイプ 2: 123、34、23、12345 タイプ 4: ccccc ccccc ccccc タイプ 1: sssssssssssssss

タイプ 4 では、aaaaa bbbbb ccccc になるはずですが、そうではありません。また、タイプ 1 では表示されませんが、この奇妙な四角いグリッチのように印刷されます。

これが私のコードです。3つのファイルに入っています

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include "lab5.h"
#include "lab5dispatch.c"
#define MAX_ENTRIES 16

int main()
{
MESSAGE cache[MAX_ENTRIES];
int i = 0;
printf("Please enter in 1 for characters, 2 for ints, 3 for floats and 4 for words\n");


while (scanf("%d", &cache[i].messageType) != EOF && i < MAX_ENTRIES) 
{
   switch(cache[i].messageType)
    {
    case 1:
        scanf("%16s", &cache[i].MESSAGE_CONTENT.charPointer);
        break;
    case 2:
        scanf("%d", &cache[i].MESSAGE_CONTENT.theInts[0]);
        scanf("%d", &cache[i].MESSAGE_CONTENT.theInts[1]);
        scanf("%d", &cache[i].MESSAGE_CONTENT.theInts[2]);
        scanf("%d", &cache[i].MESSAGE_CONTENT.theInts[3]);
        break;
    case 3:
        scanf("%f",  &cache[i].MESSAGE_CONTENT.theFloats[0]);
        scanf("%f",  &cache[i].MESSAGE_CONTENT.theFloats[1]);
        scanf("%f",  &cache[i].MESSAGE_CONTENT.theFloats[2]);
        scanf("%f",  &cache[i].MESSAGE_CONTENT.theFloats[3]);
        break;
    case 4:
        scanf("%s", &cache[i].MESSAGE_CONTENT.word1);
        scanf("%s", &cache[i].MESSAGE_CONTENT.word2);
        scanf("%s", &cache[i].MESSAGE_CONTENT.word3);
        break;
    }
    i++;
}
message_dispatcher(cache, i);
}

2つ目のファイルです。

#ifndef LAB5_H_ /* to prevent re-definitions */
#define LAB5_H_ /* that cause errors */
#include <stdio.h>
#include <stdlib.h>
#include <string.h>

//the unsigned short is to store what type of data the main struct will store

typedef struct MESSAGE
{
unsigned short int messageType;
union
{
    char * charPointer; //this is for the string
    int theInts[4];
    float theFloats[4];
    char word1[5]; //can probably use a 2d array here but that's too complicated right now haha
    char word2[5];
    char word3[5];
} MESSAGE_CONTENT;
} MESSAGE;

そしてこれが最後のファイルです。

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include "lab5.h"
void message_dispatcher( MESSAGE msg[], int j ) {
int i;
for (i = 0; i < j; i++)
{
    switch(msg[i].messageType)
    {
    case 1:
        printf("Type 1: %s\n", &msg[i].MESSAGE_CONTENT.charPointer);
        break;

    case 2:
        printf("Type 2: %d, %d, %d, %d\n", msg[i].MESSAGE_CONTENT.theInts[0],
                msg[i].MESSAGE_CONTENT.theInts[1], msg[i].MESSAGE_CONTENT.theInts[2],
                msg[i].MESSAGE_CONTENT.theInts[3]);
        break;

    case 3:
        printf("Type 3: %f/ %f/ %f/ %f \n", msg[i].MESSAGE_CONTENT.theFloats[0], msg[i].MESSAGE_CONTENT.theFloats[1], msg[i].MESSAGE_CONTENT.theFloats[2], msg[i].MESSAGE_CONTENT.theFloats[3]);
        break;

    case 4:
        printf("Type 4: %s %s %s\n", msg[i].MESSAGE_CONTENT.word1, msg[i].MESSAGE_CONTENT.word2, msg[i].MESSAGE_CONTENT.word3);
        break;
    }
}
}
4

2 に答える 2

2

word1、word2、および word3 はすべて同じユニオン MESSAGE_CONTENT にあるため、同じメモリに格納されます。したがって、それらのいずれかを読み取るたびに、最後に読み取った値によって上書きされたように見えます。

http://en.cppreference.com/w/cpp/language/unionをご覧ください

「char word[3][6];」を使用してみてください。組合で。そうすれば、3 つの単語すべてを保存できます。また、各単語に NULL 文字を入れる余地があることも確認する必要があります。普通はそれで逃げることはできなかったでしょうが、組合にはもっと余裕がありました.

これらのいずれかにより、必要なものに近づくことができます (もちろん、コード内の他のバグは無視されます):

代替＃1

typedef struct MESSAGE
{
    unsigned short int messageType;
    union
    {
        char * charPointer;
        int theInts[4];
        float theFloats[4];
        char theWords[3][6];
    } MESSAGE_CONTENT;
} MESSAGE;

代替＃2

typedef struct MESSAGE
{
    unsigned short int messageType;
    union
    {
        char * charPointer;
        int theInts[4];
        float theFloats[4];
        struct {
            char word1[6];
            char word2[6];
            char word3[6];
        } WORDS;
    } MESSAGE_CONTENT;
} MESSAGE;
于 2013-02-25T03:32:30.400 に答える
0

コードの次の部分が原因で、間違った（期待どおりではない）入力が発生します。

case 4:
        scanf("%s", &cache[i].MESSAGE_CONTENT.word1);
        scanf("%s", &cache[i].MESSAGE_CONTENT.word2);
        scanf("%s", &cache[i].MESSAGE_CONTENT.word3);
        break;

scanf関数の構文によると、パラメータは次
のとおり です。1.パラメータ文字列。
2.オブジェクトのアドレス。

2番目（オブジェクトのアドレス）
をcache [i] .MESSAGE_CONTENT.word == char word1 [5] ==配列上のconstポインターとして注意してください。これはすでにアドレスであるため、コードでは＆を省略してください。

case 4:
            scanf("%s", cache[i].MESSAGE_CONTENT.word1);
            scanf("%s", cache[i].MESSAGE_CONTENT.word2);
            scanf("%s", cache[i].MESSAGE_CONTENT.word3);
            break;
于 2013-02-25T03:50:43.687 に答える