Перевод из инфиксной в постфиксную запись

@DevilRigoll · Регистрация: 09.04.2014

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

Суть моей программы: перевод из инфиксной в постфиксную запись.
Но проблема в том, что функция типа string не возвращает значение. В чём ошибка?
Main.cpp

C++

#include<iostream>
#include<string>
#include<conio.h>
#include<cstdlib>
 
#include"ClassStack.h"
 
using namespace std;
 
 
 
int PrioritySign(char w)
{
    switch(w)
    {   
        case '^':
            return 4;
        case '/':
        case '*':
            return 3;
        case '+':
        case '-':
            return 2;
        case '(':
            return 1;       
    }
}
 
void InficsToPostfics(string &Str)
{
    Stack Stc;
    string OutStr;
       for(int i = 0; i < Str.size(); i++)
       {
           if(Str[i] == '(')
              Stc.AddHead(Str[i]);
           if(Str[i] == ')')
           { 
              while(Stc.Head != 0)
              {
                 if(Stc.Head->Text == '(')
                 {
                    Stc.DeleteHead();
                    break;
                 }
                 OutStr += Stc.Head->Text;
                 Stc.DeleteHead();  
              } 
           }  
           if((Str[i] == '*')||(Str[i] == '-')||(Str[i] == '+')||(Str[i] == '/')||(Str[i] == '^'))
           {
              if(Stc.Head == NULL)
                 Stc.AddHead(Str[i]);
              else
              {  
                 while((Stc.Head != NULL)&&(PrioritySign(Stc.Head->Text) > PrioritySign(Str[i])))
                 {
                    OutStr += Stc.Head->Text;
                    Stc.DeleteHead();
                 }
                 Stc.AddHead(Str[i]);
              }
           }
           else
           {
              OutStr += Str[i];
           }
       }
     while(Stc.Head != 0)
     {
        OutStr += Stc.Head->Text;
        Stc.DeleteHead();
     }  
     delete(&Stc);
     Str=OutStr;       
}
 
int main()
{
    string Str;
    cout << "Enter string" << endl;
    cin >> Str;
    InficsToPostfics(Str);
    cout << Str;
    getch();
    return 0;
}

ClassStack.h

C++

#ifndef STACK
#define STACK
 
 
struct List
{
    public:char Text;
    List *Next;
    List(char T):Next(NULL),Text(T){}
};
 
class Stack
{
   public:
   List *Head;
   void DeleteHead();
   void AddHead(char);
   friend int PrioritySign(char );
};
 
 
#endif

FuncStack.cpp

C++

#include<iostream>
 
#include"ClassStack.h"
 
using namespace std;
 
 
 
void Stack::AddHead(char Text)
{
   List* newHead = new List(Text);
   newHead->Next = Head;
   Head = newHead; 
}
 
void Stack::DeleteHead()
{
     List *newHead = Head;
     Head = Head->Next;
     delete(newHead);
}

@nmcf · 14.02.2015, 08:48

Она вообще ничего не возвращает - void. Внутри функции OutStr правильно формируется?
74-я строка не понятно для чего, память же не выделялась. 69-73 должны быть в деструкторе класса Stack.

@zss · 14.02.2015, 10:07

Строка 75

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

Str=OutStr;

Это разве не скрытый возврат значения (по ссылке)?

@nmcf · 14.02.2015, 10:15

Да, но выражение "функция возвращает значение" имеет вполне конкретный смысл.

@zss · 14.02.2015, 10:22

Ну напишите вид возвращаемого значения не void, а string
и в конце функции

C++

1	return OutStr;

Вызов

C++

1	string result=InficsToPostfics(Str);

@DevilRigoll · 14.02.2015, 15:46 **[ТС]**

zss,

C++

#include<iostream>
#include<string>
#include<conio.h>
#include<cstdlib>
 
#include"ClassStack.h"
 
using namespace std;
 
 
 
int PrioritySign(char w)
{
    switch(w)
    {   
        case '^':
            return 4;
        case '/':
        case '*':
            return 3;
        case '+':
        case '-':
            return 2;
        case '(':
            return 1;       
    }
}
 
string InficsToPostfics(string &Str)
{
    Stack Stc;
    string OutStr;
       for(int i = 0; i < Str.size(); i++)
       {
           if(Str[i] == '(')
              Stc.AddHead(Str[i]);
           if(Str[i] == ')')
           { 
              while(Stc.Head != 0)
              {
                 if(Stc.Head->Text == '(')
                 {
                    Stc.DeleteHead();
                    break;
                 }
                 OutStr += Stc.Head->Text;
                 Stc.DeleteHead();  
              } 
           }  
           if((Str[i] == '*')||(Str[i] == '-')||(Str[i] == '+')||(Str[i] == '/')||(Str[i] == '^'))
           {
              if(Stc.Head == NULL)
                 Stc.AddHead(Str[i]);
              else
              {  
                 while((Stc.Head != NULL)&&(PrioritySign(Stc.Head->Text) > PrioritySign(Str[i])))
                 {
                    OutStr += Stc.Head->Text;
                    Stc.DeleteHead();
                 }
                 Stc.AddHead(Str[i]);
              }
           }
           else
           {
              OutStr += Str[i];
           }
       }
     while(Stc.Head != 0)
     {
        OutStr += Stc.Head->Text;
        Stc.DeleteHead();
     }    
     return OutStr;
      
}
 
int main()
{
    string Str;
    cout << "Enter string" << endl;
    cin >> Str;
    
    string result = InficsToPostfics(Str);
    cout << result;
    getch();
    return 0;
}

сделал как вы сказали, но всё равно ничего не выводит

@nmcf · 14.02.2015, 15:50

Так может дело в самом алгоритме, а не в строках? Отладчиком проверяй.

@nmcf 7803 / 6567 / 2988 Регистрация: 14.04.2014 Сообщений: 28,706
	14.02.2015, 08:48	2
	Она вообще ничего не возвращает - void. Внутри функции OutStr правильно формируется? 74-я строка не понятно для чего, память же не выделялась. 69-73 должны быть в деструкторе класса Stack. 0

@zss Модератор 13701 / 10904 / 6472 Регистрация: 18.12.2011 Сообщений: 29,110
	14.02.2015, 10:07	3
	Строка 75 Сообщение от DevilRigoll Str=OutStr; Это разве не скрытый возврат значения (по ссылке)? 0

@nmcf 7803 / 6567 / 2988 Регистрация: 14.04.2014 Сообщений: 28,706
	14.02.2015, 10:15	4
	Да, но выражение "функция возвращает значение" имеет вполне конкретный смысл. 0

@nmcf 7803 / 6567 / 2988 Регистрация: 14.04.2014 Сообщений: 28,706
	14.02.2015, 15:50	7
	Так может дело в самом алгоритме, а не в строках? Отладчиком проверяй. 0

Опции темы