Программа компилируется, но не запускается

@TiredStars · Регистрация: 04.09.2015

Студворк — интернет-сервис помощи студентам

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
#include <iostream>
 
using namespace std;
 
int main()
{
    int x = 0, y = 0;
    cout<<"Set interval:"<<endl;
    cin>>x>>y;
    if (x<y) {
             for (x,y; x <= y; x++)
             cout<<x<<endl;
    }
    else {
         for (x,y; y <= x; y++)
         cout<<y<<endl;
    }      
    return 0;
}

Компилируется, но при попытке запустить ничего не происходит. Ткните в ошибку плз

@Kerry_Jr · 25.10.2015, 09:21

TiredStars, а вы числа пытались вводить?

@TiredStars · 25.10.2015, 09:44 **[ТС]**

я не могу ввести числа, ибо она тупо не запускается)

Добавлено через 18 минут
Разобрался, в чем была проблема, оказалось при компилировании указал не консольный проект, а оконный

Добавлено через 1 минуту
Кстати, есть какой-нибудь более простой способ определения меньшего числа из нескольких вводимых, чем тот который я использовал?

@Melg · 25.10.2015, 10:04

TiredStars, Приветствую, в Вашем случае, вам необходимо знание не только меньшего, но и большего значений. Поэтому имеет смысл использовать стандартный алгоритм std::minmax.
Ссылка на описание: http://en.cppreference.com/w/cpp/algorithm/minmax

Пример реализации:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
#include <iostream>
#include <algorithm>
 
using std::cout;
using std::cin;
using std::endl;
 
int main() {
  int x = 0, y = 0;
  cout << "Set interval:" << endl;
  cin >> x >> y;
  auto minmax = std::minmax(x, y);
 
  auto distance = minmax.second - minmax.first;
  for (int i = 0; i <= distance; ++i) {
    cout << minmax.first + i << endl;
  }
}

Пруф работоспособности
Для прямого задания интервала: http://ideone.com/olEOsp
Для инвертированного задания интервала: http://ideone.com/R6XvH2

Данный код я привел единым блоком в мейне, чтобы не пугать Вас чрезмерно. В общем случае, стоило разделить этот код на подфункции:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
#include <iostream>
#include <algorithm>
 
using std::cout;
using std::cin;
using std::endl;
 
void AskUserForInterval() { cout << "Set interval:" << endl; }
 
std::pair<int, int> GetIntervalFromUser() {
  AskUserForInterval();
  int x = 0;
  int y = 0;
  cin >> x >> y;
  return std::minmax(x, y);
}
 
void ShowInterval(const std::pair<int, int> &minmax) {
  auto distance = minmax.second - minmax.first;
  for (int i = 0; i <= distance; ++i) {
    cout << minmax.first + i << endl;
  }
}
 
int main() {
  auto interval = GetIntervalFromUser();
  ShowInterval(interval);
}

Пруф работоспособности:
http://ideone.com/ZTiSk5

