Задача на массивы в StringGrid

@Мун · Регистрация: 03.10.2011

Студворк — интернет-сервис помощи студентам

Уважаемые форумчане, требуется ваша помощь, изучаю C++ совсем недолго, строго за нубовопросы не судите
Ниже обработчик для кнопки вывода в мемо наименьших значений по столбикам массива в SG (fixed Cols/Rows 1), значения находятся верно, но при попытке вывести их индексы и переможить их, считает через раз, чаще выдает неверно или значения вроде "1231241", укажите на ошибку

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
void __fastcall TForm1::Button2Click(TObject *Sender)
{
int index1, index2, proiz, min;
min=StrToInt(StringGrid1->Cells[1][1]);
 for (j = 1; j < StringGrid1->ColCount; j++) {
    min = StrToInt(StringGrid1->Cells[j][1]) ;
        for (i = 1; i < StringGrid1->RowCount; i++) {
            if (StrToInt(StringGrid1->Cells[j][i]) < min) {
                min = StrToInt(StringGrid1->Cells[j][i]);
                index1 = i;
                index2 = j;
                proiz = i*j;
        }
          }
Memo1->Lines->Add("Минимальный элемент в столбце  " + IntToStr(j) + " = " +
            IntToStr(min));
Memo2->Lines->Add("Индексы минимальных элементов:  " + IntToStr(index1) + " и " +
            IntToStr(index2));
Memo3->Lines->Add("Произведения индексов:  " + IntToStr(proiz));

@Lord_Voodoo · 29.07.2013, 23:19

во-первых, такая форма записи не является приемлемой, каждая переменная должна объявляться на новой строке:
Мун,

Сообщение от Мун

int index1, index2, proiz, min;

во-вторых, крайне нехорошо объявлять переменные без инициализации, тоже есть не очень хорошо

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

AntonChik · 30.07.2013, 05:53

Сообщение от Lord_Voodoo

во-первых, такая форма записи не является приемлемой, каждая переменная должна объявляться на новой строке:

странно. впервые такое слышу)

меня смутило что i n j не объявлены в теле обработчика (видать где-то выше глобально объявлены), вот это точно не гуд.

@nick42 · 30.07.2013, 10:28

Основную ошибку я вижу в том, что в мэмо строки добавляются ПОСЛЕ обработки всей таблицы, а нужно после завершения цикла по i (проход по колонке). И опять же, - будет показано только первое минимальное значение, а их одинаковых может быть несколько (теоретически; структуру данных в таблице вы не показали).

@Lord_Voodoo · 30.07.2013, 10:32

AntonChik,

http://www.possibility.com/Cpp... ndard.html

правда там говорится про ситуацию:

C++
1
String* pName, name, address;

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

AntonChik · 30.07.2013, 10:32

Сообщение от nick42

а нужно после завершения цикла по i

я тоже сначала про это подумал, но там все нормально, мемо как раз после цикла i, просто в конце не хватает скобки для цикла j

@nick42 · 30.07.2013, 10:49

.. верно. И правильнее было бы инициализировать индексы как J 1 , а то при минимуме в верхней строчке будет неверный результат.

@Мун · 30.07.2013, 22:33 **[ТС]**

Сообщение от AntonChik

странно. впервые такое слышу)

меня смутило что i n j не объявлены в теле обработчика (видать где-то выше глобально объявлены), вот это точно не гуд.

я объявил глобально, потому что они используются ранее при выводе массива в SG

Добавлено через 2 минуты

Сообщение от AntonChik

я тоже сначала про это подумал, но там все нормально, мемо как раз после цикла i, просто в конце не хватает скобки для цикла j

скобка в конце в самом есть закрывающаяся, поигрался с ними, все только хуже) где-то с циклом потерялся видимо

Добавлено через 1 минуту

Сообщение от nick42

.. верно. И правильнее было бы инициализировать индексы как J 1 , а то при минимуме в верхней строчке будет неверный результат.

вы о дополнительном значении J1?

@nick42 · 30.07.2013, 23:29

Сообщение от Мун

вы о дополнительном значении J1?

.. ну вот вы закончили цикл по i, увеличили на единицу j и начали следующий цикл по i..., - вы "освежили" только значение min, установив его равным значению самой верхней ячейки. Значения index1 и index2 остались от прежнего цикла. А теперь допустите, что значение верхней ячейки и есть самое минимальное... . Значит условие if в этом цикле по i ни разу не выполнится, и в конце цикла в Memo отправится правильное значение min и... старые значения индексов. Если бы вы в начале цикла по i позаботились одновременно с установкой min на самое "верхнее" значение установить и соответствующие этой ячейке индексы, - запись в мэмо была бы корректной.

Новые блоги и статьи Все статьи Все блоги /
Подстановка значения реквизита справочника в табличную часть документа Maks 10.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "ПланированиеПерсонала", разработанного в конфигурации КА2. Задача: при выборе сотрудника (справочник Сотрудники) в ТЧ документа. . .	Очистка реквизитов документа при копировании Maks 09.04.2026 Алгоритм из решения ниже применим как для типовых, так и для нетиповых документов на самых различных конфигурациях. Задача: при копировании документа очищать определенные реквизиты и табличную. . .	модель ЗдравоСохранения 8. Подготовка к разному выполнению заданий anaschu 08.04.2026 https:/ / github. com/ shumilovas/ med2. git main ветка * содержимое блока дэлэй из старой модели теперь внутри зайца новой модели 8ATzM_2aurI	Блокировка документа от изменений, если он открыт у другого пользователя Maks 08.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа, разработанного в конфигурации КА2. Задача: запретить редактирование документа, если он открыт у другого пользователя. / / . . .
Система безопасности+живучести для сервера-слоя интернета (сети). Двойная привязка. Hrethgir 08.04.2026 Далее были размышления о системе безопасности. Сообщения с наклонным текстом - мои. А как нам будет можно проверить, что ссылка наша, а не подделана хулиганами, которая выбросит на другую ветку и. . .	Модель ЗдрввоСохранения 7: больше работников, больше ресурсов. anaschu 08.04.2026 работников и заданий может быть сколько угодно, но настроено всё так, что используется пока что только 20% kYBz3eJf3jQ	Дальние перспективы сервера - слоя сети с космологическим дизайном интефейса карты и логики. Hrethgir 07.04.2026 Дальнейшее ближайшее планирование вывело к размышлениям над дальними перспективами. И вот тут может быть даже будут нужны оценки специалистов, так как в дальних перспективах всё может очень сильно. . .	Горе от ума kumehtar 07.04.2026 Эта мне ментальная установка, что вот прямо сейчас, мол, мне для полного счастья не хватает (нужное вписать), и когда я этого достигну - тогда и полный кайф. Одна из самых сильных ловушек на пути. . . .

@nick42 Практикантроп 4841 / 2726 / 534 Регистрация: 23.09.2011 Сообщений: 5,798
	30.07.2013, 10:28
	Основную ошибку я вижу в том, что в мэмо строки добавляются ПОСЛЕ обработки всей таблицы, а нужно после завершения цикла по i (проход по колонке). И опять же, - будет показано только первое минимальное значение, а их одинаковых может быть несколько (теоретически; структуру данных в таблице вы не показали). 0

@nick42 Практикантроп 4841 / 2726 / 534 Регистрация: 23.09.2011 Сообщений: 5,798
	30.07.2013, 10:49
	.. верно. И правильнее было бы инициализировать индексы как J 1 , а то при минимуме в верхней строчке будет неверный результат. 0