[C#] StringToInt funktioniert nicht...

Ich bin zwar kein C#-Programmierer, ein paar C++ Kentnisse habe ich jedoch. Beim Anschauen des Sourcecodes sind mir 3 Probleme aufgefallen WENN du es mit dem System machen möchtest, dass du in deinem Sourcecode verwendest:

1. Du gehst den String von vorne nach hinten durch. Das ist falsch rum, da beispielsweise in der Zahl "15" die "1" mit 10 und die "5" mit 1 multipliziert werden müssten. Du musst also den String von rechts nach links durchgehen, so, wie auch unsere Zahlen aufgebaut sind.
2. Die for-Schleife geht von 0 los. Die erste Stelle von rechts muss aber mit 1 multipliziert werden, da sonst an der Stelle immer 0 addiert wird und nicht die richtige Ziffer. Du müsstest somit an der Stelle eine If-Abfrage einbauen, die, wenn i 0 ist, mit 1 multipliziert Siehe Punkt 4
3. Du multiplizierst den Char mit 1, 10, 20. Der Char hat aber, wenn es der Buchstabe '0' ist, nicht den Wert null, sondern einen anderen (Im ASCII-Zeichensatz z.B. 48). Somit müsstest du zu erst den Wert von '0' davon abziehen und ihn dann mit 1, 10 usw. multiplizieren.
4. von Drasko: Du multiplizierst 10 mit 1, 2, 3 usw. Du musst es aber potenzieren, damit du nicht auf 10, 20, 30 usw. kommst, sonder auf 10, 100, 1000 usw. Damit ist dann auch Punkt 2. gelöst, da 10^0 1 ist.

All diese Probleme hat der oben genannte Code nicht, da er leicht anders funktioniert und damit auch gleichzeitig effektiver und einfacher ist.

Edit: Mit oben genannt meine ich den von Changer, in Draskos Code wurden die Probleme nicht beseitigt, er hat nur die Chars durch ihre Int-Werte ersetzt, was aber eigentlich unnötig ist.
Edit2: Punkt 4 von Drasko eingeführt.
Edit3: An alle: Er möchte seine Kenntnisse verbessern. Ich denke nicht, dass es ihm hilft, wenn ihm hier fertiger Source-Code präsentiert wird. Ich denke, solchen Leute sollte lieber beigebracht kriegen, wie man Programme richtig debuggt und Fehlerquellen findet.
Somit mein Tipp: Wenn man den Code einmal im Kopf durchgeht oder das Programm mithilfe von Konsolenausgaben den Effekt jeder Zeile anzeigen lässt, dann lassen sich alle Fehler ganz leicht identifizieren. Du solltest daher bei solchen Problemen erst probieren, den Fehlerpunkt einzugrenzen (hier gibt es leider mehrere) und selber eine Lösung dafür zu suchen, da es sonst sehr anstrengen werden kann, größere Programme zu schreiben ;).

Zitat von Drasko

Spoiler anzeigen

Da testbereiter Code:

C#

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;


namespace Sharpadox
{
    class Program
    {
        static void Main(string[] args)
        {
            int i = StringToInt("99454");
            Console.WriteLine(i.ToString());
            Console.Read();
        }
        private static int StringToInt(string input)
        {
            int num = 0;
            int count = 0;
            bool succ = true; //unnötig, wenn es auf falsch gesetzt wird kann man auch direkt falsch zurückgeben.
            for (int i = input.Length-1; i > -1; i--)
            {
                if ((int)input[i] < 48 || (int)input[i] > 57) //nimm doch '0' und '9', da weiß man eher, was das machen soll, wenn man es sieht. So muss man erst die Werte nachschauen, wenn man sie nicht weiß.
                {
                    succ = false;
                    break;
                }
                else
                {
                    if (count == 0) [color=#ff0000] //unnötig[/color]
                    {
                        num += GetNumberByChar(input[i]);
                        count++;
                    }
                    else
                    {
                        num += GetNumberByChar(input[i]) * (int)(Math.Pow(10, (double)count)); // hier einfach anstatt GetNumberByChar(input[i]) input[i] - '0' einsetzen
                        count++;
                    }
                }
            }
            return succ == false ? -1 : num;
        }


        private static int GetNumberByChar(char c) { //fällt weg
            switch ((int)c) {
                case 48: return 0;
                case 49: return 1;
                case 50: return 2;
                case 51: return 3;
                case 52: return 4;
                case 53: return 5;
                case 54: return 6;
                case 55: return 7;
                case 56: return 8;
                case 57: return 9;
                default: return -1;
            }
        }


    }


}

Alles anzeigen

Somit wird daraus:

private static int StringToInt(string input)
        {
            int num = 0;
            int count = 0;
            for (int i = input.Length - 1; i >= 0; i--)
            {
                if (!('0' <= (int)input[i] <= '9'))
                {
                    return false;
                }
                else
                {
                    num += ((int)input[i] - '0') * (int)(Math.Pow(10, (double)count)); 
                    count++;
                }
            }
            return num;
        }

Wenn ich mich da nicht irgendwo vertippt habe - ist ungetestet.
Edit: Getestet, funktioniert