Funktion für nur Zahlen und nur Buchstaben

Power Off

MFK schrieb:

Mehrere Leute haben dir an mehreren Stellen gesagt, was sie an deinem Code stört. In diesem und auch in anderen Threads. Du verstehst die Argumente nicht.

Du kannst Dich nach Deinen Aussagen nicht einfach auf die auf die Aussagen anderer berufen.

Gib mir konkrete Beispiele und Begruendungen, dann reden wir weiter.

Power Off

volkardus schrieb:

manchmal hat man "by design" bestimmte sonderwünsche ausgeschlossen. natürlich in absprache mit dem kunden. und genau die will er dann doch haben.

Das ist Unsinn. Man kann Programme so schreiben, dass sie von vornherein in jede Richtung erweiterbar sind.

Nur uebelster Spaghetti-Code laesst sich nicht erweitern. Du willst doch nicht behaupten, dass Du das machst?

Deine Beispiele sind etwas aus der Luft gegriffen, uebrigens.

volkardus schrieb:

du meinst kene zyklischen abhängigkeiten? äh, das kommt aber schon mal vor. warum die forderung?

Nein, in gutem Code kommt das nicht vor. Das ist Spaghetti-Code (also verworrener Code).

Wenn es aus Design-Gruenden so erscheint, als ob es nicht anders geht, sollte man sobald moeglich, das Design entsprechend anpassen.

volkardus schrieb:

aber ich hab ne andere: jede funktion erfüllt nur einen zweck. dein monster liest whitespaces davor weg und danach, kümmert sich ums vorzeichen und liest die ziffernfolge selbst ein. das wäre für mich mindestens 4 funktionen, für jeden einfachen zweck eine und eine, die alles zusammenpappt.

Die Funktion erfuellt den Zweck, dass sie eine Zahl aus einer Zeile liest.

Man kann's mit dem Aufbrechen von Funktionalitaet auch uebertreiben.

MFK

Gib mir konkrete Beispiele und Begruendungen, dann reden wir weiter.

Das bringt bei dir nichts. Du bist in deinen Ansichten anscheinend so festgefahren, dass davon bei dir nichts ankommt.

Lass dir eine Merkbefreiung ausstellen.

volkard

Power Off schrieb:

Das ist Unsinn. Man kann Programme so schreiben, dass sie von vornherein in jede Richtung erweiterbar sind.

man kann nicht alle programme so schreiben. schon die sprachauswahl setzt einem sehr enge grenzen.

Nur uebelster Spaghetti-Code laesst sich nicht erweitern. Du willst doch nicht behaupten, dass Du das machst?

"spaghetti-code" ist ein schlagwort aus alter zeit. das wort benutze ich gar nicht, weil man keinen spaghetti-code baut. seit jahrzehnten nicht mehr. bisher ist dein code am dichtesten dran, aber er ist noch "strukturiert", das muß man zugeben.

volkardus schrieb:

du meinst keine zyklischen abhängigkeiten? äh, das kommt aber schon mal vor. warum die forderung?

Nein, in gutem Code kommt das nicht vor. Das ist Spaghetti-Code (also verworrener Code).

es ging um klassen. haste schonmal forward-deklaraionen machen müssen?

Wenn es aus Design-Gruenden so erscheint, als ob es nicht anders geht, sollte man sobald moeglich, das Design entsprechend anpassen.

das paradebeispiel für zyklische abhängigkeiten von funktionen geschieht doch regelmäßig beim rekursiven abstiegscompiler. und du bist doch compilerbauprofi? wie kann dir das nicht sofort einfallen?

Die Funktion erfuellt den Zweck, dass sie eine Zahl aus einer Zeile liest.

sie macht viele verschiedenen dinge NACHEINANDER. gut mit namen benennbare dinge, die sogar unabhängig voneinander funktionieren. also wer die nicht trennt, hat echt die letzten 20 jahre nix dazugelernt. früher war es in der tat üblich, so zu programmieren. da war es ja auch schneller, denn es gar beim inline. aber das hat sich total geändert, daß muß ich dir mal sagen. schau doch am besten mal auch in die enschlägigen newsgroups und die anderen foren (villeicht nicht gerade nicht zfx oder heise) und schau, was die so machen.