Общие принципы:
Не используем using namespace std; - это плохая практика, подключает слишком много всего сразу, подключаем только то, что реально используется.
Не итерируемся по интервалу в не каноничной форме. Цикл for желательно начинать с нулевого значения и заканчивать максимальным. Мы итерируемся от 0 смещения до дистанции интервала включительно, а внутри цикла просто прибавляем к минимальному значению интервала текущее смещение. Такой код читается намного проще и не содержит заполнителей, как у Вас for(x,y;...
Разделяем любые задачи на подзадачи, подазадчи выносим в отдельные функции с говорящими именами - это обеспечит читаемость и более простую модификацию.

@nmcf · 25.10.2015, 10:05

Куда проще?

@Melg · 25.10.2015, 10:13

Кроме того, используем префиксный инкремент ++i в цикле - хороший тон, может пригодится на объектах итераторах и сэкономить время на копировании итераторов (которое происходит при постфиксном инкременте i++).
Каждую переменную объявляем отдельно, явно указывая тип переменной, поскольку объявления через запятую могут в конечном счете приводить в заблуждение и выполнять не те действия, которые ожидает программист. Плюс такой код подвержен меньшему количеству ошибок при внесении изменений.
Объявляем переменные как можно ближе к месту их использования. Это позволяет с одной стороны - выделять код в более маленькие функции, с другой стороны - улучшает читаемость, поскольку объявление переменной и её использование не разделены кучей кода, который на самом деле не использует данную переменную.

Добавлено через 4 минуты
Наконец в Вашем коде Вы дважды продублировали код выведения цикла на экран. Дублирующийся код дорог с точки зрения поддержки, внесения изменений и сопровождения. На самом деле вывод на экран диапазона значений однозначно определяется двумя значениями - минимальным и максимальным значениями диапазона. Если параметризировать функцию переменными содержащими эти значения то можно добиться от неё того же поведения, как в Вашем коде, без дублирования. (в нашем случае их объединяет в одну сущность тип

C++
1
std::pair<int,int> minmax

, где minmax.first - минимальное, а minmax.second - максимальное значения).
Забыл упомянуть, используется стандарт языка с++11. (В частности для удобства написания - тип некоторых переменных не записывается целиком, а объявлен auto и выводится компилятором автоматически на этапе компиляции из типа выражения стоявшего справа от знака присваивания)

@TiredStars · 25.10.2015, 10:20 **[ТС]**

Спасибо, даже не ожидал получить настолько информативный и развернутый ответ)

@zss · 25.10.2015, 10:51

Хоть это уже и не актуально, осмелюсь обратить внимание на

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

for (x,y; x <= y; x++)

Похоже Вы не понимаете синтаксиса for

for(Что_делать_перед_циклом;условие_выполнения_тела_цикла;что_делать_в_конце_цикла)

Обязательными являются только точки с запятыми(выделены красным).
А теперь попробуйте объяснить Ваше x,y , которое соответствует Что_делать_перед_циклом

@TiredStars · 25.10.2015, 11:17 **[ТС]**

Сначала думал что в первой части должны быть используемые в цикле переменные)
А как быть, если в первой части цикла for вообще не нужно ничего делать?)

@zss · 25.10.2015, 11:33

Ничего не писать

C++
1
for (; x <= y; x++)

В принципе можно даже

C++
1
for ( ; ; )

Это будет то же самое, что и

C++
1
while(true)

@TiredStars · 25.10.2015, 11:37 **[ТС]**

Ясно, спасибо

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами 8Observer8 20.02.2026 Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .	Конвертировать закладки radiotray-ng в m3u-плейлист damix 19.02.2026 Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .	Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI Eddy_Em 18.02.2026 Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .
И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.	«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»	SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 12.02.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 11.02.2026 Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .

@Kerry_Jr 3106 / 2591 / 1219 Регистрация: 14.05.2014 Сообщений: 7,236 Записей в блоге: 1
	25.10.2015, 09:21
	TiredStars, а вы числа пытались вводить? 0

@TiredStars 2 / 2 / 1 Регистрация: 04.09.2015 Сообщений: 76
	25.10.2015, 09:44 [ТС]
	я не могу ввести числа, ибо она тупо не запускается) Добавлено через 18 минут Разобрался, в чем была проблема, оказалось при компилировании указал не консольный проект, а оконный Добавлено через 1 минуту Кстати, есть какой-нибудь более простой способ определения меньшего числа из нескольких вводимых, чем тот который я использовал? 0

@nmcf 7804 / 6568 / 2988 Регистрация: 14.04.2014 Сообщений: 28,705
	25.10.2015, 10:05
	Куда проще? 1

@TiredStars 2 / 2 / 1 Регистрация: 04.09.2015 Сообщений: 76
	25.10.2015, 10:20 [ТС]
	Спасибо, даже не ожидал получить настолько информативный и развернутый ответ) 0

@TiredStars 2 / 2 / 1 Регистрация: 04.09.2015 Сообщений: 76
	25.10.2015, 11:17 [ТС]
	Сначала думал что в первой части должны быть используемые в цикле переменные) А как быть, если в первой части цикла for вообще не нужно ничего делать?) 0

@TiredStars 2 / 2 / 1 Регистрация: 04.09.2015 Сообщений: 76
	25.10.2015, 11:37 [ТС]
	Ясно, спасибо 0