Остановка выполнения программы при scanf? - PullRequest
Новая
       17

Остановка выполнения программы при scanf?

2 голосов
/ 18 апреля 2010

main.c (со всеми заголовками, такими как stdio, stdlib и т. Д.): 

int main()
{
int input;

while(1)
{
    printf("\n");
    printf("\n1. Add new node");
    printf("\n2. Delete existing node");
    printf("\n3. Print all data");
    printf("\n4. Exit");
    printf("Enter your option -> ");
    scanf("%d", &input);

    string key = "";
    string tempKey = "";
    string tempValue = "";
    Node newNode;
    Node temp;
    switch (input) {
        case 1:
            printf("\nEnter a key: ");
            scanf("%s", tempKey);
            printf("\nEnter a value: ");
            scanf("%s", tempValue);          //execution ternimates here

            newNode.key = tempKey;
            newNode.value = tempValue;

            AddNode(newNode);
            break;
        case 2:
            printf("\nEnter the key of the node: ");
            scanf("%s", key);
            temp = GetNode(key);
            DeleteNode(temp);
            break;
        case 3:
            printf("\n");
            PrintAllNodes();
            break;
        case 4:
            exit(0);
            break;
        default:
            printf("\nWrong option chosen!\n");
            break;
    }
}

return 0;
}

storage.h: 

#ifndef DATABASEIO_H_
#define DATABASEIO_H_

//typedefs
typedef char *string;

/*
 * main struct with key, value,
 * and pointer to next struct
 * Also typedefs Node and NodePtr
 */
typedef struct Node {
    string key;
string value;
struct Node *next;
} Node, *NodePtr;

//Function Prototypes
void AddNode(Node node);
void DeleteNode(Node node);
Node GetNode(string key);
void PrintAllNodes();

#endif /* DATABASEIO_H_ */

Я использую Eclipse CDT, и когда я ввожу 1, я ввожу ключ. Тогда консоль говорит. Я использовал GDB и получил эту ошибку: 

Program received signal SIGSEGV, Segmentation fault.
0x00177024 in _IO_vfscanf () from /lib/tls/i686/cmov/libc.so.6

Есть идеи, почему?

Ответы [ 4 ]

5 голосов
/ 18 апреля 2010

вы должны выделить достаточно памяти для ваших строк (typedef char * string) для чтения с помощью scanf ()

3 голосов
/ 18 апреля 2010
  • Как сказали Джонатан и Патрик, Сначала выделите память , а затем передайте указатели / массивы в scanf.

//Change the value here to change the size of Ur strings (char-arrays actually)
#define MAX_LENGTH 20

char key[MAX_LENGTH];

char tempKey[MAX_LENGTH];

char tempValue[MAX_LENGTH];

Возможно, вы также захотите проверить небольшое прохождение по GDB, которое у меня здесь:

> 2600 Гц / Взлом в любой исполняемый файл с использованием gdb /

GOODLUCK !!

CVS @ 2600Hertz

2 голосов
/ 18 апреля 2010

Вы должны выделить все хранилище, которое scanf() собирается использовать, прежде чем вызывать его; scanf() не выделяет память. Ваши строки пусты; для значений выделено минимальное хранилище. Ввод любых дополнительных данных является катастрофой.

Кроме того, scanf () ожидает символьный указатель, а не строковую ссылку или значение - поскольку это переменная функция, компилятор может предупреждать об этом только в ограниченном количестве.

0 голосов
/ 18 апреля 2010

Хм, вы уверены, что scanf может использовать предоставленную строку для хранения данных?

Я бы попробовал использовать достаточно большой буфер символов или переключиться на реальные функции C ++ для чтения ввода.

...