C Anfänger kommt nicht weiter

Hy

Kleine Einleitung:
Ich bin jetzt im 3.Semester in der Abend HTL für Elektrotechnik
und wir haben heuer mit C-Programmieren angefangen (leider haben wir nur jede zweite woche am Montag eine Einheit und am Mittwoch zwei)
Am Anfang haben wir uns in Informatik nochmal mit Binär und Hex Zahlen beschäftigt dann ist Hello World gekommen.
Das war noch kein Problem
Nach ein paar kleinen Programmen die auch noch nicht wirklich ein Problem dargestellt haben hat der Lehrer dann Vollgas gegeben
das war vor 2 Wochen
Wir haben die Erstellung und von Struktogrammen und deren Verwendung durchgenommen kein Problem
Dann haben wir einen Source Code von Ihm bekommen "Tippt das ab und versteht es" ohne jegliche Auskunft zu den schleifen ect.

Jetzt zu meinem Problem:
Diese Woche haben wir ein Struktogramm bekommen ohne Infos und Anhaltspunkte wie wir das lösen sollten/können

Es geht um einen Binär zu Dezimal Umrechner der ein Nibble umrechnen kann
und bei jeder Eingabe sofort die Zahl auf einen Fehler (sprich 0 od. 1 sonst Fehler) prüft
am ende gibt es dann eine Abfrage ob man noch eine Zahle umrechnen will oder ob das Programm beendet werden soll

Das wären mal meine Überlegungen gewesen nur sagt mir schon der Compiler da passt was nicht "Syntax Fehler":
Übrigens die IDE ist Bloodsheed Dev. C/Cpp unter Win 7

Code: C

#include <stdio.h>
#include <stdlib.h>

int main(void)
{
  int         a, b, c, d, erg;
  char        bin0, bin1, bin2, bin3, janein;
  
  printf      ("Binaer Nibbel in Dezimalzahl umwandeln\n");
  
  do                                                    // beginn der programm-schleife
  {
                                                        
      do
      {                                                  
              printf   ("4 Binaer-Ziffern eingeben: ");
              
              bin0 = getche();                          //erste zahl einlesen
              
              for (bin0 != 0 && bin0 != 1)
              {             //überprüfen ob wirklich nur 0 oder 1 eingegben wurde
              printf ("Nur Null oder Eins eingeben");  //fehlermeldung wenn eine andere zahle eigegeben wurde
              }
              
              bin1 = getche();                          //zweite zahl einlesen
              
              for (bin1 != 0 && bin1 != 1)              //überprüfen ob wirklich nur 0 oder 1 eingegben wurde
              {
              printf ("Nur Null oder Eins eingeben");  //fehlermeldung wenn eine andere zahle eigegeben wurde
              }
              
              bin2 = getche();                          //dritte zahl einlesen
              
              for (bin2 != 0 && bin2 != 1)             //überprüfen ob wirklich nur 0 oder 1 eingegben wurde
              {
              printf ("Nur Null oder Eins eingeben");  //fehlermeldung wenn eine andere zahle eigegeben wurde
              }
              
              bin3 = getche();                          //vierte zahl einlesen
              
              for (bin3 != 0 && bin3 != 1)             //überprüfen ob wirklich nur 0 oder 1 eingegben wurde
              {
              printf ("Nur Null oder Eins eingeben");  //fehlermeldung wenn eine andere zahle eigegeben wurde
              }
      }while ( (bin0 !=0 || bin0 !=1) && (bin1 !=0 || bin1 !=1) && (bin2 !=0 || bin2 !=1) && (bin3 !=0 || bin3 !=1));      
      
      
              do
              {
               erg = ((var0 * 8 ) + (var1 *4) + (var2 * 2) + var3 )
               printf    ("Das Ergebniss = %i\n", erg);
               printf    ("\n");
               printf    ("Wollen Sie noch eine Berechnung durchführen? (Ja...j eingeben oder Nein...n eingeben");
               
               janein = getche();
               janein = toupper(janein);  
               
               if (janein != 'N' && janein != 'J')
               {
                          printf ("Es wurde ein falschen Zeichen eingegeben!!!");
                          
               }  
               
              }while (janein != 'J' && janein != 'N');
  }while (janein == 'J'); // ende des programms

  
  system("PAUSE");	
  return 0;
}

