Предупреждение при компиляции "control reaches end of non-void function"

@maytreya · Регистрация: 05.02.2021

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

Добрый день, коллеги. Продолжаю решать задачки с целью подготовки к экзамену. Задачу про НОД решил попробовать двумя способами. И если первый вариант я подсмотрел теорию за .. какой то там класс, то второй - это, так сказать, мысли мои и не самые удачные, видимо. Короче .... при запуске второго варианта VS выдает " cpp:72:1: warning: control reaches end of non-void function [-Wreturn-type]
72 | } ". Ответ при этом не соответствует действительности. Помогите разобраться и набраться опыта. Спасибо.

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
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
#include <iostream>
#include <cmath>
#include <ctime>
using namespace std;
 
//Задание 1   ***********************
//вариант 1
int nod(int n, int m);
//вариант 2
int delitel (int m, int n, int x, int i);
 
 
//mainmainmainmainmainmainmainmainmainmainmainmainmainmainmainmainmainmainmainmain
//mainmainmainmainmainmainmainmainmainmainmainmainmainmainmainmainmainmainmainmain
int main()
{
    
 
//*******************************************************
 
//Задание 1
 
  int a,b,k;
  int x=0;
  int i=0;
  cout<< "enter two numbers a and b: \n";
  cin>> a;
  cout<<endl;
  cin>> b;
 
  k=delitel(a,b,x,i);
 
        cout<<"the greatest divisor of the numbers a and b: ";
        cout<<k; 
        cout<<endl;
 
 
return 0;
}
//out_main_out_main_out_main_out_main_out_main_out_main_out_main_out_main_
//out_main_out_main_out_main_out_main_out_main_out_main_out_main_out_main_
 
//Задание 1   ***********************
 
int nod(int n, int m)
{
// Алгоритм Евклида
    if (n<m){
    swap(n,m);
    }
    if (m == 0) 
        return n; 
    else 
        return nod(m, n % m);   
}
 
 
//вариант 2
int delitel (int m, int n, int x, int i)
{
  // n=x+i; 
if (n>m)
swap (n,m);
if(x!=0)
{
  if (m%x==0 && n%x==0)
  {
  return x;
  }
delitel (m,n,x-1, i+1);
}
}

Добавлено через 3 минуты
Переменную i удалил не везде. Не обращайте внимание)).

Байт · 21.02.2021, 11:50

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

cpp:72:1: warning: control reaches end of non-void function [-Wreturn-type]

Ерунда. Предупреждение. Функция должна что-то возвращать. Поставь в строчку 71 return 0; Или
void delitel (int m, int n, int x, int i)

Добавлено через 2 минуты
На работу программы влияния оказывать не должно.
Хотя надо стремиться к "чистой" трансляции, без варнингов.

@maytreya · 21.02.2021, 13:10 **[ТС]**

.......... delitel (m,n,x-1, i+1);
}
return 0;
}
Вот так? но тогда ответ k=0 всегда.
Вообще то функция действительно работает, но ответ дает не верный. Например при вводе 1 и 2 ответ 6422240.

@oleg-m1973 · 21.02.2021, 13:46

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

Вот так? но тогда ответ k=0 всегда.

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
int delitel (int m, int n, int x, int i)
{
    // n=x+i; 
    if (n>m)
        swap (n,m);
 
    if(x!=0)
    {
        if (m%x==0 && n%x==0)
            return x;
 
 
        return delitel (m,n,x-1, i+1);
    }
    return 0;
}

Байт · 21.02.2021, 15:18

maytreya, все мои замечания касается только твоего

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

cpp:72:1: warning: control reaches end of non-void function [-Wreturn-type]

Про ответ ничего не знаю, это отдельно смотреть...

@maytreya · 21.02.2021, 23:27 **[ТС]**

Спасибо за подсказку. Нашел еще одну логическую нестыковку, попробую добить до рабочего состояния)).

Добавлено через 3 часа 5 минут
Еще раз спасибо всем кто откликнулся... Второй вариант стал "рабочим"... считает верно правда от варнинга не избавился. Конечно 1-й вариант гораздо изящней, но для тренировки 2-й то же не плох. )) Кому интересно вот код:

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
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
#include <iostream>
#include <cmath>
#include <ctime>
using namespace std;
 
