Как упростить проверку на правильность круглых скобок?

Question

[carbone]

Слишком большое условие, за код извиняюсь ,заранее спасибо

#include <iostream>
#include <fstream>

using namespace std;



int main() {
    const int n = 255;
    char text[n];
    ifstream file;
    file.open("Text.txt");
    while (file.get(text, n))
    {
        cout << "Text in txt file: " << text;
    }
    file.close();
    cout << text << endl;
    int sum = 0, open = 0, close = 0;
    for (int i = 0; i < n; i++)
    {
        if (text[i] == '(')
        {
            open = open + 1;
        }
        else if (text[i] == ')')
        {
            close = close + 1;


            if (close == open && close != 0 && open != 0)
            {
                sum = sum + close, close = 0, open = 0;

            }

        }
        else
            if (open > close)
            {
                sum = sum + close, close = 0, open = 0;
            }

           if (open < close)
          {
               sum = sum + open, close = 0, open = 0;
          }


    }
    cout << "Correct brackets: " << sum;


}

Comments

[Игорь Кузьмич]

вычеркни close != 0 или open != 0
одна из этих проверок лишняя

[Игорь Кузьмич]

если сомневаешься, преврати вычеркнутое в ассерт

[Игорь Кузьмич]

еще можно присваивание исправить
close = open = 0;
не зря же оператор = возвращает ссылкой значение

[carbone]

Игорь Кузьмич, Спасибо, мне больше интересно как можно количество условий меньше сделать, а то 6 условий как-то много

[Игорь Кузьмич]

а стой а где проверка, что ты делаешь если я получишь вот это: )))))(((((

[Игорь Кузьмич]

или вот это ))))
или вот это (((((

[carbone]

Игорь Кузьмич, я прям ооочень запутано сделал

if (close == open && close != 0 && open != 0)
            {
                sum = sum + close, close = 0, open = 0;

            }

        }
        else
            if (open > close)
            {
                sum = sum + close, close = 0, open = 0;
            }

           if (open < close)
          {
               sum = sum + open, close = 0, open = 0;
          }

я в коде просто считаю количество правильных пар скобок, а остальные игнорирую

Answer 1

[Игорь Кузьмич]

попробуй это

#include <iostream>
#include <cassert>

using namespace std;

int main()
{
	string input;
	cin >> input;
	int unclosed_count = 0;
	for (char c : input)
	{
		if (c == '(')
		{
			++unclosed_count;
		}	
		else if (c == ')')
		{
			--unclosed_count;
			assert(unclosed_count >= 0);//не знаю как надо реагировать на неправильные скобки, но вот тут нужно отслеживать корректность.
		}
	}
}

Comments

[carbone]

а в дальнейшем какие условия поставить чтоб вывести число правильных скобок?

[mayton2019]

Один парень в youtube красиво решает похожую задачу в С++ и многих других языках. Даю его решение. Хотя не уверен что автору оно подходит в силу особенностей. Может эта лаба должна демонстрировать базовые возможности С++.

https://www.youtube.com/watch?v=zrOIQEN3Wkk

Answer 2

[res2001]

Просто считайте количество открывающих скобок и количество закрывающих. Причем, если на закрывающей скобке количество открывающих <= количества закрывающих, то ее не учитываем (или учитываем в ошибках закрывающих скобок), т.к. нет парной открывающей.
Ошибки по открывающим скобкам будут, если в конце количество открывающих > количества закрывающих, количество ошибок открывающих скобок в этом случае = количество открывающих - количества закрывающих.
Количество правильных скобок == количеству закрывающих (т.к. не правильные закрывающие мы не считаем).