создать функцию меняющую местами четные элементы в нечетных строках

[Anya]

Помогите, пожалуйста... я написала программу, которая должна менять местами четные элементы в нечетных строках, но у меня какая то ошибка, которая дает непонятный цикл в ответе, я никак не могу понять что такое...посмотрите пожалуйста, очень нужно, мне ее завтра сдать надо((
Вот код:

Код: (C++)

1. #include<iostream.h>
2. #include<conio.h>
3. #include<stdio.h>
4. #include<stdlib.h>
5.  
6. void zamena(int A[20][20]){
7.     int x,i,j,s,t;
8.     for (i=0; i<s; i++){
9.         if(i%2==1){
10.             for (j=0; j<t; j=j+2){
11.                 if (j%2==0){
12.                     x=A[i][j];
13.                     A[i][j]=A[i][j+2];
14.                     A[i][j+2]=x;
15.                 }
16.             }
17.         }
18.     }
19. };
20.  
21. void main(){
22.     void zamena(int[20][20]);
23.     int A[20][20];
24.     int i,j,s,t;
25.     clrscr();
26.     printf("vvedite razmer matricy;\n");
27.     scanf("%i%i",s,t);
28.     randomize();
29.     for (i=0; i<s; i++){
30.         for(j=0; j<t; j++){
31.             A[i][j]=random(10);
32.         }
33.     }
34.  
35.     printf("Massiv:\n");
36.     for (i=0; i<s; i++){
37.         for(j=0; j<t; j++){
38.             printf("%i",A[i][j]);
39.         }
40.     }
41.  
42.     zamena(A);
43.     for (i=0; i<s; i++){
44.         for(j=0; j<t; j++){
45.             printf("%i\t",A[i][j]);
46.         }
47.         printf("\n");
48.     }
49.     getch();
50. }

[Dimka]

Anya, поясни, что такое двумерный массив, и как с ним работать?

[Anya]

Это объединение символов одного типа, это что то вроди матрицы размером M*N, где M колличество строк, а N колличество столбцов., а элементы этой матрицы находятся на пересечении номера строки и столбца, получаем, что элемент это у нас a[i][j],где i строка, j столбец. Работать можно либо в цикле, т.е практически со всей строкой, столбцом или с конкретными элементами

Добавлено через 1 час, 31 минуту и 1 секунду:
я там кстати описалась:

Код:

x= A[i][j];
A[i][j]=A[i][j+2];
A[i][j+2]=x;

[Вад]

Anya, в следующий раз вставляй код в теги [code][/code] и форматируй, пожалуйста, а то понять ничего невозможно.

Я в исходном сообщении подстветил 22ю строку - это так и задумывалось, или копипаста неудачная?

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

Добавлено через 8 минут и 43 секунды:
Посмотрел код. Anya, почему у тебя в функции zamena переменные s,t, которые должны представлять собой размерность массива, нигде не инициализируются? Там может происходить что угодно, в этой функции.

[Anya]

Вад, ну 22-ой строки в принципе может и не быть, а то что я не инициализировала s,t сильно влияет на ход роботы программы, ошибок же компилятор не выдает

[Вад]

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

[Anya]

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

[Dale]

Anya, в языке C параметры в подпрограмму всегда передаются только по значению. Если функция обладает побочным эффектом, т.е. меняет значение своих параметров, эти параметры явно должны передаваться как ссылки. Поэтому в строке

Код: (C)

scanf("%i%i",s,t);

ничего (хорошего) не происходит. Замените ее на

Код: (C)

scanf("%i%i", &s, &t);

Чтобы убедиться, что это именно так, поставьте контрольную печать значений s и t после ввода и проверьте, действительно ли в них содержится то, что ожидается.

[Вад]

Anya, тогда или у тебя странный компилятор, или ты не читаешь сообщения во время сборки программы. Современные компиляторы для такого рода кода по умолчанию должны выдавать предупреждение о неинициализированных переменных. Программа от это собираться, конечно, не перестанет.

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

[Dale]

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

Компилятор C к таким вещам относится с хладнокровным пофигизмом, по принципу: "вы же просили именно этого - так получайте". Нужно пользоваться вспомогательными инструментами, вроде lint'а. Я для C обычно использую splint.

[Вад]

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

Компилятор C к таким вещам относится с хладнокровным пофигизмом, по принципу: "вы же просили именно этого - так получайте". Нужно пользоваться вспомогательными инструментами, вроде lint'а. Я для C обычно использую splint.

Судя по заголовку iostream.h, это не мог быть компилятор C. Если только он не отнёсся с пофигизмом и к инклуду.

[Anya]

а вообще сама функция замена правильно написана для задания? потому что я сейчас кое-что подправила и у меня теперь получается первоначальная матрица выводится, а вторичная, которая должна получится в результате функции нет, может я вызов не правильно осуществляю?

[Dale]

Судя по заголовку iostream.h, это не мог быть компилятор C.

Точно, проглядел. Хотя затесался он туда явно случайно.

Добавлено через 3 минуты и 49 секунд:

первоначальная матрица выводится, а вторичная, которая должна получится в результате функции нет

Не выводится совсем ничего, или не то, что надо?

[Anya]

Вообще ничего, оно просто останавливается

[Вад]

Anya, показывай текущий вариант - мы ж не знаем, что там подправлено

[Dale]

И понаставьте контрольных печатей побольше. Чтобы знать, какие точки в программе пройдены, а до каких дело не дошло.

[Anya]

Код:

#include<iostream.h>
#include<conio.h>
#include<stdio.h>
#include<stdlib.h>
void zamena(int A[5][5]){
int x,i,j,s,t;
for (i=0; i<s; i++){
if(i%2==1){
for(j=0; j<t; j=j+2){
if (j%2==0){
x=A[i][j];
A[i][j]=A[i][j+2];
A[i][j+2]=x;}}}}
};
void main(){
int A[5][5];
int i,j,s,t;
clrscr();
printf("vvedite razmer matricy;\n");
scanf("%i%i",&s,&t);
randomize();
for (i=0; i<s; i++){
 for(j=0; j<t; j++){
 A[i][j]=random(10);}}
printf("Massiv:\n");
for (i=0; i<s; i++){
 for(j=0; j<t; j++){
 printf("%i\t",A[i][j]);}
 printf("\n");}           \\получается проходит только до этой точки доходит, а к функции не обращается
zamena(A);
 for (i=0; i<s; i++){
 for(j=0; j<t; j++){
 printf("%i\t",A[i][j]);}
 printf("\n");}
 getch();
 }

[Вад]

Anya, ну так, как я в самом начале сказал, в функции zamena нигде не инициализируются s и t - и вообще, реальные размерности массива в неё никак не передаются.
То, что имена переменных в этой функции и в main совпадают - ровно ни о чём не говорит. В результате, цикл по замене работает некорректно.

[Anya]

то есть, мне сам ввод размерности перенести в функцию замена?

[Вад]

Нет, лучше передать эти значения в функцию в виде параметров. Так же, как ты передаёшь массив.

[Anya]

хорошо, сейчас попробую

Добавлено через 12 минут и 56 секунд:
Вад, что то я запуталась, мы передаем s и t в виде параметров, как массив, массив я передаю как параметр функции замена, а s и t мне как параметр функции main передавать?

[Вад]

Нет, тебе ведь и размерности тоже нужно передать параметром в функцию замена - ведь именно эта функция ничего не знает про введённые размеры. Так же, как она не знала бы и про значения, записанные в массив в теле функции main, если бы ты массив не передала через параметр.

[Dimka]

Anya, s и t - параметры функции zamena.

[Anya]

я уже поняла, параметры я написала, программа практически работает, правда выводит не то что мне надо, выводит матрицу, а меняет не то

[Вад]

Anya, значит, в алгоритме замены ошибка
Опиши поточнее, как именно должна происходить замена. По какому принципу меняются чётные элементы в нечётных строках? Не совсем понятно.

[Anya]

ну как я поняла эту задачу, необходимо менять четный по индексу элемент, в нечетной по индексу строке, т.е у меня есть строка и я в ней меняю 2-й и 4-й, 6-й и 8-й местами, как бы попарно, а если у нас нечетное колличество элементов, последнее отбрасывается

[Вад]

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

[Anya]

Спасибо всем за помощь программа работает