Man kann's mit dem Aufbrechen von Funktionalitaet auch uebertreiben.

eigentlich nicht.
aber sagmal, isdigit und RANGE kannste doch einsehen?
warum nicht wenigstens noch eatwhite?

Power Off

volkard schrieb:

man kann nicht alle programme so schreiben. schon die sprachauswahl setzt einem sehr enge grenzen.

Wovon sprichst Du?

Man kann so gut wie in jeder Sprache so gut wie alles programmieren.

Dass man in Java z.B. keine hochperformanten, hardwarenahen Programme schreiben kann ist klar, aber dafuer gibt's ja JNI.

In C oder C++ bin ich noch auf kein unmoegliches Problem gestossen. Ausser es war zur der Zeit tatsaechlich technisch voellig unmoeglich (z.B. sowas wie "Bagger, der in der Luft schwebt und das Raum-Zeit-Kontinuum verzerrt" ).

volkard schrieb:

"spaghetti-code" ist ein schlagwort aus alter zeit. das wort benutze ich gar nicht, weil man keinen spaghetti-code baut. seit jahrzehnten nicht mehr. bisher ist dein code am dichtesten dran, aber er ist noch "strukturiert", das muß man zugeben.

Nach dem bisschen Code, was Du von mir gesehen hast, kannst Du das wohl kaum behaupten. Und rein objektiv gesehen, ist er sauber, strukturiert, und hat keine zyklischen Abhaengigkeiten.

Das Schlagwort "Spaghetti-Code" ist immer aktuell, wenn's ums programmieren geht.

Auch bei objektorientierten Sprachen (gerade dort, wo viel Quark fabriziert wird).

volkard schrieb:

haste schonmal forward-deklaraionen machen müssen?

Klar, z.B. fuer Rekursionen und Listen.

Aber bei Klassen z.B. koennen zyklische Abhaengigkeiten "toedlich" sein.

volkard schrieb:

das paradebeispiel für zyklische abhängigkeiten von funktionen geschieht doch regelmäßig beim rekursiven abstiegscompiler. und du bist doch compilerbauprofi? wie kann dir das nicht sofort einfallen?

Weil bei Rekursionen die Abhaengigkeiten vorgegeben sind. Bei einem Compiler mit rekursivem Abstieg geht das ja gar nicht anders. Bei einem Compiler sind auch alle Abhaengigkeiten z.B. eines Syntax-Analyzers in der EBNF dokumentiert. D.h. der Syntax-Analyzer implementiert die EBNF 1:1.

Ich spreche eigentlich von zyklischen Abhaengigkeiten auf hoeherer Ebene, also z.B. "Komponente X erstellt Komponente Y. Komponente Y modifiziert Komponente X. Komponenten X und Y modifizieren sich gegenseitig. Thread T modifiziert Komponente X. Thread U modifiziert Komponente Y und X. Thread V ruft Code aus Thread T und U auf. Thread V modifiziert manchmal Komponente Y und manchmal X. Thread U beendet Thread V. Thread V beendet manchmal Thread T. Thread T erstellt manchmal Thread W, der auf Thread V wartet."

So was in der Art.

volkard schrieb:

warum nicht wenigstens noch eatwhite?

Weil readnum() nicht nur white-space ueberliest. Vielleicht solltest Du vor ausfuehrlicher Kritik den Code nochmal genau ansehen.

volkard

Power Off schrieb:

volkard schrieb:

man kann nicht alle programme so schreiben. schon die sprachauswahl setzt einem sehr enge grenzen.

Wovon sprichst Du?

symbolisch differenzieren geht mit lisp in ein paar zeilen. in c wirste alt dabei.
haste nen interpreter drunter kannste dir unter umständen einen mathematischen parser schenken.
haste keine lokalen variablen, wirds mit komplexeren programmen nicht gerade leicht.

Man kann so gut wie in jeder Sprache so gut wie alles programmieren.

halt mit mehr oder weniger aufwand.

