[C#] How to modify a Hashtable entry value in-place?!

[Vovka]

Вроде простейшая проблема, но что-то я ничего не понимаю. Можно, конечно, удалить и вставить новое, но это очень криво. Сначала попробовал так:

Code: Select all

Hashtable map = new Hashtable();

map["first"] = 10;
map["second"] = 20;

foreach(DictionaryEntry e in map)
{
   e.Value = 100;
   Console.WriteLine("{0}: {1}", key, map[key]);
}

Вообще не компилит, причём выдаёт совершенно невразумительное сообщение об ошибке.

Вот так вот тоже не работает:

Code: Select all

Hashtable map = new Hashtable();

map["first"] = 10;
map["second"] = 20;

foreach(object key in map.Keys)
{
   map[key] = 100;
   Console.WriteLine("{0}: {1}", key, map[key]);
}

Бросает исключение, что коллекция была изменена и она не может дальше енумерэйтить.

Так я же не ключи меняю?!

[blanko27]

Бросает исключение, что коллекция была изменена и она не может дальше енумерэйтить.
Так я же не ключи меняю?!

Ну правильно, запрещается изменять коллекцию во время енумерации. Это будет работать:

Code: Select all

map["first"] = 100; 
map["second"] = 100;

foreach(DictionaryEntry e in map) 
{
    Console.WriteLine("{0}: {1}", e.Key, map[e.Key]); 
}

[Vovka]

Ну правильно, запрещается изменять коллекцию во время енумерации.

Интересно, какая светлая голова это придумала и почему.

Это будет работать:

Code: Select all

map["first"] = 100; 
map["second"] = 100;

foreach(DictionaryEntry e in map) 
{
    Console.WriteLine("{0}: {1}", e.Key, map[e.Key]); 
}

Так здесь вы ничего не меняете!
Или вы демонстрируете, что я могу читать элементы во время енумерации? Ну слава богу, хоть это можно.

[Palych]

Vot ona - prichina pochemu nel'zya Itaretors pryatat' za fancy operators!

A ved' oni eto govno i v Java taschat...

[blanko27]

Можно применить такую последовательность:

Code: Select all

         Hashtable map = new Hashtable(); 

         map["first"] = 10; 
         map["second"] = 20;

         ArrayList items = new ArrayList();
         foreach (string key in map.Keys)
         {
            items.Add(key);
         }

         foreach(string key in items) 
         {
            map[key] = 100;
            Console.WriteLine("{0}: {1}", key, map[key]); 
         }

[Strannik223]

Странно, но за 2 года програмирования на шарпе не наступал на эту особенность.
А зачем вам модифицировать все элементы в hashtable?

[blanko27]

...А зачем вам модифицировать все элементы в hashtable?

Он, по-видимому, упростил ситуацию для наглядности. Ему не надо модофоцировать все, а только некоторые, подпадающие под определенное условие. Это тоже невозможно при энумерации.

[Bobo]

Ну надо еще понимать, что такое mutable и immutable.

Вот так будет работать:

Code: Select all

class Q
{
  public Q(int x){X=x;}
  public int X;
}

void Main()
{
  Hashtable t = new Hashtable()
  t.Add("a", new Q(20))
  t.Add("b", new Q(30))
  foreach (object key in t.Keys)
    ((Q)t[key]).X = 100;
}

А вот если сделать так:

Code: Select all

Arraylist a = new Arraylist();
a.Add(10);
a.Add(20);
foreach (object o in a)
  o = 100;

то ошибки не будет, но и значения, естественно не поменяются.

[blanko27]

Вот так будет работать:

Code: Select all

class Q
{
  public Q(int x){X=x;}
  public int X;
}
... ...
}

Да это решение с классом очень хорошее. Но одна существующая проблма этого решения, что оно не позволяет добавлять и удалять элементы из коллекции по необходимому условию, типа:

Code: Select all

foreach (object key in t.Keys)
{
    t.Remove(key);
}

[Bobo]

Но одна существующая проблма этого решения,...

Естественно, итератор по определению запрешает изменять коллекцию.
Я просто хотел обратить внимание на то, что надо различать случаи, когда нам надо менять коллекцию, а когда - ее елементы.

[Vovka]

Ну надо еще понимать, что такое mutable и immutable.

И что это такое?
В Language Reference такого понятия не нашёл.

Ваше решение, возможно, работает, но это чистейшей воды workaround (я бы даже назвал hack).

Никакой мало-мальски убедительной причины для того, чтобы не позволять в таких случая менять Value я по-прежнему не вижу. Скорее всего, это design bug.

[leprechaun]

Ну надо еще понимать, что такое mutable и immutable.

И что это такое?
В Language Reference такого понятия не нашёл.

Google would be definitely helpful here. Some of the links that came first:

http://www.javaworld.com/javaworld/java ... table.html
http://www.rh.edu/~kousek/cish4020/ppt/Chapter15.ppt
http://www.cs.man.ac.uk/arch/people/j-s ... ode30.html

[Strannik223]

Никакой мало-мальски убедительной причины для того, чтобы не позволять в таких случая менять Value я по-прежнему не вижу. Скорее всего, это design bug.

Согласен.
Я понимаю почему нельзя менять Key, но Value за что под раздачу попало непонятно.

[Bobo]

Ну надо еще понимать, что такое mutable и immutable.

И что это такое?
В Language Reference такого понятия не нашёл.

В C# Language Reference ети понятия упоминаются только в связи с типом string. (который reference but immutable)
А разьяснения про отличия value types ot reference types разжеваны повсеместно.

Ваше решение, возможно, работает, но это чистейшей воды workaround (я бы даже назвал hack).

ето чистой воды демонстрация того, что в вашем случае проблема не с итератором, а с тем, что значение value type поменять в принципе невозможно. Его можно только заменить на новый обьеkт.

Никакой мало-мальски убедительной причины для того, чтобы не позволять в таких случая менять Value я по-прежнему не вижу.

Я надеюсь, причину не позволять менять коллекцию вы видите. А в случае с immutable types изменение value - ето вставка нового обьекта.

[Sam Adams]

Code: Select all

foreach(object key in map.Keys)
{
   map[key] = 100;
   Console.WriteLine("{0}: {1}", key, map[key]);
}

Бросает исключение, что коллекция была изменена и она не может дальше енумерэйтить.

Похоже, что разработчики немного недаработали этот момент. Я в похожих случаях (если производительность не критична) делаю так -

Code: Select all

public static IList CreateList(IEnumerator ie);   // для использования итератора в конструкции foreach

foreach (object key in CreateList(map.Keys.GetEnumerator()))
{
   map[key] = 100;
}