[C++]Error Frage

['Aleo]

Ich habe ein kleines Problem und zwar möchte ich einen Taschenrechner in C++ basteln und es kommt abe rimmer ein Fehler wenn ich 1*3 eintippe, bei 1+3 geht alles. Fehler: debug assertion failed vector subscript out of range
Code:
Den Klassennamen Funktionen und die Funktionsnamen auch bitte ignorieren war nur ein Versuch für Klassen

 Spoiler

#include "stdafx.h"
#include "Funktionen.h"
#include <string>
#include <iostream>
#include <algorithm>
#include <vector>
#include <boost\algorithm\string.hpp>

int main()
{
	Funktionen f;
	

	while (true)
	{
		std::vector<std::string> results;
		std::string s;
		std::cin >> s;

		if(s.find('+'))
		{
			boost::split(results, s, boost::is_any_of("+"), boost::token_compress_on);
			std::cout << f.Addieren(atoi(results[0].c_str()), atoi(results[1].c_str())) << std::endl;
		}
		if (s.find('*'))
		{
			boost::split(results, s, boost::is_any_of("*"), boost::token_compress_on);
			std::cout << results.capacity();
			std::cout << f.Multiplizieren(atoi(results[0].c_str()), atoi(results[1].c_str())) << std::endl;
		}
	}
	return 0;
}

Ich habe auch ein anderes Problem undzwar würde ich es auch gerne so machen das wenn man 1 + 2 eingibt die abstände gelöscht werden und erst dann gesplittet wird, jedoch bekomme ich das nicht hin, habe es mit s.erase(boost::remove_if(......)....); versucht es ging aber nicht.

Ich bedanke mich schonmal im Vorraus,
'Aleo

[Delinquenz]

Zum einen sollte man std::stoi und nicht atoi benutzen.

Andere Frage:

[adolfschaerf]

Falls es nicht klappen sollte könntest du ja alternativ mit std::cin eben integer, double etc. einlesen und die Rechenoperation gesondert behandeln mit switch-case. Imho kann man die Geschwindigkeit bei solch kleinen Projekten vernachlässigen.

[Schlüsselbein]

Lexer und Parser schreiben und tada, damit ganz einfach auch komplexe Strings auswerten.

[Tasiro]

Wenn schon Boost verwendet wird, kannst du dir auch gleich einen Parser mit schreiben. Warum einfach, wenn es auch kompliziert geht?

Davon abgesehen lässt sich ein einfacher Parser auch problemlos selbst erstellen:

 Spoiler

#include <iostream>
#include <string>
#include <cctype>

template <typename iterator>
int expression (iterator & begin, iterator end);
template <typename iterator>
int term (iterator & begin, iterator end);
template <typename iterator>
int factor (iterator & begin, iterator end);
template <typename iterator>
int number (iterator & begin, iterator end);

int main () {
	std::cout << "expression: ";
	std::string input;
	getline (std::cin, input);
	try {
		auto begin = input.begin ();
		auto end = input.end ();
		int value = expression (begin, end);
		std::cout << "value: " << value;
		if (begin != end)
			std::cout << "\ninput not completely parsed";
	} catch (const char * exception) {
		std::cout << "error: " << exception;
	}
	std::cout << '\n';

	std::cin.get ();
}

template <typename iterator>
int expression (iterator & begin, iterator end) {
	int sum = term (begin, end);
	while (begin != end)
		if (* begin == '+')
			sum += term (++begin, end);
		else if (* begin == '-')
			sum -= term (++begin, end);
		else
			return sum;
	return sum;
}

template <typename iterator>
int term (iterator & begin, iterator end) {
	int product = factor (begin, end);
	while (begin != end)
		if (* begin == '*')
			product *= factor (++begin, end);
		else if (* begin == '/') {
			int denominator = factor (++begin, end);
			if (! denominator)
				throw "division by zero";
			product /= denominator;
		} else
			return product;
	return product;
}

template <typename iterator>
int factor (iterator & begin, iterator end) {
	if (begin == end)
		throw "unexpected end of input";
	if (* begin == '(') {
		int value_inside = expression (++begin, end);
		if (begin == end || * begin != ')')
			throw "missing ')'";
		++begin;
		return value_inside;
	}
	return number (begin, end);
}

template <typename iterator>
int number (iterator & begin, iterator end) {
	int value = 0;
	if (begin == end || ! std::isdigit (* begin))
		throw "expected a number";
	do {
		value *= 10;
		value += * begin - '0';
		++begin;
	} while (begin != end && std::isdigit (* begin));
	return value;
}

Korrektheit nicht bewiesen - das Programm stürzte bei zehn zufälligen Eingaben nicht ab, ist also korrekt.
Was die einzelnen Funktionen tun, darf gerne erraten werden.
Die Funktionen könnten in eine eigene Klasse kommen, wie auch der Rückgabetyp parametrisiert werden könnte oder stattdessen ein Syntaxbaum erstellt werden könnte (nach Möglichkeit gleich ein abstrakter). Weiterhin wäre es sinnvoll, noch eine eigene Fehlerklasse erstellen, ebenso das Erstellen eines eigenen Lexers und einer entsprechenden Token-Klasse. Aber dann wiederum wäre die Nutzung von Spirit vielleicht doch nicht verkehrt, sofern man weiß, wie man damit richtig umgeht.

Aber wenn dich dieses Thema interessiert, kannst du dir natürlich das Drachenbuch kaufen, wenn du neben einfacher Syntaxanalyse auch den ganzen Rest eines Compilers kennen lernen möchtest. Oder du suchst einfach im Internet danach...