Путаница с функциями и указателями. Платина. Ansi c

@Yn94k0 · Регистрация: 10.01.2013

Author24 — интернет-сервис помощи студентам

Прошу прощения за очередную лишнюю тему, где школьник запутался в указателях и не может передать структуру в функцию, но нужна помощь, ибо перелопачивать нельзя, надо исправить.
Мне нужно было заполнить и вывести динамический массив структур в функциях. Очевидно, я от души налажал с передачей данных в функцию. Методом тыка исправить не вышло. Если отправите читать мануалы про указатели и прочая, я пойму, но проблемы не решит: по ходу, я неполноценный.
Помогите найти ошибку, пожалуйста.
Инбифо "в ДНК у тебя ошибка".
И да, если кроме помощи в нахождении ошибки мне укажут, почему именно это - отвратительный говнокод, никому не нужная моя благодарность будет и вовсе бесконечной.

C++

#include "stdafx.h"
#include <stdlib.h>
#include <string.h>
#include "conio.h"
 
//создание структуры
struct publications
    {
        int id;
        char name[20];
        char address[20];
        int numofpub;
    };
 
void enter(publications *arr, int count)
{
    arr = (publications *)realloc(arr,(count+1)*sizeof(publications));//увеличение выделенной памяти на 1 элемент
    printf("enter id, name, address and number of publications\n ");
    scanf_s("%d",arr[count].id);//ввод
    _flushall();//очистка буфера ввода
    gets(arr[count].name);//ввод 
    _flushall();//очистка буфера ввода
    gets(arr[count].address);//ввод 
    _flushall();//очистка буфера ввода
    scanf_s("%d",arr[count].numofpub);//ввод 
    count++;//увеличение счетчика на 1
}
 
void show (publications *arr, int count)
{
    printf("ID\tname taddress\tnumber of publications\n");
    for(int i = 0;i < count;i++)//вывод на экран данных структуры
    {
    printf ("%d\t", arr[i].id);
    printf ("%s\t\t", arr[i].name);
    printf ("%s\t\t", arr[i].address);
    printf ("%f\n", arr[i].numofpub);//вывод площади
    }
}
void main()
{
    int count = 0;
    bool key = false;
    publications *arr = NULL;//созали обнуленый указатель
    arr = (publications *) malloc(sizeof(publications));//выделение памяти под 1 элемент структуры dachnik
    printf("What you gonna do? Press F1 to enter new data. Press F2 to print. Press esc to esc\n ");
    do
    {
        switch(_getch())
            {
                case 59:system("cls");
                enter(arr, count);break; 
                case 60:system("cls");
                show(arr,count);_getch();break;
                case '\x1b':system("cls");
                key = true ;break;
            }
    }   
    while(key!= true);
 
    
    //освобождение динамической памяти
    free (arr);
    getch;
}

alsav22 · 19.06.2013, 01:06

Как-то так:

C++

//#include "stdafx.h"
#include <stdlib.h>
#include <string.h>
#include <conio.h>
#include <stdio.h>
 
//создание структуры
struct publications
{
    int id;
    char name[20];
    char address[20];
    int numofpub;
};
 
void enter(publications *&arr, int &count)
{
    arr = (publications *)realloc(arr,(count + 1) * sizeof(publications));//увеличение выделенной памяти на 1 элемент
    printf("enter id, name, address and number of publications\n ");
    scanf_s("%d", &arr[count].id);//ввод
    _flushall();//очистка буфера ввода
    gets(arr[count].name);//ввод 
    _flushall();//очистка буфера ввода
    gets(arr[count].address);//ввод 
    _flushall();//очистка буфера ввода
    scanf_s("%d", &arr[count].numofpub);//ввод 
    count++;//увеличение счетчика на 1
}
 
void show (publications *arr, int count)
{
    printf("ID\tname taddress\tnumber of publications\n");
    for(int i = 0;i < count;i++)//вывод на экран данных структуры
    {
    printf ("%d\t", arr[i].id);
    printf ("%s\t\t", arr[i].name);
    printf ("%s\t\t", arr[i].address);
    printf ("%d\n", arr[i].numofpub);//вывод площади
    }
}
 
 
void main()
{
    int count = 0;
    bool key = false;
    publications *arr = NULL;//созали обнуленый указатель
    arr = (publications *) malloc(sizeof(publications));//выделение памяти под 1 элемент структуры dachnik
    printf("What you gonna do? Press F1 to enter new data. Press F2 to print. Press esc to esc\n ");
    do
    {
        switch(_getch())
            {
                case 59:system("cls");
                    enter(arr, count); break; 
                case 60:system("cls");
                    show(arr, count);_getch(); break;
                case '\x1b':system("cls");
                    key = true ;break;
            }
    }   
    while(key!= true);
  
    //освобождение динамической памяти
    free (arr);
    getch;
}

@Yn94k0 · 19.06.2013, 01:18 **[ТС]**

Сударь, вы маг и волшебник!
Спасибо за избавление :3
Кажется, я начинаю понимать, как это должно работать.

alsav22 · 19.06.2013, 01:24

И _getch() в этой строке можно убрать (лишняя пауза):

C++

