c# - CSV 文字列を解析して整数の配列にする

Question

テキストボックスフィールドに 123,145,125 I を入力して、このフィールドを整数の配列に分割します。すべてが正しく解析されている場合は、このフィールドを true または false で検証します。

コード：

private bool chkID(out int[] val) 
{
    char[] delimiters = new char[] { ',' };
    string[] strSplit = iconeID.Text.Split(delimiters);  


    int[] intArr = null;
    foreach (string s in strSplit) //splits the new parsed characters 
    {
        int tmp;
        tmp = 0;
        if (Int32.TryParse(s, out tmp))
        {
            if (intArr == null)
            {
                intArr = new int[1];
            }
            else
            {
                Array.Resize(ref intArr, intArr.Length + 1);
            }
            intArr[intArr.Length - 1] = tmp;
        }

        if (Int32.TryParse(iconeID.Text, out tmp))
        {
            iconeID.BorderColor = Color.Empty;
            iconeID.BorderWidth = Unit.Empty;

            tmp = int.Parse(iconeID.Text);
            val = new int[1];
            val[0] = tmp;
            return true;
        }


    }
    val = null;
    ID.BorderColor = Color.Red;
    ID.BorderWidth = 2;
    return false;
}

//新しいコード: private bool chkID(out int[] val) //bool satus for checkID function { string[] split = srtID.Text.Split(new char[1] {','}); リスト番号 = new List(); int 解析済み;

        bool isOk = true;
        foreach( string n in split){
            if(Int32.TryParse( n , out parsed))
                numbers.Add(parsed);
            else
                isOk = false;
        }
        if (isOk){
            strID.BorderColor=Color.Empty;
            strID.BorderWidth=Unit.Empty;
            return true;
        } else{
            strID.BorderColor=Color.Red;
            strID.BorderWidth=2;
            return false;
        }
            return numbers.ToArray();
        }

score 8 · Accepted Answer

指定された関数はやりすぎのようです。あなたのタイトルが暗示している質問に答えるものは次のとおりです。

//int[] x = SplitStringIntoInts("1,2,3, 4, 5");

static int[] SplitStringIntoInts(string list)
{
    string[] split = list.Split(new char[1] { ',' });
    List<int> numbers = new List<int>();
    int parsed;

    foreach (string n in split)
    {
        if (int.TryParse(n, out parsed))
            numbers.Add(parsed);
    }

    return numbers.ToArray();
}

編集（質問に対するコメントに基づく）

この関数が実行する必要がある 3 つのことを定義しました。あとは、それぞれのメソッドを作成するだけです。以下は、それらをどのように実装できるかについての私の推測です。

int[] ValidateIDs(int[] allIDs)
{
    List<int> validIDs = new List<int>(allIDs);

    //remove invalid IDs

    return validIDs.ToArray();
}

void DownloadXmlData(int[] ids)
{
    ...
}

これで、新しい関数を実行するだけです:

void CheckIconeID(string ids)
{
    int[] allIDs = SplitStringIntoInts(ids);
    int[] validIDs = ValidateIDs(allIDs);

    DownloadXmlData(validIDs);
}

score 3 · Accepted Answer

@Austin Salonenの答えに本当にコメントしたかったのですが、うまくいきませんでした。質問に対する素晴らしい回答ですが、csv/int 変換部分についてもう少し一般的に議論を広げたいと思いました。

これは小さな点であり、あまり議論する価値はありませんが、foreachループを単純なforループに交換することを検討します. より単純な IL になる可能性があります (読み取りが速くなります)。( http://www.codeproject.com/KB/cs/foreach.aspx、http://msdn.microsoft.com/en-us/library/ms973839.aspx [文字列反復に For ループを使用 - バージョン 1] を参照してください。）。
私は 2 つの方法を作成します。1 つは安全で使用TryParseし、「適切な」値のみを追加する方法で、もう 1 つはそれほど安全ではありませんが高速です。

提案された「安全な」関数（悪い値を知りたくない場合のオーバーロード付き）...

    public static int[] SplitAsIntSafe (this string csvString) {
        List<string> badVals;
        return SplitAsIntSafe(csvString, ',', out badVals);
    }
    public static int[] SplitAsIntSafe (this string delimitedString, char splitChar, out List<string> badVals) {
        int         parsed;
        string[]    split   = delimitedString.Split(new char[1] { ',' });
        List<int>   numbers = new List<int>();
        badVals             = new List<string>();

        for (var i = 0; i < split.Length; i++) {
            if (int.TryParse(split[i], out parsed)) {
                numbers.Add(parsed);
            } else {
                badVals.Add(split[i]);
            }
        }
        return numbers.ToArray();
    }

提案された「速い」機能....

    public static int[] SplitAsIntFast (this string delimitedString, char splitChar) {
        string[]    strArray = delimitedString.Split(splitChar);
        int[]       intArray = new int[strArray.Length];

        if(delimitedString == null) {
            return new int[0];
        }
        for (var i = 0; i < strArray.Length; i++) {
            intArray[i] = int.Parse(strArray[i]);
        }
        return intArray;
    }

とにかく、これが誰かに役立つことを願っています。

score 1 · Accepted Answer

このFileHelperとCSV Readerもチェックしてみてください。

彼らがあなたを助けてくれることを願っています...気をつけてください、トム

score 1 · Accepted Answer

CSV ファイルを解析するための優れた無料ライブラリがあります: FileHelpers

    using FileHelpers;

    // First declare the record class

    [Delimitedrecord(";")]
    public class SampleType
    {
        public string Field1;
        public int    Field2;
    }


    public void ReadExample()
    {
        FileHelperEngine engine = new FileHelperEngine(typeof(SampleType));

        SampleType[] records;    

        records = (SampleType[]) engine.ReadFile("source.txt");

        // Now "records" array contains all the records in the
        // sourcefile and can be acceded like this:

        int sum = records[0].Field2 + records[1].Field2;
    }

score 0 · Accepted Answer

.NET 2.0 では、次のように記述できます。

string test = "123,14.5,125,151,1.55,477,777,888";

bool isParsingOk = true;


int[] results = Array.ConvertAll<string,int>(test.Split(','), 
    new Converter<string,int>(
        delegate(string num)
        {
            int r;
            isParsingOk &= int.TryParse(num, out r);
            return r;
        }));

score 0 · Accepted Answer

public bool ParseAndCheck(string source,
    out IList<int> goodItems, out IList<string> badItems)
{
    goodItems = new List<int>();
    badItems = new List<string>();

    foreach (string item in source.Split(','))
    {
        int temp;
        if (int.TryParse(item, out temp))
            goodItems.Add(temp);
        else
            badItems.Add(item);
    }

    return (badItems.Count < 1);
}

score -1 · Accepted Answer

これは簡単で、かなりうまくいくと思います。有効な数値のみを返します。

static int[] SplitStringIntoInts(string list)
{            
    int dummy;            
    return (from x in list.Split(',')
            where int.TryParse(x.ToString(), out dummy)
            select int.Parse(x.ToString())).ToArray();           
}

c# - CSV 文字列を解析して整数の配列にする

7 に答える 7

Related

Reference