Премини към съдържанието

Препоръчан отговор


Здравейте, тази задача ми се бе паднала на контролно, за съжаление се мъча твърде дълно с нея затова се налага да потърся съвет от някой с повече опит.

Та ето го условието: 

Да се създаде свързан списък от естествени числа до въвеждането на 0, чрез добавяне в края. Готовият списък да се иведе на екрана. Да се въведе от клавиатура та цяло число Х и всички елементи на списъка , по-големи от  Х , да се запишат в стек. Да се прочетат елементите на стека и да се изведат на екрана.

Кодът написан до момента. (фунциите ползвани в него са ни преподавани в лекциите). Моля да обясните защо не работи, как е правилно да бъде направено. 

#include <iostream>
 
using namespace std;
 
struct List {
	int x;
	List *next;
};
void Spush(List *& top, int i) {
	List *p = new List;
	p->x = i;
	p->next = top;
	top = p;
}
 
List * Spop(List *& top) {
	if (top == NULL) return top;
	List *p = top;
	top = top->next;
	return p;
}
 
void AddEnd(List *& first, List *& last, int i) {
	List *p = new List;
	p->x = i;
	p->next = NULL;
	if (first == NULL) first = last = p;
	else {
	  last->next=p;
	  last=p;
	}
}
 
void PrintList(List * first) {
	List *p = first;
	while (p != NULL) {
	  std::cout << p->x << " ";
	  p = p->next;
	}
	std::cout << std::endl;
}
 
 
 
 
 
int main()
{
 
    int y;
    cin>>y;
 
   List *first=NULL,*st,*last;
   st=NULL;
    List *p=first;
 
 
while(y!=0){
    AddEnd(first,last,y);
      cin>>y;
}
PrintList(first);
// Тази част работи до тук
 
st=NULL;
 
int X;
cin>>X;
if(X>=0){
 
 
while (p!=NULL){
if (p->x>X) {
    Spush (st,p->x);
    p=p->next;
 
}
}
}
else
    cout<< "X is less that 0";
while ((p=Spop(st))!=NULL){
    cout <<p->x<<" ";
    delete p;
 
}
//Елементите не се извеждат, без никакви грешки
cout<<endl;
}
 

 

Сподели този отговор


Линк към този отговор
Сподели в други сайтове

На практика никъде не добавяш елемeнтите към стека.

Самото извеждане може да го правиш и с PrintList(st), но може и да вадиш елементи с Spop() докато се изпразни стека,

само че начина по който реализиран Spop изобщо не е удобен за ползване

позволих си да ти коригирам малко програмата:

#include <iostream>
 
using namespace std;
 
struct List {
	int x;
	List *next;
};
void Spush(List *& top, int i) {
	List *p = new List;
	p->x = i;
	p->next = top;
	top = p;
}

int Spop(List *& top) 
{
	if (top == NULL) return 0;
	int n = top->x;
	List *p=top;
	top = top->next;
	delete p;
	return n;
}
 
void AddEnd(List *& first, List *& last, int i) {
	List *p = new List;
	p->x = i;
	p->next = NULL;
	if (first == NULL) first = last = p;
	else {
	  last->next=p;
	  last=p;
	}
}
 
void PrintList(List * first) {
	List *p = first;
	while (p != NULL) {
	  std::cout << p->x << " ";
	  p = p->next;
	}
	std::cout << std::endl;
}
 
 
 
 
 
int main()
{
 
    int y;
    cin>>y;
 
   List *first=NULL,*st,*last;
   st=NULL;
    List *p=first;
 
 
while(y!=0){
    AddEnd(first,last,y);
      cin>>y;
}
PrintList(first);
 
st=NULL;
 
int X;
cin>>X;
p=first;
if(X>=0)
{
   while (p)
   {
       if (p->x>X) Spush (st,p->x);
       p=p->next; 
   }
}
else
    cout<< "X is less that 0";
PrintList(st);   // извеждане  на стека  като списък
cout<<endl;
while (st) cout<<Spop(st)<<" ";
cout<<endl;
}

 

Редактирано от ined (преглед на промените)
  • Харесва ми 1

Сподели този отговор


Линк към този отговор
Сподели в други сайтове