//Задание 1   ***********************
//вариант 1
int nod(int n, int m);
//вариант 2
int delitel (int m, int n, int x);
 
 
//mainmainmainmainmainmainmainmainmainmainmainmainmainmainmainmainmainmainmainmain
//mainmainmainmainmainmainmainmainmainmainmainmainmainmainmainmainmainmainmainmain
int main()
{
    
 
//*******************************************************
 
//Задание 1
 
  int a,b,k,x;
  cout<< "enter two numbers a and b: \n";
  cin>> a;
  cout<<endl;
  cin>> b;
  //вариант 1
  //k=nod(a,b);
 
  //вариант 2
  if (b>a)
    x=a;
  else
    x=b; 
  k=delitel(a,b,x);
 
        cout<<"the greatest divisor of the numbers a and b: ";
        cout<<k; 
        cout<<endl;
 
 
return 0;
}
//out_main_out_main_out_main_out_main_out_main_out_main_out_main_out_main_
//out_main_out_main_out_main_out_main_out_main_out_main_out_main_out_main_
 
//Задание 1   ***********************
 //вариант 1
int nod(int n, int m)
{
// Алгоритм Евклида
    if (n<m){
    swap(n,m);
    }
    if (m == 0) 
        return n; 
    else 
        return nod(m, n % m);   
}
 
                      //считает верно, но постоянно выдает предупреждение. Не могу понять почему.
//вариант 2                    Untitled-1.cpp:71:1: warning: control reaches end of non-void function [-Wreturn-type]
                            // 71 | }
                            //    | ^
int delitel (int m, int n, int x)
{
if (n>m)
swap (n,m);
if(x!=0)
{
  if (m%x==0 && n%x==0)
  {
  return x;
  }
delitel (m,n,x-1);
}
}

@oleg-m1973 · 21.02.2021, 23:29

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

//считает верно, но постоянно выдает предупреждение. Не могу понять почему.
//вариант 2 Untitled-1.cpp:71:1: warning: control reaches end of non-void function [-Wreturn-type]
// 71 | }
// | ^

Я ж тебе вроде показал, что нужно сделать

Сообщение от oleg-m1973

int delitel (int m, int n, int x, int i)
{
// n=x+i;
if (n>m)
swap (n,m);
if(x!=0)
{
if (m%x==0 && n%x==0)
return x;
return delitel (m,n,x-1, i+1);
}
return 0;
}

@maytreya · 21.02.2021, 23:33 **[ТС]**

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

Добавлено через 2 минуты

Сообщение от oleg-m1973

Я ж тебе вроде показал, что нужно сделать

Я так делал... в результате всегда 0(ноль)

@S_el · 21.02.2021, 23:33

Больше пишите и больше читайте, вот и весь секрет.

@oleg-m1973 · 21.02.2021, 23:34

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

Я так делал... в результате всегда 0(ноль)

Покажи, как делал

@maytreya · 21.02.2021, 23:36 **[ТС]**

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
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
#include <iostream>
#include <cmath>
#include <ctime>
using namespace std;
 
//Задание 1   ***********************
//вариант 1
int nod(int n, int m);
//вариант 2
int delitel (int m, int n, int x);
 
 
//mainmainmainmainmainmainmainmainmainmainmainmainmainmainmainmainmainmainmainmain
//mainmainmainmainmainmainmainmainmainmainmainmainmainmainmainmainmainmainmainmain
int main()
{
    
 
//*******************************************************
 
//Задание 1
 
  int a,b,k,x;
  cout<< "enter two numbers a and b: \n";
  cin>> a;
  cout<<endl;
  cin>> b;
  //вариант 1
  //k=nod(a,b);
 
  //вариант 2
  if (b>a)
    x=a;
  else
    x=b; 
  k=delitel(a,b,x);
 
        cout<<"the greatest divisor of the numbers a and b: ";
        cout<<k; 
        cout<<endl;
 
 
return 0;
}
//out_main_out_main_out_main_out_main_out_main_out_main_out_main_out_main_
//out_main_out_main_out_main_out_main_out_main_out_main_out_main_out_main_
 
//Задание 1   ***********************
 //вариант 1
int nod(int n, int m)
{
// Алгоритм Евклида
    if (n<m){
    swap(n,m);
    }
    if (m == 0) 
        return n; 
    else 
        return nod(m, n % m);   
}
 
                      //считает верно, но постоянно выдает предупреждение. Не могу понять почему.
//вариант 2                    Untitled-1.cpp:71:1: warning: control reaches end of non-void function [-Wreturn-type]
                            // 71 | }
                            //    | ^
