Поиск количества отрицательных чисел кратно семи

[ruivit]

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

   #include <iostream>



// Эта функция работает нормуль

int vol(int as){ 
	int a;
	if(as < 0){
		if(as % 7 == 0){
		a = 1;	
			
		}	
		else{
			
		 a = 0;
		}
		
	} else
	{
		a = 0;
	}
	
	return a;
}



int main(int argc, char** argv) {
	
	
	int n, j, t;
	
	// ru = 0; // какаята ерунда....
	int ru;
	
	int *a;
		
	std :: cin >> n;
		
	// создаём список
	for(int i =0; n > i; i++){
		
	std :: cin >> a[i];		
	
	}

	// ищем в списке числа кратные семи
	for(int i =0; n > i; i++){
		
	t = a[i];
		
	ru = ru + vol(t); // тут тоже какая та ерунда получается. 
	}
	
	std :: cout << ru;	
	return 0;
}

Вообще как то странно с переменой типа int всё происходит я с таким в первой сталкиваюсь. Вроде как создал переменную потом тут же ей присваевываешь значения и воля всё перестаёт работать.

int a = 0;

[p51x]

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

но вот при поиске в массиве возникли проблемы

У вас нет поиска в массиве по заданию

int *a;

Память выделять кто будет?

// создаём список

Серьезно? Вы книжку/лекции открывали?

std :: cin >> a[i];

Куда вы пишите?

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

Вообще как то странно с переменой типа int всё происходит я с таким в первой сталкиваюсь.

И ни такое увидите, если не будете выделять память и будет писать куда попало.

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

После этого
память освободить...

Где?

[ruivit]

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

У вас нет поиска в массиве по заданию

int *a;

Память выделять кто будет?

// создаём список

Короче исправил, в общим ерунду сморозил. Всё за работало, сейчас буду пробовать отправлять на сайт.

#include <iostream>



// Эта функция работает нормуль

int vo(int as){ 
	int a;
	if(as < 0){
		if(as % 7 == 0){
		a = 1;	
			
		}	
		else{
			
		 a = 0;
		}
		
	} else
	{
		a = 0;
	}
	
	return a;
}



int main(int argc, char** argv) {
	
	
	int a[1000];
	int n, r =0;
	
	std :: cin >> n;
	
	for(int i = 0; n > i; i++){
		
	std :: cin >> a[i];		
		
	}

	for(int i = 0; n > i; i++){
		
	if(vo(a[i]) == 1){
		r++;
		
	}
					
	}
	
	std :: cout << r;
	
	delete a;
	
	return 0;
}

Серьезно? Вы книжку/лекции открывали?

std :: cin >> a[i];

Куда вы пишите?


И ни такое увидите, если не будете выделять память и будет писать куда попало.


Где?

Короче за ново переписал код, всё за работало. Буду пробовать отправлять на сайт.

 #include <iostream>



// Эта функция работает нормуль

int vo(int as){ 
	int a;
	if(as < 0){
		if(as % 7 == 0){
		a = 1;	
			
		}	
		else{
			
		 a = 0;
		}
		
	} else
	{
		a = 0;
	}
	
	return a;
}



int main(int argc, char** argv) {
	
	
	int a[1000];
	int n, r =0;
	
	std :: cin >> n;
	
	for(int i = 0; n > i; i++){
		
	std :: cin >> a[i];		
		
	}

	for(int i = 0; n > i; i++){
		
	if(vo(a[i]) == 1){
		r++;
		
	}
					
	}
	
	std :: cout << r;
	
	delete a;
	
	return 0;
}

[ruivit]

Прога работает. Теперь новая напасть нейросеть сайта https://olympic.nsu.ru/ не хотит принимать программу вроде как она не проходит третий этап. Вот код ее

#include <iostream>



// Эта функция работает нормуль

int vo(int as){ 
	int a;
	if(as < 0){
		if(as % 7 == 0){
		a = 1;	
			
		}	
		else{
			
		 a = 0;
		}
		
	} else
	{
		a = 0;
	}
	
	return a;
}



