Ошибка conversion error from string

asked May 30, 2011 at 1:14

The compiler cannot tell from that code if the function will ever reach the end and still return something. To make that clear, replace the last else if(...) with just else.

The compiler isn’t smart enough to know that <, >, and == are a «complete set». You can let it know that by removing the condition «if(val == sorted[mid])» — it’s redundant. Jut say «else return mid;«

If the function is non-void,it means it has to return something before reaching the end of function block[ _} ].So, when we give only if and else-if statements the compiler cannot tell from that code,that any of these statements will be evaluated to true and return something.Means, if all of the condition evaluates to false,then the control will reach the end of function,without returning something,which is wrong.

answered Mar 30, 2021 at 12:38

Always build with at least minimal optimization. With -O0, all analysis that the compiler could use to determine that execution cannot reach the end of the function has been disabled. This is why you’re seeing the warning. The only time you should ever use -O0 is for step-by-line debugging, which is usually not a good debugging approach anyway, but it’s what most people who got started with MSVC learned on…

I had the same problem. My code below didn’t work, but when I replaced the last «if» with «else», it works. The error was: may reach end of non-void function.

int shifted(char key_letter)
  {
        if(isupper(key_letter))
        {
            return key_letter - 'A'; 
        }

        if(islower(key_letter)   //<----------- doesn't work, replace with else

        {                                            


            return key_letter - 'a'; 
        }

  }

answered Jan 6, 2014 at 5:23

Make sure that your code is returning a value of given return-type irrespective of conditional statements

This code snippet was showing the same error

int search(char arr[], int start, int end, char value)
{
    int i;
    for(i=start; i<=end; i++)
    {
        if(arr[i] == value)
            return i;
    }
}

This is the working code after little changes

int search(char arr[], int start, int end, char value)
{
    int i;
    int index=-1;
    for(i=start; i<=end; i++)
    {
        if(arr[i] == value)
            index=i;
    }
    return index;
}

answered Oct 24, 2020 at 13:11

It means it’s searching for a function that needs to be completed.

else if(val == sorted[mid]) return mid;

so, remove the if() part and modify the code or add an else() at the end which returns an int.

answered Nov 7, 2020 at 10:22

Compiler by itself would not know that the conditions you have given are optimum .. meaning of this is you have covered all cases ..
So therefore it always wants a return statement … So either you can change last else if with else or just write return 0 after last else if ;`int binary(int val, int sorted[], int low, int high) {
int mid = (low+high)/2;

if(high < low)
    return -1;

if(val < sorted[mid])
    return binary(val, sorted, low, mid-1);

else if(val > sorted[mid])
    return binary(val, sorted, mid+1, high);

else if(val == sorted[mid])
    return mid;
return 0; }`

answered Feb 4, 2021 at 12:39

answered Apr 26, 2022 at 8:55

Add the binary function to the end of the code:

int binary(int val, int sorted[], int low, int high)
{ /* ... */ return 1;}

If not one of the conditions is not met, the int function should return int.

asked May 30, 2011 at 1:14

The compiler cannot tell from that code if the function will ever reach the end and still return something. To make that clear, replace the last else if(...) with just else.

The compiler isn’t smart enough to know that <, >, and == are a «complete set». You can let it know that by removing the condition «if(val == sorted[mid])» — it’s redundant. Jut say «else return mid;«

If the function is non-void,it means it has to return something before reaching the end of function block[ _} ].So, when we give only if and else-if statements the compiler cannot tell from that code,that any of these statements will be evaluated to true and return something.Means, if all of the condition evaluates to false,then the control will reach the end of function,without returning something,which is wrong.

answered Mar 30, 2021 at 12:38

Always build with at least minimal optimization. With -O0, all analysis that the compiler could use to determine that execution cannot reach the end of the function has been disabled. This is why you’re seeing the warning. The only time you should ever use -O0 is for step-by-line debugging, which is usually not a good debugging approach anyway, but it’s what most people who got started with MSVC learned on…

I had the same problem. My code below didn’t work, but when I replaced the last «if» with «else», it works. The error was: may reach end of non-void function.

int shifted(char key_letter)
  {
        if(isupper(key_letter))
        {
            return key_letter - 'A'; 
        }

        if(islower(key_letter)   //<----------- doesn't work, replace with else

        {                                            


            return key_letter - 'a'; 
        }

  }

answered Jan 6, 2014 at 5:23

Make sure that your code is returning a value of given return-type irrespective of conditional statements

This code snippet was showing the same error

int search(char arr[], int start, int end, char value)
{
    int i;
    for(i=start; i<=end; i++)
    {
        if(arr[i] == value)
            return i;
    }
}

This is the working code after little changes

int search(char arr[], int start, int end, char value)
{
    int i;
    int index=-1;
    for(i=start; i<=end; i++)
    {
        if(arr[i] == value)
            index=i;
    }
    return index;
}

answered Oct 24, 2020 at 13:11

It means it’s searching for a function that needs to be completed.

else if(val == sorted[mid]) return mid;

so, remove the if() part and modify the code or add an else() at the end which returns an int.

answered Nov 7, 2020 at 10:22

Compiler by itself would not know that the conditions you have given are optimum .. meaning of this is you have covered all cases ..
So therefore it always wants a return statement … So either you can change last else if with else or just write return 0 after last else if ;`int binary(int val, int sorted[], int low, int high) {
int mid = (low+high)/2;

if(high < low)
    return -1;

if(val < sorted[mid])
    return binary(val, sorted, low, mid-1);

else if(val > sorted[mid])
    return binary(val, sorted, mid+1, high);

else if(val == sorted[mid])
    return mid;
return 0; }`

answered Feb 4, 2021 at 12:39

answered Apr 26, 2022 at 8:55

Add the binary function to the end of the code:

int binary(int val, int sorted[], int low, int high)
{ /* ... */ return 1;}

If not one of the conditions is not met, the int function should return int.