String in Integer umwandeln

[baule1]

Hallo, bin neu hier und C- Anfänger und habe eine Frage:
Habe mir eine Funktion gebastelt (characterToInt), die einen vom Benutzer eingelesenen String (andere Funktion) an diese übergibt und zu einer Integerzahl konvertiert.

Code: Alles auswählen

int characterToInt(char *user_in_char)
{
  int out = 0;
  int sign_value =0;
  int count = strlen(user_in_char) - 1;
  double exponent = 0;
  
  if (strlen(user_in_char) > 11)
    return 0;

  for (count = strlen(user_in_char) - 1; count >= 0; count--)
  {
    
    if (isdigit(user_in_char[count]))  
    {
      sign_value = ((user_in_char[count] - '0') * pow(10, exponent));
      out = out + sign_value;
      exponent++;
    }

    else
      return 0;
  }
  
  return out;      
}

> 11 ist eine grobe Abschätzung nach oben, eine int kann ja max. 10 Stellen haben.

Jetzt wollte ich implementieren, dass die Zahl auch negative Werte annehmen kann (daher auch das 11).
Habe es mit Code wie

Code: Alles auswählen

    if (strncmp((user_in_char[0] - '0'), '-', 1))
               printf("negative Zahl\n");

in unterschiedlichen Varianten versucht... stehe grade an und bin über hilfe sehr dankbar...
lg, baule1

[nufan]

Hi und willkommen im Forum

Jetzt wollte ich implementieren, dass die Zahl auch negative Werte annehmen kann (daher auch das 11).
Habe es mit Code wie

Code: Alles auswählen

if (strncmp((user_in_char[0] - '0'), '-', 1))
    printf("negative Zahl\n");

Welchen Compiler verwendest du denn? Der Syntax von strcmp stimmt nämlich nicht.
Hier der Link zur Referenz: http://proggen.org/doku.php?id=c:lib:string:strcmp

Code: Alles auswählen

user_in_char[0] - '0'

Durch -'0' bekommst du den Integer-Wert einer char-_Zahl_. Also wenn das Zeichen '1' (ASCII 49) ist, und dann '0' (ASCII 48) abgezogen wird bekommst du 1 (in ASCII). Gut - stimmt so.
Du ziehst jedoch '0' ab und vergleichst dann mit einem Zeichen. Sollte das Zeichen also wirklich '-' (ASCII 45) sein und du ziehst '0' ab, erhälst du -3...

So sollte es gehen:

Code: Alles auswählen

if (strncmp (user_in_char[0], '-') == 0)

Hoffe das war verständlich

[baule1]

verständlich wars auf alle fälle, danke für die flotte hilfe.

zum compilieren verwende ich gcc - im übrigen ists kein wunder, dass das nicht die syntax von strcmp ist, es is ja auch strncmp .
wobei das in dem Fall wurscht wäre, da ja sowieso nur 1 Zeichen an der ersten Stelle des Arrays steht (oder täusch ich mich?).

werde deinen code in kürze ausprobieren

[baule1]

funktioniert leider nicht... 1.) warning: ... makes pointer of integer without a cast...
2.) hab ich das problem, dass ich, wenn ichs laufen lass, einen segmentation error krieg...

[Kerli]

Hallo, bin neu hier und C- Anfänger und habe eine Frage:

Hallo, willkommen im Forum. Ich glaub wir sollten uns ja schon irgendwoher kennen

So sollte es gehen:

Code: Alles auswählen

if (strncmp (user_in_char[0], '-') == 0)

funktioniert leider nicht...

Kein Wunder strncmp erwartet char* als Paramter und noch eine Anzahl der zu überprüfenden Zeichen. Richtig wäre also:

Code: Alles auswählen

if(  !strncmp (user_in_char, "-", 1) )

user_in_char ist einfach der Zeiger auf das erste Zeichen, also gleichwertig mit &user_in_char[0] und der zweite Parameter muss als String übergeben werden und nicht als char, dh also mit dem doppelten Anführungszeichen " und nicht mit dem einfachen '.

Und das ! finde ich einfach schöner als '== 0', ist aber eigentlich Geschmackssache.

[baule1]

ja, wir kennen uns... ich hab klarerweise an der stelle jetzt nicht strncmp verwendet, sonder strcmp... mal schauen, was dabei rauskommt, wenn ich das so mach, wie du (bisher waren deine tipps ja alle hilfreich)...

[nufan]

zum compilieren verwende ich gcc - im übrigen ists kein wunder, dass das nicht die syntax von strcmp ist, es is ja auch strncmp

*Augen auswisch* ach jetzt seh ichs auch
Eigentlich kannst du dir das mit str(n)cmp auch sparen, da du ja nur 1 Zeichen vergleichst.

Wozu ist exponent ein double? Kann doch höchstens 10 oder 11 sein. Also sollte ein int reichen

Ich hab dein Programm jetzt etwas abgeändert, so ist es leichter zu testen.

Code: Alles auswählen

#include <stdio.h>  
#include <string.h>
#include <math.h>
#include <ctype.h>
   
int main()
{

  char user_in_char[] = {"-123"};
  int out = 0;
  int sign_value =0;
  int count = strlen(user_in_char) - 1;
  int p;
  int exponent = 0;
     
  if (strlen(user_in_char) > 11)
    return 0;

  if (user_in_char[0] == '-')
  {
    printf ("\n-");
    user_in_char[0] = '0';
  }

  for (count = strlen(user_in_char) - 1; count >= 0; count--)
  {
       
    if (isdigit(user_in_char[count])) 
    {
      p = pow(10, exponent);
      sign_value = ((user_in_char[count] - '0') * p);
      out = out + sign_value;
      exponent++;
    }

        else
          return 0;
  }
     
  printf ("%d\n", out);
     
  return 0;
           
}

