С++ Почему не работает counter?

Question

[MrTomMus]

Изучаю с нуля С++. Придумал для себя таблицу умножения с помощью switch, for и if.

Цель была что бы высвечивалось два варианта. 1 Число пользователя. 2. Вся таблица умножения.
Второй вариант хотел бы сделать с пробелами между каждой 10 строкой с помощью if. Но counter отказывается делать пропуск. Знаю что можно сделать пропуск строки с помощью cout << endl; Почему counter не работает?

#include <iostream>

using namespace std;

int main() {

	int a = 1, b = 2, c = 3, d = 4, e = 5, f = 6, g = 7, h = 8, i = 9, j = 10;
	int my; // Число пользователя
	char x; // Выбор для switch
	int y; // Для for
	int counter;
	counter = 0;

	setlocale(LC_ALL, "Rus");

	cout << "Это таблица умножения." << endl;
	cout << "Выберите условие." << endl;
	cout << "1. Умножить свое число." << endl;
	cout << "2. Показать таблицу умножения." << endl;

	cin >> x;

	switch (x) {
	case '1':
		cout << "Введите Ваше число" << endl;
		cin >> my;
		cout << "\n" << endl;
		for (y = 0; y <= 10; y++)
			cout << y << " * " << my << " = " << y * my << endl;
	
		break;
	case '2': {
		
			for (y = 0; y <= 10; y++) 
				cout << a << " * " << y << " = " << a * y << endl;
			for (y = 0; y <= 10; y++)
				cout << b << " * " << y << " = " << b * y << endl;
			for (y = 0; y <= 10; y++)
				cout << c << " * " << y << " = " << c * y << endl;
			for (y = 0; y <= 10; y++)
				cout << d << " * " << y << " = " << d * y << endl;
			for (y = 0; y <= 10; y++)
				cout << e << " * " << y << " = " << e * y << endl;
			for (y = 0; y <= 10; y++)
				cout << f << " * " << y << " = " << f * y << endl;
			for (y = 0; y <= 10; y++)
				cout << g << " * " << y << " = " << g * y << endl;
			for (y = 0; y <= 10; y++)
				cout << h << " * " << y << " = " << h * y << endl;
			for (y = 0; y <= 10; y++) 
				cout << i << " * " << y << " = " << i * y << endl;
			for (y = 0; y <= 10; y++) 
				cout << j << " * " << y << " = " << j * y << endl;
				counter++;
			
				


				if (counter == 10){
					cout << "\n";
					counter = 0;
				}
				
		}
		break;
		
		


		
	
	default:
		cout << "Вы ввели неверное условие" << endl;

	}

	return 0;

}

Comments

[edward_freedom]

ну counter у тебя всегда будет 1

[Армянское Радио]

Сделайте меня развидеть этот быдлокод!
Вот это - ужасно:

int a = 1, b = 2, c = 3, d = 4, e = 5, f = 6, g = 7, h = 8, i = 9, j = 10;

Переменные должны назваться со смыслом.
Вот это - просто тошнотворно:

for (y = 0; y <= 10; y++)
cout << a << " * " << y << " = " << a * y << endl;
for (y = 0; y <= 10; y++)
cout << b << " * " << y << " = " << b * y << endl;
for (y = 0; y <= 10; y++)
cout << c << " * " << y << " = " << c * y << endl;
for (y = 0; y <= 10; y++)
cout << d << " * " << y << " = " << d * y << endl;
for (y = 0; y <= 10; y++)
cout << e << " * " << y << " = " << e * y << endl;
for (y = 0; y <= 10; y++)
cout << f << " * " << y << " = " << f * y << endl;
for (y = 0; y <= 10; y++)
cout << g << " * " << y << " = " << g * y << endl;
for (y = 0; y <= 10; y++)
cout << h << " * " << y << " = " << h * y << endl;
for (y = 0; y <= 10; y++)
cout << i << " * " << y << " = " << i * y << endl;
for (y = 0; y <= 10; y++)
cout << j << " * " << y << " = " << j * y << endl;
counter++;

Нафига накопипастили?!

[MrTomMus]

edward_freedom, С чем это связано?
Я присваиваю counter = 0
Затем в case 2 у меня идет 10 for - по 10 строк, хочу сделать между ними пробел. Что нужно сделать? И почему?

[edward_freedom]

