String in Integer umwandeln und ausgeben ohne Library

DirkB

Klar kann der Compiler optimieren, aber trotzdem ist dein my_atoi falsch.
Und multipliziern mit 10, subtrahieren der '0' machen beide Funktionen, das wars dann aber auch.

Das Reultat vom sizeof ist für diesen Zweck völlig ungeeignet.

Und probier das Ganze mal mit anderen Werten. Vor allem mit mehr oder weniger als 8 Stellen.

EOP

DirkB schrieb:

So wie die aufgerufen werden, geht das heutzutage gar nicht mehr, weilsie einen Bufferoverflow generieren können

Ich mag das. Macht mir einen Heidenspaß den Leuten ihre Programme um die Ohren zu hauen. Man hat ja sonst kein Hobby.

Wutz

Hobbyprogger schrieb:

aber wenn ich den ASM Code von my_atoi und my_atoy2 anschaue, sehe ich da keinen grossen Unterschied... Oder übersehe ich da was?

C ist keine wirkliche Hochsprache und schon gar keine Lernsprache.
C-Codes haben naturgemäß die Eigenschaft, sehr umfangreich (viele loc) zu sein, es kommt darauf an, für Verständlichkeit+Wartbarkeit die Übersicht zu behalten und dafür sind kurze Codes von Vorteil (natürlich im normalen Rahmen und nicht obfuscated).

Hobbyprogger schrieb:

Ich gehe mal davon aus, dass hier Experten sind, die u.U. auch an der Uni lehren

lol
"Experte" + "an der Uni" schließen sich gegenseitig aus.

DirkB

EOP schrieb:

DirkB schrieb:

So wie die aufgerufen werden, geht das heutzutage gar nicht mehr, weilsie einen Bufferoverflow generieren können

Ich mag das. Macht mir einen Heidenspaß den Leuten ihre Programme um die Ohren zu hauen. Man hat ja sonst kein Hobby.

Die von mir bemängelten Funktionen wurden dem TO vorgegeben.
Der Teil ist also nicht sein Programm, sondern der vom Professor/Tutor/Lehrer.

Wutz schrieb:

"Experte" + "an der Uni" schließen sich gegenseitig aus.

DirkB schrieb:

Klar kann der Compiler optimieren, aber trotzdem ist dein my_atoi falsch.
Und multipliziern mit 10, subtrahieren der '0' machen beide Funktionen, das wars dann aber auch.

Das Reultat vom sizeof ist für diesen Zweck völlig ungeeignet.

Und probier das Ganze mal mit anderen Werten. Vor allem mit mehr oder weniger als 8 Stellen.

Kann da nicht einer gleich sagen, dass man Strinlängen mit

int i = strlen(ergebnis)

ermittelt
Wieder was gelernt......
Dabke an alle!

DirkB

SeppJ schrieb:

Dein Programm hat mehrere Fehler/Schwächen:

-Die Längenermittlung in Zeile 8 ist falsch. So geht das nicht in C. Noch einmal angucken, wie das mit den C-Strings und Arrays geht.
-Selbst wenn man die Länge korrekt ermitteln würde, wäre das komplett unnötig, wegen der Natur von C-Strings.

Wir ermuntern gerne zur Selbsthilfe.
Aber beachte auch den 2. Punkt.
Da du in der Schleife danach sowieso durch den String gehst, kannst du da auch gleich auf das Ende prüfen. Du brauchst das strlen gar nicht.

Wutz

Hobbyprogger schrieb:

Kann da nicht einer gleich sagen, dass man Strinlängen mit
int i = strlen(ergebnis)
ermittelt
Wieder was gelernt......

Tja und schon wieder falsch gelernt.

Korrekt ist

size_t i = strlen(ergebnis);

Es gibt "Überraschungen" wenn du den unsigned Typ size_t beim Rechnen nicht berücksichtigst, z.B. bei

if( strlen("123")-strlen("1234") < 0 )
u.ä.

Wutz schrieb:

Korrekt ist
size_t i = strlen(ergebnis);
Es gibt "Überraschungen" wenn du den unsigned Typ size_t beim Rechnen nicht berücksichtigst, z.B. bei
if( strlen("123")-strlen("1234") < 0 )
u.ä.

int kann man ruhig nehmen. dein konstruierter fehlerfall ist quatsch.

Wutz

Dorfdeppen melden sich immer wieder ungefragt zu Wort.

Wutz schrieb:

Dorfdeppen melden sich immer wieder ungefragt zu Wort.

wow, du kannst dem dorfdeppen nicht einmal sachlich widersprechen.
bist ja voll der held.

Wutz

Geistig weniger Bemittelte mit Geltungsdrang aber ohne Ahnung bekommen Antworten auf dem gleichen Niveau, das sie offenbaren.

verano123

Hallo Leute,

sorry hatte die letzten Tage garkeine Zeit hier reinzugucken.
Wollte keine Welle hier schlagen mit der Aufgabe.

Der Thread kann gerne geschlossen werden. Hoffe, dass andere von den hier gestellten Fragen und Antworten profitieren können, die auch solche Probleme haben wie ich.

Danke für die Hilfe und die rege Diskussion

volkard

wurzelsepp schrieb:

Wutz schrieb:
Korrekt ist
size_t i = strlen(ergebnis);
Es gibt "Überraschungen" wenn du den unsigned Typ size_t beim Rechnen nicht berücksichtigst, z.B. bei
if( strlen("123")-strlen("1234") < 0 )
u.ä.
int kann man ruhig nehmen. dein konstruierter fehlerfall ist quatsch.

Ja. Tatsächlich ist es von Vorteil, wenn Nubes int für Größen nehmen genau desdawegen. Und der konstruierte Fall ist Quatsch. Ihr habt beide recht.

Oder nehmen wir mal einfach

assert(index>=0);//int
assert(index<size);//int

versus

assert(index>=0);//size_t 
assert(index<size);//size_t

. int liefert potenziell mehr bessere Fehler.

Es ist irgendwie "besser", int zu nehmen, aber irgendwie "richtiger", size_t zu nehmen. Ich denke, es ist am besten, die Mitarbeiter zu size_t zu schulen. Und wenn das nicht geht, dann freies Gefecht. Meinetwegen double.

Naja, wenn int nur 4 Byte hat und size_t 8 Byte verbietet sich int eh. Dann vielleicht ssize_t oder intptr_t??? Ist nicht lecker.

DirkB

verano123 schrieb:

Wollte keine Welle hier schlagen mit der Aufgabe.

Lass mal. Da hast du eh keinen Einfluß drauf.
Aber lieber etwas Leben als tote Hose.

verano123 schrieb:

Hoffe, dass andere von den hier gestellten Fragen und Antworten profitieren können, die auch solche Probleme haben wie ich.

Haben sie. Auch von Fragen, die gar nicht gestellt wurden.