Нарушение доступа к памяти в C

Есть несколько проблем с вашим кодом.

1. Член entrys является указателем на char, который никогда не инициализируется и не получает динамически выделяемую память. Вы объявили остальные 3 члена статически и инициализировали их значения с помощью strcpy. Однако strcpy не следует использовать для нераспределенных указателей, так как вы можете записывать в произвольное пространство памяти за пределами памяти, выделенной для запуска вашей программы. Нарушение доступа к памяти #1.

2. Вы выделяете Periodictable на основе пользовательского ввода. Итак, если пользователь вводит число меньше 89, вы пытаетесь получить доступ к другому пространству памяти, которое может быть за пределами вашей программы, когда вы ссылаетесь на Periodictable[88]. Поскольку вы знаете количество элементов с самого начала, вы можете использовать простой фиксированный массив, который будет содержать все 118 элементов (таким образом, размер 119, если вы хотите сохранить индекс массива по номеру элемента и игнорировать элемент 0). Нарушение доступа к памяти #2.

3. Когда вы выделяете массив Periodictable, вы забыли его инициализировать. Помните, что C не инициализирует массивы по умолчанию, поэтому вы должны сделать это самостоятельно. Подойдет вызов memset() для всего массива.

4. if (Periodictable[i].entrys) не будет работать, поскольку вы не разыменовываете указатель entrys из исходной реализации. Если вы решили оставить entrys в качестве указателя, вместо этого вы можете написать if (*Periodictable[i].entrys). Однако кажется, что нет необходимости использовать указатель, вместо этого можно было бы использовать простой char (на самом деле bool был бы даже более подходящим и разборчивым).

5. Вы обязательно пропускаете элемент с оператором else i++, он просто еще не отображался, потому что в массиве зарегистрировано недостаточно элементов. Вам не нужно снова увеличивать i, поскольку цикл for уже делает это.

6. В вашем коде есть два стиля инициализации, и оба они могут работать. Один — записывать значения непосредственно в массив, другой — объявлять элементы в стеке, записывать в них и копировать в массив. В приведенном ниже коде два элемента инициализируются по-разному в качестве примера.

Итак, вот как будет выглядеть код.

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

#define MAXSIZE 119

typedef struct
  {
    char Name[20];
    char Symbol[3];
    char Atomicnumber[5];
    char entrys;
  } Element;


int main(void)
{
    Element Periodictable [MAXSIZE];
    memset(&Periodictable, 0, sizeof(Element) * MAXSIZE);

    Element Aluminium;
    strcpy(Aluminium.Name, "Aluminium");
    strcpy(Aluminium.Symbol, "Al");
    strcpy(Aluminium.Atomicnumber, "13");
    Aluminium.entrys = 1;
    memcpy(&Periodictable[13], &Aluminium, sizeof(Element));

    strcpy(Periodictable[88].Name, "Radium");
    strcpy(Periodictable[88].Symbol, "Ra");
    strcpy(Periodictable[88].Atomicnumber, "88");
    Periodictable[88].entrys = 1;

    printf("Recorded elements:\n");
    printf("\n");
    for (int i=1; i<= MAXSIZE; i++)
    {
        if (Periodictable[i].entrys)
        {
            printf("Name: %s \n",Periodictable[i].Name);
            printf("Symbol: %s \n",Periodictable[i].Symbol);
            printf("Atomic number: %s \n",Periodictable[i].Atomicnumber);
            printf("\n");
        }
    }

    return (0);
}

И выводит:

Recorded elements:

Name: Aluminium
Symbol: Al
Atomic number: 13

Name: Radium
Symbol: Ra
Atomic number: 88

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