Problem Abfrage ob 1 oder 0 eingegeben wurde , C++

RBS2

@diejessi sagte in Problem Abfrage ob 1 oder 0 eingegeben wurde , C++:

Die Binäre Zahl wird eingegeben und überprüft werden soll, ob auch eine 1 oder eine 0 vorkommt.
Wenn etwas anderes als eine 1 oder 0 eingegeben worden ist, soll es mir einen Fehler ausgeben.

Pobier mal so:

    if (regex_match(bin, regex("[01]+")))
    {
        // Gültiger Binärstring
    }
    else
    {
        // Eingabe enthält falsche Zeichen
    }

Edit: #include <regex> nicht vergessen.

dirkski

@diejessi

[..]
Die Binäre Zahl wird eingegeben und überprüft werden soll, ob auch eine 1 oder eine 0 vorkommt.

Auch? Oder nur aus 0en und 1en besteht?

diejessi

@dirkski soll nur aus 0 und 1 bestehen, nichts anderes. Also eine Binäre Zahl soll eingegeben werden.

diejessi

@RBS2 #error This file requires compiler and library support for the \

das kommt dabei raus..

diejessi

@dirkski #error This file requires compiler and library support for the \

kommt dabei raus, wenn ich das so benutze

dirkski

@diejessi schau dir swordfishs if-abfrage an und vergleiche sie mit deiner. Ist kein großer Unterschied. Aber der ist halt wichtig.

Danke für die Nachformatierung deines codes.

Bin gerade mit dem smartfon dabei und ist ziemlich lästig mit der kopiererei...

diejessi

@dirkski

#include <iostream>
#include <cstdlib>
#include <iomanip>
#include <regex>

using namespace std; 

bool is_valid_binary(std::string const&str)
{
	for(auto ch : str)
		if(ch != '1' || ch != '0')
		return false; 
	return true;
}

int main(int argc, char** argv) 
{
	string bin; 
	
	cout << "BIN: " <<endl; 
	cin >> bin; 
	
		if(!is_valid_binary(bin))
		{
			cout << "f";
			laenge = 0; 
		}
		else
		{
			cout << "r";
			laenge = laenge -1;			
		}
	return 0;
}

Funktioniert nicht, ebenfalls eine Fehlermeldung..

dirkski

@diejessi und welche? Fehlermeldungen bitte auch in

```text
Fehlermeldung...
```

Bin unterwegs und habe nur termux-clang

dirkski

@diejessi wenn dein Compiler regex nicht kennt solltest du das entsprechende include raus nehmen...

RBS2

@diejessi sagte in Problem Abfrage ob 1 oder 0 eingegeben wurde , C++:

@RBS2 #error This file requires compiler and library support for the \

das kommt dabei raus..

Neuen Compiler nehmen. Meiner ist GCC 7.4.0

dirkski

Die liebe Jessie trollt, Beweis:

Sie hat swordsfishs code geändert das es nicht klappen kann. Ich bin raus

bool is_valid_binary(std::string const&str)
{
	for(auto ch : str)
		if(ch != '1' || ch != '0')
		return false; 
	return true;
}

Edit: ist ihr eingefügter code, original hat 2 &

diejessi

@dirkski

// Copyright (C) 2007-2014 Free Software Foundation, Inc.
//
// This file is part of the GNU ISO C++ Library.  This library is free
// software; you can redistribute it and/or modify it under the
// terms of the GNU General Public License as published by the
// Free Software Foundation; either version 3, or (at your option)
// any later version.

// This library is distributed in the hope that it will be useful,
// but WITHOUT ANY WARRANTY; without even the implied warranty of
// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
// GNU General Public License for more details.

// Under Section 7 of GPL version 3, you are granted additional
// permissions described in the GCC Runtime Library Exception, version
// 3.1, as published by the Free Software Foundation.

// You should have received a copy of the GNU General Public License and
// a copy of the GCC Runtime Library Exception along with this program;
// see the files COPYING3 and COPYING.RUNTIME respectively.  If not, see
// <http://www.gnu.org/licenses/>.

/** @file bits/c++0x_warning.h
 *  This is an internal header file, included by other library headers.
 *  Do not attempt to use it directly. @headername{iosfwd}
 */

#ifndef _CXX0X_WARNING_H
#define _CXX0X_WARNING_H 1