anläßlich eines programmierwettbewerbs, bei dem alle sprachen erlaubt waren, leistete ich mir den spaß, die aufgabe, nach dem gregorianischen kalnender aus tag, monat, jahr, die der user zuerst einzugeben hatte, den wochenzag zu berechnen, in brainfuck zu erstellen. zum testen nimm http://koti.mbnet.fi/villes/php/bf.php
und natürlich muß ich betonen, daß der code klar und einfach ist. und wer den nicht kapiert oder nicht lesen mag, hat natürlich keine ahnung von brainfuck. ich such dir gerne einen link mit der sprachbeschreibung raus. oder ich zeig die ne kommentierte version.

>++++[->++++<]>[-<++++>]<+++++.>++[->++++<]>[-<++++>]<[-<+>]<++++.+++++.-------.
------.+.+++.>++[->++++<]>[-<++++>]<.[-]<++++.++++.>++[->++++<]>[-<++++>]<.[-]++
[->++++<]>[-<++++>]<[-<->]<-------.>++[->++++<]>[-<++++>]<[-<+>]<+++++++++.+++.-
----.------------.+++++++++++++++++++.>++[->++++<]>[-<++++>]<.[-]<[-]+++[->++++<
]>[-<++++>]<.+++++.-----.++++++++.-------.++++++++.--.-.++++.>++[->++++<]>[-<+++
+>]<.[-]>>,>+++[->++++<]>[-<++++>]<[-<->]<[->++<]>[-<+++++>],>+++[->++++<]>[-<++
++>]<[-<->]<[-<+>],>+++[->++++<]>[-<++++>]<[-<->]<[->++<]>[-<+++++>],>+++[->++++
<]>[-<++++>]<[-<->]<[-<+>],>+++[->++++<]>[-<++++>]<[-<->]<[->++<]>[-<+++++>],>++
+[->++++<]>[-<++++>]<[-<->]<[-<+>],>+++[->++++<]>[-<++++>]<[-<->]<[->++<]>[-<+++
++>],>+++[->++++<]>[-<++++>]<[-<->]<[-<+>]<[->>+>+<<<]>>>[-<<<+>>>]++++[->>+>+<<
<]>>>[-<<<+>>>]<-<<<[>>>[->+>+<<]>>[-<<+>>]+<[[-]>-<]>[-<+>]<[[-]<<+<[->>+>+<<<]
>>>[-<<<+>>>]]<-<<<-]>>>[-<<->>]<[-]<-[-<+>]+<[[-]>-<]>[-<+>]<[-<+>]<<[->>+>+<<<
]>>>[-<<<+>>>]+<[[-]>-<]>[-<+>]<[-<->]<<<[->>>+>+<<<<]>>>>[-<<<<+>>>>]++++[->>+>
+<<<]>>>[-<<<+>>>]<-<<<[>>>[->+>+<<]>>[-<<+>>]+<[[-]>-<]>[-<+>]<[[-]<<+<[->>+>+<
<<]>>>[-<<<+>>>]]<-<<<-]>>>[-<<->>]<[-]<-[-<+>]<<<[->>>+>+<<<<]>>>>[-<<<<+>>>>]<
[-<+>]+<[[-]>-<]>[-<+>]<[-<+>]<<<[->>>+>+<<<<]>>>>[-<<<<+>>>>]<<<[->>>+>+<<<<]>>
>>[-<<<<+>>>>]<[-<+>]+<[[-]>-<]>[-<+>]<[-<<<<<<<+>>>>>>>]<<<<[->>>>>+>+<<<<<<]>>
>>>>[-<<<<<<+>>>>>>]<[->+>+<<]>>[-<<+>>]<[[-]<->++++[-<<++++++++>>]<<->>]<[->+>+
<<]>>[-<<+>>]<[[-]<-><<--->><<<[->>>>+>+<<<<<]>>>>>[-<<<<<+>>>>>]<[-<<<+>>>]<]<[
->+>+<<]>>[-<<+>>]<[[-]<-><<+++>><<<[->>>>+>+<<<<<]>>>>>[-<<<<<+>>>>>]<[-<<<->>>
]<]<[->+>+<<]>>[-<<+>>]<[[-]<-><<->>]<[->+>+<<]>>[-<<+>>]<[[-]<-><<+>>]<[->+>+<<
]>>[-<<+>>]<[[-]<-><<->>]<[->+>+<<]>>[-<<+>>]<[[-]<-><<+>>]<[->+>+<<]>>[-<<+>>]<
[[-]<->]<[->+>+<<]>>[-<<+>>]<[[-]<-><<->>]<[->+>+<<]>>[-<<+>>]<[[-]<-><<+>>]<[->
+>+<<]>>[-<<+>>]<[[-]<-><<->>]<[->+>+<<]>>[-<<+>>]<[[-]<-><<+>>]<[->+>+<<]>>[-<<
+>>]<[[-]<-><<[-]>>]<[-]<[->+>+<<]>>[-<<+>>]+<[[-]>-<]>[-<+>]<[[-]<<<<<<<<+>>>>>
>>>]<<<<<<[->>>>>>+>+<<<<<<<]>>>>>>>[-<<<<<<<+>>>>>>>]<[<[->>+>+<<<]>>>[-<<<+>>>
]+<[[-]>-<]>[-<+>]<[[-]<<+>><<<<<<<<<+>>>>>>>>>]<<->-]<[-]<[-]+<<<[->>>>+>+<<<<<
]>>>>>[-<<<<<+>>>>>]<[-[-[-<->[-]]]]<[[-]<[->+>+<<]>>[-<<+>>]+<[[-]>-<]>[-<+>]<[
[-]<++++++++++[->++<]>[-<+++++>]<<->>]<->]<[->+>+<<]>>[-<<+>>]+++++++[->>+>+<<<]
>>>[-<<<+>>>]<-<<<[>>>[->+>+<<]>>[-<<+>>]+<[[-]>-<]>[-<+>]<[[-]<<+<[->>+>+<<<]>>
>[-<<<+>>>]]<-<<<-]>>>[-<<->>]<[-]<-[-<+>]<[-<<<<<+>>>>>]<[->+>+<<]>>[-<<+>>]+++
+[->>+>+<<<]>>>[-<<<+>>>]<-<<<[>>>[->+>+<<]>>[-<<+>>]+<[[-]>-<]>[-<+>]<[[-]<<+<[
->>+>+<<<]>>>[-<<<+>>>]]<-<<<-]>>>[-]<[-<<+>>]<[-]+++++++[->>+>+<<<]>>>[-<<<+>>>
]<-<<<[>>>[->+>+<<]>>[-<<+>>]+<[[-]>-<]>[-<+>]<[[-]<<+<[->>+>+<<<]>>>[-<<<+>>>]]
<-<<<-]>>>[-<<->>]<[-]<-[-<+>]<[-<<<<<+>>>>>]<<[->>+>+<<<]>>>[-<<<+>>>]+++++++[-
>>+>+<<<]>>>[-<<<+>>>]<-<<<[>>>[->+>+<<]>>[-<<+>>]+<[[-]>-<]>[-<+>]<[[-]<<+<[->>
+>+<<<]>>>[-<<<+>>>]]<-<<<-]>>>[-<<->>]<[-]<-[-<+>]<[-<<<<<+++++>>>>>]+++++++[-<
<<<<+>>>>>]<<[->>+>+<<<]>>>[-<<<+>>>]++++[->>+>+<<<]>>>[-<<<+>>>]<-<<<[>>>[->+>+
<<]>>[-<<+>>]+<[[-]>-<]>[-<+>]<[[-]<<+<[->>+>+<<<]>>>[-<<<+>>>]]<-<<<-]>>>[-]<[-
<<+>>]<[-]+++++++[->>+>+<<<]>>>[-<<<+>>>]<-<<<[>>>[->+>+<<]>>[-<<+>>]+<[[-]>-<]>
[-<+>]<[[-]<<+<[->>+>+<<<]>>>[-<<<+>>>]]<-<<<-]>>>[-<<->>]<[-]<-[-<+>]<[-<<<<<+>
>>>>]<<<[->>>>+>+<<<<<]>>>>>[-<<<<<+>>>>>]<[->+>+<<]>>[-<<+>>]<[[-]<->]<[->+>+<<
]>>[-<<+>>]<[[-]<-><<+++>>]<[->+>+<<]>>[-<<+>>]<[[-]<-><<->>]<[->+>+<<]>>[-<<+>>
]<[[-]<-><<+++>>]<[->+>+<<]>>[-<<+>>]<[[-]<-><<++>>]<[->+>+<<]>>[-<<+>>]<[[-]<->
<<+++>>]<[->+>+<<]>>[-<<+>>]<[[-]<-><<++>>]<[->+>+<<]>>[-<<+>>]<[[-]<-><<+++>>]<
[->+>+<<]>>[-<<+>>]<[[-]<-><<+++>>]<[->+>+<<]>>[-<<+>>]<[[-]<-><<++>>]<[->+>+<<]
>>[-<<+>>]<[[-]<-><<+++>>]<[->+>+<<]>>[-<<+>>]<[[-]<-><<++>>]<[-]+++++++[->>+>+<
<<]>>>[-<<<+>>>]<-<<<[>>>[->+>+<<]>>[-<<+>>]+<[[-]>-<]>[-<+>]<[[-]<<+<[->>+>+<<<
]>>>[-<<<+>>>]]<-<<<-]>>>[-<<->>]<[-]<-[-<+>]<[-<<<<<+>>>>>]<<<<[->>>>+>+<<<<<]>
>>>>[-<<<<<+>>>>>]<[-<<<<<+>>>>>]<[-]<[-]<[-]<[-]+++++++[->>+>+<<<]>>>[-<<<+>>>]
<-<<<[>>>[->+>+<<]>>[-<<+>>]+<[[-]>-<]>[-<+>]<[[-]<<+<[->>+>+<<<]>>>[-<<<+>>>]]<
-<<<-]>>>[-<<->>]<[-]<-[-<+>]<[->+<]<[->+<]>>[-<<+>>]<[[-]<[-]+++++++>]+[->+>+<<
]>>[-<<+>>]<[[-]<<[->>>+>+<<<<]>>>>[-<<<<+>>>>]+<[[-]>-<]>[-<+>]<[[-]>++++[->+++
+<]>[-<++++>]<+++++++++++++++++++.>++[->++++<]>[-<++++>]<[-<+>]<----.-..++++++.-
------------------.++++++.[-]<<<->>]<]<[->+>+<<]>>[-<<+>>]<[[-]<<->><<[->>>+>+<<
<<]>>>>[-<<<<+>>>>]<>+<[[-]>-<]>[-<+>]<[[-]>++++[->++++<]>[-<++++>]<++++++++++++
+.>++[->++++<]>[-<++++>]<[-<+>]<++.-.++++++.-------------------.++++++.[-]<<<->>
]<]<[->+>+<<]>>[-<<+>>]<[[-]<<->><<[->>>+>+<<<<]>>>>[-<<<<+>>>>]+<[[-]>-<]>[-<+>
]<[[-]>++++[->++++<]>[-<++++>]<++++.>++[->++++<]>[-<++++>]<[-<+>]<+++++.----.+++
++++++.+++++.+.-------------------.++++++.[-]<<<->>]<]<[->+>+<<]>>[-<<+>>]<[[-]<
<->><<[->>>+>+<<<<]>>>>[-<<<<+>>>>]+<[[-]>-<]>[-<+>]<[[-]>++++[->++++<]>[-<++++>
]<+++++++++++++.>++[->++++<]>[-<++++>]<[-<+>]<----.+++++++++++..+++.--------.---
---------.+++++.[-]<<<->>]<]<[->+>+<<]>>[-<<+>>]<[[-]<<->><<[->>>+>+<<<<]>>>>[-<
<<<+>>>>]+<[[-]>-<]>[-<+>]<[[-]>++++[->++++<]>[-<++++>]<++++.>++[->++++<]>[-<+++
+>]<[-<+>]<+++++++++++.-..---------.+++++++++++++.+.+.-------------------.++++++
.[-]<<<->>]<]<[->+>+<<]>>[-<<+>>]<[[-]<<->><<[->>>+>+<<<<]>>>>[-<<<<+>>>>]+<[[-]
>-<]>[-<+>]<[[-]>++++[->++++<]>[-<++++>]<++++++.>++[->++++<]>[-<++++>]<[-<+>]<++
++++++++++.-------------.++++.+++++++++++.-------------------.++++++.[-]<<<->>]<
]<[->+>+<<]>>[-<<+>>]<[[-]<<->><<[->>>+>+<<<<]>>>>[-<<<<+>>>>]+<[[-]>-<]>[-<+>]<
[[-]>++++[->++++<]>[-<++++>]<+++++++++++++++++++.>++[->++++<]>[-<++++>]<[-<+>]<-
-----------------.++++++++++++.++++++.+.-------------------.++++++.[-]<<<->>]<]<
[->+>+<<]>>[-<<+>>]<[[-]<<->><<[->>>+>+<<<<]>>>>[-<<<<+>>>>]+<[[-]>-<]>[-<+>]<[[
-]>++++[->++++<]>[-<++++>]<++++++.>++[->++++<]>[-<++++>]<[-<+>]<-.+++.++++.-----
--.+++++++++++++.[-]<<<->>]<]<[-]<[-]++++++++++.[-]<

