Beiträge von Kaliber

Zitat von Squall

Einige dieser Zahlen sind VehicleID's von Flugzeugen diese sollen natürlich nicht erstellt werden

Nur Flugzeuge oder auch Helikopter?

Was ist mit Booten?

Was ist mit Zügen?

Was ist mit Anhängern? 😅

Es gibt Utils-Funktionen, da hatte jemand Langeweile und hat das mal alles rausgeschrieben: https://github.com/WoutProvost/samp-util/tree/master

samp-util/vehicle.inc at master · WoutProvost/samp-util

A set of useful utility functions for SA-MP. Contribute to WoutProvost/samp-util development by creating an account on GitHub.

github.com

new model, type;
for(;;) { //Dirty
    model = randomEx(400, 609);
    type = GetVehicleModelType(model);
    if(type != VEHICLE_TYPE_PLANE) break;
}
//Hier kannst dann model nutzen

Endlosschleifen sind immer bisschen dirty.

Besser wäre es, einfach ein konstantes Array zu haben, wo du alle modelids, welche gültig sind speicherst und daraus dann eine zufällige ID wählst.

Zum Beispiel:

stock const ValidVehicles[] = {400, 401, 402, 403}; //Oben im Skript

//Dann im Befehl:
new model = ValidVehicles[random(sizeof(ValidVehicles))];
//Bekommst direkt ein zufälliges Modell aus dem Array

Zitat von Squall

random(609) + 400

Das gibt dir eine Zahl zwischen:

(0-608)+400...also wird dein Wert zwischen: 400-1008 sein.

Zitat von Squall

400-609

Dann musst du:

400+random(210) verwenden.

Oder du definierst dir eine einfacherer Funktion (tust du ganz oben ins Skript):

#define randomEx(%0,%1) (random((%1) - (%0) + 1) + (%0))

Da kannst du einfach:

randomEx(400, 609)

verwenden

Zitat von Ioannis20x

Und das mehrere Male.

Excuse me, what

Wie zur Hölle bekommt man das denn hin. Das bedeutet du hast Funktionen, die sehen wie folgt aus:

stock test(a, b, c, d, e, f, g, h, i, j, k, l, m, n, o, p, q, r, s, t, u, v, w, x, y, z, a, b, c, d, e, f, g)

oder

public test(a, b, c, d, e, f, g, h, i, j, k, l, m, n, o, p, q, r, s, t, u, v, w, x, y, z, a, b, c, d, e, f, g)

Also dass die mehr als 32 Parameter haben.

Keine Ahnung, wie man das schafft xD Musst du das Design etwas anpassen.

Kannst ja mal eine Posten, dann können wir mal draufschauen

Ioannis20x gibt hier ein Wiki dazu: https://team.sa-mp.com/wiki/MySQL_R40.html

Die neuen Funktionen heißen ähnlich, musst du quasi nur umbenennen und die Parameter neu anordnen

#Handy

Zitat von Mogly1

Ein Lock reicht ungünstigerweise nicht aus, da das Sichern eines kritischen Abschnittes selbst kritisch ist.

Yes mb, es war nicht atomar. Sry, es ist 10 Jahre her, dass ich manuell so sections sichern musste...

Genau aus dem Grund macht das auch keiner manuell xD

Zitat von Mogly1

nicht die Korrektheit ändert

Hat ja nichts mit Clean Code zu tun. Was eine Troll Ergänzung.

Zitat von Mogly1

Also der Code von dir funktioniert nicht, dort fahren mehrere Autos über die Brücke, da der kritische Abschnitt nicht richtig ist gesichert ist

Naja, ich sag mal, eigentlich sollte der genauso funktionieren wie dein Code.

Wieso meinst du, dass er nicht funktioniert?

Das heißt in beiden Fällen kann es zu einer Race-Condition kommen, da die Variablen nicht richtig synchronisiert werden zwischen den Threads.

Dafür sollte man z.B. stdatomic.h oder Mutexes nutzen.

Beides wird hier nicht getan, somit ist das so oder so kritisch...

Mogly1 dann versuch ich mal etwas Licht ins Dunkel zu bringen.

Wieso ist das bad code?

Okay, nun zum Rest.

Ich verstehe nicht, wofür du die status Variable benötigst. Ich verstehe weiterhin nicht, wofür du die enter Variable brauchst.

Weiterhin verstehe ich nicht, wofür du die "Kollision Variable" brauchst. Der Plan ist doch, dass sowas nicht stattfinden kann, oder etwa nicht?

Wieso dann nicht z.B. einfach so lösen?

//Edit: Funktioniert nicht, außer als wenn ticket eine atomare Variable ist.

