Anfänger Release [Selfmade]

Der Gm an sich eig Klasse 8/10

Für Anfänger ein guters start

Mfg

Machst du weitere Updates?

Nirgends steht, dass es ein 100% Selfmade ist?
Lesen, bevor man postet, bitte!

Zitat von PrimeTime.

ich finde es nicht so gut für anfänger da er nicht gerade Resourcensparend ist....

Ehm ja ganz toller Kommentar, als ob Anfänger genau wüssten wie sie Resourcensparend/schonend scripten -.-'

Wäre aber schön wenn du das ma updaten könntest, unteranderem sind publics total unnötig wenn du sie nicht per Timer oder CallLocalFunction oder CallRemoteFunction aufrufst
Du könntest auch noch den schnipsel einbauen den ich gepostet hab bzw die anderen daran anpassen

wie mach mih auf den server admin

Zitat von magnussoft

Eine frage was bringt CallLocalFunction ?!?

Kein Problem

Ich versuchs ma zu erklären
Wenn du eine Include schreibst kannst du ja nicht auf die Funktionen des GMs zugreifen, aber mit der CallLocalFunction geht das.
Du greifst sozusagen auf das aktive Script zu, in dem die Include eingebunden ist.

wie kann mal sich auf den server admin machen mit rcon geht das ja net hmm

Zitat von Dixon

Nirgends steht, dass es ein 100% Selfmade ist?
Lesen, bevor man postet, bitte!

Es steht aber auch nirgends im Script, dass er teile kopiert hat - ganz im Gegenteil er schreibt dort sogar, dass alles von ihm ist:

Zitat

// Die Credits "müssen" drinne bleiben
// Jeder darf das Script bearbeiten usw aber man "Soll" erwähnen das der Grundbau von mir ist. Nun wünsche ich dir Viel Spaß mit diesem Script.
// Mit freundlichen grüßen
//Tobi aka magnussoft

Zitat

if (strcmp("/Credits", cmdtext,true, 10) == 0)
{
SendClientMessage (playerid, LIGHTGREEN, "~~~Credits~~~");
SendClientMessage (playerid, LIGHTGREEN, "[Server Owner] Tobi");
SendClientMessage (playerid, LIGHTGREEN, "[Scripter] Tobi");
SendClientMessage (playerid, LIGHTGREEN, "[Mapper] Tobi");
return 1;
}

Alles anzeigen

Ausserdem ist die Ueberschrift bei pastebin (http://pastebin.com/68RM42m2) 'Magnussoft Selfmade!!!' - wo ist da bitte irgendwo der Hinweis darauf, dass er das Script teilweise einfach kopiert hat anstatt selbst zu scripten?
Er selbst besteht darauf, dass man seinen Namen drinnen laesst, gibt anderen Leuten aber keine Credits.

Ich hab mir das Script nochmal genauer angesehen und mir ist eine Sache besonders ins Auge gefallen, die besonders schlecht umgesetzt wurde

nämlich dieser Teil:
public OnPlayerStateChange(playerid, newstate, oldstate)
{
new newcar = GetPlayerVehicleID(playerid);
if(newstate == PLAYER_STATE_DRIVER)
{
if(LSPDCar(newcar))
{
if (pInfo[playerid][Fraktion] == 1)
{}
else
{
RemovePlayerFromVehicle(playerid);
SendClientMessage(playerid, ROT, "Du bist kein Polizist");
return 1;
}
}
}
if(newstate == PLAYER_STATE_DRIVER)
{
if(FBICar(newcar))
{
if (pInfo[playerid][Fraktion] == 4)
{}
else
{
RemovePlayerFromVehicle(playerid);
SendClientMessage(playerid,ROT,"Du bist kein Mietglied der FBI");
return 1;
}
}
}
if(newstate == PLAYER_STATE_DRIVER)
{
if(SFPDCar(newcar))
{
if (pInfo[playerid][Fraktion] == 5)
{}
else
{
RemovePlayerFromVehicle(playerid);
SendClientMessage(playerid,ROT,"Du bist kein Mietglied des SFPD");
return 1;
}
}
}
if(newstate == PLAYER_STATE_DRIVER)
{
if(MEDICCar(newcar))
{
if (pInfo[playerid][Fraktion] == 4)
{}
else
{
RemovePlayerFromVehicle(playerid);
SendClientMessage(playerid,ROT,"Du bist kein Mietglied der Medics");
return 1;
}
}
}
return 1;
}
Was soll das werden ne Party?
Alter grauenhaft, nicht ma als ich gecopiet und gepastet hab (damals :D) sah das so aus

Ich nehm den Code ma kurz auseinander
als erstes nehmen wir mal an der Spieler ist in der Fraktion Medic
public OnPlayerStateChange(playerid, newstate, oldstate)
{
new newcar = GetPlayerVehicleID(playerid);
if(newstate == PLAYER_STATE_DRIVER) // Du fragst hier ab ob der Spieler Fahrer ist
{
if(LSPDCar(newcar))
{
if (pInfo[playerid][Fraktion] == 1) { } // unnötige if else verschachtlung if(pInfo[playerid][Fraktion] != 1) reicht voll kommen aus
else
{
RemovePlayerFromVehicle(playerid);
SendClientMessage(playerid, ROT, "Du bist kein Polizist");
return 1;
}
}
}
if(newstate == PLAYER_STATE_DRIVER) // Da wir ja annehmen das der Spieler in der Fraktion Medic ist läuft der Code weiter und du fragst wieder ab ob Fahrer ist -> du machst das 2 mal wozu?
{
if(FBICar(newcar))
{
if (pInfo[playerid][Fraktion] == 4) { } // Gleiche wie beim LSPD
else
{
RemovePlayerFromVehicle(playerid);
SendClientMessage(playerid,ROT,"Du bist kein Mietglied der FBI");
return 1;
}
}
}
if(newstate == PLAYER_STATE_DRIVER) // Hier schon wieder ne abfrage es reicht einmal vollkommen aus
{
if(SFPDCar(newcar))
{
if (pInfo[playerid][Fraktion] == 5) { } // Gleiche wie oben -.-'
else
{
RemovePlayerFromVehicle(playerid);
SendClientMessage(playerid,ROT,"Du bist kein Mietglied des SFPD");
return 1;
}
}
}
if(newstate == PLAYER_STATE_DRIVER) // Hier haben wir nummer 4 deiner Abfrage... Es wird langsam langweilig...
{
if(MEDICCar(newcar))
{
if (pInfo[playerid][Fraktion] == 4) // Gleiche wie vorher auch -.-'
{}
else
{
RemovePlayerFromVehicle(playerid);
SendClientMessage(playerid,ROT,"Du bist kein Mietglied der Medics");
return 1;
}
}
}
return 1;
}
Du bist jetzt 4 Mal die Abfrage durchgegangen ob der Spieler Fahrer ist -.-'
Der Server wird sich nachher bei größeren Script bedanken...
Und Newbies lernen falsche Techniken

Das mit den car Funktionen hab ich dir ja schon erklärt