Ich habe es so versucht

bool test (string str){
   bool a = true;

   for (int k=0; k<str.length();++k){
	if (str[k]!=('A'||'G' ){
	a = false;
	break;
	}
  }
   return a;
}

Das funktioniert leider so nicht, und ich bekomm ein false obwohl nur As und Gs vorkommen. Also irgendwie verstehe ich diese logischen operatoren nicht so ganz. Wie realisier ich denn ein weder noch?

str[k]!='A' && str[k]!='G')

Wie kann das kompilieren?

bool test (string str)
{
   return str.find_first_not_of("AG") == std::string::npos;
}

Edit: Rechtschreibfehler.
Edit³: Man, ich mach die ganze Zeit Flüchtigkeitsfehler...

if (str[k]!=('A'||'G')
{
    a = false;
    break;
}

Wie kann das kompilieren?

Das ist zwar nicht sinnvoll, aber gültiges C++. Zuerst wird 'A' mit 'G' logisch verODERt, was true ergibt. true ist in den Zahlenwert 1 konvertierbar. Dieser wird mit str[k] verglichen. Das ist den Compilern die ich kenne nicht einmal eine Warnung wert.

FreakY<3Cpp schrieb:

if (str[k]!=('A'||'G')
{
    a = false;
    break;
}

Wie kann das kompilieren?

Das ist zwar nicht sinnvoll, aber gültiges C++. Zuerst wird 'A' mit 'G' logisch verODERt, was true ergibt. true ist in den Zahlenwert 1 konvertierbar. Dieser wird mit str[k] verglichen. Das ist den Compilern die ich kenne nicht einmal eine Warnung wert.

Ich meinte eher die Klammersetzung.

SeppJ schrieb:

FreakY<3Cpp schrieb:

if (str[k]!=('A'||'G')
{
    a = false;
    break;
}

Wie kann das kompilieren?

Das ist zwar nicht sinnvoll, aber gültiges C++. Zuerst wird 'A' mit 'G' logisch verODERt, was true ergibt. true ist in den Zahlenwert 1 konvertierbar. Dieser wird mit str[k] verglichen. Das ist den Compilern die ich kenne nicht einmal eine Warnung wert.

Ich meinte eher die Klammersetzung.

Aha! Hier haben wir also einen klaren Fall von NICHT KOPIERT!

FreakY<3Cpp schrieb:

SeppJ schrieb:

FreakY<3Cpp schrieb:

if (str[k]!=('A'||'G')
{
    a = false;
    break;
}

Wie kann das kompilieren?

Das ist zwar nicht sinnvoll, aber gültiges C++. Zuerst wird 'A' mit 'G' logisch verODERt, was true ergibt. true ist in den Zahlenwert 1 konvertierbar. Dieser wird mit str[k] verglichen. Das ist den Compilern die ich kenne nicht einmal eine Warnung wert.

Ich meinte eher die Klammersetzung.

Aha! Hier haben wir also einen klaren Fall von NICHT KOPIERT!

Ja, damit man weiß was ich meine. Übrigens ist deine Lösung falsch.

Übrigens ist deine Lösung falsch.

weil..........
Edit: Jetzt ohne Spaß, die ist völlig korrekt.

die einen String darauf testet ob nur Buchstaben A und G drin vorkommen.

Sone schrieb:

FreakY<3Cpp schrieb:

Übrigens ist deine Lösung falsch.

weil..........
Edit: Jetzt ohne Spaß, die ist völlig korrekt.

Der TE will, dass beide Buchstaben vorkommen sollen.

bool test(const string& str)
{
	return str.find_first_not_of("AG") == std::string::npos;
}

int main()
{
	cout << boolalpha << test("AAAA") << endl; // sollte false sein.
	cout << boolalpha << test("GGGG") << endl; // sollte false sein.
	cout << boolalpha << test("AAGG") << endl; // nur das sollte true sein.
}

Mein Gott, nimm doch einfach einen Algorithmus

Er bastelt sich doch nen Algorithmus. Du rufst einfach nur ne Funktion auf.

Gugelmoser schrieb:

Der TE will, dass beide Buchstaben vorkommen sollen.

Wo steht das?

Gugelmoser schrieb:

Der TE will, dass beide Buchstaben vorkommen sollen.

Wo steht das?

quote - String darauf testet ob nur Buchstaben A und G drin vorkommen. - end quote

Zumindest verstehe ich das unter und . :p

Also die aufgabe war einen DNA strang darauf zu überprüfen ob nur A T G und C drin vorkommen und dann aufgaben die darauf aufbauen. zB. dass der Strang durch drei teilbar ist usw. Die klammarsetzung war natürlich völliger blödsinn, aber bin ja auch anfänger :p

Bin immer wieder erstaunt welche diskussionen einfache fragen entfachen

Ich bin immer wieder erstaunt darüber, wie unpräzise man Fragen stellen kann.

Namal schrieb:

Also die aufgabe war einen DNA strang darauf zu überprüfen ob nur A T G und C drin vorkommen

Ein DNA Strang hat immer nur A T G und C "drin". Du willst wahrscheinlich einen String (Zeichenkette) überprüfen, ob sie einen DNA Strang repräsentiert (rate ich jetzt mal).

Ich bin immer wieder erstaunt darüber, wie unpräzise man Fragen stellen kann.

Dein nick ist bei dir wohl das programm...

Das einzige was der OP verbessern kann ist der Threadtitel. Die Frage war sonnenklar.

Nach 3 mal editieren schon.

Tut mir Leid, aber dein Beitrag war 5 Minuten nach meinem letzten Edit.

Aber sry, ich muss weniger Editieren.

@Michael: HMM? Wurde ihm vorgeschrieben, dass er selber einen so banalen Algorithmus programmieren soll?
Anstatt sich also die Hände schmutzig zu machen, tut er, was alle Programmierer tun: faul sein

@Michael: HMM? Wurde ihm vorgeschrieben, dass er selber einen so banalen Algorithmus programmieren soll?

Was Michael meint ist, dass es bereits ein Algorithmus war. Du hast nur eine Funktion aufgerufen.

Und ich kapiere nicht, wie man die Frage des OP falsch verstehen kann.

Das einzige was der OP verbessern kann ist der Threadtitel. Die Frage war sonnenklar.

Vorallem wenn man sich dann selbst mit sowas "ownt"

krümelkacker schrieb:

Ein DNA Strang hat immer nur A T G und C "drin". Du willst wahrscheinlich einen String (Zeichenkette) überprüfen, ob sie einen DNA Strang repräsentiert (rate ich jetzt mal).

Wo ist bei deinem "nur A T G und C" zu meinem "nur A und G" ?? Soweit mir bekannt, müssen nicht alle diese buchstaben vorkommen.

Wo ist bei deinem "nur A T G und C" zu meinem "nur A und G" ??

Namal schrieb:

Also die aufgabe war einen DNA strang darauf zu überprüfen ob nur A T G und C drin vorkommen

Und was zum Henker soll denn bitte außerdem noch in einem DNA-strang vorkommen?
Demnächst hier: eine Funktion die prüft, ob in einem boolschen Wert nur true oder false vorkommt.

Demnächst hier: eine Funktion die prüft, ob in einem boolschen Wert nur true oder false vorkommt.

constexpr bool isTrueOrFalse(bool)
{
    return true;
}

Namal schrieb:

krümelkacker schrieb:

Ich bin immer wieder erstaunt darüber, wie unpräzise man Fragen stellen kann.

Dein nick ist bei dir wohl das programm...

Das würde ich nicht sagen. kk ist IMO mitunter einer der kompetentesten hier im Forum.

Namal schrieb:

Shade schrieb:

Das einzige was der OP verbessern kann ist der Threadtitel. Die Frage war sonnenklar.

Vorallem wenn man sich dann selbst mit sowas "ownt"

Was erwartest du eigentlich, wie man dir hilft, wenn du alle nur runtermachst?

Nicht so fies

Sone schrieb:

@Michael: HMM? Wurde ihm vorgeschrieben, dass er selber einen so banalen Algorithmus programmieren soll?

Was Michael meint ist, dass es bereits ein Algorithmus war. Du hast nur eine Funktion aufgerufen.

Ahh. Sehr gut bemerkt, Michael.
Ich weiß übriges sehr wohl, was ein Algorithmus ist, brauchst nichts zu verlinken.