#define _DEFAULT_SOURCE
#include <unistd.h>
#include <stdlib.h>
#include <stdio.h>
#include <pthread.h>
#include <time.h>

#define NUM_THREADS   5
#define sle 0.01
#define crosses 100

clock_t start_t, end_t;

volatile long ticket;
int bridge_crosses;

void lock (long tid) {
    while(ticket != -1)
    {
        printf("Thread [%ld] waiting...\n",tid);
        sleep(sle);
    }
    ticket = tid;
}

void unlock (long tid) {
     printf("Thread [%ld] unlocked!\n", tid);
     ticket=-1;
}


void *bridge_scenario (void *threadid)
{
  long tid = (long) threadid;
  printf ("Hello World! It's me, thread #%ld !\n", tid);
  while (bridge_crosses < crosses) {
        lock(tid);
        printf("Car [%ld] crossing the road...\n", tid);
        sleep(sle);
        unlock(tid);
        bridge_crosses += 1;
        sleep(sle);
    }

  pthread_exit (NULL);
}

int main()
{
  start_t = clock();    
  pthread_t threads[NUM_THREADS];
  long t;

  // init data
  bridge_crosses = 0;
  ticket = -1;

  for (t = 0; t < NUM_THREADS; t++) {
     printf ("In main: creating thread %ld\n", t);
     int rc = pthread_create(&threads[t], NULL, bridge_scenario, (void *)t);
     if (rc) {
        printf ("ERROR; return code from pthread_create () is %d\n", rc);
        exit (-1);
     }
  }

  // joining threads
  for (t = 0; t < NUM_THREADS; t++) {
    pthread_join(threads[t], NULL);
  }

  // output
  end_t = clock();
  double total_t = (double)(end_t - start_t) / CLOCKS_PER_SEC;
  printf ("bridge crosses: %d\n time: %f \n", bridge_crosses, total_t);

  /* Last thing that main() should do */
  pthread_exit (NULL);
  return 0;
}

So könntest z.B. auch ganz einfach zählen, wie oft welches Auto gerade drübergefahren ist.

Und es kann so zu keinen Kollisionen kommen.

Der Grund btw, wieso das ohne das printf bei dir nicht funktioniert ist der, dass der Compiler die Variablen und Abfragen wegoptimiert 😄 Auf -O2 denkt der sich, ach, das brauch der nicht, weg damit xD

Damit das nicht passiert, musst du sie volatile setzen, dann lässt der Compiler die Finger davon

Hoffe es hilft etwas.

Mogly1 Noch schlechter hätte man die Frage glaube wirklich nicht formulieren können.

Was genau möchtest du erreichen.

Erkläre doch mal mehr zu deinem Code. Was genau geht denn nicht? Die printf sind ja ausgeklammert, da wird also nichts ausgegeben. Wieso geht denn das nicht?

Was soll das random Semikolon da? Soll das eine Endlosschleife sein? Dieser Code ist maximal dirty.

Eventuell mehr auf Clean-Code achten...

Achso.

Ja, das sind nur Warnings, die kannst du eig ignorieren.

Beheben tust du sie entweder, indem du deine MySQL Version updatest (empfohlen).

Oder du bearbeitest das a_mysql.inc, denn da kommen die Warnings her, das hat nichts mit deinem Code zu tun.

Dort werden Strings in den Funktionen übergeben, als Beispiel:

native MySQL:mysql_connect(host[], user[], password[], database[], MySQLOpt:option_id = MySQLOpt:0);//So steht es da

native MySQL:mysql_connect(const host[], const user[], const password[], const database[], MySQLOpt:option_id = MySQLOpt:0); //So müsste es aussehen

Also immer wenn Strings (bzw Arrays) übergeben werden, also wenn diese eckigen Klammern bei den Parametern verwendet werden, sollte man ein const verwenden, wenn diese in der Funktion nicht verändert werden

Zitat von Ioannis20x

weiß jemand woran das liegt und kann mir evtl helfen?

Wieso in aller Wellt sollte das query const sein?

const bedeutet, der Wert ist konstant und ändert sich nicht. Aber da du das hier bei der Deklaration verwendest und der "Variable" keinen Wert zuweist, wäre das immer leer...

Also mach aus dem const ein new

Um welches Game handelt es sich denn?

Also SA:MP, MTA, GTA IV, GTA V (Rage:MP/Alt:V)?

Zitat von Andosius

Sie kann in ihrem aktuellen Stand keine komplexeren Programmieraufgaben lösen.

Achso, das klang so, als ob du generell von KI sprichst. Dass das ChatGPT aktuell noch nicht vermag - ok.

Zitat von Andosius

Schlussendlich wird sie aber niemals den Menschen ersetzen können.

Hui ui ui, mächtige Worte.

Wie kommst du zu dieser Annahme?

