C Guter Programmierstil?

[Crazy Driver]

Hallo Leute,

Ich wollte gerne mal wissen ob folgender Code zum guten Programmierstil gehört.
(Ziel: char zu int)

char x[] = "6";
int y;
y = text[0] - '0';

 

[Pawel23]

Nein.

 

[fRaNkLiN]

Das ist eigentlich gar kein Programmierstil.
Erstmal erstellst du ein char-Array/String und belegst es mit "6" vor - meckert da nicht schon der Compiler...?
Dann definierst du y als Integer. Danach weist du y den Wert text[0] (diese Variable wurde übrigens weder deklariert noch initialisiert) minus den Char '0' zu. Spätestens hier gibt sich der Compiler die Kugel und deinstalliert sich selbst.
Was du brauchst, ist entweder eine explizite Typumwandlung mit vorgestelltem (char) - ein so genannter Cast - oder eine Funktion wie itoa (enthalten in stdlib.h).

char umgewandelt;
char als_string[2];
int y;

y = 5;
umgewandelt = (char)y;
itoa(y, als_string, 10);

 

[dichter-karl]

ähm wieso?

klassischer Type cast

char k = 0;
int  i;

i = (int) k;

 

[Whiz-zarD]

benutz doch die atoi()-Funktion
http://www.cplusplus.com/reference/clibrary/cstdlib/atoi/

 

[7H3 N4C3R]

Hallo Leute,

Ich wollte gerne mal wissen ob folgender Code zum guten Programmierstil gehört.
(Ziel: char zu int)

char x[] = "6";
int y;
y = text[0] - '0';

Naja... was soll dieses Code-Fragment überhaupt machen? Sieht in diesem Kontext erstmal ziemlich eigenartig aus. Davon abgesehen:

Der Code ist absolut standardkonform und korrekt.

Die Subtraktion zweier char's ist übrigens schon von vornherein vom typ int, selbst wenn auf der linken Seite kein int stände. (Durch die Subtraktion werden die usual arithmetic conversions angewendet, wodurch wiederum auf die beiden char's die integer promotion angewendet wird - heißt, alle Typen mit einem kleineren Rang als int werden implizit in ints gewandelt, sofern in int darstellbar, sonst unsigned int).

 

[Whiz-zarD]

Naja... was soll dieses Code-Fragment überhaupt machen?

Eine Konvertierung von Char nach Int. Steht auch im Eingangsbeitrag.
Dafür gibt es aber die atoi Funktion.

 

[7H3 N4C3R]

Aaaaaachh.... char* nach int ist gemeint. Okay, jetzt macht's klick.

Okay, das ist dann äußerst gruseliger Code (wäre mal interessant, ob bei '6' - '0' überhaupt 6 rauskommen muss... mal den Standard wühlen, wenn ich Langeweile habe).

Naja, lieber direkt strtol(). atoi ist schlampig was die Fehlererkennung angeht. Oder sscanf (ruft intern in aller Regel auch nur strtol auf, übernimmt aber die Konvertierungsarbeit).

 

[Whiz-zarD]

Stimmt. Mit sscanf wäre sowas ja auch möglich ^^"

 

[7H3 N4C3R]

Erstmal erstellst du ein char-Array/String und belegst es mit "6" vor - meckert da nicht schon der Compiler...?

Warum sollte er?

Dann definierst du y als Integer. Danach weist du y den Wert text[0] (diese Variable wurde übrigens weder deklariert noch initialisiert) minus den Char '0' zu. Spätestens hier gibt sich der Compiler die Kugel und deinstalliert sich selbst.

Text ist ein Typo, ist wohl offensichtlich... Vom Typo abgesehen - warum sollte der Compiler das tun?

Was du brauchst, ist entweder eine explizite Typumwandlung mit vorgestelltem (char) - ein so genannter Cast - oder eine Funktion wie itoa (enthalten in stdlib.h).

char umgewandelt;
char als_string[2];
int y;

y = 5;
umgewandelt = (char)y;
itoa(y, als_string, 10);

itoa ist nicht in stdlib.h enthalten (vielleicht in der Version deines Library-Herstellers - in ISO9899 wird es nicht erwähnt). Nebenbei legt itoa irgendwie nahe, dass der frisch nach char konvertierte int wieder nach int konvertiert wird beim Aufruf.