Двумерный массив на С++, Проверка на правильность написания кода Опции

[Екатерина]

Всем добрый вечер!
Есть задание к лабораторной работе
Написать функцию, для поиска максимального элемента в указанной строке двумерного массива. Сдвинуть в двумерном массиве все строки циклически вправо на количество элементов равное максимальному элементу в этой строке.
Могу я попросить специалистов проверить правильность написания кода?

#include <stdio.h>
#include <tchar.h>
#include <stdlib.h>
#define SIZE_ARRAY 10       //размер массива
#define MAX_INTEGER 10      //максимально допустимое число
int myarray[SIZE_ARRAY][SIZE_ARRAY]; // масив с которым работаем
int _tmain(int argc, _TCHAR* argv[])
{
	int NeedString=0; //необходимая строка - строка с которой работаем
	int max = 0;      //с помощью этой переменной определяем максимальный элемент в строке
	printf("Array:\n");

	for (int i = 0; i < SIZE_ARRAY; i++) //заполняем массив случайными числами и одновременно выводим его на экран
    {
        for (int ii = 0; ii < SIZE_ARRAY; ii++)
        {
            myarray[i][ii] = rand()%MAX_INTEGER;
            printf("%3d ", myarray[i][ii]);
        };
        printf("\n");
    }
	printf("Vvedi nomer stroki: ");   //запрашиваем необходимую строку у пользователя..
    scanf("%d", &NeedString);
	while (NeedString <= 0 || NeedString > SIZE_ARRAY) //проверяем
        {
			printf("Vvedi v diapazone ot 1 do %d:  ", SIZE_ARRAY);
            scanf("%d", &NeedString);
            printf("\n");
		}
    NeedString--;

    for (int i = 0; i < SIZE_ARRAY; i++)  //ищем максимальное число в строке
    {
        if (myarray[NeedString][i] > max) max = myarray[NeedString][i];
    };
	printf("Max integer in string is %d\n", max); //и выводим его на экран
	printf("Sdvig %d strok na %d element vpravo\n", NeedString+1, max);
	for (int i = 0; i < max; i++) //сдвигаем на нужное число элементов
    {
        int save = myarray[NeedString][SIZE_ARRAY-1];
        printf("%d: ", i+1);
        for (int ii = SIZE_ARRAY-1; ii > 0; ii--)
        {
            myarray[NeedString][ii] = myarray[NeedString][ii-1];
        };
        myarray[NeedString][0] = save;
        for (int i = 0; i < SIZE_ARRAY; i++) printf("%3d ", myarray[NeedString][i]);
        printf("\n");
    };

    scanf("%d", &NeedString);
    return 0;
}

Слегка терзают смутные сомнения...

[Krjuger]

Ну собственно я немного переделал ваше.

#include <stdio.h>
#include <tchar.h>
#include <stdlib.h>
#define SIZE_ARRAY 10       //размер массива
#define MAX_INTEGER 10      //максимально допустимое число
int myarray[SIZE_ARRAY][SIZE_ARRAY]; // масив с которым работаем
int _tmain(int argc, _TCHAR* argv[])
{
	int NeedString=0; //необходимая строка - строка с которой работаем
	int max = 0;      //с помощью этой переменной определяем максимальный элемент в строке
	printf("Array:\n");

	for (int i = 0; i < SIZE_ARRAY; i++) //заполняем массив случайными числами и одновременно выводим его на экран
    {
        for (int ii = 0; ii < SIZE_ARRAY; ii++)
        {
            myarray[i][ii] = rand()%MAX_INTEGER;
            printf("%3d ", myarray[i][ii]);
        };
        printf("\n");
    }
	printf("Vvedi nomer stroki: ");   //запрашиваем необходимую строку у пользователя..
    scanf("%d", &NeedString);
	while (NeedString <= 0 || NeedString > SIZE_ARRAY) //проверяем
        {
			printf("Vvedi v diapazone ot 1 do %d:  ", SIZE_ARRAY);
            scanf("%d", &NeedString);
            printf("\n");
		}
    NeedString--;

    for (int i = 0; i < SIZE_ARRAY; i++)  //ищем максимальное число в строке
    {
        if (myarray[NeedString][i] > max) max = myarray[NeedString][i];
    };
	printf("Max integer in string is %d\n", max); //и выводим его на экран
	printf("Sdvig %d strok na %d element vpravo\n", NeedString+1, max);
	for (int pos=0;pos<max;pos++)
	{
	for (int i = 0; i < SIZE_ARRAY; i++) //сдвигаем на нужное число элементов
    {
        int save = myarray[i][SIZE_ARRAY-1];
        for (int j=SIZE_ARRAY-1;j > 0  ; j--)
		{
			myarray[i][j]=myarray[i][j-1];
		}
        myarray[i][0]=save;
	}
	}
	for (int i = 0; i < SIZE_ARRAY; i++)
	{
		printf("%d: ", i+1);
	    for (int j = 0; j < SIZE_ARRAY; j++) 
		    printf("%3d ", myarray[i][j]);
        printf("\n");
	}
    scanf("%d", &NeedString);
    return 0;
}

И как подмечено ранее это чистый С,все прелести С++ не используются и я бы еще избавился на вашем месте от #include <tchar.h> ну и сообветственно переделал строку int _tmain(int argc, _TCHAR* argv[]) так как библиотека подключается только ради _TCHAR типа, а argc и argv[] вообще никак в вашей программе не используются и поэтому в их наличии нет необходимости,посоветовать,как переделать не буду,так как не знаю, как в борланде это делается.У меня в визуал студио можно убрать библиотеку и заменить описываемую строку на int main().

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

И на будущее обычно координаты у матрици обозначают i,j так еще с математических обозначений повелось и придерживаться таких канонов неплохо было бы)))Это конешно же не обязательно,но так легче разбираться если некоторые стандартизированные вещи есть.

Сообщение отредактировано: Krjuger - 13.06.2011 23:08