Ich bitte um kleine richtungsweisende Tips nicht um die Komplettlösung
Wenn ich total falsch bin könnt ihr das gerne sagen
in dem Struktogramm ist zwar eine case switch verzeichnet ich weis aber nicht wirklich was ich damit anfangen sollte in diesem fall

Hier noch das Struktogramm



Gruß
noir

der standard for-statement ausdruck schaut anders aus, wie du es es in deinem for-statement hast. kann mir sogar vorstellen (hab das jetzt in deinem text gelesen, hab vorher nur den code angeschaut), dass ein error geworfen wird, weil der syntax nicht passt.

entweder while oder if würden passen.

was ich auch nicht genau versteh ist, wieso hast du bin0, etc als char, wenn es nur 0 oder 1 sein soll, hab jetzt den code noch ein wenig überflogen, wenn ein chast kommt, dann passts.

müsste doch ein error dann ausgeben, wenn du nur 0 oder 1 eingegeben werden soll und dann kommt vllt ein 'a' oder 'b', dann passt es ja nicht mehr laut deinem Kommentar bei der Abfrage.

mit einem int und einem if kannst es so schreiben: if(bin0 < 0 && bin0 > 1) ... da hast bei int nur 0 oder 1. kannst zwar auch 0.5 eingeben, sollte aber auch vorher dann abgefangen werden mit try & catch, weil ja "int".

Die fors in den Zeilen 21-42 sollten ifs sein

Code: C

if((bin0 != 0) && (bin0 != 1)) {           
//überprüfen ob wirklich nur 0 oder 1 eingegben wurde
//fehlermeldung wenn eine andere zahle eigegeben wurde
printf ("Nur Null oder Eins eingeben");
}

...double pwnd...war klar

btw. Struktogramme werden imho gar nicht mehr verwendet. Flow Charts sind da zeitgemäßer.

Die 'for' in if's umwandeln ist nicht der Weisheits letzter Schluss, da man sich so eine erneute direkte Eingabe verbaut. Deshalb ist hier ein do-while einfacher. (While selbst geht nicht, da die variable überprüft wird, bevor was drinnen steht bzw. nicht mehr überprüft wird, wenn etwas falsches drinnen steht. Dsa muss man sich immer vor Augen halten)

Da du noch echt ein paar Defizite hast (gibst du ja selbst zu - vor allem wegen der Kontrollstrukturen), würde ich Vorschlagen du ackerst das mal etwas durch:
http://openbook.galileocomputing.de/c_von_a_bis_z/

everything what that said + "geh leck ein struktogramm".

Wird schon werden

Nebenbei Anekdote: Ich hab den Code zuerst mit GCC kompiliert und, bist du grauslich, da waren die Fehlermeldungen ... dürftig. Hab gehört das "clang" beim Fehlerreporten viel besser ist, also "apt-get install clang", das sind Welten unterschied O_O. Und dann auch glauch colored console output, w00t!

Mir wäre auch Code Blocks @ Ubuntu lieber aber der Lehrer hat von meiner Idee nicht viel gehalten
Shit happens

gcc -Wall -lm shit.c -o shit

http://pastebin.com/Gt5cNhGk
Funktioniert solange Integer eingegeben werden.

Ist ein Aufruf von main erlaubt? #120

Kann man gegen die Warnung etwas tun?
shit.c:27:7: warning: implicit declaration of function ‘__fpurge’ [-Wimplicit-function-declaration]

Ob es bei printf("\a"); wirklich piept konnte ich noch nicht testen.