Mikrowelle Praktikum

Th

Außerdem solltest du die Aufgabenstellung beachten:

Keine Methode der beiden Klassen liest irgendwelche Daten von Tastatur ein oder gibt etwas auf den Bildschirm aus. Dafür ist allein das Anwendungsprogramm zuständig.

Also kein "cout" in deinen Klassen-Methoden verwenden!!!

Bastel dir am besten eine Test-Klasse, welche die ganzen Methoden aufruft und dann die Werte über Tastatur einliest und die Rückgabewerte der Methoden am Bildschirm ausgibt.

ThaRealMatix

Okay ich hab nun mal folgende Änderungen:

FrozenMeal.h

#pragma once
#include <string>
#include "Microwave.h"
using namespace std;

class FrozenMeal
{
private:
	string name;
	int volume;
	double temperature;
	double coefficient;

public:
	FrozenMeal(string name, int volume);
	FrozenMeal(FrozenMeal&);
	~FrozenMeal();
	void changeTemperature(int ct);
	string getName();
	int getVolume();
	int getCoeffizient();
	int getTemperature();
};

FrozenMeal.cpp

#include "FrozenMeal.h"

FrozenMeal::FrozenMeal(string aName, int aVolume)
{
	name = aName;
	volume = aVolume;
	temperature = -18;
	coeffizient = 0.24*temperature/;
}

FrozenMeal::FrozenMeal(FrozenMeal& f)
{
	name = f.name;
	volume = f.volume;
	temperature = l.temperature;
	coeffizient = l.coeffizient;
}

FrozenMeal::~FrozenMeal(void)
{
}

void FrozenMeal::changeTemperature()
{
	temperature= temperature + coeffizient * Power* /volume;
}

void FrozenMeal::getName()
{
	return name;
}

int FrozenMeal::getVolume() const
{
	return volume;
}

double FrozenMeal::getCoeffizient() const
{
	return coeffizient;
}

void FrozenMeal::getTemperature()
{
	return temperature;
}

Microwave.h

#pragma once
#include <string>
#include <iostream>
#include <cassert>
using namespace std;

