[SAMMELTHREAD] Kleine Scripting Fragen

Zitat von Cireyses

Vielleicht hilft dir das.

Inwiefern soll mir das denn helfen?

Ja, und? Ist doch auch richtig so.

Zum eigentlichen Problem: Das liegt daran, dass bei jedem OnPlayerUpdate die Geschwindigkeit wieder runtergesetzt wird, da der Spieler ja eigentlich nicht so schnell laufen kann. Wenn du es GENAU so machst wie ich es dir gegeben habe, auch mit der GetPlayerSpeed Funktion, dann geht es, mit kleinen Rucklern, aber er ist merklich schneller.

Zitat von Jeffry

Ja, und? Ist doch auch richtig so.

Zum eigentlichen Problem: Das liegt daran, dass bei jedem OnPlayerUpdate die Geschwindigkeit wieder runtergesetzt wird, da der Spieler ja eigentlich nicht so schnell laufen kann. Wenn du es GENAU so machst wie ich es dir gegeben habe, auch mit der GetPlayerSpeed Funktion, dann geht es, mit kleinen Rucklern, aber er ist merklich schneller.

Hmm.. Aber wenn ich es genau so mache, wie du es mir gegeben hast, dann wird "Speed" mit dem Wert 0 ausgegeben.
Die Funktion "GetPlayerSpeed" hat nur einen Parameter, nämlich den, für die Player-ID.

Zitat von Jeffry

Hmm.. Aber wenn ich es genau so mache, wie du es mir gegeben hast, dann wird "Speed" mit dem Wert 0 ausgegeben.
Die Funktion "GetPlayerSpeed" hat nur einen Parameter, nämlich den, für die Player-ID.

Wie ich schon sagte, mit der GetPlayerSpeed Funktion die ich dir gegeben habe, benenn die von mit aus um, zu GetPlayerSpeed2, wenn du schon eine GetPlayerSpeed Funktion hast. Dann wirst du sehen, dass es geht. Ich weiß ja nicht was für Werte deine GetPlayerSpeed-Funktion zurück gibt.

Zitat von Jeffry

Wie ich schon sagte, mit der GetPlayerSpeed Funktion die ich dir gegeben habe, benenn die von mit aus um, zu GetPlayerSpeed2, wenn du schon eine GetPlayerSpeed Funktion hast. Dann wirst du sehen, dass es geht. Ich weiß ja nicht was für Werte deine GetPlayerSpeed-Funktion zurück gibt.

Ähm...

Zitat von ErikSon

Aber wenn ich es genau so mache, wie du es mir gegeben hast

Ich benutze den GANZEN Code von dir ..
Es klappt nicht, wenn ichs so mache, "Speed" wird immer der Wert 0.0000 ( ist ja Float ) zugeteilt.

Dann poste bitte den Code wie du ihn hast, wenn du den Code vor mir benutzt.