Много ти благодаря ined , без твоята помощ едва ли в скоро време тази програма щеше да бъде написана коректно. Благодаря и ти пожелавам лека вечер /ден.

Сподели този отговор


Линк към този отговор
Сподели в други сайтове
#include <iostream>
#include <climits>
using namespace std;

struct node
{
	int data;
	node* next;
};

struct list
{
	int size;
	node* head;
	node* tail;
	node* conductor1;
	node* conductor2;
	list();
	void push_back(int n);
	void push_front(int n);
	~list();
};

list::list()
{
	size = 0;
	head = new node;
	tail = new node;
	head->next = tail;
	tail->next = head;
}

void list::push_front(int n)
{
	conductor1 = new node;
	conductor1->data = n;
	conductor1->next = head->next;
	head->next = conductor1;
	size++;
}

void list::push_back(int n)
{
	conductor1 = new node;
	conductor1->data = n;
	conductor1->next = tail;
	tail->next->next = conductor1;
	tail->next = conductor1;
	size++;
}



list::~list()
{
	if (head != 0)
	{
		conductor2 = head->next;
		conductor1 = head;
	}
	else return;
	while (conductor2 != tail)
	{
		conductor1 = conductor2;
		conductor2 = conductor2->next;
		delete conductor1;
	}
	delete head;
	delete tail;
}

struct stack :list
{
	int top();
	int pop();
	void push(int n);
	~stack();
};

int stack::top()
{
	if (size != 0) return head->next->data;
	else return INT_MIN;
}

int stack::pop()
{
	int answer;
	if (size != 0) answer = head->next->data;
	else return INT_MIN;
	conductor1 = head->next;
	head->next = head->next->next;
	delete conductor1;
	size--;
	return answer;
}

void stack::push(int n)
{
	push_front(n);
}

stack::~stack()
{
	while (size != 0)
	{
		pop();
	}
	delete head;
	delete tail;
	head = 0;
	tail = 0;
}


int main()
{
	list container;
	stack lifo;
	int n;
	cin >> n;
	while (n != 0)
	{
		container.push_back(n);
		cin >> n;
	}
	int x;
	cin >> x;
	container.conductor1 = container.head->next;
	while (container.conductor1 != container.tail)
	{
		if (x < container.conductor1->data)
		{
			lifo.push(container.conductor1->data);
		}
		container.conductor1 = container.conductor1->next;
	}
	while (lifo.size > 0)
	{
		cout << lifo.top() << endl;
		lifo.pop();
	}
    return 0;
}

 

Сподели този отговор


Линк към този отговор
Сподели в други сайтове

Регистрирайте се или влезете в профила си за да коментирате

Трябва да имате регистрация за да може да коментирате това

Регистрирайте се

Създайте нова регистрация в нашия форум. Лесно е!

Нова регистрация

Вход

Имате регистрация? Влезте от тук.