ok, das programm ist gut. es ist sogar vorbildlich, würde ich sagen. es ist auch noch wartbar. für brainfuck-verhältnisse optimal. es gibt aber andere sprachen, in denen kalenderprogramme besser wartbar sind.

edit: der oben genannte interpreter schafft's nicht, weil er nur 30 sek laufzeit erlaubt.
http://justice.loyola.edu/~mcoffey/pr/5k/i.html ist besser.

Power Off

volkard schrieb:

anläßlich eines programmierwettbewerbs, bei dem alle sprachen erlaubt waren, leistete ich mir den spaß, die aufgabe, nach dem gregorianischen kalnender aus tag, monat, jahr, die der user zuerst einzugeben hatte, den wochenzag zu berechnen, in brainfuck zu erstellen. zum testen nimm http://koti.mbnet.fi/villes/php/bf.php
und natürlich muß ich betonen, daß der code klar und einfach ist. und wer den nicht kapiert, hat natürlich keine ahnung von brainfuck. ich such dir gerne einen link mit der sprachbeschreibung raus. oder ich zeig die ne kommentierte version.

edit: der oben genannte interpreter schafft's nicht, weil er nur 30 sek laufzeit erlaubt.
http://justice.loyola.edu/~mcoffey/pr/5k/i.html ist besser.