int main(int argc, char** argv) {
	
	// объявить массив указателей
	int *a;	
	a = new int[1000]; // выделить память для 1000 элементов
	
	int n, r =0;
	
		
	std :: cin >> n;
		
	for(int i = 0; n > i; i++){		
	std :: cin >> a[i];	// заполняю массив	
	if(vo(a[i]) == 1){ // ищу отрицательное кратное семи 
		r++; // если оно есть добовляю еденицу		
	}					
	}

	
	std :: cout << r; // вывожу на экран количество отрицательных кратных семи чисел 
	
	delete[] a; // освобождаю память.


	
	return 0;
}

[BDA]

Раз запрашиваете n, то и выделяйте n:

std :: cin >> n;
a = new int[n];

Только массив это не односвязный список. Пример - первая картинка в статье https://neerc.ifmo.ru/wiki/index.php...81%D0%BE%D0%BA.

[ruivit]

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

Раз запрашиваете n, то и выделяйте n:

std :: cin >> n;
a = new int[n];

Только массив это не односвязный список. Пример - первая картинка в статье https://neerc.ifmo.ru/wiki/index.php...81%D0%BE%D0%BA.

Вот своял новую программку с односвязный списком снова не прошёл

 #include <iostream>
#include <forward_list>
/* run this program using the console pauser or add your own getch, system("pause") or input loop */


// Эта функция работает нормуль

int vo(int as){ 
	int a;
	if(as < 0){
		if(as % 7 == 0){
		a = 1;	
			
		}	
		else{
			
		 a = 0;
		}
		
	} else
	{
		a = 0;
	}
	
	return a;
}



int main(int argc, char** argv) {
	


int r,s, ro;

std :: cin >> r; // длина списка
	
std::forward_list<int> arr(r);
// Заполняем список значениями
int n = 1;
for (int &el : arr) {
	
std :: cin >> s;	
   el = s; // заносим значения в список
}


// Выводим значения
for (int &el : arr) {
 
 	if(vo(el) == 1){  // ищем отрицательные числа

	// ищу отрицательное кратное семи 
		ro++; // если оно есть добовляю еденицу		
	}					
 
}



std::cout << ro; // 1 2 3	
	
	return 0;
}

[BDA]

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

снова не прошёл

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

struct list {
    int val;
    struct list *next;
};

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

[Пётр Седов]

ruivit, если не заморачиваться на exception safety, можно так:

#include <iostream>

using namespace std;

int main() {
  // односвязный список
  struct item_t {
    item_t* next;
    int value;
  };
  item_t* items_first = nullptr;

  int count;
  cin >> count;
  for (int k = 0; k < count; k++) {
    item_t* i = new item_t;
    cin >> i->value;
    // добавляем *i в начало списка
    i->next = items_first;
    items_first = i;
  }

  int biz_count = 0;
  // перебираем все элементы списка
  for (const item_t* i = items_first; i != nullptr; i = i->next) {
    if ((i->value < 0) && (i->value % 7 == 0)) {
      biz_count++;
    }
  }
  cout << biz_count << endl;

  // освобождаем список
  while (items_first != nullptr) {
    item_t* i = items_first;
    items_first = i->next;
    delete i;
  }
  return 0;
}

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

Теперь новая напасть нейросеть сайта https://olympic.nsu.ru/ не хотит принимать программу вроде как она не проходит третий этап.

Зачем вам эта хрень?

[Lapita12]

#include <iostream>

int vol(int as) {
    int a;
    if (as < 0) {
        if (as % 7 == 0) {
            a = 1;
        }
        else {
            a = 0;
        }
    }
    else {
        a = 0;
    }

    return a;
}

int main(int argc, char** argv) {
    int n, j, t;
    int ru = 0;
    int* a;

    std::cin >> n;

    // Выделяем память для массива
    a = new int[n];

    // Создаем список
    for (int i = 0; i < n; i++) {
        std::cin >> a[i];
    }

    // Итерация по списку и подсчет количества отрицательных целых чисел, кратных 7
    for (int i = 0; i < n; i++) {
        t = a[i];
        ru += vol(t);
    }

    std::cout << ru << std::endl;

    // Освободить выделенную память
    delete[] a;

    return 0;
}