Форум программистов [Powered by Invision Power Board]

Версия для печати темы
Нажмите сюда для просмотра этой темы в оригинальном формате

Форум программистов > C/C++: Общие вопросы > проблема с структурами

Автор: Robin 31.5.2007, 13:46

Код



#include <stdio.h>
#include <stdlib.h>
#define MAX 15

struct myaim {
    char *glob;
    char *local;
    char *net;
}inf[MAX];

int freef();
void add();
void print();


void main()
{
int a;

printf("Hey! Here i write my aims\n");
printf("SELECT:\n");
printf("1 - Add\n");
printf("2 - Show\n");

scanf("%d", &a);

if(a == 1){
    add();
}
else
    if(a == 2){
        print();
    }
    else
        printf("Unknown command\n");
}

int freef()
{
register int i;

for(i = 0; inf[i].glob[0] && i<MAX; i++);

if(i == MAX) {
    return -1;
}
return i;
}

void add()
{
    int type;
   int slot;
    char c[80];
    puts("Enter type of aim: \n");
    puts("1 - global\n");
    puts("2 - local\n");
    puts("3 - net\n");
    scanf("%d", &type);

    slot = freef();
    if(slot == -1){
        printf("\nNo free memory");
    }
else
    {
switch(type)
{
    case 1:
    printf("Enter aim\n");
    scanf("%s", inf[slot].glob);
    break;
    case 2:
    printf("Enter aim\n");
    scanf("%s", inf[slot].local);
    break;
    case 3:
    printf("Enter aim\n");
    scanf("%s", inf[slot].net);
    break;
    default: puts("Unknown format\n");
}
}

printf("Added\n");

}

void print()
{
    register int i;

    for(i = 0; i<MAX; ++i){
        if(inf[i].glob[0]){
            printf("Here what i have to do\n");
            printf("%s\n", inf[i].glob);
            printf("%s\n", inf[i].local);
            printf("%s\n", inf[i].net);
        }
    }
    printf("\n");
}

Код


C:\Documents and Settings\Стас>C:\lcc\lcc\servvvv.exe
Hey! Here i write my aims
SELECT:
1 - Add
2 - Show
1
Enter type of aim:

1 - global

2 - local

3 - net

2

abnormal program termination

просьба указать на ошибки

Автор: FiMa1 31.5.2007, 14:11

К моменту выполнения for(i = 0; inf[i].glob[0] && i<MAX; i++); в функции freef поле glob не проинициализировано ни каким значением.
А, если не секрет, каков смысл этого цикла?

Автор: JackYF 31.5.2007, 14:17

Цитата(Robin @ 31.5.2007, 13:46

)

2
abnormal program termination

Что-то я не вижу кода функции print(), которая вызывается при выборе 2-го пункта меню.

Автор: Robin 31.5.2007, 14:17

FiMa1 следовательно там 0, и запишется в первый слот, не так ли?

Автор: v_nikolaev 31.5.2007, 14:21

Цитата(Robin @ 31.5.2007, 14:17)

FiMa1 следовательно там 0, и запишется в первый слот, не так ли?

проблема в том, что glob - указатель. он проинициализирован нулём, а когда ты к нему как к массиву обращаешься, он и смотрит первый элемент массива, начинающегося с адреса 0x0 - вот и проблема.

Автор: FiMa1 31.5.2007, 14:21

Цитата(Robin @ 31.5.2007, 14:17)

FiMa1 следовательно там 0, и запишется в первый слот, не так ли?

Не так. С какой стати? Не зависимо ни от какого условия мы, при указании типа aim '1', т.е. Add, мы идем на выполнение функции freef: slot = freef(); К этому моменты указатель char *glob; не инициализирован ни каким значение (Bad Ptr), мы же с ним что-то пытаемся проделать...

Автор: v_nikolaev 31.5.2007, 14:24

Цитата(FiMa1 @ 31.5.2007, 14:21)

Цитата(Robin @ 31.5.2007, 14:17)

FiMa1 следовательно там 0, и запишется в первый слот, не так ли?

даже, если проинициализировать нулём - проблема не исчезнет smile

Автор: Robin 31.5.2007, 14:31

Код



void init()
{
register int i;
for(i = 0; i<MAX; ++i) inf[i].glob[0]= '\0';    
}

не помогло

Автор: FiMa1 31.5.2007, 14:32

Цитата(v_nikolaev @ 31.5.2007, 14:24)

даже, если проинициализировать нулём - проблема не исчезнет smile

Да, согласен. Не понятен в принципе смысл цикла в приведенном выше контексте...

Автор: FiMa1 31.5.2007, 14:50

Цитата(Robin @ 31.5.2007, 14:31)

Код

void init()
{
register int i;
for(i = 0; i<MAX; ++i) inf[i].glob[0]= '\0';    
}

не помогло

Robin, понимаешь, после объявления char *glob; есть указатель, который, по-умолчанию, указывает на адрес 0x00000000 в памяти. Т.о, когда мы пишем for(i = 0; i<MAX; ++i) inf[i].glob[0]= '\0'; , мы пытаемся обратиться к glob как к массиву (массиву символов в твоем понимании здесь) вернее к первому элементу данного массива, т.е. все к тому же адресу 0x00000000 (уже сказал v_nikolaev).
Видимо, следует сделать так (т.е. предварительно выделить память под MAX символов + проинициализировать указатель валидным адресом в памяти):

Код

void init()
{
    register int i = 0;
    inf[0].glob = (char*)malloc(MAX);
    
    while( i < MAX )
        inf[0].glob[i++] = '\0';
}

Автор: Robin 31.5.2007, 15:10

спс, только там в 7 срочке вместо inf[0].glob[i++] = '0'; - inf[0].glob[i++] = '\0';, тоесть не ноль а нулевой символ

Автор: FiMa1 31.5.2007, 15:22

Цитата(Robin @ 31.5.2007, 15:10)

только там в 7 срочке не ноль а нулевой символ

Я видел, не суть...
Не забудь, что выделенную память придется где-то освобождать!!!