Cool!

Aber fuer jemanden, der Brainfuck kann, ist das doch kein Problem zu lesen, oder?

Hier ist z.B. der Output eines Lexical Analyzers, den ich neulich geschrieben habe:

#2:$2:c$8:I5:Alpha$E:{
#4:$5:f$E:I7:beeswax$L:($N:I5:omega$S:,$U:I5:delta$a:)$b:;
#6:$2:}$3:;
#8:$2:M$C:I5:Alpha$H:.$I:I7:beeswax$Q:{
#9:$5:I4:redo$9:4
#A:$5:F$8:I5:omega$E:=$G:I5:delta$M:{
#B:$8:I5:omega$E:A$H:I5:omega$N:/$P:N2:$Q:;
#C:$8:I5:delta$E:A$H:I5:delta$N:+$P:N1:$Q:;
#D:$8:J$D:I4:redo$H:;
#E:$5:}
#F:z
#G:$8:y$D:{
#H:$B:w$G:I1:i$H:;
#I:$B:I1:i$D:A$G:N1:$H:;
#J:$8:}
#K:Y$I:I1:i$K:<$M:N5:
#L:$8:d$B:{
#M:k$G:I8:printnum$P:I1:i$Q:;
#N:$8:}
#O:$2:}
#Q:c$8:I5:Alpha$E:I1:a$F:;
#R:$2:k$7:I1:a$8:.$9:I7:beeswax$G:($I:N5:$J:,$L:N7:$N:)$O:;

