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

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

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

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

C++

#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++

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++

#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++

#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++

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++

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

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

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

@oleg-m1973 6770 / 4564 / 1843 Регистрация: 07.05.2019 Сообщений: 13,726
	21.02.2021, 23:38	12
	Сообщение от maytreya if (m%x==0 && n%x==0) { return x; } return delitel (m,n,x-1); } return 0; } Твоя проблема не в том, что ты коряво пишешь, а в том, что не слушаешь, то что тебе говорят. Корявый код - это следствие. 0

@oleg-m1973 6770 / 4564 / 1843 Регистрация: 07.05.2019 Сообщений: 13,726
	21.02.2021, 23:40	14
	Жирным пометил, на случай, если ты цвета не различаешь Добавлено через 56 секунд Сообщение от maytreya 78 строчка return delitel (m,n,x-1) сделал? 0

Байт Диссидент 27709 / 17325 / 3811 Регистрация: 24.12.2010 Сообщений: 38,979
	21.02.2021, 11:50	2
	Сообщение от 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 минуты На работу программы влияния оказывать не должно. Хотя надо стремиться к "чистой" трансляции, без варнингов. 1

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

Байт Диссидент 27709 / 17325 / 3811 Регистрация: 24.12.2010 Сообщений: 38,979
	21.02.2021, 15:18	5
	maytreya, все мои замечания касается только твоего Сообщение от maytreya cpp:72:1: warning: control reaches end of non-void function [-Wreturn-type] Про ответ ничего не знаю, это отдельно смотреть... 0

@oleg-m1973 6770 / 4564 / 1843 Регистрация: 07.05.2019 Сообщений: 13,726
	21.02.2021, 23:29	7
	Сообщение от 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; } 0

@maytreya 0 / 0 / 0 Регистрация: 05.02.2021 Сообщений: 28
	21.02.2021, 23:33 [ТС]	8
	Кстати, не рискнул выделить в отдельную тему. Подскажите где можно узнать о правильном написании кода, а то мне тут давеча намекнули, что мол пишу коряво))) Добавлено через 2 минуты Сообщение от oleg-m1973 Я ж тебе вроде показал, что нужно сделать Я так делал... в результате всегда 0(ноль) 0

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

@oleg-m1973 6770 / 4564 / 1843 Регистрация: 07.05.2019 Сообщений: 13,726
	21.02.2021, 23:34	10
	Сообщение от maytreya Я так делал... в результате всегда 0(ноль) Покажи, как делал 0

@maytreya 0 / 0 / 0 Регистрация: 05.02.2021 Сообщений: 28
	21.02.2021, 23:39 [ТС]	13
	Сообщение от oleg-m1973 Покажи, как делал 78 строчка 0

@maytreya 0 / 0 / 0 Регистрация: 05.02.2021 Сообщений: 28
	21.02.2021, 23:44 [ТС]	15
	Сообщение от oleg-m1973 return delitel (m,n,x-1) сделал? да... опять ноль Добавлено через 2 минуты вот когда убираю "return 0; " тогда считает правильно, но опять предупреждение 0

@oleg-m1973 6770 / 4564 / 1843 Регистрация: 07.05.2019 Сообщений: 13,726
	21.02.2021, 23:51	18
	Сообщение от maytreya Весь код не передаю только концовку return 0 в конце добавь и всё будет правильно. Добавлено через 23 секунды Сообщение от maytreya return 0; если на 13 строке, то ответ ноль При каких параметрах? Добавлено через 1 минуту Сделай return 1. Это когда общего делителя нет 1

@maytreya 0 / 0 / 0 Регистрация: 05.02.2021 Сообщений: 28
	22.02.2021, 00:00 [ТС]	19
	Сообщение от oleg-m1973 Сделай return 1. Это когда общего делителя нет ух ты... получилось. теперь бы понять почему....Спасибо огромное ... буду разбираться))) Добавлено через 4 минуты еще раз спасибо... все понял 0

@oleg-m1973 6770 / 4564 / 1843 Регистрация: 07.05.2019 Сообщений: 13,726
	22.02.2021, 00:06	20
	Сообщение от maytreya еще раз спасибо... все понял Для спасибо есть специальная кнопка 3