0

多くの文字列変数を持つオブジェクトがあり、それらは配列から入力されます。これは非常に効率が悪いように思えますが、改善したい時間が少しあるので、すぐに理解できる唯一の方法でしたディクショナリを使用してすべての値を格納します。

これは私が今持っているものです：

Product product = new Product();
                string[] row = productsList[i];

                product.row1 = row;

                product.PRODUCTID = row[0];
                product.PRODUCTCODE = row[1];
                product.PRODUCT = row[2];
                product.CLEAN_URL = row[3];
                product.WEIGHT = row[4];
                product.LIST_PRICE = row[5];
                product.DESCR = row[6];
                product.FULLDESCR = row[7];
                product.KEYWORDS = row[8];
                product.TITLE_TAG = row[9];
                product.META_KEYWORDS = row[10];
                product.META_DESCRIPTION = row[11];
                product.AVAIL = row[12];
                product.RATING = row[13];
                product.FORSALE = row[14];
                product.SHIPPING_FREIGHT = row[15];
                product.FREE_SHIPPING = row[16];
                product.DISCOUNT_AVAIL = row[17];
                product.MIN_AMOUNT = row[18];
                product.LENGTH = row[19];
                product.WIDTH = row[20];
                product.HEIGHT = row[21];
                product.LOW_AVAIL_LIMIT = row[22];
                product.FREE_TAX = row[23];
                product.CATEGORYID = row[24];
                product.CATEGORY = row[25];
                product.POS = row[26];
                product.MEMBERSHIP = row[27];
                product.PRICE = row[28];
                product.THUMBNAIL = row[29];
                product.IMAGE = row[30];
                product.TAXES = row[31];
                product.ADD_DATE = row[32];
                product.VIEWS_STATS = row[33];
                product.SALES_STATS = row[34];
                product.DEL_STATS = row[35];
                product.SMALL_ITEM = row[36];
                product.SEPARATE_BOX = row[37];
                product.ITEMS_PER_BOX = row[38];
                product.GENERATE_THUMBNAIL = row[39];
                product.RETURN_TIME = row[40];
                product.MANUFACTURERID = row[41];
                product.MANUFACTURER = row[42];

特に、最終的に読み返す必要があり、この方法でそれを行うことをあまり楽しみにしていないため、非常に長くて時間がかかります。さらに、 Products クラスは文字列変数の混乱です。

配列をループしてそれらすべてを Dictionary に追加するための迅速で簡単な方法はありますか?

4

5 に答える 5

1 
// you only have to do this once
string[] labels = new string [] { "PRODUCTID", "PRODUCTCODE", ...

次に、辞書を用意して、次のようにロードできます。

Dictionary<string,string> d = new Dictionary<...>

for (int i=0; i<labels.Count; i++) {
    d[labels[i]] = row[i];
}
于 2013-03-22T00:14:26.237 に答える
1

これを行うことができるように、そのロジックを Product のオーバーロードされたコンストラクターに入れます。

Product product = new Product(productsList[i]);

このロジックを回避するためにできることはあまりありません。行にさらにメタデータをエンコードできない限り、どこかに存在する必要があります。

于 2013-03-22T00:15:43.930 に答える
1

各値に一意のキーがあることを考えると、ここでループを使用することは自明ではありません。どうやっても、どこかにすべてのキーをリストする必要があります。つまり、静的配列内のすべてのキーを定義して、次のようにループを使用できます。

string[] keys = new [] {"PRODUCTID", "PRODUCTCODE", ... };
Dictionary<string, string> ProductAttributes = new Dictionary<string,string>();

for(int i = 0, i < row.Length, i++)
{
    ProductAttributes[keys[i]] = row[i];
}

ただし、これを行うと、プロパティを持つことで得られるすべての静的チェックが失われることに注意してください。すべてをディクショナリに配置することで、キー名の入力ミスによる実行時エラーが発生する可能性があります。

于 2013-03-22T00:16:07.973 に答える
1

キーを配列に入れます。

string[] keys = {
  "PRODUCTID", "PRODUCTCODE", "PRODUCT", "CLEAN_URL",
  "WEIGHT", "LIST_PRICE", "DESCR", "FULLDESCR",
  "KEYWORDS", "TITLE_TAG", "META_KEYWORDS", "META_DESCRIPTION",
  "AVAIL", "RATING", "FORSALE", "SHIPPING_FREIGHT",
  "FREE_SHIPPING", "DISCOUNT_AVAIL", "MIN_AMOUNT", "LENGTH",
  "WIDTH", "HEIGHT", "LOW_AVAIL_LIMIT", "FREE_TAX",
  "CATEGORYID", "CATEGORY", "POS", "MEMBERSHIP",
  "PRICE", "THUMBNAIL", "IMAGE", "TAXES",
  "ADD_DATE", "VIEWS_STATS", "SALES_STATS", "DEL_STATS",
  "SMALL_ITEM", "SEPARATE_BOX", "ITEMS_PER_BOX", "GENERATE_THUMBNAIL",
  "RETURN_TIME", "MANUFACTURERID", "MANUFACTURER"
};

これで、それから辞書を作成できます。

string[] row = productsList[i];

product.row1 =
  Enumerable.Range(0, row.Length).ToDictionary(i => keys[i], i => row[i]);
于 2013-03-22T00:39:03.400 に答える