Конец Списка

@Ramix57 · Регистрация: 01.12.2012

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

Создаю список и в последнем элементе всегда мусор, а не 0.
next CXX0030: Error: expression cannot be evaluated
В буфере последнего элемента HHHH...
Буду благодарен на ссылку примера списков, где последний элемент занулен, или за совет, что подправить, чтобы разрешить данную проблему. P.S. Удаление выделенной памяти еще не занимался.

@Kuzia domovenok · 21.09.2013, 18:19

Не по теме:

качать это никто не будет

@Ramix57 · 21.09.2013, 19:16 **[ТС]**

Kuzia domovenok, поэтому в первую очередь я буду благодарен за ссылку с правильной реализацией списков, где последний элемент правильно задат.
Так как при записи Node->this = NULL; он всё равно не может занулить указатель.

@Kuzia domovenok · 21.09.2013, 20:10

Не по теме:

что мы обсуждаем? я кода не вижу!

alsav22 · 21.09.2013, 20:28

Ramix57, где, конкретно, в коде, в последнем элементе муссор? И файл, с которым работаете.

@Ramix57 · 21.09.2013, 21:26 **[ТС]**

alsav22, когда идет последние ";" он создает последний элемент в списке, у которого должен быть указатель нулевым. Да, там мусор судя по дебагу.

C++

if (c=='\n' && _c==';')
        {
            ListF->MyStack.PutInStack(cBufSymbol, cBufOperator);
            ListF->next = new CNodeStack;
            ListF->next->next = NULL;
            ListF = ListF->next;
            . . .
        }

alsav22 · 21.09.2013, 22:47

Так попробуйте:

C++

    do
    {
        c = fgetc(fIn);
        
        if (feof(fIn)) break;
        
        if (isdigit(c))
        {
            *cBS++=c;
        }
        else if(isalpha(c) && isdigit(_c))
        {
            *cBS++=',';
        }
        else if(c=='{' && _c=='}')
        {
            *cBS++=',';
        }
        else if(isspace(c) && _c=='}')
        {
            *cBS++='|';
        }
        else if(isspace(c) && (_c=='+' || _c=='-' || _c=='*' || _c=='/' || _c=='^' || _c=='S' || _c=='(' || _c==')'))
        {
            *cBO++=_c;
        }
        if (c == ';')
        {
            ListF->MyStack.PutInStack(cBufSymbol, cBufOperator);
            
            FreeBuf(cBufSymbol);
            FreeBuf(cBufOperator);
            cBS = cBufSymbol;
            cBO = cBufOperator;
            
            c = fgetc(fIn);
            
            if (c != EOF)
            {
                ungetc(c, fIn);
                ListF->next = new CNodeStack;
                ListF->next->next = NULL;
                ListF = ListF->next;
            }
            else break;
        }
        _c = c;
    } while(true);

@Ramix57 · 22.09.2013, 00:53 **[ТС]**

C++

if(!(fIn=fopen("in.txt","r"))) EMyError.CriticalError(3);
    do
    {
        c = tolower(fgetc(fIn));
        if (CountErrorLex!=0) EMyError.CriticalError(4);
        if (isdigit(c))
        {
            *cBS++=c;
        }
        else if(isalpha(c) && isdigit(_c) || c=='{' && _c=='}')
        {
            *cBS++=',';
        }
        else if(isspace(c) && _c=='}')
        {
            *cBS++='|';
        }
        else if(isspace(c) && (_c=='+' || _c=='-' || _c=='*' || _c=='/' || _c=='^' || _c=='S' || _c=='(' || _c==')'))
        {
            *cBO++=_c;
        }
        if ((c=='\n' || c==EOF) && _c==';')
        {
            ListF->MyStack.PutInStack(cBufSymbol, cBufOperator);
            ListF->next = NULL;
            FreeBuf(cBufSymbol);
            FreeBuf(cBufOperator);
            cBS = cBufSymbol;
            cBO = cBufOperator;
            if(c!=EOF)
            {
                ListF->next = new CNodeStack;
                ListF->next->next = NULL;
                ListF = ListF->next;
            }
            else
            {
                
            }
        } 
        _c = c;
    }while(!feof(fIn));
    fclose(fIn);
    ungetc(c,fIn);
    ListF = Start;

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

alsav22 · 22.09.2013, 00:55

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

Изменил вот так.

Зачем? Чем не устроил предложенный вариант?

alsav22 · 22.09.2013, 01:03

Вот файл fract.cpp (добавил два метода для вывода) и результат работы:

C++

#include "Fraction.h"
#include "NodeStack.h"
 
void FreeBuf(char *buf);
 
