Member 14719581 Ответов: 2

Я не могу понять свой ошибочный код на языке Си


Привет, ребята! что случилось?
Я пытаюсь пару часов решить досадную ошибку и получить ожидаемый результат, но каждая попытка терпит неудачу.
Не могли бы вы мне помочь, пожалуйста?
Это мой исходный код:
#define _CRT_SECURE_NO_WARNINGS

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

#define READ_ONLY "r"
#define WRITE_BINARY "wb"
#define READ_BINARY "rb"

#define MEM_ALLOC_ERR 1234
#define NO_SUCH_PATH 1235

#define NEW(type, elements) ((type* )calloc(elements, sizeof(type)))
#define LOG(message, code) {printf("%s", message); exit(code);}

typedef struct Time {
	int hh, mm, ss;
}*p_time_t;

typedef struct Log 
{
	char		date[11];
	p_time_t	time;
	uint32_t	index;
	double		voltage,
				logger_temperature,
				sun_shooting,
				sixth_field,
				seventh_field,
				env_temperature;
}*p_log_t;

FILE* OpenFile(const char* path, const char* type) {
	FILE* fptr = NULL;
	fptr = fopen(path, type);
	if (!fptr)LOG("File cant be found", NO_SUCH_PATH);
	return fptr;
}

p_log_t CreateLog()
{
	p_log_t new_log = NULL;
	new_log = NEW(Log, 1);
	if (!new_log)LOG("Memmory allocation failed", MEM_ALLOC_ERR);
	
	new_log->time = NEW(Time, 1);
	if (!new_log->time)LOG("Memmory allocation failed", MEM_ALLOC_ERR);

	return new_log;
}

void InitLog(FILE* fptr, p_log_t log){
	if(fscanf(fptr, "%s %d:%d:%d %u %lf %lf %lf %lf %lf %lf",
		log->date,
		&(log->time->hh),
		&(log->time->mm),
		&(log->time->ss),
		&log->index,
		&log->voltage,
		&log->logger_temperature,
		&log->sun_shooting,
		&log->sixth_field,
		&log->seventh_field,
		&log->env_temperature));
}

void PrintLog(p_log_t log)
{
	printf("%s %d %d %d %u %lf %lf %lf %lf %lf %lf",
		log->date,
		log->time->hh,
		log->time->mm,
		log->time->ss,
		log->index,
		log->voltage,
		log->logger_temperature,
		log->sun_shooting,
		log->sixth_field,
		log->seventh_field,
		log->env_temperature);
}

int main(int argc, char** argv)
{
	FILE* input_file = OpenFile("DataMeteo-E5.txt", READ_ONLY);
	
	p_log_t log = CreateLog();
	
	p_log_t* logs = NULL;
	int count = 0;
	FILE* output_file = OpenFile("DataMeteo-E5.MyExtension", WRITE_BINARY);
	while (!feof(input_file))
	{
		InitLog(input_file, log);
		if (log->env_temperature < 26.5 || log->env_temperature > 32.5)
		{
			fwrite(log, sizeof(Log), 1, output_file);
			count++;
			if (count == 1)logs = NEW(p_log_t, 1);
			else logs = (p_log_t*)realloc(logs, sizeof(Log)*count);
		}
	}
	fclose(output_file);
	
	output_file = OpenFile("DataMeteo-E5.MyExtension", READ_BINARY);

	fread(logs, sizeof(Log), count, output_file);
	printf("%d", count);
	for (int i = 0; i < count-1; i++)
	{
		PrintLog(logs[i]);
	}

	return EXIT_SUCCESS;
}


Содержимое моего файла выглядит следующим образом:
2015-07-22 09:02:00 1346139 13.03 25.19 6.477 3.851 0.836 26.02
2015-07-22 09:03:00 1346140 13.03 25.22 6.493 3.879 0.841 26.07
2015-07-22 09:04:00 1346141 13.02 25.25 6.516 3.91 0.846 26.01
2015-07-22 09:05:00 1346142 13.03 25.29 6.537 3.94 0.846 25.99
2015-07-22 09:06:00 1346143 13.02 25.32 6.574 3.976 0.843 26.19
2015-07-22 09:07:00 1346144 13.02 25.36 6.606 4.012 0.845 26.3
2015-07-22 09:08:00 1346145 13.02 25.38 6.656 4.057 0.848 26.4
2015-07-22 09:09:00 1346146 13.02 25.43 6.667 4.086 0.849 26.45
2015-07-22 09:10:00 1346147 13.02 25.46 6.663 4.109 0.851 26.44
2015-07-22 09:11:00 1346148 13.02 25.5 6.657 4.131 0.856 26.51
2015-07-22 09:12:00 1346149 13.02 25.53 6.693 4.17 0.862 26.53
2015-07-22 09:13:00 1346150 13.02 25.56 6.723 4.205 0.865 26.71
2015-07-22 09:14:00 1346151 13.02 25.6 6.734 4.233 0.866 26.64
2015-07-22 09:15:00 1346152 13.02 25.63 6.774 4.269 0.863 26.57
2015-07-22 09:16:00 1346153 13.02 25.67 6.768 4.283 0.865 26.63
2015-07-22 09:17:00 1346154 13.02 25.7 6.749 4.293 0.868 26.63
2015-07-22 09:18:00 1346155 13.02 25.73 6.725 4.302 0.87 26.8
2015-07-22 09:19:00 1346156 13.02 25.78 6.728 4.327 0.871 26.85


