Leerzeichen komprimieren

user369

Das Problem was ich habe ist, das ich nicht weiß wie ich zwei Werte an zwei verschiedenen Stellen einlese.

btw ... Diese Aufgabe ist als "leicht" klassifiziert aber bis jetzt konnte ich alle anderen Aufgaben und darunter auch als schwer klassifizierte innerhalb von 10-15min oder 30min alleine lösen. Hier sitz ich schon seit 2std und probiere mit dem vorhandenen Wissen was zu erreichen aber ich komm auf keinen grünen Zweig.
Oder ich stehe einfach nur gewaltig auf dem Schlauch.
Ich denke einfach, ich werde die Aufgabe erstmal sein lassen.

CStoll

Dann liest du halt nur ein Zeichen ein und merkst dir das jeweils letzte. (ich würde ja std::unique() verwenden)

user369

#include <iostream>
using namespace std;

int main () {
	char c1, c2;
	while (cin.get(c1)) {
		c2 = c1;
		if (c1 == ' ' && c2 == ' ') {
			cout << "";
		}
		else {
			if (c1 == ' ' && c2 != ' ') {
				cout << " ";
			}
			else {
				 cout << c1;
			}
		}
	}

	system("pause");
	return 0;
}

Das ist das was ich habe und ich denke das hier halt nur fehlt wie ich c2 den Wert des vorherigen c1 Wertes verpasse. Also mir ist klar das

c2 = c1

falsch ist aber ich weiß auch nicht wie ich es anders machen soll.

wxSkip

Setze doch mal den Pseudocode von SeppJ um. Der ist doch schon sehr ausführlich.

CStoll

user369 schrieb:

Also mir ist klar das
c2 = c1
falsch ist aber ich weiß auch nicht wie ich es anders machen soll.

Die Zuweisung ist schon richtig, nur steht sie an der falschen Stelle in der Schleife

user369

Ok, jetzt habe ich was gegessen und tadaaa

#include <iostream>
using namespace std;

bool c2 = false;

int main () {
	char c1;
	bool c2 = false;
	while (cin.get(c1)) {
		if (c1 == ' ' && c2) {
		}
		else {
			if (c1 == ' ') {
				cout << " ";
				c2 = true;
			}
			else {
				 cout << c1;
				 c2 = false;
			}
		}
	}

	system("pause");
	return 0;
}

Aber ich bräuchte für mich zum Verständnis noch eine Erklärung warum man c2 am Anfang als Konstant false ansieht?

Belli

Schau mal, so geht es etwas einfacher:

#include <iostream>
using namespace std;

int main () {
    char letztes = 'x';
    char aktuelles;

    while (cin.get(aktuelles))
    {
        if(!(aktuelles == ' ' && letztes == ' '))
        	  cout << aktuelles;

        letztes = aktuelles;
    }

}

zu Deiner Frage:
c2 steht auf false, wenn das zuletzt gelesene Zeichen kein Leerzeichen war, deshalb wird es zu Programmbeginn mit false initialisiert. Im Programmablauf wird es auf true gesetzt, wenn ein Leerzeichen gefunden wurde, und wieder auf false, wenn ein anderes Zeichen gelesen wurde.

wxSkip

user369 schrieb:

Ok, jetzt habe ich was gegessen und tadaaa
#include <iostream>
using namespace std;

bool c2 = false;

int main () {
	char c1;
	bool c2 = false;
	while (cin.get(c1)) {
		if (c1 == ' ' && c2) {
		}
		else {
			if (c1 == ' ') {
				cout << " ";
				c2 = true;
			}
			else {
				 cout << c1;
				 c2 = false;
			}
		}
	}

	system("pause");
	return 0;
}
Aber ich bräuchte für mich zum Verständnis noch eine Erklärung warum man c2 als Konstant false ansieht?

Du meinst wohl Default false? Das ist deshalb so, weil am Anfang der Eingabe ja noch keine Leertaste eingegeben wurde.

Tipp 1: Benenne deine Variablen vernünftig
Tipp 2: Die If-Bedingungen können SeppJ entsprechend vereinfacht werden.

user369

Belli schrieb:

Schau mal, so geht es etwas einfacher:
#include <iostream>
using namespace std;

int main () {
    char letztes = 'x';
    char aktuelles;
    
    while (cin.get(aktuelles))
    {
        if(!(aktuelles == ' ' && letztes == ' '))
        	  cout << aktuelles;
        	  
        letztes = aktuelles;
    }

}
zu Deiner Frage:
c2 steht auf false, wenn das zuletzt gelesene Zeichen kein Leerzeichen war, deshalb wird es zu Programmbeginn mit false initialisiert. Im Programmablauf wird es auf true gesetzt, wenn ein Leerzeichen gefunden wurde, und wieder auf false, wenn ein anderes Zeichen gelesen wurde.

Ok, cool. Ich denke ich habs jetzt.

Fettes Danke an alle

wxSkip

user369 schrieb:

Fettes Danke an alle

Bei dir ist aber das Fett fett und nicht das Danke

user369

Das war ausnahmsweise mal Absicht

Obligatorischer Verweis auf die Standardbibliothek:

#include <algorithm>
#include <iostream>
#include <iterator>

bool both_spaces(char c, char d) {
  return c == ' ' && d == ' ';
}

int main() {
  std::cin.unsetf(std::ios::skipws);
  std::unique_copy(std::istream_iterator<char>(std::cin),
                   std::istream_iterator<char>(),
                   std::ostream_iterator<char>(std::cout, ""),
                   both_spaces);
}