вообщем задача такая, есь массив L1,L2,L3. Массивы L1 и L2 заполняются случайными числами. Есть процедура, которая поочередно сравнивает каждое значение обоих массивов, и если значения равны, то это значение записывается в L3...

проблема в следующем, если забить массивы L1,L2 в ручную то все работает, а если массивы L1, L2 забиваются рандомно, то ничего не работает...


вот код забивания массивов рандомом




а вот код сравнение элементов массивов L1 и L2





P.s. под "массивами" в данном случае имею ввиду listbox

Так ведь если массивы заполняются случайными числами от 0 до N, не факт что обязательно найдётся попарное совпадение. Например, в одном массиве не будет числа 4, а в другом - числа 8.

И что вообще означает:


  

Что именно "не работает" ? Поконкретнее.

mr.DUDA, массивы у меня заполняются не от 0 до N, а от 0 до 10-ти...и самих элементов каждого массива 10. так что там не может не быть хотя бы одного совпадения...
а не работает буквально - нажимаю на кнопку на которую повешен код сравнения L1 и L2 - и как будто я на нее и не нажимал...вообщем эффекта ноль...в то же время - если руками забить ТЕ ЖЕ числа которые при предыдущем запуске сгенерировал рандом - то все работает...

такое ощущение что  в коде заполнения массивов рандомными значениями,в listbox'ы просто добаляются строчки со случайными числами, а item'ам соостветвующие значение не присваиваются...т.е...массив значениями не заполняются - не знаю как это может быть....



Это сообщение отредактировал(а) derek - 29.10.2006, 22:54

Тьфу ты. С каких пор ListBox называют массивом?! Этим ты всех и запутал.
В ListBox'е хранятся ссылки на объекты. Поэтому, когда ты добавляешь в него число, оно "оборачивается" (boxing) в System.Object и в этом виде добавляется в коллекцию Items. Таким образом, сравнивая два элемента, фактически, ты сравниваешь ссылки на объекты, а не их значения. Используй if ((int)L1[i] == (int)L2[j]), и будет тебе счастье  .

0..N общий случай для 0..10, мы говорим об одном и том же


хотя бы одно будет


оч. странно... чем отличается код, когда int-ы руками забиваются в листбокс, от кода с random-ом ? можно его посмотреть ?


но другой пример то работает !   

тут я немного неправильно выразился. под "вводом значений руками" я подразумевал "ввод значений через конструктор"...т.е. кода собственно и не было





да это работает...но есть вопрос - что это вообще за метод такой (int)L1[i] - имею ввиду (int) - это ведь не конвертирование, ведь нельзя объект переконвентировать в int...вообщем это работает...но КАК это работает мне не ясно...

Почему? При выполнении Add мы упаковываем value-тип int (System.Int32) в object (System.Object), а потом приводим object к типу int и распаковываем. По сути, операция приведения - это и есть конвертация.

mr.DUDA, волшебство, однако. Действительно, почему работает ручное заполнение?

Exception, 


Потому как товарищ derek, сам ответил на свой вопрос 

ListBox он заполняет, а где он присваивает значения своим массивам я не увидел.

Он заполняет значениями только listBox L1 L2. А где массивы?

Когда руками добавляем, в дизайнере - в листбоксе будут строки. А в примере сравниваются числа. Пример вообще не должен работать при таких условиях.

Привет всем.

Мне думается, должно быть так


PS извините переменную по фламандски назвал getal=число.   


Это сообщение отредактировал(а) thomas - 30.10.2006, 18:14

mr.DUDA
В том-то и дело!
Как тебе известно, строки - ссылочные объекты, а потому сравнение одинаковых строк будет работать и без приведения типов: ссылки на одинаковые строки будут одинаковыми. А вот для value-типов следует выполнять приведение, так как ссылки всегда будут разными.
Именно поэтому заполнение строк в дизайнере срабатывало.

добавление в дизайнере от программного добавления отличается немногим...если заглянуть в *.designer.cs, то можно увидеть list1.Items.Add ("...")...и что выдаст такой код?1-исключение, 2-исключение, 3-ок...вывод? виноват string pooling. 

Это сообщение отредактировал(а) archimed7592 - 31.10.2006, 02:10

У меня такой же код выдаёт Ok:



хммм ?..   

он у тебя не такой же  у тебя вместо throw new Exception () стоит Console.WriteLine ("ok") потому и выдаёт OK 

Ну это несерьёзно  , почему string pooling работает-то ?... К примеру, если абстрагироваться от листбокса (оффтоп пошёл) и запихнуть одинаковые стринговые литералы в два разных object-а, то сравнивать будет корректно. А почему у derek работает сравнение строки с int-ом, я так и не въехал, видимо неправильная у меня трава   

  хде такое увидел?
вообще 
контрпример:единственный метод, которым можно проверить равны объекты или нет - это Equals. и при сравнении объектов типа Object оператором == он НЕ вызывается.выведет что немного иной результат, нежели вариант выше.
другое дело, когда сравниваешь s1 и s2 оператором == там всё ок потому, что его перегружают.

Ну строки со строками...


Дык, это если присваивать не константную строку а результат действия над строкой (а-ля Remove), то да. Не-константы не интернируются и в пул не попадают. А вот если мы их так:


то совсем другое дело, выведет "s1 == s2"   

P.S. оффтоп пошёл, уважаемые.

Добавлено @ 18:03 
З.Ы. для интернированных строк сравниваются уже не сами строки, а ссылки на объект string.

для объектов типа string всегда сравниваются строки (если конечно ссылки не равны...если равны, то сравнивать нечего). для объектов типа object всегда сравниваются ссылки. добавляя строку в пул ты получишь ссылку идентичную s1 и соответственно результат сравнения ссылок будет очевидным.

ну ты же модер  вынеси обсуждение в отдельный тред. вопрос очень интересный на самом деле.

опять наступаю на грабли listbox'а
почему не работает такой простой код - где я опять тупанул  ?



ошибки не выдает при компиляции. Просто не работает - ноль эффекта...

не <=, a >=

Я пьяный, могу ошибаться...   

так вылетает с ошибкой
ArgumentOutOfRangeException
InvalidArgument=Value of '5' is not valid for 'index'.

ну тогда for (i = 5 - 1; i >= 0; --i) ...

archimed7592, так работает. вообще не думал что ошибка в построении цикла. спасибо

derek, не сочтите за хамство, но


Означает "давайте будем отсчитывать от 5 до 0(или меньшего) до тех пор, пока у нас меньше или 0", а это ни разу не выполнится. Потому что отсчитывать начинаем от 5. А это больше чем 0. Не пройдёт ни одной итерации цикла. 

mr.DUDA, да-да, мне если честно стыдно     

во-первых: заполнение от 0 до 9, если быть точнее.

во-вторых: теоретически совпадения может и не быть вообще, причем вероятность не такая уж и мизерная. Random совсем не гарантирует неповторение одинакового числа, тем более диапазон в нашем случае довольно узок.

в-третьих: если уж хотим сравнивать целые, используя ListBox, то используйте для сравнения хотя бы ListBox.IntegerCollection. Хотя ListBox по большому счету это вообще контрол и он предназначен совсем для других целей. По сути дела началось обсуждение работы самого контрола и вопроса как этим членом расколоть орех.
В нашем случае L1.Items[ i ] это object, давайте тогда и сравнивать объекты как и положено, например
 хоть и это тоже оффтоп и глупость.