Что я уже пробовал:

Я пытался сделать это для разных значений в цикле for.
Также я знаю, что счет равен 35.

Richard MacCutchan

"Я пытался сделать это для разных значений в цикле for.
Также я знаю, что счет равен 35."

Что это значит и какое отношение имеет к вашей проблеме?

2 Ответов

Рейтинг:
2

Patrice T

Цитата:
Я не могу понять свой ошибочный код на языке Си

Вы забыли сказать, что должен делать ваш код ! Мы не можем догадаться !
Вы забыли сказать нам, как код идет не так (как вы знаете, что это неправильно).

Единственный разумный совет-научиться пользоваться отладчиком.
-----
Ваш код ведет себя не так, как вы ожидаете, или вы не понимаете, почему !

Существует почти универсальное решение: запускайте свой код на отладчике шаг за шагом, проверяйте переменные.
Отладчик здесь, чтобы показать вам, что делает ваш код, и ваша задача-сравнить с тем, что он должен делать.
В отладчике нет никакой магии, он не знает, что должен делать ваш код, он не находит ошибок, он просто помогает вам, показывая, что происходит. Когда код не делает того, что ожидается, вы близки к ошибке.
Чтобы увидеть, что делает ваш код: просто установите точку останова и посмотрите, как работает ваш код, отладчик позволит вам выполнять строки 1 на 1 и проверять переменные по мере их выполнения.

Отладчик - Википедия, свободная энциклопедия[^]

Освоение отладки в Visual Studio 2010 - руководство для начинающих[^]
Базовая отладка с помощью Visual Studio 2010 - YouTube[^]

1.11 — отладка программы (пошаговое выполнение и останова) | выучить C++[^]

Отладчик здесь только для того, чтобы показать вам, что делает ваш код, и ваша задача-сравнить его с тем, что он должен делать.


Рейтинг:
0

OriginalGriff

Мы понятия не имеем, что вы ожидаете от этого: даже с данными, сообщающими о том, что "это не работает", никому не помогает - мы не можем видеть ваш экран, не можем получить доступ к вашему жесткому диску или читать ваши мысли. И мы не знаем, о чем просил тебя твой наставник!

Так что все будет зависеть от тебя.
К счастью, у вас есть инструмент, который поможет вам выяснить, что происходит: отладчик. Если вы не знаете, как его использовать, то быстрый поиск в Google названия вашей IDE и "отладчика" должен дать вам необходимую информацию.

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

Извините, но мы не можем сделать это за вас - пришло время вам освоить новый (и очень, очень полезный) навык: отладку!


Member 14719581

Я обнаружил, что проблема возникает в строке, где я вызываю функцию PrintLog.
Я обнаружил, что он не выделяет память должным образом, используя функцию realloc.
Он возвращает 3 разных адреса "подсчета" раз

k5054

Вы неправильно используете realloc. Если новый параметр size для перераспределения больше текущего размера передаваемого указателя, то происходит одно из трех событий:
1) Если есть свободное место, то размер текущего элемента увеличивается. Будет возвращено текущее значение указателя
2) если новый размер больше будет переполнен другим выделенным элементом, но для нового элемента есть свободное место, то выделяется новое пространство, содержимое текущего указателя копируется в новое пространство, а старое пространство освобождается. Возвращается значение новой области памяти
3) если больше нет доступной памяти, возвращается NULL, содержимое текущего указателя не изменяется и не освобождается

Что это значит, так это то, что код похож

ptr = realloc(ptr, new_size);
может потерпеть неудачу неожиданным образом. В частности, если realloc не может найти новое пространство, то ptr теперь будет равен нулю, что, вероятно, означает, что вы потеряли указатель, который у вас был, и у вас происходит утечка памяти.Вот что вам следует сделать
void *tmp_ptr = realloc(ptr, size);
if(tmp_ptr == NULL) {
    // handle error somehow 
}
else
    ptr = tmp_ptr;
// continue processing