Я новичок в языке C, поэтому в этом коде могут быть логические ошибки, которых я еще не знаю. Есть структура для карты, которая имеет значение и масть как поля.

Мой мыслительный процесс для этого кода был следующим:

  • Сначала создайте указатель на карточку.
  • Во-вторых, создайте карточку в цикле for и назначьте поля этой карточки.
  • Наконец, укажите на созданную карточку и увеличьте указатель.

Повторите этот процесс, чтобы создать подряд 52 карты в адресах памяти. В основном я собирался создать колоду карт для каждых 8 байтов в памяти, но строка card_ptr++; внутри цикла не работает, как я думал. Есть идеи, что здесь не так?

Я протестировал прокомментированный раздел, и он увеличивает указатели на 8 байтов, как я хотел, но цикл этого не делает. Я также добавил операторы печати, чтобы помочь вам понять, о чем я думаю.

#include <stdio.h>

typedef enum {
  SPADES,
  HEARTS,
  DIAMONDS,
  CLUBS,
  NUM_SUITS
} suit_t;

struct card_tag {
  unsigned value;
  suit_t suit;
};
typedef struct card_tag card_t;

int main(){ 

    card_t *card_ptr;
    printf("Initial card pointer created. %d\n", card_ptr);
    for(int i =SPADES; i < NUM_SUITS; i++){
        for(int j = 1; j < 14; j++){
            card_t card;
            card.value = j;
            card.suit = i;
            printf("Card -> Value = %d Suit = %d, is created.\n", card.value, card.suit);
            card_ptr = &card;
            printf("%d points to the last card.\n", card_ptr);
            card_ptr++;
            printf("Pointer is incremented to %p\n\n", card_ptr);
        }   
    }

    /*card_t *card_ptr;
    printf("%d\n", card_ptr);
    card_ptr++;
    printf("%d\n", card_ptr);
    card_ptr++;
    printf("%d\n", card_ptr);
    */
}
1
gunesevitan 15 Сен 2018 в 20:12

2 ответа

Лучший ответ

Ваш код явно ведет к неопределенному поведению. Если вы хотите сделать это с помощью указателей, рассмотрите приведенный ниже код.

1: Сначала вам нужно объявить указатель на указатели.

card_t **card_ptr = malloc(sizeof(card_t*)*NUM_SUITS);

2: Затем выделите память для каждого указателя.

card_ptr[i] = malloc(sizeof(card_t)*14);

3: Увеличьте указатель, как показано ниже.

card_ptr[i]++;

4: освободите память с помощью free после завершения работы.

Образец кода:

#include <stdio.h>
#include<stdlib.h>

typedef enum {
  SPADES,
  HEARTS,
  DIAMONDS,
  CLUBS,
  NUM_SUITS
} suit_t;

struct card_tag {
  unsigned value;
  suit_t suit;
};
typedef struct card_tag card_t;

int main(){ 

    card_t **card_ptr = malloc(sizeof(card_t*)*NUM_SUITS);
    if (card_ptr == NULL) return 0;

    printf("Initial card pointer created. %d\n", card_ptr);

    for(int i =SPADES; i < NUM_SUITS; i++){

        card_ptr[i] = malloc(sizeof(card_t)*14);
        if (card_ptr[i] == NULL) return 0;

        card_t *tempPtr = card_ptr[i];

        for(int j = 1; j < 14; j++){

            tempPtr->value = j;
            tempPtr->suit = i;

            printf("Card -> Value = %d Suit = %d, is created.\n", tempPtr->value, tempPtr->suit);

            printf("%p points to the last card.\n", tempPtr);
            tempPtr++;
            printf("Pointer is incremented to %p\n\n", tempPtr);
        }   
    }

    /*card_t *card_ptr;
    printf("%d\n", card_ptr);
    card_ptr++;
    printf("%d\n", card_ptr);
    card_ptr++;
    printf("%d\n", card_ptr);
    */

     for(int i =SPADES; i < NUM_SUITS; i++){
        free(card_ptr[i]);
        card_ptr[i] = NULL;
     }
    free(card_ptr);
    card_ptr = NULL;

}
3
kiran Biradar 15 Сен 2018 в 17:55

Более серьезным вопросом здесь является объем декларации карты. Локальные переменные выделяются в стеке, и когда они выходят за пределы области видимости, память в некотором смысле «освобождается» (доступна для использования в другом месте, хотя, вероятно, не будет повторно инициализирована до нуля). Ваш цикл постоянно использует одну и ту же область памяти, потому что card выходит за пределы области видимости и воссоздается на следующей итерации цикла.

Как указывали другие, вам нужно либо статически объявить массив карт, либо использовать malloc, чтобы они оставались в куче.

Используя ваш код в качестве отправной точки, вот пример:

Пример (объявить массив карточек и выполнить итерацию с помощью указателя) :

card_t deck[52];

card_t *card_ptr = deck; /* Set pointer to the beginning of the array of cards */

printf("Initial card pointer created. %d\n", card_ptr);
for(int i =SPADES; i < NUM_SUITS; i++){
    for(int j = 1; j < 14; j++){
        card_ptr->value = j;
        card_ptr->suit = i;
        printf("Card -> Value = %d Suit = %d, is created.\n", card_ptr->value, card_ptr->suit);
        printf("%d points to the last card.\n", card_ptr);
        card_ptr++;
        printf("Pointer is incremented to %p\n\n", card_ptr);
    }   
}
1
Lasidar 15 Сен 2018 в 22:12