#if __cplusplus < 201103L
#error This file requires compiler and library support for the \
ISO C++ 2011 standard. This support is currently experimental, and must be \
enabled with the -std=c++11 or -std=gnu++11 compiler options.
#endif

#endif

diejessi

@dirkski ja er soll auch nicht überprüfen ob beides drin ist sondern mind einer..

auch mit dem & funktioniert das nicht

diejessi

@diejessi
Für alle die es interessiert, hab meinen Fehler gefunden... hier ist der richtige Code:

#include <iostream>
#include <cstdlib>
#include <iomanip>

using namespace std; 

int main(int argc, char** argv) 
 {
	string bin; 

	cout << "BIN: " <<endl; 
	cin >> bin; 
	
	int laenge = bin.length();
	
	while(laenge > 0 )
	{
		laenge = laenge -1;
		if(bin[laenge] == '1' || bin[laenge] == '0')
		{
			cout << "r";
				
		}
		else
		{
			cout << "f";
			laenge = 0; 		
		}
	}
	return 0;
}

hab nur eine Sache getauscht.. xD

Swordfish

Das schreit doch förmlich nach einer for-Schleife.
laenge = 0; ~> break;.
std::string::length() gibt keinen int zurück.
Die Parameter für main() und das return 0; am Ende sind überflüssig.
<string> fehlt, <cstdlib> und <iomanip> brauchst du nicht.

//edit: Eigentlich schreit's nach einer range-based-for:

#include <iostream>
#include <string>

int main()
{
    std::string bin;
    std::cin >> bin;
    for (auto ch : bin) {
        if (ch == '0' || ch == '1') {
            std::cout.put('r');
        } else {
            std::cout.put('f');
            break;
        }
    }
    std::cout.put('\n');
}

RBS2

Noch eine Möglichkeit:

#include <algorithm>
#include <iostream>
#include <string>

using namespace std;

int main()
{
    string bin;
    cin >> bin;

    bin.erase(remove_if(bin.begin(),bin.end(),[](char x){return x=='0'||x=='1';}),bin.end());
    cout << (bin.empty() ? "gute binärzahl!" : "falsche zeichen mit drinne!") << endl;
}

Swordfish

@RBS2 sagte in Problem Abfrage ob 1 oder 0 eingegeben wurde , C++:

bin.erase(remove_if(bin.begin(),bin.end(),[](char x){return x-'0'<2;}),bin.end());

auto new_end{ remove_if(bin.begin(), bin.end(), [](char x){ return x - '0' > 1; }) };
if (new_end != bin.end()) {
    // garbage
}

... dann könnte man mit bin danach sogar noch etwas anfangen ^^

Finnegan

Nochwas hierzu:

@wob sagte in Problem Abfrage ob 1 oder 0 eingegeben wurde , C++:

Man darf auf string[length_of_string] zugreifen, das ist '\0'.

Das ist so eine typische Information in einem Anfängerthread, die zwar nicht falsch ist, aber dennoch bestens geeignet, um Anfängern beim Lernen Knüppel zwischen die Beine zu werfen. Siehe hier:

@diejessi sagte in Problem Abfrage ob 1 oder 0 eingegeben wurde , C++:

@wob ich verstehe nich ganz...

Das gilt ab C++11 und nur für Strings (!). Für fast alle anderen array-ähnlichen Datenstrukturen ist diese Information falsch und sogar gefährlich. Man lernt unbeschwerter C++ wenn man sowas nicht weiss, und auch fortgeschrittene C++-Programmierer können ein Leben lang wunderbaren und korrekten Code schreiben, ohne jemals davon gehört zu haben

Spencer

Als Wiedereinsteiger in C++ (zuvor Python) ist mir folgende Lösung in den Sinn gekommen. Hierbei habe ich mich an der Python-Funktion all orientiert, welche glücklicherweise ein Äquivalent hat.:

#include <algorithm>
#include <iostream>
#include <string>


bool is_binary_digit(char c)
{
	return c == '0' or c == '1';
}

bool is_binary_number(const std::string& xs)
{
	return std::all_of(xs.begin(), xs.end(), is_binary_digit);
}

void test(const std::string& input)
{
	if (is_binary_number(input)) {
		std::cout << input << " is a valid binary number." << std::endl;
	}
	else {
		std::cout << input << " is NOT a valid binary number." << std::endl;
	}
}

int main()
{
	const std::string valid_input{ "01001000100" };
	const std::string invalid_input{ "0101a01c" };

	test(valid_input);
	test(invalid_input);
}