MrTomMus, ты вызываешь итерацию в конце всех циклов 1 раз
counter++;

[MrTomMus]

edward_freedom, А как сделать что бы работало?

[edward_freedom]

MrTomMus, без фигурных скобок, for выполняет одну следующую строку

for (y = 0; y <= 10; y++) {
cout << j << " * " << y << " = " << j * y << endl;
counter++;
}

[MrTomMus]

edward_freedom,

break;
	case '2': {
		
		for (y = 0; y <= 10; y++) 
			{
				cout << a << " * " << y << " = " << a * y << endl;
				counter++;
			}
			for (y = 0; y <= 10; y++) {
				cout << b << " * " << y << " = " << b * y << endl;
				counter++;
			}
			for (y = 0; y <= 10; y++) {
				cout << c << " * " << y << " = " << c * y << endl;
				counter++;
			}
			for (y = 0; y <= 10; y++) {
				cout << d << " * " << y << " = " << d * y << endl;
				counter++;
			}
			for (y = 0; y <= 10; y++) {
				cout << e << " * " << y << " = " << e * y << endl;
				counter++;
			}
			for (y = 0; y <= 10; y++) {
				cout << f << " * " << y << " = " << f * y << endl;
				counter++;
			}
			for (y = 0; y <= 10; y++) {
				cout << g << " * " << y << " = " << g * y << endl;
				counter++;
			}
			for (y = 0; y <= 10; y++) {
				cout << h << " * " << y << " = " << h * y << endl;
				counter++;
			}
			for (y = 0; y <= 10; y++) {
				cout << i << " * " << y << " = " << i * y << endl;
				counter++;
			}
			for (y = 0; y <= 10; y++) {
				cout << j << " * " << y << " = " << j * y << endl;
				counter++;
			}


			if (counter == 11) 
				cout << "\n";
				counter = 0;
			}
		
				
		break;

Вот так сделал, верно? Не делает пропуск строки

[edward_freedom]

MrTomMus, проверь counter увеличивается или нет, если увеличивается, то тема решена

[edward_freedom]

MrTomMus, и у тебя это условие

if (counter == 11) 
        cout << "\n";
        counter = 0;
      }

стоит в самом низу, когда 10 циклов, прибавят к counter около 100 и 11 он никогда не будет

[MrTomMus]

edward_freedom, Да, я просто делал упражнение с \n и использовал переменную counter. Решил попробовать сделать свою программу простую с использованием счетчика строк и пропуском строки после каждой 10 строки. Как сделать что бы он делал пропуск строки? Нужно в каждом цикле for делать if?

[maaGames]

MrTomMus, тебе не нужен никакой counter, просто после каждого цикла добавь перевод строки.
И, пожалуйста, ставь фигурные скобки. Пок аты не умеешь программировать, тебе категорически нельзя "упрощать" код, удаляя необязательные структурные элементы. Ты сам себя путаешь и обманываешь. По отступу в строке counter++ очевидно, что ты себя обманул. Ставь фигурные скобочки. Всегда.

[MrTomMus]

maaGames, У меня было задание

#include <iostream>
using namespace std;

int main() {
	setlocale(0, "");
	int a;
	int counter;
	counter = 0;

	for (a = 1; a <= 100; a++) {
		cout << a << endl;
		counter++;


		if (counter == 2) {
			cout << "\n" << endl;
			counter = 0;
		}
	}

	return 0;

}

Я не хотел как то обманывать себя, просто хотел применить вот это задание в свою программу. То что можно было сделать банальный отступ через cout << endl; я понял. Вот со скобками что ты имеешь ввиду? Со скобками я путаюсь.

[maaGames]

MrTomMus,

// вот это  
    for (y = 0; y <= 10; y++) 
           cout << j << " * " << y << " = " << j * y << endl;
           counter++;

// работает вот так
      for (y = 0; y <= 10; y++) 
      {
              cout << j << " * " << y << " = " << j * y << endl;
      }
      counter++;

// а не вот так
     for (y = 0; y <= 10; y++) 
     {
          cout << j << " * " << y << " = " << j * y << endl;
          counter++;
     }

[MrTomMus]

maaGames, Понял, спасибо

Answer 1

[Артемий Кошар]

Создай для него тело:
for (y = 0; y <= 10; y++) {
cout << j << " * " << y << " = " << j * y << endl;
counter++;
}