Fuer mich ist das kein Problem, zu lesen.

Also Lesbarkeit ist immer relativ.

volkard

Power Off schrieb:

volkard schrieb:

warum nicht wenigstens noch eatwhite?

Weil readnum() nicht nur white-space ueberliest. Vielleicht solltest Du vor ausfuehrlicher Kritik den Code nochmal genau ansehen.

ok, eatLine.
warum das nicht?

volkard

mich beschleicht die befürchtung, daß mein code nur spaß war (weil ich damit garantiert den ersten (und einzigen) platz bei den brainfuckern kriege) und du deinen ernst meinst.

otze

Weil readnum() nicht nur white-space ueberliest. Vielleicht solltest Du vor ausfuehrlicher Kritik den Code nochmal genau ansehen.

ahh, danke dir. hab völlig vergessen das zu kritisieren.

wenn der user ne false eingabe macht, soll er bitteschön ne fehlermeldung bekommen, und eine neue Eingabe tätigen, anstatt dass die funktion einfach alle zeichen bis zum ende der zeile killt(sofern nicht auf dem weg doch noch ne zahl oder ein '-' auftaucht ). Im endeffekt sollte die funktion sofort abbrechen, sofern ein anderes zeichen ausser leerzeichen/zeilenumbrüchen/zahlen/'-' kommt.