Wichtig ist, dass im if in dem auf - geprüft das 1. Zeichen auf 0 gesetzt wird. Sonst multiplizierst du mit 45...
Jedenfalls funzt es bei mir. Willst du eine "echte" negative Zahl, also nicht einfach ein printf mit '-', dann musst du eine Variable die auf Minus prüft setzen, und wenn die am Ende des Unterprogrammes wahr ist, wird die Zahl mit -1 multipliziert.

Also etwa so:

Code: Alles auswählen

int negativ = 0;

...

  if (user_in_char[0] == '-')
  {
    negativ = 1;
    user_in_char[0] = '0';
  }

...

  if (negativ)
    out  *= -1; 

  return out;

[baule1]

bin gestern auf ein weiteres (sehr blödes!) problem draufgekommen, vielleicht habt ihr eine idee, mir sind sie nämlich ausgegangen...

hab zweimal den returnwert "0" angegeben; einmal wenn der benutzer mehr als 11 Zeichen eingibt, das andere mal, wenn er etwas eingibt, das keine zahl ist...
jetzt hab ich das problem: wenn der benutzer "0" eingibt, ist der wert, der variable "out", die ich zurückliefere, auch 0, sprich, wenn ich die funktion auf ihren Rückgabewert == 0 überprüfe, dann krieg ich ein problem, wenn der Benutzer 0 eingegeben hat... --> auf sowas kommt man als anfänger während des programmierens natürlich nicht... hab schon ausprobiert, einen char zu übergeben, was allerdings nicht geht, der hat ja auch den "wert" des ascii codes; dann hab ich versucht, mit EXIT_FAILURE zu arbeiten, das hat den wert 1; sprich, dann krieg ich ein problem, wenn der benutzer 1 eingibt.

bitte um hilfe... dankeschöne

[Kerli]

bin gestern auf ein weiteres (sehr blödes!) problem draufgekommen, vielleicht habt ihr eine idee, mir sind sie nämlich ausgegangen...

hab zweimal den returnwert "0" angegeben; einmal wenn der benutzer mehr als 11 Zeichen eingibt, das andere mal, wenn er etwas eingibt, das keine zahl ist...
jetzt hab ich das problem: wenn der benutzer "0" eingibt, ist der wert, der variable "out", die ich zurückliefere, auch 0, sprich, wenn ich die funktion auf ihren Rückgabewert == 0 überprüfe, dann krieg ich ein problem, wenn der Benutzer 0 eingegeben hat...

In C kann man das leider nicht sehr schön machen. Aber es gibt eigentlich zwei Möglichkeiten:

1.) Du verwendest für deine Fehler einen Wert der sehr wahrscheinlich nie verwendet wird. Es ist nicht unüblich dafür den größten oder kleinsten Wert zu nehmen, den der Rückgabewert annehmen kann.

2.) Du führst noch einen weiteren Parameter ein, den du per-reference übergibst und in den du hineinschreibst ob eine gültige Ausgabe vorliegt.

Üblicherweise wird man in diesem Fall aber doch eher zu 1.) tendieren...

auf sowas kommt man als anfänger während des programmierens natürlich nicht

Aus so etwas lernt man am meisten. Ich glaub so schnell wird dir das nicht wieder passieren

EXIT_FAILURE zu arbeiten, das hat den wert 1; sprich, dann krieg ich ein problem, wenn der benutzer 1 eingibt.

Siehe 1.) Wenn du es so machst, dann nur mit einem Wert der üblicherweise keiner normalen Eingabe entspricht.

[Xin]

In C kann man das leider nicht sehr schön machen. Aber es gibt eigentlich zwei Möglichkeiten:

1.) Du verwendest für deine Fehler einen Wert der sehr wahrscheinlich nie verwendet wird. Es ist nicht unüblich dafür den größten oder kleinsten Wert zu nehmen, den der Rückgabewert annehmen kann.

2.) Du führst noch einen weiteren Parameter ein, den du per-reference übergibst und in den du hineinschreibst ob eine gültige Ausgabe vorliegt.

Von 1.) rate ich bei einfachen Datentypen, wie int oder float dringend ab. Da kann man gleich Bug als Kommentar dran schreiben. Heute funktionierts, in 2 Jahren denkt keiner dran und behandelt was immer er bekommt als Ergebnis.

Bleibt 2.)

In C:

Code: Alles auswählen

char div( int a, int b, int * result )
{
  if( b == 0 ) 
    return FALSE;

  *result = a / b;
  return TRUE;
}

int main( void )
{
  int ergebnis;
  int wert1 = 10;
  int wert2 = 5;

  if( div( wert1, wert2, &ergebnis ) )
  {
    /* alles super */
    printf( "Das Ergebnis ist %d\n", ergebnis );
  }
  else printf( "Division fehlgeschlagen; kein Ergebnis.\n" );
}

Etwas aufwendiger in der Verwendung, aber dafür eine klare Aufforderung an den Programmierer sich darum zu kümmern, wenn was schief geht.
In C++ gibt es eine etwas schönere Variante (und ich meine nicht Exceptions). Außerdem kann man dort bool (Wahrheitswert: hat's geklappt?) statt char (kleiner Datentyp) oder int oder was auch immer als Rückgabewert verwenden.