Warum muss diese C++ Funktion nicht zurückgegeben werden?

Question

sah ich einige einen

int r, odd_divisors(int a, int b) { 
    r = sqrt(b) + 1; 
    r -= sqrt(a); 
} 

AFAIK diesen Code schreiben, wird der Compiler automatisch return 0; am Ende dieses Codes hinzufügen, aber in diesem Fall gibt es den Wert von r. Könnte mir bitte jemand helfen, zu erklären, warum das passiert. Vielen Dank.

UPDATE: Diese Funktion funktioniert tatsächlich in dieser Codefights-Site: https://codefights.com/challenge/eu4zLJDcv88B2mcCp. Sie können nach sir_ementaler Lösung suchen.

UPDATE 2: Danke für alle, die darauf hingewiesen, diese Funktion ist ill Format. Ich wusste, dass. Der Grund, warum ich hier gefragt habe, ist, dass es die Lösung des Gewinners auf der Seite ist, die ich im vorherigen Update erwähnt habe. Es sieht für mich so aus, dass Codefights dem Compiler ein anderes Feature hinzufügen müssen.

Answer 1

Die "implizite int" -Regel, die Sie möglicherweise aus Vor-Standard C kennen, gilt nicht für C++. Dies ist ein ungültiger Code.

Korrigieren Sie Ihren kaputten Code.

Answer 2

Der Code, den Sie gepostet haben, ist nicht legal C++, aber es ist sehr nah an legal. Die folgende ist legal:

// Example program 
#include <iostream> 
#include <string> 
#include <math.h> 

int r, odd_divisors(int a, int b); 
int odd_divisors(int a, int b) { 
    r = sqrt(b) + 1; 
    r -= sqrt(a); 
    return 0; 
} 

int main() 
{ 
    odd_divisors(16,25); 
    std::cout << "Hello, " << r; 
} 

Es gibt zwei Dinge hier los ist. Erstens können Sie ein int definieren und eine Funktion in derselben Anweisung deklarieren, aber Sie können die Funktion nicht definieren. Zulassen ist dies eine offensichtliche Compiler-Erweiterung.

Zweitens ist die Rückgabe eines Werts aus einer Funktion zur Rückgabe von Werten ein undefiniertes Verhalten. Dass kann einen Crash bedeuten - aber es kann auch bedeuten, genau das zu tun, was Sie erwarten. Wenn Sie nicht versuchen, den Wert aus der Funktion zu verwenden, ist es sehr wahrscheinlich (aber nicht sicher), dass nichts passiert.

Es ist natürlich viel sauberer zu machen odd_divisors eine Void-Funktion sein, oder besser noch, es den Wert zurückgeben und keine statische überhaupt verwenden.