Hast du das aus diesem Tutorial entnommen?

Hast du da am Anfang die Änderungen an der Client-Config auch vorgenommen?

Wenn nicht, check das mal aus und teste evtl das auch mit dem Chat. Vermutlich stimmt da etwas bei der Einrichtung nicht.

Sagen wir du hast 3 Felder in der Datenbank.

id, level, geld

Und weiterhin sagen wir in der Datenbank gibt es 2 Spieler, die du abfragst.

Dann ist der row_count = 2 und der field_count = 3

Also wenn Rows = 0 ist, dann wissen wir, dass kein Spieler gefunden wurde. Das ist die wichtige Information.

Wie viele Felder zurückkommen, wenn Rows zurückkommen, wissen wir eig eh schon immer, da wir sie im Query spezifizieren.

Aus diesem Grund ist diese Info eig unnötig und wurde dann später ausgelagert in eine andere Funktion.

Falls du aus irgendwelchen Gründen das auch wissen magst, gibt es die: cache_get_field_count Funktion, welche genauso funktioniert wie die row_count Funktion

Zitat von Klemme

wie muss das hier bei r41 4 aussehen hänge dort fest und werde nicht ganz aus der include schlau

Einfach:

new row_count;
cache_get_row_count(row_count);
if(!row_count)

Zitat von Klemme

wie müsste es denn nun sein das er nicht mehr mekkert?

Also eigentlich ist es richtig so.

Keine Ahnung, was qawno hat, wenn ich den Code bei mir verwende, bekomme ich keine Warnings.

Allerdings ein paar Hinweise:

1. Du kannst die Schleife mit break abbrechen, wenn du jmd gefunden hast, wozu soll er weiter suchen.
2. Du brauchst die Variable nicht mehr auf 0 setzen am Ende, da sie eh zerstört wird.
3. Den Code könntest du womöglich öfter gebrauchen und ist eine Funktionalität, welche du auslagern solltest.

Ich empfehle dir sowas:

//Nutze das Pressed Makro:
#define PRESSED(%0) \
    (((newkeys & (%0)) == (%0)) && ((oldkeys & (%0)) != (%0)))

stock IsFahrlehrerOnline() //Klar, am Besten nicht Denglisch, aber who cares
{
    for(new i; i < MAX_PLAYERS; i++)
    {
        if(IsPlayerConnected(i) && SpielerInfo[i][pFraktion] == 10) return true;
    }
    return false;
}

//Und dann unten:
if(PRSSED(KEY_SECONDARY_ATTACK) && IsPlayerInRangeOfPoint(playerid,3,1020.2769,-1348.1006,13.3283) && !IsPlayerInAnyVehicle(playerid))
{
    return ShowInfoDialog(playerid,"Information", IsFahrlehrerOnline() ? ("Es ist ein Fahrlehrer online /service!") : ("Es ist leider kein Fahrlehrer online!"));
}

Kannst auch einfach dein Skript in Notepad++ öffnen und oben auf "Codierung" klicken und schauen, dass du es nach ANSI kodierst.

Zitat von XonarZ

Kannst du mir das vielleicht erklären? Ich benutze den pawncc von open.mp.

Das hat damit nichts zu tun.

Die haben das in der Standard-Lib geändert, dass man sich nicht vertippen kann.

Du solltest da nicht TEXT_DRAW_FONT:1 nutzen, sondern die Werte, welche im enum definiert wurden.

In diesem Fall TEXT_DRAW_FONT_1

Genauso solltest du oben nicht progressbar_direction:0 nutzen, sondern direkt: BAR_DIRECTION_RIGHT

Beim Align, sollst du dann auch nicht TEXT_DRAW_ALIGN:1 nutzen, sondern direkt: TEXT_DRAW_ALIGN_LEFT

Dadurch kann man das besser lesen, als einfach nur eine 0 oder 1. Da weiß man direkt, um was es sich handelt.

Kannst ja einfach mal in das Include von omp reinschauen, das ist das omp_textdraw.inc Da siehst du die ganzen enums.

enum TEXT_DRAW_ALIGN:__TEXT_DRAW_ALIGN
{
    TEXT_DRAW_ALIGN_LEFT                       = 1,
    TEXT_DRAW_ALIGN_CENTRE,
    TEXT_DRAW_ALIGN_CENTER = TEXT_DRAW_ALIGN_CENTRE,
    TEXT_DRAW_ALIGN_RIGHT
}

Joa...ist wohl n Bug im Include.

Die "aktuelle" Version wurde ja auch nicht als Stable Release gekennzeichnet...vermutlich wurde da nicht mehr so viel getestet.

Allerdings habe ich dir mal die stable Version angehangen, dort funktioniert es.