Verschiedene Tankarten Scripten

ENEF · 9. März 2015

Hallo leute, ich habe mal ein paar Fragen übrig.

Da ich nicht nur so ein normales langweiliges Tanksystem möchte, dachte ich mir dass ich mehrere Tankarten zur verfügung stelle.
Meine Frage ist es jetzt, wie weise ich die verschiedenen Fahrzeuge den verschiedenen Tankarten zu?
Es soll:
Benzin, Diesel, Kerosin und Gas vorhanden sein.
Das nächste wäre, wie ich die Zeit für die jeweiligen Tankart bestimme, indem sie ihre Literzahl verlieren.
Bsp: Diesel brauch 20 Sekunden bis ein Liter weg geht oder 20 Meter je nachdem,
Benzin 15 Sekunden/Meter

Versteht ihr? Logischerweise sollen sie dann auch unterschiedlich viel kosten.
Und jedes Auto hat seine vorgegebene Tankmenge also bsp. Sultan 50liter Tank und Infernus nur 43 Liter.

Bis jetzt habe ich mein Tanksystem so:

Defines:

Spoiler anzeigen

Timer:

Spoiler anzeigen

Tanken:

Spoiler anzeigen

Public's

Spoiler anzeigen

Das war der Code.
Ich hoffe ihr könnt mir auf die Sprünge helfen, wäre echt sehr Dankbar dafür

Mit freundlichen Grüßen

TVPikachu

Jeffry:

Kaliber · 9. März 2015

Zitat von TVPikachu

Ich hoffe ihr könnt mir auf die Sprünge helfen, wäre echt sehr Dankbar dafür

Du musst ein Array verwenden (oder eine Datenbank) und dann jeder Modelid, die Werte zuweisen:

stock const VehData[][2] = {
{0, 50},
{1, 43},
{0, 50}
};

#define GetVehicleTankArt(%0) VehData[(%0)-400][0]
#define GetVehicleMaxTank(%0) VehData[(%0)-400][1]

new veh = GetPlayerVehicleID(playerid);
new m = GetVehicleModel(veh); //Angenommen modelid == 400
switch(GetVehicleTankArt(m)) {
case 0: print("Dieses Auto tankt Diesel!"); //<< das wird geprintet
case 1: print("Dieses Auto tankt Benzin!");
//usw..
}

printf("Max Tank: %dL",GetVehicleMaxTank(m)); //Output: Max Tank: 50

switch(GetVehicleTankArt(m)) {
case 0: Tank[veh] -= 20; //da Diesel -= 20
case 1: Tank[veh] -= 15; //Da Benzin -= 15
//usw
}

mfg.

ENEF · 9. März 2015

Zitat von Kaliber

stock const VehData[][2] = { {0, 50}, {1, 43}, {0, 50} };

Wie wäre das jetzt aufgebaut?
0,50//0 ist jetzt benzin und 50 die liter Zahl und das Auto Landstalker?

Ich check grad voll nicht durch

Kaliber · 9. März 2015

Zitat von TVPikachu

Ich check grad voll nicht durch

Schau dir doch unten die Funktionen an...mein Gott das sind 2 Variablen...

ENEF · 10. März 2015

Ich habe jetzt jedem Fahrzeug seinen Tank zugewiesen, aber wenn ich

[pwn=284]new v;
v = GetPlayerVehicleID(playerid);
new m;
m = GetVehicleModel(v);
switch(GetVehicleTankArt(m)) {
case 0: print("Dieses Auto tankt Diesel!"); //<< das wird geprintet
case 1: print("Dieses Auto tankt Benzin!");
}

switch(GetVehicleTankArt(m)) {
case 0: Tank[veh] -= 20; //da Diesel -= 20
case 1: Tank[veh] -= 15; //Da Benzin -= 15 //usw
}[/pwn]

Einfüge bekomme ich Errors/Warnings

Spoiler anzeigen

Und ich bin noch ein ziemlicher Anfänger im Scripten, sorry

Ich füge das switch ja alles oben ins GM ein gell?

Kann ich ein Command machen, oder du mir an dem ich mich Orientieren kann..