Вход

  • Разглеждащи това в момента   0 потребители

    Няма регистрирани потребители разглеждащи тази страница.

  • Горещи теми в момента

  • Подобни теми

    • от Chris Panov
      Здравейте, 
      В момента пиша една програма, в която в една от функциите ми се налага да излезна по-рано при определено условие. Функцията връща string впрочем.
      Ето кодът:
       
      #include "stdafx.h" #include <iostream> #include <string> #include <sstream> #include <vector> #define CONSOLE_LOG(x) std::cout << x #define COMMAND_END command[0] #define COMMAND_SUM command[1] #define COMMAND_SUBTRACT command[2] #define COMMAND_CONCAT command[3] #define COMMAND_DISCARD command[4] #define COMMAND_DISPSEQ command[5] template <typename T> void top_back(std::vector<T>& v) {     v.erase(v.end() - 1); } std::string sum(std::string& x, std::string& y, std::vector<std::string>& vec) {     int xInt = atoi(x.c_str());     int yInt = atoi(y.c_str());     int result = xInt + yInt;     std::ostringstream os;     std::string strResult;     os << result;     strResult = os.str();     top_back(vec);     top_back(vec);     return strResult; } std::string subtract(std::string& x, std::string& y, std::vector<std::string>& vec) {     int xInt = atoi(x.c_str());     int yInt = atoi(y.c_str());     int result = xInt - yInt;     std::ostringstream os;     std::string strResult;     os << result;     strResult = os.str();     top_back(vec);     top_back(vec);     return strResult; } std::string concat(std::string& x, std::string& y, std::vector<std::string>& vec) {     if (atoi(x.c_str()) < 0)     {         return;     }     else {         std::string concatStr = y + x;         top_back(vec);         top_back(vec);         return concatStr;     } } void dispseq(std::vector<std::string>& vec) {     std::vector<std::string>::const_iterator iter;     for (iter = vec.begin(); iter != vec.end(); iter++)     {         std::cout << *iter << std::endl;     } } void enterSequence() {     std::vector<std::string> command =     {         "end",         "sum",         "subtract",         "concat",         "discard",         "dispseq"     };     std::vector<std::string> sequence;     std::string input;     std::string a;     std::string b;     do     {         std::cin >> input;         sequence.emplace_back(input);         if (input == COMMAND_END)         {              top_back(sequence);         }         if (input == COMMAND_SUM)         {             top_back(sequence);             a = sequence[sequence.size() - 1];             b = sequence[sequence.size() - 2];             sequence.emplace_back(sum(a, b, sequence));         }         if (input == COMMAND_SUBTRACT)         {             top_back(sequence);             a = sequence[sequence.size() - 1];             b = sequence[sequence.size() - 2];             sequence.emplace_back(subtract(a, b, sequence));         }         if (input == COMMAND_CONCAT)         {             top_back(sequence);             a = sequence[sequence.size() - 1];             b = sequence[sequence.size() - 2];             sequence.emplace_back(concat(a, b, sequence));         }         if (input == COMMAND_DISCARD)         {             top_back(sequence);             top_back(sequence);         }         if (input == COMMAND_DISPSEQ)         {             top_back(sequence);             dispseq(sequence);         }     } while (input != COMMAND_END); } int main() {     enterSequence();     std::cin.get();     std::cin.get();     return 0; } И ето проблемната функция:
       
      std::string concat(std::string& x, std::string& y, std::vector<std::string>& vec) {     if (atoi(x.c_str()) < 0)     {         return;     }     else {         std::string concatStr = y + x;         top_back(vec);         top_back(vec);         return concatStr;     } }  
      Както виждате ако стрингът x(който естествено го превръщам в интиджър) е по-малък от 0, теоритично трябва да излезна от функцията, и това е единственият начин за който се сетих, с return;, обаче пък компилаторът иска функцията да връща стойност, понеже е от тип стринг.
      Как да съчетая двете? - да връща стринг ако всичко е по план, и да излиза ако е по-малко от 0.
    • от Chris Panov
      Здравейте,
      В момента имам сериозен проблем със взимането на броят на елементи в даден вектор.
      vec.size(); и  
      sizeof(vec) / sizeof(vec[0]); не работят. Програмата те пита за брой ключове за даден тест, въвеждаш ключовете, после въвеждаш и отговорите, които са били дадени, и програмата ги сравнява и в зависимост от това колко верни отговори имаш ти дава точки. Ключовете и дадените отговори са тип char. ("A", "B", "C" etc.)
      Ето го и кода, като повечето съм го направил на коментар за да си тествам само функцията която извежда броя на елементите.
      #include "stdafx.h" #include <iostream> #include <vector> #define CONSOLE_LOG(x) std::cout << x; std::vector<char> keys = {'A', 'B', 'C', 'D', 'E'}; std::vector<char> studentAnswers; int n, points = 0; char key, answer; template <typename T> T vecSize(std::vector<T>& vec) { T size = vec.size(); return size; } /* template <typename T1> void enterKeys(std::vector<T1>& k) { CONSOLE_LOG("Please enter the number of keys: "); std::cin >> n; for (int i = 0, counter = 1; i < n, counter <= n; i++, counter++) { CONSOLE_LOG("Key " << counter << ": "); std::cin >> key; k.emplace_back(key); } } template <typename T2> void enterAnswers(std::vector<T2>& stAns) { for (int i = 0, counter = 1; i < n, counter <= n; i++, counter++) { CONSOLE_LOG("Answer " << counter << ": "); std::cin >> answer; stAns.emplace_back(answer); } } template <typename T3> void getGrade(std::vector<T3>& x, std::vector<T3>& y) { for (int i = 0; i < n; i++) { if (x[i] == y[i]) { points++; } } CONSOLE_LOG("Points: " << points); } */ int main() { /* enterKeys(keys); enterAnswers(studentAnswers); getGrade(studentAnswers, keys); */ std::cout << keys.size() << std::endl; std::cout << vecSize(keys) << std::endl; std::cin.get(); std::cin.get(); std::cin.get(); return 0; } Опитах всевъзможни начини, и пак не става. Идеята е да заместя n променливата със броя на елементите от вектора. Програмата върви по един и същи начин, защото все пак в n променливата запазваме големината на вектора, но бих искал да си го направя с функция.
      Както виждате на края на програмата си извеждам тестове. Първият, който си е по конвенционалният метод си работи сам по себе си, но пък във for цикъл не бачка.
      А вторият е функцията която съм направил. Проблемът е че ми извежда непознат символ - "�"
      Проблемът е че типът на елементите във вектора е char, защото като го направих с int тип си работеше както трябва.
      Бих бил изключително благодарен ако някой може да ме насочи и да ми бутне едно рамо :)
    • от Goshko
      Да се напише програма, която създава структура "Book" като имате следните полета - Title(заглавието на книгата), Автор(Author), Цена(Price) и уникален номер на книгата(ISBN-num). Да се ваведе цяло число n и след него n на брой данни за ученика. Да се изведе на монитора данните за книгата с цена по-висока от предварително зададена.
    • от Нели Николова
      Здравейте, имам две готови задачи, но не мога да ги компилирам. Дали може да ги проверите?
      зад.1 Да се състави програма, която да сортира едномерен масив от цели числа тип short  с име X състоящ се от 19 елемента. Сортирането да се извърши във възходящ ред чрез метода пряка селекция.
      #include <iostream>
      using namespace std;
      void sortAsc(short[]);
      int main() {
          short x[] = {123,13,23,31,1,55,36,17,8,9,10,11,6,12,14,15,16,35,184,19};
           
          sortAsc(x);
          
          return 0;
      }
      void sortAsc(short x[]){
          for(short i = 0; i < 19 ; i++) {
              for(short j = i; j < 19; j++) {
                  if(x[j] < x) {
                      swap(x[j], x);
                  }
              }    
          }
          
          for(int i=0; i<19; i++)
          {
              cout << x << endl;
          }
      }
      зад. 2 Да се състави програма,която реализира динамичен стек от реални числа тип float. Стекът да се преобразува в два нови стека, един стек P за четните и един стек O за нечетните числа от стек едно. Стековете да се извеждат на екрана.
      int main() {
          stack<float>numbers;
          stack<float>p;
          stack<float>o;
          
          for(short i = 1; i <= 200; i++) {
              numbers.push(i);
          }  
          
          numbers.push(200.64);
          numbers.push(203.34);
          
          while(!numbers.empty()) {
              int number = numbers.top();
              
              if (number % 2 == 0) {
                  p.push(number);
              } else {
                  o.push(number);
              }
              numbers.pop();
          }
          
          cout << "EVEN NUMBERS:" << endl;
          
          while(!p.empty()) {
              cout << p.top() << endl;
              p.pop();
          }
          
          cout << "ODD NUMBERS:" << endl;
          
          while(!o.empty()) {
              cout << o.top() << endl;
              o.pop();
          }
          
      }
      Много Ви благодаря :)
       
    • от Georgi Kirchev
      Здравейте имам да предам курсова работа утре ,но не мога да я реша , ще бъда изключително благодарен ако може някой да ми помогне.

      Дефинирайте клас Телевизор, който да е родител на клас Самсунг и клас Сони. Клас Телевизор да има цена и брой продадени за година в сектор private. Класовете Самсунг и Сони да имат в сектор public функции за определяне на общата сума от продажби за година (цена * брой продадени за година). Всички класове да имат конструктори по подразбиране.
       
      Благодаря предварително !
  • Дарение

×

Информация

Поставихме бисквитки на устройството ви за най-добро потребителско изживяване. Можете да промените настройките си за бисквитки, или в противен случай приемаме, че сте съгласни с нашите условия за ползване.