Inventar System

Sehr nice, gut gemacht.

Very nice. Finde dieses Include für ein paar Nebenjobs nützlich. Danke!

Kannst Du mir mal erklären was das machen soll, ich begreifs echt nicht. Warum ein retvalue, wenn er nicht benutzt werden kann - Unabhängig davon, ob das Item erfolgreich oder nicht erfolgreich erstellt worden ist returnst Du denselben Wert?

stock GivePlayerItem(playerid, item[]) {
new slot = GetFreeSlot(playerid);
if(slot == -1) return 0xF;
format(Item[playerid][slot],32,item);
return 0xF;
}

Item wird erfolgreich erstellt -> retvalue 0xF
Item wird nicht erfolgreich erstellt, da GetFreeSlot -1 returnt -> retvalue 0xF

Ich würde eigentlich gerne wissen, wie GivePlayerItem abgelaufen ist, ohne gleich GetItem benutzen zu müssen. Würd mich über ne Erklärung freuen. Besser was 100% korrektes zu releasen als was halbfertiges, bei Unklarheiten PN

Gruss

Kaliber dann schreib doch:

if(foobar == -1) return;

Oder was schöner ist, hau das ganze in ne if Abfrage (im Code returnen ist nicht so schön). Also zB so:

GivePlayerItem(playerid, item[])
{
new slot = GetFreeSlot(playerid)
if(slot != INVALID_SLOT)
{
format(Item[playerid][slot],32,item);
}
}

INVALID_SLOT kannste als globale Konstante definieren, denn man kann raten, was -1 bedeutet, eine Stringbezeichnung ist da schon viel eindeutiger

Kaliber
Schlussendlich ist alles möglich. Ich sprech aber aus eigener Erfahrung. Irgendwann haste ein C Projekt mit 15'000 Zeilen und merkst, dass dies und jenes nicht stimmt, veränderst returnvalues und schlussendlich haste ein Stück Scheisse und kannst Teile neu schreiben. Sprich Du erleichterst dir das Leben, wenn Du dir einen einheitlichen Stil angewöhnst. Auch anderen Entwicklern kannste so das Leben leichter machen (namentlich in Open-Source Projekten)

Ich würde es zB so machen. Grund für das Grosschreiben der globalen Werte ist, dass sie globale Werte darstellen. Ich verwende bewusst kein #define, da dieses nur begrenzt typensicher ist:

// Global
new INVALID_SLOT = -1;
new VALID_SLOT_ACTION = 1;

GivePlayerItem(playerid, item[])
{
new slot = GetFreeSlot(playerid)
if(slot != INVALID_SLOT)
{
format(Item[playerid][slot],32,item);
return VALID_SLOT_ACTION; // Oder alternativ die slotid
}
return INVALID_SLOT;
}

Gruss

Sorry das ich mich grade Dumm anstelle aber versteh ich grade nicht also das ist ein Inventar System wo halt Sachen gespeichert sind aber wie speicher ich etwas wird es automatisch oder wie ähhh?. SORRY!

Mit freundlichen Grüßen

Coole Sache,würdest du mir sowas als Kofferraumsystem machen gg Taschengeld?

Schönes System aber eigentlich keine Grosse Arbeit
Trotzdem Danke

Zitat von Kaliber

Nein, die Sachen werden nicht automatisch gespeichert, aber du kannst das ganz simpel speichern...

Halt mit deinem Speicherungssystem und GetItem(playerid, slot); Das sähe dann so aus (Beispiel mit dini):

dini_Set(Spielerdatei,"Item1",GetItem(playerid,0));
dini_Set(Spielerdatei,"Item2",GetItem(playerid,1));
//usw bis MAX_SLOTS erreicht ist

Was genau meinst du...erkläre das mal genauer

mfg.

Alles anzeigen

Achso verstehe danke!

Die Slots fangen mit 0 an oder?

Und man arbeitet hier nur mit Strings oder? Also auch so speichern in der DB?

- Tomsen

Zitat von Tomsen

Die Slots fangen mit 0 an oder?

Und man arbeitet hier nur mit Strings oder? Also auch so speichern in der DB?

- Tomsen

Jap

Warum machst du bitte nichts ordentlich?

Dass du static verstanden hast, find ich sogar gut
Nur was mich irgendwie ankotzt ist, das du Systeme schreibst sie päsentierst, aber nicht drüber nachdenkst, sie sinnvoll zu gestalten.

Beispiel:
stock GetItem(playerid,slot) {
if(slot > MAX_SLOTS) return ERROR;
return Item[playerid][slot];
}
Das wird erfolgreich sein immer, egal was kommt, ob ich jetzt über dem "Limit" liege oder nicht.
Was nützt es, wenn der Rückgabewert "Fehler" ist?
Weißt du warum kalcor die Funktion GetPlayerName so gestaltet hat wie er es getan hat?

Nicht weil er keinen anderen Lösungsweg wusste oder weil PAWN da einwenig rummeckert.
Sondern, weil man damit wunderbar auch prüfen kann, ob der Spieler auch online ist.

sprich if(GetPlayerName(playerid,name,MAX_PLAYER_NAME)) // wenn online - was dann?

Das auf dein "Sytsem" zu übertragen wäre überhaupt nicht das Problem.
Du gibst dann nicht "Fehler" zurück, sondern 0.

Funktionsheader würde dann so aussehen
stock GetItem(playerid,slot, name[],len)
Es ist viel einfacher zu handhaben, als das
if(!strcmp(GetItem(playerid,25),"Fehler"))

Man kann es direkt weiterverarbeiten ohne unnötiges Format.
Auch eins der Gründe warum der Spielername stock - kompletter Müll ist

Dazu ist deine Deklaration, absoulut belustigend
static ...,ERROR[32]="FEHLER",INVALID_SLOT=-1;
Warum es dem Compiler in Mund schütten, wenn man auch den RAM belasten kann?

Item[playerid][s] = "";
Ein ganz schlimmer Fehler, der dir das "beigebracht" hat - gehört verprügelt.
Du sorgst hier nur für memory leaks.

Nutze gleich format oder strdel, dafür sind diese da

Auch wenn ich mich wiederhole, denk bitte nächstesmal einfach deine Systeme durch,
nicht das man beim anschauen schon Kopfschmerzen bekommt

Wieso wird der Dialog nicht angezeigt?

new string[256];
format(string,sizeof(string),"%s\n%s\n%s\n%s\n%s",GetItem(playerid,1),GetItem(playerid,2),GetItem(playerid,3),GetItem(playerid,4),GetItem(playerid,5));
ShowPlayerDialog(playerid,DialogInventar,DIALOG_STYLE_LIST,"Dein Inventar",string,"Ok","Abbrechen");

- Tomsen

Unglaubliche 32 Zeilen Effektivcode und trotzdem noch so viel Mist. Das grenzt schon an die Möglichkeit des Unmöglichen. Und für sowas erhält man noch positive Bewertungen? Die Leute haben sich den Code wahrscheinlich nichtmal angeschaut. Ich meine... wenn du wenigstens noch eine tolle Speicherung oder sowas hättest... aber das ist total sinnlos.