Power Off

Was haltet ihr hiervon?

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

int            chr = EOF;
int            dig = EOF;
int            val = 0;
int            sgn = 0;

void readchr( void ) {
   do {
      chr = getchar();
   } while ( chr == '\r' );
}

bool readdigit( void ) {
   if ( chr < '0' || chr > '9' ) return false;
   dig = chr;
   readchr();
   return true;
}

bool readdec( void ) {
   if ( !readdigit() ) return false;
   val = 0;
   do {
      val = val * 10 + ( dig - '0' );
   } while ( readdigit() );
   return true;
}

bool readint( void ) {
   if ( chr == '-' ) {
      readchr();
      sgn = -1;
   }
   else {
      sgn = 1;
   }
   if ( !readdec() ) return false;
   val *= sgn;
   return true;
}

void skipspc( void ) {
   while ( chr == ' ' || chr == '\t' ) readchr();
}

bool readnum( int* result ) {
   readchr();
   skipspc();
   if ( !readint() ) return false;
   skipspc();
   if ( chr != '\n' ) return false;
   *result = val;
   return true;
}

void skiptoeol( void ) {
   while ( chr != '\n' && chr != EOF ) readchr();
}

void promptnum( const char* name, int* result ) {
RETRY:
   printf( "%s eingeben (ganze Zahl): ", name );
   fflush( stdout );
   if ( !readnum( result ) ) {
      skiptoeol();
      if ( chr == EOF ) {
         printf( "Eingabe abgebrochen (EOF)\n" );
         exit( EXIT_FAILURE );
      }
      printf( "Eingabefehler: %s muss ganze Zahl sein\n", name );
      goto RETRY;
   }
   printf( "%s = %d\n", name, *result ); 
}

int main( int argc, char** argv ) {
   int w;
   promptnum( "Watt", &w );
   return EXIT_SUCCESS;
}

volkard

Power Off schrieb:

Was haltet ihr hiervon?

viel besser
jetzt kriegste auch nicht mehr allgemeines gemäkel, sondern konkrete mecker. naja das augenfällige goto und die globalen variablen wohl. wobei ich aber eher meine, daß die gar nicht so unhübsch in diesem fall sind.
jetzt ruft das program eigentlich am lautestes danach, ein paar kleinigkeiten einzubauen, die mit c++ besser gehen.

ich nehme mal ein kleines stück raus und gucke nach, ob die globalen vars vermeidbar sind, ohne daß man sich weh tut.

bool readint( void ) {
   if ( chr == '-' ) {
      readchr();
      sgn = -1;
   }
   else {
      sgn = 1;
   }
   if ( !readdec() ) return false;
   val *= sgn;
   return true;
}

mache ich zu