int delitel (int m, int n, int x)
{
if (n>m)
swap (n,m);
if(x!=0)
{
  if (m%x==0 && n%x==0)
  {
  return x;
  }
delitel (m,n,x-1);
}
return 0;
}

@oleg-m1973 · 21.02.2021, 23:38

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

if (m%x==0 && n%x==0)
{
return x;
}
return delitel (m,n,x-1);
}
return 0;
}

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

@maytreya · 21.02.2021, 23:39 **[ТС]**

Сообщение от oleg-m1973

Покажи, как делал

78 строчка

@oleg-m1973 · 21.02.2021, 23:40

Жирным пометил, на случай, если ты цвета не различаешь

Добавлено через 56 секунд

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

78 строчка

return delitel (m,n,x-1) сделал?

@maytreya · 21.02.2021, 23:44 **[ТС]**

Сообщение от oleg-m1973

return delitel (m,n,x-1) сделал?

да... опять ноль

Добавлено через 2 минуты
вот когда убираю "return 0; " тогда считает правильно, но опять предупреждение

@oleg-m1973 · 21.02.2021, 23:46

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

да... опять ноль

Да ладно

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
int delitel(int m, int n, int x)
{
    if (n>m)
        std::swap(n,m);
    
    if(x!=0)
    {
        if (m%x==0 && n%x==0)
        {
            return x;
        }
        return delitel(m,n,x-1);
    }
    return 0;
}
int main()
{
    std::cout << delitel(10, 20, 2) << std::endl;
}

Добавлено через 1 минуту

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

вот когда убираю "return 0; " тогда считает правильно, но опять предупреждение

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

@maytreya · 21.02.2021, 23:49 **[ТС]**

Весь код не передаю только концовку

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
int delitel (int m, int n, int x)
{
if (n>m)
swap (n,m);
if(x!=0)
{
  if (m%x==0 && n%x==0)
  {
  return x;
  }
return delitel (m,n,x-1);
}
 
}

Добавлено через 57 секунд
return 0; если на 13 строке, то ответ ноль

@oleg-m1973 · 21.02.2021, 23:51

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

Весь код не передаю только концовку

return 0 в конце добавь и всё будет правильно.

Добавлено через 23 секунды

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

return 0; если на 13 строке, то ответ ноль

При каких параметрах?

Добавлено через 1 минуту
Сделай return 1. Это когда общего делителя нет

@maytreya · 22.02.2021, 00:00 **[ТС]**

Сообщение от oleg-m1973

Сделай return 1. Это когда общего делителя нет

ух ты... получилось. теперь бы понять почему....Спасибо огромное ... буду разбираться)))

Добавлено через 4 минуты
еще раз спасибо... все понял

@oleg-m1973 · 22.02.2021, 00:06

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

еще раз спасибо... все понял

Для спасибо есть специальная кнопка

Новые блоги и статьи Все статьи Все блоги /
PhpStorm 2025.3: WSL Terminal всегда стартует в ~ and_y87 14.12.2025 PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .	Access VikBal 11.12.2025 Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.	Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro	Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .	От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .
Поиск "дружественных имён" СОМ портов Argus19 22.11.2025 Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .	Сколько Государство потратило денег на меня, обеспечивая инсулином. Programma_Boinc 20.11.2025 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .	Ломающие изменения в C#.NStar Alpha Etyuhibosecyu 20.11.2025 Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .	Мысли в слух kumehtar 18.11.2025 Кстати, совсем недавно имел разговор на тему медитаций с людьми. И обнаружил, что они вообще не понимают что такое медитация и зачем она нужна. Самые базовые вещи. Для них это - когда просто люди. . .	Создание Single Page Application на фреймах krapotkin 16.11.2025 Статья исключительно для начинающих. Подходы оригинальностью не блещут. В век Веб все очень привыкли к дизайну Single-Page-Application . Быстренько разберем подход "на фреймах". Мы делаем одну. . .

@maytreya 0 / 0 / 0 Регистрация: 05.02.2021 Сообщений: 28
	21.02.2021, 13:10 [ТС]
	.......... delitel (m,n,x-1, i+1); } return 0; } Вот так? но тогда ответ k=0 всегда. Вообще то функция действительно работает, но ответ дает не верный. Например при вводе 1 и 2 ответ 6422240. 0

@S_el 2444 / 1842 / 406 Регистрация: 15.12.2013 Сообщений: 8,243
	21.02.2021, 23:33
	Больше пишите и больше читайте, вот и весь секрет. 0