Problem mit isdigit()

[ProgBeginner]

Hallo

Ich habe auch mal wieder ein Problem ...

Grundsätzlich "lauffähig" ist mein Programm - aber isdigit() trifft Entscheidungen, die für mein Verständnis unlogisch sind.

Es soll gemäss Gauss'scher Summenformel die Summe aller natürlichen Zahlen bis zur ( per Kommandozeilenparameter übergebenen ) Zahl bestimmt werden.

Quellcode:

Code: Alles auswählen

#include <ctype.h>
#include <stdio.h>
#include <string.h>

int main( int argc, char* argv[] )
{
  if( argc != 2 ) // Wächter
  {
    printf( "Falsche Anzahl von Argumenten\n" );
    return 0;
  }
 
  int zahl = atoi( argv[1] );

  if (isdigit(zahl))
  {
    printf( "Ergebnis: %d\n", zahl * (zahl + 1) / 2);
  }

  else
  {
    printf( "FEHLER!\n" );
  }

  return 0;
}

Bildschirmausgaben vom Experimentieren mit diesem Progrämmchen :



Was läuft hier falsch?

[Xin]

Grundsätzlich "lauffähig" ist mein Programm - aber isdigit() trifft Entscheidungen, die für mein Verständnis unlogisch sind.

Da gibt es jetzt natürlich zwei Möglichkeiten... -)

Code: Alles auswählen

int main( int argc, char* argv[] )
{
  ... 
  int zahl = atoi( argv[1] );

  if (isdigit(zahl))

Was läuft hier falsch?

Digit bedeutet Ziffer. '0' ist eine Ziffer, '1' ist eine Ziffer... '9' ist eine Ziffer.
Du wandelst eine Zahl(!) in ein int. Wenn die Zahl zufälligerweise '0'-'9' (48-57) entspricht, dann ist der ASCII-Code eine Ziffer. Wenn nicht... nicht.
Auch wenn isdigit() ein Integer erwartet, so erwartet es eben einen ASCII-Code und keine beliebige Zahl.

[ProgBeginner]

Das heisst ich kann den if(isdigit()) - Käse weglassen?

[mfro]

Das heisst ich kann den if(isdigit()) - Käse weglassen?

Nun, wenn Du eine anständige Fehlerbehandlung haben willst (schließlich könnte ja auch jemand dein Programm mit "./gauss x" aufrufen), solltest Du schon prüfen, ob das erste Argument nur aus Ziffern besteht. Das sollte aber geschehen _bevor_ atoi() aufgerufen wird.

Der bessere Ansatz wäre allerdings, gleich strtol() (statt atoi()) zu verwenden. atoi() ist erstens deprecated, zweitens kann man mit strtol() direkt eine fehlerhafte Eingabe erkennen/abfangen.

[ProgBeginner]

Dieser Quellcode hier :

Code: Alles auswählen

#include <ctype.h>
#include <stdio.h>
#include <string.h>

int main( int argc, char* argv[] )
{
  if( argc != 2 ) // Wächter
  {
    printf( "Falsche Anzahl von Argumenten\n" );
    return 0;
  }
 
  int zahl = atoi( argv[1] );
  if (zahl > 0)
  {
    printf( "Ergebnis: %d\n", zahl * (zahl + 1) / 2);
  } else {
    printf( "FEHLER!\n");
  }

  return 0;
}

liefert bei negativen Zahlen "FEHLER!" ( wie beabsichtigt ) , aber auch bei Buchstaben. Für den Buchstabenfall versteh ich nur noch nicht so ganz, warum - aber scheint zu funktionieren.

[oenone]

If the first sequence of non-whitespace characters in str is not a valid integral number, or if no such sequence exists because either str is empty or it contains only whitespace characters, no conversion is performed and zero is returned.