bsp: mit einer SendClientMessage "Tankart, Kraftstoff und Zeit"
Zeit = Die zeit in der das Auto 1 Liter benzin verliert, mit abfragen halt daran könnt ich mich orientieren..

Sorry wenns zu umständlich für dich ist oder so ...

Mit freundlichen Grüßen

Kaliber · 10. März 2015

Zitat von TVPikachu

Ich füge das switch ja alles oben ins GM ein gell?

Nein..

Das sind doch Beispiele, wie du das nutzen kannst...

Lies dir das doch einmal durch, Zeichen für Zeichen, dann findest du doch auch schon Beispiele, wo ich Tank abziehe, wenn das Fahrzeug Diesel ist...etc

ENEF · 10. März 2015

Okey ein bisschen habe ich jetzt gecheckt.
Zum beispiel im Textdraw es anzeigen zu lassen klappt, aber nur ohne mein System

Mein System:
100 % Tank ist er Grün
50% Tank ist er Gelb
25% Tank ist er Rot..

ich musste es jetzt so machen weil ich es anders nicht gerafft hab:

new v = GetPlayerVehicleID(playerid);
new m = GetVehicleModel(v);
new string[125];
switch(GetVehicleTankArt(m)) {
case 0: format(string,sizeof string,"Tankart: ~g~Diesel~n~~w~Tank: ~g~%i ~w~liter~n~Km: 0000012~g~%d ~w~",Tank[vehicleid]);
case 1: format(string,sizeof string,"Tankart: ~g~Benzin~n~~w~Tank: ~g~%i ~w~liter~n~Km: 0000012~g~%d ~w~",Tank[vehicleid]);
}
PlayerTextDrawSetString(playerid,CarTankart[playerid],string);

Kann ich es aber nicht einfach so machen und bei den Case irgendwie schreiben
case 0: "Diesel"
und dann :
if(Tank[vehicleid] < 25)format(string,sizeof string,"Tankart: %i~n~~w~Tank: ~r~%i ~w~liter~n~Km: 0000012~g~%d ~w~",GetVehicleTankArt(m),Tank[vehicleid]);
else if(Tank[vehicleid] < 50)format(string,sizeof string,"Tankart: %i~n~~w~Tank: ~y~%i ~w~liter~n~Km: 0000012~g~%d ~w~",GetVehicleTankArt(m),Tank[vehicleid]);
else if(Tank[vehicleid] <= 100)format(string,sizeof string,"Tankart: %i~n~~w~Tank: ~g~%i ~w~liter~n~Km: 0000012~g~%d ~w~",GetVehicleTankArt(m),Tank[vehicleid]);

Wenn du bei dem helfen kannst, wär ich dir schon dankbar

//Edit: Habs selbst geschafft

Jeffry · 10. März 2015

Doch, klar.

Kannst es so machen:
new v = GetPlayerVehicleID(playerid);
new m = GetVehicleModel(v);
new string[125];
switch(GetVehicleTankArt(m))
{
case 0: string = "Diesel";
case 1: string = "Benzin";
}
format(string,sizeof string,"Tankart: ~g~%s~n~~w~Tank: ~g~%i ~w~liter~n~Km: 0000012~g~%d ~w~",string,Tank[vehicleid]);
PlayerTextDrawSetString(playerid,CarTankart[playerid],string);

ENEF · 10. März 2015

Hi,

Ich hab jetzt bisschen weiter angepasst, da alle Autos ja mit 100 Tank gespawnt werden aber ja nur bestimmte Anzahl an Tank haben.
Deswegen hab ich jetzt es so gemacht, dass jedes Auto mit seiner MAximalen Tankfüllung spawnt...

for (new i=0; i< MAX_VEHICLES; i++)
{
for(new p; p<MAX_PLAYERS; p++) {
new v = GetPlayerVehicleID(p);
new m = GetVehicleModel(v);
GetVehicleParamsEx(i,engine,lights,alarm,doors,bonnet,boot,objective);
SetVehicleParamsEx(i,VEHICLE_PARAMS_OFF,lights,alarm,doors,bonnet,boot,objective);
Motor[i] = false;
Tank[i] = GetVehicleMaxTank(m); }
}