...
case 60:system("cls");
       show(arr, count); break;
...

@Yn94k0 · 19.06.2013, 01:34 **[ТС]**

Хотя нет, не совсем понимаю.
Почему в этой
void enter(publications *&arr, int &count) //функция ввода
строке одновременно и операция взятия адреса, и переход по нему? Разве это не должно привести к тому же результату, что и просто void enter(publications arr, int &count)? И почему у count`а мы просто переходим по адресу (хотя в подпрограмму передавалось значение), а у arr`а его ещё и берём?

Гетч там к месту, просто в исправленном варианте теперь создаётся пауза между просмотром результатов и возвращением в главное меню, стиранием всего на экране и выводом просьбы нажать на кнопку) Но спасибо!)

alsav22 · 19.06.2013, 01:54

Сообщение от Yn94k0

Гетч там к месту,

Не к месту.

Сообщение от Yn94k0

Почему в этой
void enter(publications *&arr, int &count) //функция ввода
строке одновременно и операция взятия адреса, и переход по нему?

Это не взятие адреса, а передача в функцию указателя на начало массива и переменной count по ссылке.

Добавлено через 3 минуты
Если как было:

C++

1	void enter(publications *arr, int count);

, то в функцию передаются копии указателя и переменной, поэтому то, что происходит в функции с ними, не меняет значение переменных, объявленных в main().

@Yn94k0 · 19.06.2013, 02:06 **[ТС]**

>Не к месту.
Теперь к месту :3
>Это не взятие адреса, а передача в функцию указателя на начало массива и переменной count по ссылке.
Спасибо большое, попробую когда-нибудь это осмыслить

alsav22 · 19.06.2013, 03:10

Можно так ещё передавать в функцию:

C++

//#include "stdafx.h"
#include <stdlib.h>
#include <string.h>
#include <conio.h>
#include <stdio.h>
 
//создание структуры
struct publications
{
    int id;
    char name[20];
    char address[20];
    int numofpub;
};
 
void enter(publications **arr, int *count)
{
    *arr = (publications *)realloc(*arr, (*count + 1) * sizeof(publications));//увеличение выделенной памяти на 1 элемент
    printf("enter id, name, address and number of publications\n ");
    scanf_s("%d", &((*arr)[*count].id));//ввод
    _flushall();//очистка буфера ввода
    gets((*arr)[*count].name);//ввод 
    _flushall();//очистка буфера ввода
    gets((*arr)[*count].address);//ввод 
    _flushall();//очистка буфера ввода
    scanf_s("%d", &((*arr)[*count].numofpub));//ввод 
    ++(*count);//увеличение счетчика на 1
}
 
void show (publications *arr, int count)
{
    printf("ID\tname taddress\tnumber of publications\n");
    for(int i = 0;i < count;i++)//вывод на экран данных структуры
    {
    printf ("%d\t", arr[i].id);
    printf ("%s\t\t", arr[i].name);
    printf ("%s\t\t", arr[i].address);
    printf ("%d\n", arr[i].numofpub);//вывод площади
    }
}
 
 
void main()
{
    int count = 0;
    bool key = false;
    publications *arr = NULL;//созали обнуленый указатель
    arr = (publications *) malloc(sizeof(publications));//выделение памяти под 1 элемент структуры dachnik
    printf("What you gonna do? Press F1 to enter new data. Press F2 to print. Press esc to esc\n ");
    do
    {
        switch(_getch())
            {
                case 59:system("cls");
                    enter(&arr, &count); break; 
                case 60:system("cls");
                    show(arr, count);  break;
                case '\x1b':system("cls");
                    key = true ; break;
            }
    }   
    while(key!= true);
  
    //освобождение динамической памяти
    free (arr);
    _getch();
}

@Yn94k0 0 / 0 / 0 Регистрация: 10.01.2013 Сообщений: 18
	19.06.2013, 01:18 [ТС]	3
	Сударь, вы маг и волшебник! Спасибо за избавление :3 Кажется, я начинаю понимать, как это должно работать. 0

@Yn94k0 0 / 0 / 0 Регистрация: 10.01.2013 Сообщений: 18
	19.06.2013, 01:34 [ТС]	5
	Хотя нет, не совсем понимаю. Почему в этой void enter(publications *&arr, int &count) //функция ввода строке одновременно и операция взятия адреса, и переход по нему? Разве это не должно привести к тому же результату, что и просто void enter(publications arr, int &count)? И почему у count`а мы просто переходим по адресу (хотя в подпрограмму передавалось значение), а у arr`а его ещё и берём? Гетч там к месту, просто в исправленном варианте теперь создаётся пауза между просмотром результатов и возвращением в главное меню, стиранием всего на экране и выводом просьбы нажать на кнопку) Но спасибо!) 0

@Yn94k0 0 / 0 / 0 Регистрация: 10.01.2013 Сообщений: 18
	19.06.2013, 02:06 [ТС]	7
	>Не к месту. Теперь к месту :3 >Это не взятие адреса, а передача в функцию указателя на начало массива и переменной count по ссылке. Спасибо большое, попробую когда-нибудь это осмыслить 0

	19.06.2013, 03:10