Programm funktioniert nur beim Debugen

Enitin

Also noergel... Du hast einen Fehler gemacht nicht ich. Lies bitte noch eimal genau. Wie oft soll ich es eigentlich noch sagen? Ich habe gesagt das es im Debug und Release Modus funktioniert aber nicht von Ordner aus. Und ich bin mir zu 100% sicher das es dieser Code ist weil ich ihn kopiert habe und noch zur sicherheit durchgelesen habe. Und pumuckel: Ich habe schon gesagt das der Code schlecht ist und wenn ich jetzt noch etwas verändere würde das nur noch verwirren. Soll ich ein Video machen und es auf YouTube stellen? Ihr könnt euch selber überzeugen wenn ihr wollt.

daddy_felix

Ich tippe mal, dass mit dem goto da was nicht stimmt. Warum willst du das denn partout nicht rausnehmen?

Du bist ja ganz schön beratungsresistent. Über das goto wurde ja schon genug gesagt. Aber ich verstehe die Zeile "system(("ping " + IP).c_str());" nicht. Was soll die schließende Klammer nach IP?
Gruß

Enitin

Hab ich ja schon versucht aber es geht nicht... Macht mir doch jetzt bitte mal ein Codebeispiel. Dann würde ich bemerken was ich falsch gemacht habe.

daddy_felix

Enitin schrieb:

Hab ich ja schon versucht aber es geht nicht... Macht mir doch jetzt bitte mal ein Codebeispiel. Dann würde ich bemerken was ich falsch gemacht habe.

was hast du schon versucht und was geht dann nicht? Zeig uns doch mal deinen Versuch.

Enitin

Wieso sollten die Klammern nicht stimmen? Oder warum sollte ich sie nicht brauchen?

Enitin

Und jaaa ich weiß das dieser Code auch schlecht geschrieben ist...

#include <iostream>
#include <windows.h>
#include <string>
using namespace std;

int main()
{
	string IP;
	int A = 0;

	cout << "Examples:" << endl;
	cout << "Link = enitin.co.de" << endl;
	cout << "IP = 127.0.0.1\n" << endl;
	cout << "Link/IP: ";
	cin >> IP;

	while(A==0)
	{
	system("cls");
	cout << "Testing ping..." << endl;
	system(("ping " + IP).c_str());
	cout << "\nPing will be tested again in 5 seconds for " << IP << "." << endl;
	Sleep(5000); //Warte 5 Sekunden
	}
}

#include <iostream>
#include <windows.h>
#include <string>
using namespace std;

int main()
{
	string IP;
	cout << "Examples:" << endl;
	cout << "Link = enitin.co.de" << endl;
	cout << "IP = 127.0.0.1\n" << endl;
	cout << "Link/IP: ";
	cin >> IP;

	for(;;)
	{
		system("cls");
		cout << "Testing ping..." << endl;
		system(("ping " + IP).c_str());
		cout << "\nPing will be tested again in 5 seconds for " << IP << "." << endl;
		Sleep(5000);
	}
}

knivil

Jaja, oder while(true) ...

Ich habe dein Orginalprogramm kopiert, es sowohl im Debug und Release-Mode uebersetzt und aus der IDE gestartet. Es funktioniert. Ich habe aber 127.0.0.1 eingegeben. Klicke ich auf das Programm im Ordner und gebe ebenfalls 127.0.0.1 ein, so funktioniert es genauso. Ich stelle keinen Unterschied fest.

PS: Falls das ein Versuch ist, IP-Adressen zu sammeln. Die nuetzen nur bedingt was.

Enitin

Hier habt ihr jetzt das Video...
https://www.youtube.com/watch?v=DicJFL4P5Ss&feature=youtu.be

Enitin

Hahaha. Nein das soll kein Programm werden um IP-Adressen zu sammeln.

Versuch duch einfach mal das:
1. Clean Solution
2. Rebuild Solution

Enitin

Könntest du mir auch sagen was Clean- und Rebuild Solution ist? Wie gesagt ich bin ein totaler Anfänger.

pumuckl

Enitin schrieb:

Ich habe schon gesagt das der Code schlecht ist und wenn ich jetzt noch etwas verändere würde das nur noch verwirren.

Noch viel schlimmer: Wenn du jetzt was veränderst könnte besserer Code bei rauskommen und gleichzeitig der Fehler verschwinden. Das wäre die reinste Katastrophe, nicht zu wissen, warum der Schrottcode, bei dem sich allen die Fußnägel hochrollen, unter bestimmten Umständen nicht funktioniert hat.

Enitin schrieb:

Könntest du mir auch sagen was Clean- und Rebuild Solution ist? Wie gesagt ich bin ein totaler Anfänger.

Links oben in Visual Studio steht Projektmappe Ping .
1. Rechtsklick ---> Clean Solution.
2. Visual Studio schließen und neu starten.
3. Rechtsklick ---> Rebuild Solution

Und das jeweils für Debug- und Release-Modus.

Enitin

Nein geht leider immer noch nicht... Das ist doch komisch oder? Bei allen geht es nur bei mir nicht?

_matze

Auf deutsch: Projekt bereinigen und Projekt neu erstellen.

Enitin schrieb:

Nein geht leider immer noch nicht... Das ist doch komisch oder? Bei allen geht es nur bei mir nicht?

Die Ping.exe die die du da öffnest, was hat die Datei für ein Änderungsdatum ?

daddy_felix

Mach mal eine Änderung in deinem Code, gib bspw. mal einen anderen Text aus.

Wird der geänderte Text auch wirklich angezeigt?

Skym0sh0

ich denke auch, dass die exe die er ausführt eine falsche ist, und nicht die aktuelle...