Keine Errors aber wenn ich Motor ingame anmachen möchte sagt er mir ich habe kein Tank

Jeffry · 10. März 2015

Sollte so passen:
for (new v=0; v< MAX_VEHICLES; v++)
{
GetVehicleParamsEx(v,engine,lights,alarm,doors,bonnet,boot,objective);
SetVehicleParamsEx(v,VEHICLE_PARAMS_OFF,lights,alarm,doors,bonnet,boot,objective);
Motor[v] = false;
Tank[v] = GetVehicleMaxTank(GetVehicleModel(v));
}

ENEF · 10. März 2015

Leider immer noch nicht, komischerweise wird die Tankfarbe (ID 402 mit Tank max. 50) auch in Rot angezeigt :o?
mein Motor starten befehl ist ja korrekt?

Spoiler anzeigen

MfG

Kaliber · 10. März 2015

Schreib das mal so:

//unter OnGameModeInit
ManualVehicleEngineAndLights();
SetTimer("@idle",1000,0);

//Außerhalb davon
@idle();@idle() {
for(new v,m; v<MAX_VEHICLES; v++) {
m = GetVehicleModel(v);
if(!m) continue;
Tank[v] = GetVehicleMaxTank(m);
}
}

Zitat von Jeffry

Sollte so passen:

Da kann es zu Fehlern kommen, z.B. wenn er darunter noch Autos erstellt, oder m (was so kommen wird) eine ungültige vehicleid hat...dann greift er auf einen ungütligen Index im Array zu.

mfg.

ENEF · 10. März 2015

Geht auch nicht.

Ich schick jetzt einfach mal mein Tanksystem vielleicht hackt es irgendwo..

Spoiler anzeigen

Mit freundlichen Grüßen

Jeffry · 10. März 2015

Zitat von Kaliber

Da kann es zu Fehlern kommen, z.B. wenn er darunter noch Autos erstellt, oder m (was so kommen wird) eine ungültige vehicleid hat...dann greift er auf einen ungütligen Index im Array zu.

Stimmt, du fragst es in deinem define nicht ab. Wäre besser das in der Funktion entsprechend abzufragen, dann muss man es nicht überall nochmal hinschreiben, wo man die Funktion verwendet.
stock GetVehicleTankArt(model)
{
if(model < 400 || model > 611) return -1;
return VehData[model-400][0];
}
stock GetVehicleTankArt(model)
{
if(model < 400 || model > 611) return -1;
return VehData[model-400][1];
}

Oder eben in den define klatschen, davon bin ich aber kein Freund, da viele das dann nicht mehr verstehen, vor allem wenn es länger wird. Ein Code sollte ja auch noch lesbar sein.

ENEF · 10. März 2015

Hi,

Habe den Stock jetzt eingefügt, muss ich jetzt bei den anderen Codes noch irgendwas beachten?

[pwn=2814]
stock GetVehicleTankArt(model)
{
if(model < 400 model > 611) return -1;
return VehData[model-400][0];
}
stock GetVehicleTankArt(model)
{
if(model < 400 model > 611) return -1;
return VehData[model-400][1];
}[/pwn]

C:\Users\Tobi\Desktop\SAMP\samp037_svr_RC1_win32\gamemodes\newrl.pwn(2814) : error 017: undefined symbol "model"
C:\Users\Tobi\Desktop\SAMP\samp037_svr_RC1_win32\gamemodes\newrl.pwn(2816) : error 010: invalid function or declaration
C:\Users\Tobi\Desktop\SAMP\samp037_svr_RC1_win32\gamemodes\newrl.pwn(2817) : error 010: invalid function or declaration
C:\Users\Tobi\Desktop\SAMP\samp037_svr_RC1_win32\gamemodes\newrl.pwn(2819) : error 017: undefined symbol "model"
C:\Users\Tobi\Desktop\SAMP\samp037_svr_RC1_win32\gamemodes\newrl.pwn(2821) : error 010: invalid function or declaration
C:\Users\Tobi\Desktop\SAMP\samp037_svr_RC1_win32\gamemodes\newrl.pwn(2822) : error 010: invalid function or declaration
Pawn compiler 3.2.3664 Copyright (c) 1997-2006, ITB CompuPhase