class Microwave
{
private:
	int power;		
	int volume;		
	double period;

public:
	Microwave(int power, int volume);
	Microwave(Microwave&);
	~Microwave(void);
	void morePower();
	void lessPower();
	void setPeriod(double);
	void heatMeal(FrozenMeal);
	int getPower( ;
	double getPeriod();
};

microwave.cpp

#include "Microwave.h"

Microwave::Microwave(int aPower)
{
	power = aPower;
	assert ((power >= 200) && (power <= 1000) && (power % 100 == 0));
}

Microwave::~Microwave(void)
{
	cout << "Die Mikrowelle hat eine eingestellte Leistung von: " << power << "W" << endl

}

void Microwave::morePower()
{
	if (power <=800)
		{
			power=power+200;
		} 
}

void Microwave::lessPower()
{
	if (power <=400)
		{
			power=power-200;
		} 
}

void setPeriod()
{

}

void Microwave::heatMeal()
{

}

int getPower()
{
	return power;
}

double getPeriod()
{
	return period;
}

Hier gabs schon einige interessante Thread dazu.
http://www.c-plusplus.net/forum/viewtopic-var-t-is-128265.html
http://www.c-plusplus.net/forum/viewtopic-var-t-is-128355.html
http://www.c-plusplus.net/forum/viewtopic-var-t-is-128353.html
http://www.c-plusplus.net/forum/viewtopic-var-t-is-128356.html
http://www.c-plusplus.net/forum/viewtopic-var-t-is-128371.html
http://www.c-plusplus.net/forum/viewtopic-var-t-is-128385.html

Konrad Rudolph

Hach ja, das schöne englische Wort "coeffizient". "so much" is natürlich auch nicht schlecht als Bezeichner.

Und was bitte soll folgender Code?!

correctInput = ((soMuch >= 200) && (power + soMuch <= 2000) && (soMuch % 200 == 0)) ? true : false;
correctInput == true ? power += soMuch : soMuch = 0;

Das ist ja pure Obfuscation.

ten

ThaRealMatix schrieb:

void Microwave::lessPower()
{
	if (power <=400)
		{
			power=power-200;
		} 
}

void Microwave::lessPower()
{
   power -= 200;
   if (power < 200)
      power = 200;
}

:xmas2:

Tipp: Konstanten

ten

FachmannFürMikrowellen schrieb:

Tipp: Konstanten

#define POWER_MAX 1000
#define POWER_MIN 200
#define POWER_INCREMENT 200
:xmas2:

ThaRealMatix

ähm mal ganz piano... ihr wertft mir jetzt Sachen um die Ohren ohne das mir das wirklich weiterhilft ^^'

Wieso wäre das von ten besser? Es ist ja zumindest schonmal mehr zeilen code.

Was hätte ich von define Power_max etc... eine definiertes maximum von power okay. aber ich muss immer noch einen vergleichsoperator benutzen, um zu sehen ob es übertreten wird oder nicht?

Weiss jemand vielleicht wie ich die heat funktion umsetzen könnte?

ten

ThaRealMatix schrieb:

Wieso wäre das von ten besser?

weil deine funktion nicht das tut, was du vielleicht glaubst

ThaRealMatix

Hm also ich habs nun wie folgt, aber immer noch keine Heat Funktion (ich weiss auch nicht, wozu ich eine change temperature funktion brauche, wenn ich eine heat, also erwärmen funktion habe)

FrozenMeal.h

#pragma once
#include <string>
#include "Microwave.h"
using namespace std;

class FrozenMeal
{
private:
	string name;
	int volume;
	double temperature;
	double coefficient;

public:
	FrozenMeal(string name, int volume);
	FrozenMeal(FrozenMeal&);
	~FrozenMeal();
	void changeTemperature(int ct);
	string getName();
	int getVolume();
	int getCoeffizient();
	int getTemperature();
};

FrozenMeal.cpp

#include "FrozenMeal.h"

FrozenMeal::FrozenMeal(string aName, int aVolume)
{
	name = aName;
	volume = aVolume;
	temperature = -18;
	coeffizient = 0.24*temperature/power*period;
}

FrozenMeal::FrozenMeal(FrozenMeal& f)
{
	name = f.name;
	volume = f.volume;
	temperature = l.temperature;
	coeffizient = l.coeffizient;
}

FrozenMeal::~FrozenMeal(void)
{
}

void FrozenMeal::changeTemperature()
{
	temperature= temperature + (coeffizient * Power* period/volume);
}

void FrozenMeal::getName()
{
	return name;
}

int FrozenMeal::getVolume() const
{
	return volume;
}

double FrozenMeal::getCoeffizient() const
{
	return coeffizient;
}

void FrozenMeal::getTemperature()
{
	return temperature;
}

microwave.h

#pragma once
#include <string>
#include <iostream>
#include <cassert>
using namespace std;

class Microwave
{
private:
	int power;		
	int volume;		
	double period;

public:
	Microwave(int power, int volume);
	Microwave(Microwave&);
	~Microwave(void);
	void morePower();
	void lessPower();
	void setPeriod(double);
	void heatMeal(FrozenMeal);
	int getPower( ;
	double getPeriod();
};

Microwave.cpp

#include "Microwave.h"

Microwave::Microwave(int aPower)
{
	power = aPower;
	assert ((power >= 200) && (power <= 1000) && (power % 100 == 0));
}

Microwave::~Microwave(void)
{
	cout << "Die Mikrowelle hat eine eingestellte Leistung von: " << power << "W" << endl

}

void Microwave::morePower()
{
	if (power <=800)
		{
			power=power+200;
		} 
}

void Microwave::lessPower()
{
   power -= 200;
   if (power < 200)
      power = 200;
}

void setPeriod()
{

}

void Microwave::heatMeal()
{

}

int getPower()
{
	return power;
}

double getPeriod()
{
	return period;
}

CStoll

Der Punkt mit der heatMeal() ist ganz einfach - power und period sind Member deiner Mikrowelle, da kommst du von deiner Mahlzeit aus nicht dran (besonders da die Mahlzeit nicht weiß, womit du sie kochen willst). Umgekehrt kann die Mikrowelle aber auch die Anfangstemperatur und den Wärmekoeffizienten des Essens erfragen und dann die Temperatur aktualisieren:

void FrozenMeal::changeTemperature(double zuwachs)
{ temperature += zuwachs; }

void MicroWave::heatMeal(FrozenMeal& meal)
{
  //Eventuell benötigst du noch eine Abfrage, ob das Mahl in den Ofen passt
  meal.changeTemperature(meal.getCoefficient()*power*period/meal.getVolume());
}

ThaRealMatix

Okay Danke...

Das Volumen soll, sowie ich das sehe nicht überprüft werden.

Hm... ich verstehe nur nicht, wie ich eine main aufbaue. Erhlich gesagt hab ich erst einmal mit Klassen und Co gearbeitet. Dort hatte ich Handy und Ladestation, bei denen ich bei beiden mehrere Objekte hatte. Wenn ich diese Aufgabe richtig verstehe, habe ich aber nur 1 Mikrowelle. Deswegen verstehe ich nicht so ganz wie ich sie aufrufen soll. ganz einfach wie jedes Objekt auch?

also z.B

Microwave m1( 400 );

?

Und wie bekomme ich die Bezeichner von power und period, die ja in der Microwave.h definiert sind in die ForzenMeal, da brauche ich Sie ja um den Koeffizienten auszugeben?

ThaRealMatix schrieb:

Was hätte ich von define Power_max etc... eine definiertes maximum von power okay. aber ich muss immer noch einen vergleichsoperator benutzen, um zu sehen ob es übertreten wird oder nicht?

Ja, aber du oder auch ein anderer kann es viel schneller ändern. Und wenn du mal ein größeres Programm hast, indem 100 mal 200 steht und das 30 verschiedene Bedeutungen hat, dann wird es lustig.

CStoll

Mal ein kleines Programmfragment:

MicroWave mw(400);//Mikrowelle anlegen
FrozenMeal dinner("Thunfisch",250);//Essen bereitlegen

//Mikrowelle einstellen:
while(mw.getPower()<1000) mv.morePower();
mv.setPeriod(25);

//und schließlich - Essen zubereiten:
mv.heatMeal(dinner);

Aber du scheinst massive Probleme mit den Grundlagen von C++ zu haben - eventuell solltest du mal die vorigen Kapitel deines Leerbuches durcharbeiten

ThaRealMatix

Hm sorry ich hatte auch die Aufgabenstellung falsch verstanden. Der Coefficient ist wohl konstant 0.24 und das dahinter nur ein Beispiel, wie e errechnet wurde.

-.-

Leider bekomme ich folgende Fehlermeldung beim kompilieren:

Die Zwischen- und Ausgabedateien für das Projekt "microwave" mit der Konfiguration "Debug|Win32" werden gelöscht.
Kompilieren...
FrozenMeal.cpp
d:\eigene dateien\visual studio 2005\projects\programmieren1_praktikum5\microwave\frozenmeal.cpp(45) : error C2470: "FrozenMeal::getCoefficient": Sieht wie eine Funktionsdefinition aus, es ist aber keine Parameterliste vorhanden; sichtbarer Funktionstext wird übersprungen.
d:\eigene dateien\visual studio 2005\projects\programmieren1_praktikum5\microwave\frozenmeal.cpp(51) : warning C4244: 'return': Konvertierung von 'double' in 'int', möglicher Datenverlust

double FrozenMeal::getCoefficient
{
	return coefficient;
}

P.S mit deinem PRogramm läuft es auch nicht

CStoll

d:\eigene dateien\visual studio 2005\projects\programmieren1_praktikum5\microwave\frozenmeal.cpp(45) : error C2470: "FrozenMeal::getCoefficient": Sieht wie eine Funktionsdefinition aus, es ist aber keine Parameterliste vorhanden; sichtbarer Funktionstext wird übersprungen.

Auch wenn die Funktion keine Parameter entgegennehmen soll, mußt du eine (leere) Parameterliste (entweder als () oder als (void) ) bei der Definition angeben.

d:\eigene dateien\visual studio 2005\projects\programmieren1_praktikum5\microwave\frozenmeal.cpp(51) : warning C4244: 'return': Konvertierung von 'double' in 'int', möglicher Datenverlust

[/quote]Welche Methode betrifft das denn? Du hast dort vermutlich (ziemlich sicher) ein double-Element (Gleitkomma-Zahl) als int (Ganzzahl) zurückgegeben.

PS: Das da oben ist kein Programm, sondern nur ein Ausschnitt, wie du die Klassen verwenden kannst. Das Programm darfst du gerne selber schreiben.

PPS: Und noch einmal: LERN DIE GRUNDLAGEN VON C++!!

ThaRealMatix

Hm... ich weiss ja das ich nachholbedarf habe, aber ich muss diese Aufgaben halt abgeben

was ich nciht verstehe, ist, dass etwas ziemlich ähnliches, das in einer aufgabe funktionierte, nun nicht mehr zu funktionieren scheint.

ich hab nun wie flgt abgeändert

Microwave::Microwave(int aPower, int aVolume)
{
	power = aPower;
        volume= aVolume;
}

ich bekomme nun folgende Fehlermeldung:

d:\eigene dateien\visual studio 2005\projects\programmieren1_praktikum5\microwave\microwave.h(26) : error C2061: Syntaxfehler: Bezeichner 'FrozenMeal'

die header hat den eintrag

void heatMeal(FrozenMeal);

CStoll

Kannst du mal bitte etwas mehr posten als die paar Fragmente? (und häufig liegt der tatsächliche Fehler ein paar Zeilen über der gemeldeten Stelle)

Mir könnte als Grund nur einfallen, daß der Bezeichner "FrozenMeal" (noch) nicht bekannt ist - entweder du vertauschst die Klassendefinitionen im Header oder du setzt eine Forward-Deklaration class FrozenMeal; vor die MicroWave-Klasse.

Hm... ich weiss ja das ich nachholbedarf habe, aber ich muss diese Aufgaben halt abgeben

Dann mußt du halt ein leeres Blatt abgeben und deinem Prof erklären, daß du im Unterricht nicht aufgepasst hast

ThaRealMatix

Was hieße ein halbes Jahr für die Katz. Versteh mich nciht falsch, ich weiss das Du/ihr Mühe in Euer C++ Können gesteckt habt und wenn ihr mir nicht helfen wollt, weil ihr mich für Faul haltet, und es für unfair haltet mir zu helfen ist das ok.

Ich habe halt eine Weile wegen Krankheit gefehlt. Jetzt hinke ich hinterher, Die Praktikas sind aber Bedingung, um an der Prüfung teilzunehmen.

Ich will ja alles nachholen, aber kein ganzes halbes Jahr / Semester hinschmeissen, weil ich das Praktikum heute nicht schaffe. Also versuche ich es eben. Wenn ich ducrhfalle weiss ich wenigstens, ich habe alles versucht.

Wie gesagt, wenn ihr mir nicht helfen möchtet weil ihr meint selber Schuld, mache ich Euch gewiss keinen Vorwurf. ICh kann mir nru auch keinen machen, weil ich nicht lernen konnte.

Ich poste mal nun alles, wie ich es nun soweit habe:

FrozenMeal.h

#pragma once
#include <string>
#include "Microwave.h"
using namespace std;

class FrozenMeal
{
private:
	string name;
	int volume;
	double temperature;
	double coefficient;

public:
	FrozenMeal(string name, int volume);
	FrozenMeal(FrozenMeal&);
	~FrozenMeal();
	void changeTemperature(double zuwachs);
	string getName();
	int getVolume();
	double getCoefficient();
	double getTemperature();
};

FrozenMeal.cpp

#include "FrozenMeal.h"
#include "Microwave.h"

FrozenMeal::FrozenMeal(string aName, int aVolume)
{
	name = aName;
	volume = aVolume;
	temperature = -18;
	coefficient = 0.24;
}

FrozenMeal::FrozenMeal(FrozenMeal& f)
{
	name = f.name;
	volume = f.volume;
	temperature = f.temperature;
	coefficient = f.coefficient;
}

FrozenMeal::~FrozenMeal(void)
{
}

void FrozenMeal::changeTemperature(double zuwachs)
{
	temperature += zuwachs;
} 

string FrozenMeal::getName()
{
	return name;
}

int FrozenMeal::getVolume()
{
	return volume;
}

double FrozenMeal::getCoefficient()
{
	return coefficient;
}

double FrozenMeal::getTemperature()
{
	return temperature;
}

Microwave.h

#pragma once
#include <string>
#include <iostream>
#include <cassert>
using namespace std;

class Microwave
{
private:
	int power;		
	int volume;		
	double period;

public:
	Microwave(int power, int volume);
	Microwave (Microwave&);
	~Microwave(void);
	void morePower();
	void lessPower();
	void setPeriod(double);
//	void heatMeal(FrozenMeal);
//	int getPower();
//	double getPeriod();
};

Microwave.cpp

#include "FrozenMeal.h"
#include "Microwave.h"

Microwave::Microwave(int aPower, int aVolume)
{
	power = aPower;
	volume = aVolume;
	//assert ((power >= 200) && (power <= 1000) && (power % 100 == 0));
}

Microwave::~Microwave(void)
{
	cout << "Die Mikrowelle hat eine eingestellte Leistung von: " << power << "W" << endl;

}

void Microwave::morePower()
{
	if (power <=800)				//wenn Leistung nicht groesser als 800 Watt 
		{							// erhöhe um 200. 800 + 200 = 1000 = max 
			power=power+200;
		} 
}

void Microwave::lessPower()
{
   power -= 200;					// Leistung wird um 200 verringert
   if (power < 200)					// Wenn LEistung kleiner als 200 wird
      power = 200;					// Leistung auf 200 gesetzt da minimum
}

void setPeriod()
{

}
/*
void Microwave::heatMeal(FrozenMeal& meal)
{
    meal.changeTemperature(meal.getCoefficient()*power*period/meal.getVolume());
}

int getPower()
{
	return power;
}

double getPeriod()
{
	return period;
}*/

main.cpp

#include "Microwave.h"
#include "FrozenMeal.h"

using namespace std;

void main()
{
	FrozenMeal f1 ("Truthahn", 250); //Erstellung des Gerichts
	cout << "Name des Gerichts: " << f1.getName() <<endl //Ausgabe des Namens
		 << "Temperatur des Gerichts: " << f1.getTemperature() << endl // Ausgabe der Temperatur
		 << "Volumen des Gerichts: " << f1.getVolume() << "in ccm" <<endl; //Ausgabe Volumen

	system("Pause");
}

Derzeit läuft das Programm, aber halt ohne einige Funktionen und nur weil ich einiges auskommentiert habe.

CStoll

Da solltest du als erstes die verbleibenden Methoden deiner Klassen anlegen. Und dann solltest du sie auch als Klassenmethoden definieren (da gehört in der CPP-Datei das "MicroWave::" dazu).

Als nächstes solltest die Methoden mit Inhalt füllen (setPeriod() setzt das Element period auf den übernommenen Wert etc.).

Und ich nehme mal an, das Hauptprogramm sollte auch die MicroWave-Klasse nutzen (siehe dazu das Programmfragment in einem meiner letzten Beiträge)

PS: Wenn heatMeal() seinen Beitrag per Kopie übergeben bekommt, wirkt es sich nicht auf dein Mittagessen aus - du benötigst dort eine Referenz ( void heatMeal(FrozenMeal&); ). Und die Aufgabe verlangt, daß der MicroWave-Ctor nur einen int-Wert übernimmt (was der darstellen soll, kann ich nicht genau feststellen - vermutlich das Volumen).

PPS:

Was hieße ein halbes Jahr für die Katz. Versteh mich nciht falsch, ich weiss das Du/ihr Mühe in Euer C++ Können gesteckt habt und wenn ihr mir nicht helfen wollt, weil ihr mich für Faul haltet, und es für unfair haltet mir zu helfen ist das ok.

Falls du es nicht bemerkt hast, ich will dir ja helfen. Aber dafür mußt du auch etwas tun