bool readint(int* val) {
   if ( chr == '-' ) {
      readchr();
      sgn = -1;
   }
   else {
      sgn = 1;
   }
   if ( !readdec(val) ) return false;
   *val *= sgn;
   return true;
}

und das zu

bool readint(int* val) {
   if ( chr == '-' ) {
      readchr();
      if ( !readdec(val) ) return false;
      *val = -*val;
   }
   else {
      if ( !readdec(val) ) return false;
   }
   return true;
}

naja, man hat jetzt massig möglichkeiten, was zu ändern und es macht wieder spaß. auch wenn meine änderung wohl nicht so sinnvoll war. ich denke, es würde sich gut anfühlen, die globalen vars alle zu übergabeparametern zu machen. oder nach c++ wechseln und die werte zurückgeben und bei fehler exceptions werfen.

edit:
aber was mir voll gefällt, ist

bool readnum( int* result ) { 
   readchr(); 
   skipspc(); 
   if ( !readint() ) return false; 
   skipspc(); 
   if ( chr != '\n' ) return false; 
   *result = val; 
   return true; 
}

bis auf den seltsamen readchr() am anfang, kann das ein halbidiot flüssig lesen und ich auch.

Power Off

volkard schrieb:

jetzt ruft das program eigentlich am lautestes danach, ein paar kleinigkeiten einzubauen, die mit c++ besser gehen.

vergiss aber nicht, das wir im C Forum sind!

Power Off

volkard schrieb:

naja, man hat jetzt massig möglichkeiten, was zu ändern und es macht wieder spaß. auch wenn meine änderung wohl nicht so sinnvoll war. ich denke, es würde sich gut anfühlen, die globalen vars alle zu übergabeparametern zu machen.

Hmmmm ... fuehrt das nicht dazu, dass Parameter ueber mehrere Ebenen hinweg uebergeben werden?
Waere da ein struct nicht besser?

volkard schrieb:

bis auf den seltsamen readchr() am anfang, kann das ein halbidiot flüssig lesen und ich auch.

Soso.

Der erste readchr() dient dazu, um das erste Zeichen einzulesen. Dadurch habe ich immer ein "aktuelles Zeichen", auf das ich pruefen kann und weitere Zeichen lesen kann. Das ist die Standardvorgehensweise bei recursive descendant Parsern.

volkard

Power Off schrieb:

Hmmmm ... fuehrt das nicht dazu, dass Parameter ueber mehrere Ebenen hinweg uebergeben werden?

jup.

Waere da ein struct nicht besser?

naja, wenn man ganz konsequent ist, müßte man ja auch den chr im struct mitführen oder übergeben.

bool readnum( int* result, int* chr )

fühlt sich wieder minderlustig an. statt der werte dig und val und sgn global könnte man die immer zurückgeben und dafür statt den error zurückzugeben den global machen. oder dazu noch...

Der erste readchr() dient dazu, um das erste Zeichen einzulesen. Dadurch habe ich immer ein "aktuelles Zeichen", auf das ich pruefen kann und weitere Zeichen lesen kann. Das ist die Standardvorgehensweise bei recursive descendant Parsern.

klar. man muß ein zeichen angucken können, ohne es wegzulesen. hab zu getchar() kein passendes ungetchar() gefunden. also muß man das eine zeichen puffern.

...würde man error und den chr in eine struct stecken, dann wäre das ja ein normaler eingabestrom. ich höre die objektorientierung leise anklopfen.

hehejo

Power Off schrieb:

volkard schrieb:

aber Power Off hat die klappe zu weit aufgerissen und so getan, als sei er so perfekt wie TGGC

Gar nicht wahr! Du bist doch hier der, der permanent die Klappe aufreisst!

WEIL ER ES KANN!

Power Off

hehejo schrieb:

Power Off schrieb:

volkard schrieb:

aber Power Off hat die klappe zu weit aufgerissen und so getan, als sei er so perfekt wie TGGC

Gar nicht wahr! Du bist doch hier der, der permanent die Klappe aufreisst!

WEIL ER ES KANN!

... die Klappe aufreissen!