int main()
{
   
   
    
    FILE *fIn = NULL, *fOut = NULL;
    Error EMyError;
    CNodeStack *Start;
    CNodeStack *ListF = new CNodeStack;
    ListF->next = NULL;
    Start = ListF;
    char c, _c, cBufSymbol[255] = {0}, *cBS = cBufSymbol, cBufOperator[255] = {0}, *cBO = cBufOperator;
    if(!(fIn=fopen("in.txt","r")))EMyError.CriticalError(3);
    do
    {
        c = fgetc(fIn);
        
        if (feof(fIn)) break;
        
        if (isdigit(c))
        {
            *cBS++=c;
        }
        else if(isalpha(c) && isdigit(_c))
        {
            *cBS++=',';
        }
        else if(c=='{' && _c=='}')
        {
            *cBS++=',';
        }
        else if(isspace(c) && _c=='}')
        {
            *cBS++='|';
        }
        else if(isspace(c) && (_c=='+' || _c=='-' || _c=='*' || _c=='/' || _c=='^' || _c=='S' || _c=='(' || _c==')'))
        {
            *cBO++=_c;
        }
        if (c == ';')
        {
            ListF->MyStack.PutInStack(cBufSymbol, cBufOperator);
            
            FreeBuf(cBufSymbol);
            FreeBuf(cBufOperator);
            cBS = cBufSymbol;
            cBO = cBufOperator;
            
            c = fgetc(fIn);
            
            if (c != EOF)
            {
                ungetc(c, fIn);
                ListF->next = new CNodeStack;
                ListF->next->next = NULL;
                ListF = ListF->next;
            }
            else break;
        }
        _c = c;
    } while(true);
    
    fclose(fIn);
    ListF->next = NULL;
    ListF = Start;
    while(ListF)
    {
        cout << ListF ->MyStack.GetInStackSymbols() << endl;
        cout << ListF ->MyStack.GetInStackOperator() << endl;
        ListF=ListF->next;
    }
    
    system("pause");
    return 0;
}
 
void FreeBuf(char *buf)
{
    while(*buf) *buf++=0;
}

Что ещё нужно?

@Ramix57 · 22.09.2013, 01:45 **[ТС]**

Наверное, я не так выразился, поэтому извиняюсь за потраченное время.
Посмотри приложенный мной скриншот из дебагера Visual Studio 2008.
У последнего элемента списка есть указатель на следующий элемент и там написано
next CXX0030: Error: expression cannot be evaluated
Эта ошибка меня смутила и не понимаю как убрать её и как она возникает...С самого начала темы.

alsav22 · 22.09.2013, 02:58

И что, не понял? После последнего считанного символа в файле - ;, опять создаётся новый объект и добавляется в список, хотя он уже не нужен. В своём коде я это исправил. Ваш вариант (8 пост) тоже рабочий, а скрин... Что вы хотите увидеть по нулевому указателю?

@Ramix57 · 22.09.2013, 16:25 **[ТС]**

alsav22, хотел увидел 0х000000, или так и должно быть?

alsav22 · 22.09.2013, 17:04

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

alsav22, хотел увидел 0х000000, или так и должно быть?

На скрине четыре next, в списке три объекта, next пследнего (третьего) содержит 0 (т.е. за ним нет объектов), дальше смотрится, что находится по адресу 0? Естественно, ничего. Ошибка, адрес в никуда.

@Ramix57 · 22.09.2013, 22:12 **[ТС]**

alsav22, ясно, спасибо. Моя невнимательность.

@Ramix57 0 / 0 / 0 Регистрация: 01.12.2012 Сообщений: 49
	22.09.2013, 01:45 [ТС]	11
	Наверное, я не так выразился, поэтому извиняюсь за потраченное время. Посмотри приложенный мной скриншот из дебагера Visual Studio 2008. У последнего элемента списка есть указатель на следующий элемент и там написано next CXX0030: Error: expression cannot be evaluated Эта ошибка меня смутила и не понимаю как убрать её и как она возникает...С самого начала темы. 0

alsav22 5498 / 4893 / 831 Регистрация: 04.06.2011 Сообщений: 13,587
	22.09.2013, 17:04	14
	Сообщение от Ramix57 alsav22, хотел увидел 0х000000, или так и должно быть? На скрине четыре next, в списке три объекта, next пследнего (третьего) содержит 0 (т.е. за ним нет объектов), дальше смотрится, что находится по адресу 0? Естественно, ничего. Ошибка, адрес в никуда. 1

@Ramix57 0 / 0 / 0 Регистрация: 01.12.2012 Сообщений: 49
	22.09.2013, 22:12 [ТС]	15
	alsav22, ясно, спасибо. Моя невнимательность. 0

@Kuzia domovenok
	21.09.2013, 18:19 #2
	Не по теме: качать это никто не будет 0

@Ramix57 0 / 0 / 0 Регистрация: 01.12.2012 Сообщений: 49
	21.09.2013, 19:16 [ТС]	3
	Kuzia domovenok, поэтому в первую очередь я буду благодарен за ссылку с правильной реализацией списков, где последний элемент правильно задат. Так как при записи Node->this = NULL; он всё равно не может занулить указатель. 0

@Kuzia domovenok
	21.09.2013, 20:10 #4
	Не по теме: что мы обсуждаем? я кода не вижу! 0

alsav22 5498 / 4893 / 831 Регистрация: 04.06.2011 Сообщений: 13,587
	21.09.2013, 20:28	5
	Ramix57, где, конкретно, в коде, в последнем элементе муссор? И файл, с которым работаете. 0

alsav22 5498 / 4893 / 831 Регистрация: 04.06.2011 Сообщений: 13,587
	22.09.2013, 00:55	9
	Сообщение от Ramix57 Изменил вот так. Зачем? Чем не устроил предложенный вариант? 0

alsav22 5498 / 4893 / 831 Регистрация: 04.06.2011 Сообщений: 13,587
	22.09.2013, 02:58	12
	И что, не понял? После последнего считанного символа в файле - ;, опять создаётся новый объект и добавляется в список, хотя он уже не нужен. В своём коде я это исправил. Ваш вариант (8 пост) тоже рабочий, а скрин... Что вы хотите увидеть по нулевому указателю? 0

@Ramix57 0 / 0 / 0 Регистрация: 01.12.2012 Сообщений: 49
	22.09.2013, 16:25 [ТС]	13
	alsav22, хотел увидел 0х000000, или так и должно быть? 0