Не меняет местами элементы матрицы, язык C++

[Саша Степанов]

код вроде бы правильный, но почему-то вместо того чтобы менять местами элементы B[imin][jmin] и B[3][2], просто заменяет на B[imin][jmin], помогите пожалуйста разобраться в чём дело, практики у меня мало, но раньше в подобных задачах проблем не было, пробовал тремя разными способами менять местами и свапом и без третьей переменной, всё равно только заменяет.

int main (){
    
double P,B[4][4],min,temp;
int i,j,imin,jmin;

min=1e10;
P=1;
    
for(i=0;i<4;i++)
for(j=0;j<4;j++)    
cin>>B[i][j];

cout<<"\n \n";

for(i=0;i<4;i++){
for(j=0;j<4;j++){    
cout<<B[i][j]<<setw(4);}
cout<<"\n \n";} 

cout<<"\n \n";

for(i=0;i<4;i++){
for(j=0;j<4;j++){
if(i+j<3&&B[i][j]>0){P=P*B[i][j];}  
}}

cout<<"proizvedenie polozhitelnih elementov vishe pobochnoy diagonali="<<P<<"\n \n"; 

for(i=0;i<4;i++){
j=1;
if(B[i][j]<min){
min=B[i][j];
imin=i;
jmin=j;                
}}

cout<<"\n \n";

cout<<"minimalniy element vo vtorom stolbce="<<min<<"\t ego adres=["<<imin+1<<"]["<<jmin+1<<"]";

cout<<"\n \n";

for(i=0;i<4;i++){
for(j=0;j<4;j++){
temp=B[imin][jmin];
B[imin][jmin]=B[3][2];
B[3][2]=temp;                                                                                                     
cout<<B[i][j]<<setw(4);}
cout<<"\n \n";}
 


            getchar();
             getchar();
             
             return 0;

}

[was3110]

В тех случаях, когда компиляция проходит нормально, а результат не правильный выход один: провести пошаговую отладку (трассировку) с отслеживанием значений переменных на каждом шаге. И увидите из-за чего Ваш В[3,2] не получает значения

[Mandrivnyk]

min=1e10;

Оригинально, конечно... Но обычно присваивают значение первого элемента:

min = B[0][0];

,
и, соответственно, цикл начинают со второго... И, кстати, тут же второй вопрос:

cout<<"minimalniy element vo vtorom stolbce="

Минимум надо искать только во втором столбце? Тогда ты делаешь много лишних "телодвижений"...
И вообще -- неплохо было бы знать, что именно должна делать программа.

Ну, и главное:

for(i=0;i<4;i++){
    for(j=0;j<4;j++){
        temp=B[imin][jmin];
        B[imin][jmin]=B[3][2];
        B[3][2]=temp;                                                                                                     
        cout<<B[i][j]<<setw(4);}
    cout<<"\n \n";}

Цикл-то тут зачем? Именно тут?
Ты меняешь значения B[imin][jmin] и B[3][2] 4*4=16 раз!
И выводишь его же вместо самого массива -)
Вынеси выделенную часть за пределы циклов -- до циклов, а не после.

[Саша Степанов]

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

Ну, и главное:

Цикл-то тут зачем? Именно тут?
Ты меняешь значения B[imin][jmin] и B[3][2] 4*4=16 раз!
И выводишь его же вместо самого массива -)
Вынеси выделенную часть за пределы циклов -- до циклов, а не после.

Если вынести эту часть перед циклом,а потом просто вывести цикл, ничего ровным счётом не изменится, выведет начально заданный цикл и всё, хотя ты прав по-этому скорее всего и не меняет, но тогда выхода я не вижу.

[Mandrivnyk]

Внимательно на свой код посмотри.
imin и jmin ты находишь в другом цикле.
В этом, который я выделил, ты просто выводишь массив на экран.
Так вот _перед_ выводом -- должны стоять эти три строчки, которые меняют _уже найденный_ тобой минимум B[imin][jmin] с B[3][2].
Понимаешь?
Первый цикл -- ввод массива;
Второй цикл -- вывод исходного массива;
Третий цикл -- нахождение индексов минимального элемента;
--> Обмен местами минимального элемента с B[3][2];
Четвертый цикл -- вывод полученного массива.

[Саша Степанов]

Mandrivnyk, спасибо, всё работает, но когда я делал это раньше не менялось, может быть что-то с системой было.