6 Errors.

Und soll ich jetzt den Code von Kaliber behalten?

Verstehe grad nicht mehr so alles..

MfG

Jeffry · 10. März 2015

Nein, die zwei defines kannst du löschen.
#define GetVehicleTankArt(%0) VehData[(%0)-400][0]
#define GetVehicleMaxTank(%0) VehData[(%0)-400][1]
//Löschen.

ENEF · 10. März 2015

Okey, hab ich und keine Errors.
Doch jetzt geht es trotzdem noch nicht

Hab jetzt so wie ihr es gesagt habt gemacht ..

//Edit;

Habe mal eins an der Tankstelle hingeschoben und getankt, selbst da sagt er mir ich hätte kein Tank ?

Spoiler anzeigen

Jeffry:
MfG

Jeffry · 10. März 2015

Du hast keinen Tank? Die Nachricht sehe ich nicht.

Aber ändere mal:
else if (GetVehicleMaxTank(car) < 1)
zu:
else if(Tank[car] < 1)

Dann zeige nochmal wie du die Fahrzeuge lädst und den Tank auf voll setzt.
Und zeige den Tanken Befehlt.

ENEF · 10. März 2015

Doch hier :

else if (Tank[car] < 1)
{
SendClientMessage(playerid, error, "Der Tank ist leer. Du musst erst /tanken.");
}

Das erscheint beim Versuch meinen Motor zu starten..

Es gibt keinen Befehl, ich habe es mit der Taste 2 gemacht:

Spoiler anzeigen

Hier wird es auf Maximal gestellt:
SetTimer("BenzinVerlust", VERLUST, true);
for (new v=0; v< MAX_VEHICLES; v++)
{
GetVehicleParamsEx(v,engine,lights,alarm,doors,bonnet,boot,objective);
SetVehicleParamsEx(v,VEHICLE_PARAMS_OFF,lights,alarm,doors,bonnet,boot,objective);
Motor[v] = false;
Tank[v] = GetVehicleMaxTank(GetVehicleModel(v));
}

Hier mal meine Publics:

Spoiler anzeigen

Soll ja auch nicht über den max. Tank getankt werden und am besten verschieden abgezogen werden(nach 20 Metern bei diesel - 1 und bei 15 Metern Benzin - 1) wenn es geht..

Falls du noch was brauchst, schreib es mir
Danke dir Jeffry: (und auch dir Kaliber)

Mit freundlichen Grüßen

TV

Jeffry · 10. März 2015

Schreibe die Callbacks auf jeden Fall mal so:
public BenzinVerlust()
{
for(new i=0; i<MAX_VEHICLES; i++)
{
if (Motor[i] == true)
{
Tank[i]-=1;
if (Tank[i]<1)
{
Tank[i] = 0;
GetVehicleParamsEx(i,engine,lights,alarm,doors,bonnet,boot,objective);
SetVehicleParamsEx(i,VEHICLE_PARAMS_OFF,lights,alarm,doors,bonnet,boot,objective);
Motor[i] = false;
}
}
}
return 1;
}

public Auftanken(playerid)
{
new car = GetPlayerVehicleID(playerid);
new m = GetVehicleModel(car);
Tank[car]+=1;
if(Tank[car] >= GetVehicleMaxTank(m))
{
TanktGerade[car] = false;
Tank[car] = GetVehicleMaxTank(m);
SendClientMessage(playerid, erfolg, "Dein Fahrzeug wurde %i€ vollgetankt.");
TogglePlayerControllable(playerid,1);
KillTimer(tanktimer[playerid]);
}
return 1;
}

Das mit den unterschiedlichen Tanks schauen wir an, wenn es erst mal so funktioniert.

Kannst du so das Fahrzeug auftanken?
Kannst du den Motor starten?
Hast du das setzen des vollen Tanks am Anfang vor oder nach dem erstellen der Fahrzeuge?