public OnPlayerSpawn(playerid)
{
SetCameraBehindPlayer(playerid);
StopAudioStreamForPlayer(playerid);
new Random = random(sizeof(RandomSpawns));
SetPlayerPos(playerid, RandomSpawns[Random][0], RandomSpawns[Random][1], RandomSpawns[Random][2]);
SetPlayerFacingAngle(playerid, RandomSpawns[Random][3]);

if(IsPlayerNPC(playerid))
{
new npcname[64];
GetPlayerName(playerid,npcname,64);
if(!strcmp(npcname,"CrazyMoron",true))
{
//SetSpawnInfo(playerid, team, skin, Float:x, Float:y, Float:z, Float:rotation, weapon1, weapon1_ammo, weapon2, weapon2_ammo, weapon3, weapon3_ammo);
SetSpawnInfo(playerid, 0, 60, 2492.5146, -1703.2296, 1015.5703, 328.5485, 0, 0, 0, 0, 0, 0);
new Text3D:NameEintragen = Create3DTextLabel("CrazyMoron",0xFFFFFFFF,10.0,40.0,50.0,40.0,0);
Attach3DTextLabelToPlayer(NameEintragen, playerid, 0.0, 0.0, 0.2);
SetPlayerColor(playerid,0xFFFFFFFF);

}
}
return 1;

//==============================================================================
// Registering
//==============================================================================

if(AccInfo[playerid][Registered] == 0 && ServerInfo[MustRegister] == 1) // (1030) : warning 225: unreachable code
{
#if USE_DIALOGS == true
new rstring[256];
format(rstring,256,"Welcome to %s\n\nThe Account '%s' is not registred yet.\n\n\nEnter a password to register your account:",GetServerHostName(),pName(playerid));
ShowPlayerDialog(playerid,DIALOGID+66,DIALOG_STYLE_INPUT,"Register Account",rstring,"Register","Quit");
#endif
return 1;
}

Wieso bekomme ich den unreachable code warning? Muss ich das return 1; löschen in Zeile 23?

Es gehört zu OnPlayerSpawn.

Ich verstehe nicht warum man das return 1; weglassen soll. Manchmal bleibt es und manchmal kann man es weglassen

Zitat von Fuldara

Ich verstehe nicht warum man das return 1; weglassen soll. Manchmal bleibt es und manchmal kann man es weglassen

Ein "return" beendet das Callback bzw. die Funktion genau an der Stelle und zwar immer dann, wenn es aufgerufen wird.

Als hier in Kurzform dein OnPlayerSpawn:

public ...
Code...
Code...
return 1;
Code... //wird NIEMALS erreicht.
return 1;

Nach dem ersten return ist schon Ende, das heißt der Code darunter wird NIE aufgerufen, daher die Warnung "unreachable Code" (unerreichbarer Code).

Anders hier:
public ...
Code...
if(irgendwas) return 1;
Code... //Wir erreicht, wenn "irgendwas" nicht eintrifft.
return 1;

Hier bekommst du keine Warnung, da der Code unter dem IF nur dann nicht aufgerufen wird, wenn "irgendwas" eintrifft, wenn nicht, führt er den Code unter dem IF aus.

=> Du hast als einen Code wie im ersten Beispiel, der unter keinen Umständen aufgerufen wird, daher macht das für den Compiler keinen Sinn.

Entferne das return 1, das da mittendrin steht, oder setze es in eine IF-Verschachtelung in der du es wirklich haben willst.

Aber wenn ich das return weglasse bekomme ich die warnung SERVER: Unknown command?

Zitat von Fuldara

Aber wenn ich das return weglasse bekomme ich die warnung SERVER: Unknown command?

Bei OnPlayerCommandText, ja. Weil dort ist "return 1" = keine Nachricht und "return 0" = Unknown Command. Das siehst du, weil da steht gaaaanz am Ende von OnPlayerCommandText ein return 0. Bei Befehlen arbeitest du aber ja mit der Struktur aus dem zweiten Beispiel (IF). Wird ein Befehl ausgeführt (irgendwas = wahr), dann kannst du return 1 machen, da dich der Rest dann nicht mehr interessiert.
Es kommt immer auf die Situation an, wann du was benutzen musst.

Zitat von Jeffry

Dann poste bitte den Code wie du ihn hast, wenn du den Code vor mir benutzt.

if(!IsPlayerInAnyVehicle(playerid) && Sprinted[playerid])
{
new Float:speed; GetPlayerSpeed(playerid,speed);
if(speed >= 38.0 && speed <= 40.0){ // Das ist ja klar, dass ich das bearbeitet habe..
SetPlayerSpeed(playerid, 70.0, speed); // too
}
}

//OnPlayerKey
if(newkeys & KEY_SPRINT)Sprinted[playerid] = true;
else Sprinted[playerid] = false;

stock SetPlayerSpeed(playerid, Float:_speed, Float:_isspeed){
if(playerid == INVALID_PLAYER_ID)return INVALID_PLAYER_ID;
else if(IsPlayerNPC(playerid))return 1;
else if(IsPlayerInAnyVehicle(playerid))return 1;
else if(!Sprinted[playerid])return 1;
new Float:x1, Float:y1, Float:z1, Float:a,Float:speed;
GetPlayerVelocity(playerid, x1, y1, z1);
GetPlayerFacingAngle(playerid, a);
return SetPlayerVelocity(playerid, x1*(1+(1/(1+(_speed/_isspeed)))), y1*(1+(1/(1+(_speed/_isspeed)))), z1);
}

if(speed >= 38.0 && speed <= 40.0){ // Das ist ja klar, dass ich das bearbeitet habe..
Nein, weil bei dem GetPlayerSpeed von mir wird er NIE 38 km/h rennen, sondern maximal was um die 25.

if(speed >= 20.0 && speed <= 40.0){

Deshalb sagte ich, probier es zuerst genau so wie ich es gemacht habe.

Okay, aber wie gesagt, das Problem liegt daran, dass die Variable "Speed" einen Wert von 0 hat.

Hier mal Log:
[16:13:32] Speed: 0.000000
[16:13:32] Speed: 0.000000
[16:13:32] Speed: 0.000000
[16:13:32] Speed: 0.000000
[16:13:32] Speed: 0.000000
[16:13:32] Speed: 0.000000
[16:13:32] Speed: 0.000000
[16:13:32] Speed: 0.000000
[16:13:32] Speed: 0.000000
[16:13:32] Speed: 0.000000
[16:13:32] Speed: 0.000000
[16:13:32] Speed: 0.000000
[16:13:32] Speed: 0.000000
[16:13:32] Speed: 0.000000
[16:13:32] Speed: 0.000000
[16:13:32] Speed: 0.000000
[16:13:32] Speed: 0.000000
[16:13:32] Speed: 0.000000
[16:13:32] Speed: 0.000000
[16:13:32] Speed: 0.000000
[16:13:33] Speed: 0.000000
[16:13:33] Speed: 0.000000
[16:13:33] Speed: 0.000000
[16:13:33] Speed: 0.000000
//usw ;)

Code:
if(!IsPlayerInAnyVehicle(playerid) && Sprinted[playerid])
{
new Float:speed; GetPlayerSpeed(playerid,speed);
printf("Speed: %f",GetPlayerSpeed(playerid,speed)); // auch -> "printf("Speed: %f",speed);" ausprobiert!
if(speed >= 20.0 && speed <= 40.0){
printf("Speed wird gesetzt");
SetPlayerSpeed(playerid, 40.0, speed);
}
}

stock GetPlayerSpeed_New(playerid, &Float:speed) // By: [XST]O_x
{
new Float:ST[4];
if(IsPlayerInAnyVehicle(playerid) && GetPlayerState(playerid)!=PLAYER_STATE_PASSENGER)
GetVehicleVelocity(GetPlayerVehicleID(playerid),ST[0],ST[1],ST[2]);
else GetPlayerVelocity(playerid,ST[0],ST[1],ST[2]);
ST[3] = floatsqroot(floatpower(floatabs(ST[0]), 2.0) + floatpower(floatabs(ST[1]), 2.0) + floatpower(floatabs(ST[2]), 2.0)) * 100.3;
speed = ST[3];
return 1;
}
Warum sagst du nicht, dass ich die Funktion dir noch nie gegeben habe.
Ich dachte die hätte ich dir schonmal gegeben, scheint wohl nicht so.

new Float:speed; GetPlayerSpeed_New(playerid,speed);
printf("Speed: %f",speed);

Ich hab sie gleich umbenannt, damit du nicht durcheinander kommst.

Zitat von Jeffry

stock GetPlayerSpeed_New(playerid, &Float:speed) // By: [XST]O_x
{
new Float:ST[4];
if(IsPlayerInAnyVehicle(playerid) && GetPlayerState(playerid)!=PLAYER_STATE_PASSENGER)
GetVehicleVelocity(GetPlayerVehicleID(playerid),ST[0],ST[1],ST[2]);
else GetPlayerVelocity(playerid,ST[0],ST[1],ST[2]);
ST[3] = floatsqroot(floatpower(floatabs(ST[0]), 2.0) + floatpower(floatabs(ST[1]), 2.0) + floatpower(floatabs(ST[2]), 2.0)) * 100.3;
speed = ST[3];
return 1;
}
Warum sagst du nicht, dass ich die Funktion dir noch nie gegeben habe.
Ich dachte die hätte ich dir schonmal gegeben, scheint wohl nicht so.

new Float:speed; GetPlayerSpeed_New(playerid,speed);
printf("Speed: %f",speed);

Ich hab sie gleich umbenannt, damit du nicht durcheinander kommst.

Alles anzeigen

Dachte du meintest die Funktion, die auch in dem Thread war ..
Jedenfalls klappts nun, danke dir !