Ошибка в коде

[a.n.o.n.i.m]

Именно С нужно

[Mandrivnyk]

Сообщение от a.n.o.n.i.m

добавил библиотеку #include <time.h> и строку srand(time(NULL)); перед строкой переменных,хотя сам не понял зачем добавил,но как бы работает но как то криво и хотелось бы узнать нельзя ли код "укоротить" а функционал оставить тем же,а то вроле программа простая а код слишком длинный +криво работает

srand(time(NULL));

засевает генератор (псевдо)случайных чисел; без этой строчки ты будешь получать одни и те же элементы массива при каждом выполнении программы.

Теперь -- что значит "криво работает"?
А насчет укоротить -- совмести две задачи. Ты сначала ищешь максимальное число в двойном цикле, а потом в почти идентичном цикле -- максимально длинную последовательность.
Кстати, только сейчас обратил внимание

for(int x2=2; x2<column; x2++)

Почему у тебя цикл начинается с 2?

И еще -- в конце программы надо бы освободить выделенную память.

[a.n.o.n.i.m]

for(int x2=2; x2

исправил на нуль for(int x2=0; x2

криво работает

Она когда находит строку тоесть пишет номер строки 'число' я смотрю например вторая строка а там нет одинаковых чисел все разные,хотя программа пишет её,иногда пишет правильно там например 2 одинаковых числа

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

как это сделать

p.s обновил первый пост

[Mandrivnyk]

исправил на нуль for(int x2=0; x2

не на 0, на 1. ты же сравниваешь с предыдущим элементом, поэтому цикл надо начинать не с 0-го, а с 1-го.

Она когда находит строку тоесть пишет номер строки 'число' я смотрю например вторая строка а там нет одинаковых чисел все разные,хотя программа пишет её,иногда пишет правильно там например 2 одинаковых числа

еще раз -- нумерация в С идет с 0, а не с 1, поэтому ты смотришь не на ту строчку.

как это сделать

при помощи free

[a.n.o.n.i.m]

http://s50.radikal.ru/i130/1105/c4/43de22a9da89.jpg пишет 7 строку хотя там нет одинаковых элементов

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

[Mandrivnyk]

http://s50.radikal.ru/i130/1105/c4/43de22a9da89.jpg пишет 7 строку хотя там нет одинаковых элементов

7-я строчка -- это для нас 8-я.
В С нумерация элементов массива идет с 0.
Короче, печатай не max_index, а max_index + 1

[a.n.o.n.i.m]

Короче, печатай не max_index, а max_index + 1

printf(" \nNomer stroki s samoi dlinnoi seriei odinakovih elementov:  %d",max_index+1);

Правильно??

Дак он теперь если ноль будет печатать первую строку,как раньше была проблема с нулем и в первой не будет одинаковых чисел.Корочче надо сделать так чтобы если номер строки 0 то программа пишет "Serii odinacovix elemenrov net".И еще в первой строке выходит если появятся два одинаковых числа или их не будет всеравно ответ один будет писать первую строку.Надо решить проблему с 1 строкой

Если действительно в первой строке есть одинаковые элементы то пишет "Nomer stroki s samoi dlinnoi seriei odinakovih elementov:1" а если нет одинаковых элементов то не пишет "Nomer stroki s samoi dlinnoi seriei odinakovih elementov:1" а пишет "Serii odinacovix elemenrov net" как это сделать?

[Mandrivnyk]

#include <stdio.h>
#include <stdlib.h>
#include <time.h>

using namespace std;

void fillArray(int **Array, int rows, int columns);
void showArray(int **Array, int rows, int columns);


int main()
{
    int rows, columns;
    printf("Введите количество строк:\n");
    scanf("%d", &rows);
    printf("Введите количество столбцов:\n");
    scanf("%d", &columns);

    int max_index = 0;
    int max, count, max_count, tmp;

    // Объявление массива с выделением памяти

    int **Array = (int**) malloc(rows * sizeof(int));
    for (int i = 0; i < rows; i++)
        Array[i] = (int*) malloc(columns * sizeof(int));

    fillArray(Array, rows, columns);
    showArray(Array, rows, columns);

    max_count = 1;
    max = Array[0][0];
    for (int i = 0; i < rows; i++)
    {
        count = 1;
        for (int j = 0; j < columns; j++)
        {
            if (max < Array[i][j])
                max = Array[i][j];
            if (j >= 1)
            {
                if (Array[i][j] == Array[i][j-1])
                    count++;
                else
                {
                    if (max_count < count)
                    {
                        max_count = count;
                        max_index = i;
                    }
                    count = 1;
                }
            }
        }
    }
    printf("Максимальное число -- %d\n", max);
    if (max_count > 1)
        printf("Максимальная последовательность из %d одинаковых чисел находится в строке %d\n", max_count, max_index + 1);
    else
        printf("Последовательности из одинаковых чисел нет\n");
                


    // Освобождение памяти
    for(int i = 0; i < rows; i++)
        free (Array[i]);
    free (Array);


    return 0;
}

void fillArray(int **Array, int rows, int columns)
{
    srand(time(NULL));
    for (int i = 0; i < rows; i++)
        for (int j = 0; j < columns; j++)
            Array[i][j] = rand() % 3000;
}

void showArray(int **Array, int rows, int columns)
{
    printf("\nПолученная матрица:\n");
    for (int i = 0; i < rows; i++)
    {
        for (int j = 0; j < columns; j++)
            printf("%d\t", Array[i][j]);
        printf("\n");
    }
    printf("\n");
}

ЗЫ Благодарности принимаются на R269634919062 -)

[a.n.o.n.i.m]

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

Mandrivnyk все уже голова не варит,завтро попробую доделать

[Mandrivnyk]

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

Странно...
У меня все выводит