Wird es zu Speicherlecks kommen?

Question

Ich habe ein Programm, in dem ich aus einer großen Reihe von geschachtelten For-Schleifen ausbrechen muss. Bis jetzt haben die meisten Leute mir gesagt, dass ich es tun soll, einen hässlichen Goto in meinem Code zu verwenden.

Nun, wenn ich eine Reihe von lokalen Stack erstellen (ich denke, dass sie so genannt werden, wenn nicht, meine ich nur normale Variablen ohne den neuen Befehl) Variablen in meinen Schleifen und mein Programm trifft diese Aussage triggert den goto, werde ich auf ein Speicherleck stoßen, weil mein Programm viele Schleifen nicht ordnungsgemäß beendet und die lokalen Variablen nicht aufräumt?

Answer 1

Nein, Sie werden nicht einen Speicherverlust verursachen . Die Verwendung eines goto führt nicht dazu, dass "Schleifen nicht ordnungsgemäß beendet werden". Es ist nur nicht allgemein von einem Code-Struktur-Point-of-View empfohlen.

Abgesehen davon, wenn Sie die Schleife verlassen, werden die lokalen Variablen außerhalb des Geltungsbereichs und von den Stapel (d. H. Aufgeräumt) in den Prozess.

Answer 2

Nein, werden Sie nicht.

Stellen Sie jedoch sicher, dass alle externen Ressourcen ordnungsgemäß freigegeben sind. Wenn Sie beispielsweise eine Datei öffnen, können Sie an der Stelle vorbeispringen, an der sie normalerweise geschlossen wäre.

Answer 3

Nein. Lokale Variablen müssen nicht einzeln bereinigt werden. Wenn der Stapel auftaucht, werden alle lokalen Variablen mit ihm verschwinden.

Answer 4

Nein. Sie können nur Speicher verlieren, der dynamisch zugewiesen wird.

Answer 5

Stack-Variablen werden in dem Moment definiert (und zugeordnet), in dem Sie die Funktion eingeben, und werden implizit eliminiert, sobald Sie die Funktion verlassen (da der gesamte Call-Stack-Datensatz entfernt wird). Kein Aufprallen innerhalb der Funktion kann möglicherweise Chaos mit Speicher verursachen, der die ganze Zeit zugewiesen wurde. Unabhängig davon, welchen Ausführungspfad Sie durch den Code verwenden, wird der Stack-Datensatz angezeigt, wenn die Steuerung zur aufrufenden Funktion zurückkehrt und der Speicher freigegeben wird.

Answer 6

Goto ist nicht immer schlecht, aber in Ihrem Fall sollten Sie wahrscheinlich nicht goto verwenden.

Siehe Beispiele für eine gute Verwendung von goto here und here.

Wenn Sie eine Beschriftung außerhalb des Bereichs finden, wird Ihr Objekt auf dem Stapel freigegeben.

Beispiel:

#include <iostream> 
using namespace std; 

class A 
{ 
public: 
    ~A() 
    { 
    cout<<"A destructor"<<endl; 
    } 
}; 



int main(int argc, char**argv) 
{ 
    { 
    A a; 
    cout<<"Inside scope"<<endl; 
    goto l; 
    cout<<"After l goto"<<endl; 
    } 

    cout<<"Outside of scope before l label"<<endl; 

l: 
    cout<<"After l label"<<endl; 
    return 0; 
} 

Dies drucken:

Innenumfang
A destructor
Nach l Label

Answer 7

Nein, automatische Variablen in Ihren Schleifen führen nicht zu Programmierverlusten, wenn Sie mit einer goto-Anweisung aus Ihren Schleifen ausbrechen.

Answer 8

Die anderen Antworten sind wahr .... aber, wenn Sie Loops anders verschachteln müssen, würde ich das Design in Frage stellen, das sie dorthin brachte. Diese Logik in separate Funktionen aufzuteilen wäre ein besserer Weg, um ein solches Problem zu lösen.

billy3

Answer 9

Stack-Variablen (Autos, nicht autobots) sind nicht "leaky" wie Variablen über neue zugewiesen() oder malloc().

Soweit die "Hässlichkeit" von Goto ist das nur dogmatisch. Lies Knuth, er war genauso brillant wie Dijkstra. http://pplab.snu.ac.kr/courses/adv_pl05/papers/p261-knuth.pdf Vermeiden Sie Programmierung auf Pasta-Basis, aber sorgfältiger Gebrauch wird nicht zu Spaghetti.

Dijkstra mochte sie nicht, WEIL das meiste von dem, was man mit gotos machen kann, mit anderen strukturierten Programmiertechniken gemacht werden kann und weniger Code verwendet, wodurch die andere Struktur weniger fehleranfällig wird.

Verstehen Sie, dass gotos nicht Ihre erste Lösung sein sollte, und gehen Sie nicht aus dem Weg, sie zu benutzen, aber wenn es Sinn macht, unterwerfen Sie sich nicht dogmatischen Leng Mobs. Die break-Anweisung ist ein getto getto für Fälle, in denen striktes Anhalten an das Gebot "Du sollst nicht gotos verwenden" keinen Sinn ergab.

Answer 10

Wird rückwärts zu Leckressourcen gehen? Oder irgendwelche anderen möglichen Probleme mit unterem Code?

erneut auszuführen:

 try 
     { 
      //Setup request 
      HttpWebRequest request = (HttpWebRequest)WebRequest.Create(url); 

      .... 

      //Get Response 
      HttpWebResponse response = (HttpWebResponse)request.GetResponse(); 

      if(response != HttpStatus.OK && noOfRetries < 3) 
      { 
       noOfRetries++; 
       Thread.Sleep(10 * 1000); 


       response.Close(); 
       goto Reexecute; 
      } 
      ... 

      response.Close(); 
     } 
     catch 
     { 

     }