Atuohaus will Fahrzeuge nicht createn

Jeffry · 20. September 2015

Zitat von M_McGarrett

2 und 3 aber ich hab kein fahrzeug wo mit vehicleid 3 läd sondern nur 1 und 2 und 1 läd gar nicht

Das kann schon sein, wenn die Vehicle ID 1 ein anderes Fahrzeuge bereits belegt. Du müsstest dir die dataid im Print ausgeben lassen.
Poste bitte mal den Befehl, der den Dialog anzeigen soll.

M_McGarrett · 20. September 2015

Code

public OnPlayerEnterVehicle(playerid, vehicleid, ispassenger)
{
	for(new i=1; i<MAX_AUTOHAUSCA; i++)
	{
	    if(vehicleid == AInfo[i][fahrzeugerstellt])
	    {
	        new stringauto[500];
	        printf("%d",vehicleid);
	        format(stringauto,sizeof(stringauto),"{C30000}Fahrzeugname: {FAFAFA}%s\n{C30000}Preis: {FAFAFA}%.2f$\n{C30000}Tankvolumen: {FAFAFA}%.2f\n{C30000}Tankart: {FAFAFA}%s\n\nWillst du dieses Fahrzeug kaufen?",VehicleName[GetVehicleModel(vehicleid)-400],AInfo[i][fahrzeugpreis],GetGasMax(GetVehicleModel(vehicleid)),GetPlayerVehicleType(vehicleid));
	        ShowPlayerDialog(playerid,DIALOG_KAUFAUTO,DIALOG_STYLE_MSGBOX,"Autohaus",stringauto,"Kaufen","Abbrechen");
	    }
 	}
	return 1;
}

Alles anzeigen

Jeffry · 20. September 2015

for(new i=1; i<MAX_AUTOHAUSCA; i++)
zu:
for(new i=0; i<MAX_AUTOHAUSCA; i++)

M_McGarrett · 20. September 2015

Also ich hab 4 erstellt... bei keinem ging es nach dem erstellen... nach einem Restart gingen alle bis auf das erste Fahrzeug...

Jeffry · 20. September 2015

Wie erstellst du die Fahrzeuge?
Poste dazu bitte nochmal den aktuellen Code.

Hast du die Schleife geändert?

M_McGarrett · 20. September 2015

Code

SetTimer("LoadAutohaus", 10000, false);


public LoadAutohaus(){
    for(new i=0; i<MAX_AUTOHAUSCA; i++)
	{
		new var[256],var2[128];
		format(var2, sizeof(var2),"SELECT * FROM `autohaus` WHERE `dataid` = '%d'", i);
		self_mysql_queryEx(var2);
		mysql_store_result();
		while(mysql_fetch_row_format(var2,"|")){
			mysql_get_field("dataid",var); AInfo[i][dataid] = strval(var);
			mysql_get_field("fahrzeugid",var); AInfo[i][fahrzeugid] = strval(var);
			mysql_get_field("fahrzeugpreis",var); AInfo[i][fahrzeugpreis] = floatstr(var);
			mysql_get_field("aposx",var); AInfo[i][aposx] = floatstr(var);
			mysql_get_field("aposy",var); AInfo[i][aposy] = floatstr(var);
			mysql_get_field("aposz",var); AInfo[i][aposz] = floatstr(var);
			mysql_get_field("aposa",var); AInfo[i][aposa] = floatstr(var);
			AInfo[i][fahrzeugerstellt] = CreateDynamicVehicle(AInfo[i][fahrzeugid],AInfo[i][aposx],AInfo[i][aposy],AInfo[i][aposz],AInfo[i][aposa],random(128),random(128),-1,0,0,0,50.0);
			printf("%d",AInfo[i][fahrzeugerstellt]);
			//AInfo[i][fahrzeugerstellt] = CreateVehicle(AInfo[i][fahrzeugid],AInfo[i][aposx],AInfo[i][aposy],AInfo[i][aposz],AInfo[i][aposa],random(128),random(128),-1);
		}
	}
	return 1;
}

Alles anzeigen

komisch, ich hab gerade ein fahrzeug mit /veh erstellt und dann kam da auch der Kaufdialog... das heisst vehicleid verschiebung...

Jeffry · 20. September 2015

Ahh, du hast es bei OnPlayerEnterVehicle drin.
Der Code muss in OnPlayerEnterDynamicVehicle stehen, um dem Dialog anzuzeigen. Das dürfte der Grund für die Verschiebung sein.

M_McGarrett · 20. September 2015

Nunja, alle dialoge werden nun angezeigt... nur das diesesmal leider bei dem Maverick der Bullet kommt und beim Bullet der Maverick... irgendwas will mich da ärgern

Jeffry · 21. September 2015

Versuche es so:

C

public OnPlayerEnterDynamicVehicle(playerid, vehicleid, ispassenger)
{
    for(new i=0; i<MAX_AUTOHAUSCA; i++)
    {
        printf("vehicleid: %d | fahrzeugerstellt: %d", vehicleid, AInfo[i][fahrzeugerstellt]);
        if(vehicleid == AInfo[i][fahrzeugerstellt])
        {
            new stringauto[512];
            printf("%d -> %s",vehicleid, VehicleName[GetVehicleModel(vehicleid)-400]);
            format(stringauto,sizeof(stringauto),"{C30000}Fahrzeugname: {FAFAFA}%s\n{C30000}Preis: {FAFAFA}%.2f$\n{C30000}Tankvolumen: {FAFAFA}%.2f\n{C30000}Tankart: {FAFAFA}%s\n\nWillst du dieses Fahrzeug kaufen?",VehicleName[GetVehicleModel(vehicleid)-400],AInfo[i][fahrzeugpreis],GetGasMax(GetVehicleModel(vehicleid)),GetPlayerVehicleType(vehicleid));
            ShowPlayerDialog(playerid,DIALOG_KAUFAUTO,DIALOG_STYLE_MSGBOX,"Autohaus",stringauto,"Kaufen","Abbrechen");
            break;
        }
    }
    return 1;
}

Alles anzeigen

Wenn das nicht klappt, poste bitte was im Server Log steht.

M_McGarrett · 21. September 2015

Es geht immer noch nicht...

Code

[15:36:30] vehicleid: 4 | fahrzeugerstellt: 0
[15:36:30] vehicleid: 4 | fahrzeugerstellt: 2
[15:36:30] vehicleid: 4 | fahrzeugerstellt: 3
[15:36:30] vehicleid: 4 | fahrzeugerstellt: 4
[15:36:33] vehicleid: 4 | fahrzeugerstellt: 0
[15:36:33] vehicleid: 4 | fahrzeugerstellt: 2
[15:36:33] vehicleid: 4 | fahrzeugerstellt: 3
[15:36:33] vehicleid: 4 | fahrzeugerstellt: 4
[15:36:38] vehicleid: 3 | fahrzeugerstellt: 0
[15:36:38] vehicleid: 3 | fahrzeugerstellt: 2
[15:36:38] vehicleid: 3 | fahrzeugerstellt: 3
[15:36:38] 3 -> Streifenwagen
[15:36:44] vehicleid: 2 | fahrzeugerstellt: 0
[15:36:44] vehicleid: 2 | fahrzeugerstellt: 2
[15:36:44] 2 -> Primo

Alles anzeigen

Jeffry · 21. September 2015

Ok.
Poste bitte deine GetPlayerVehicleType und die GetGasMax Funktion.

M_McGarrett · 21. September 2015

Code

GetPlayerVehicleType(vehicleid)
{
	new vtext[20];
	format(vtext, sizeof(vtext), "Unbekannt");
	new vv = GetVehicleModel(vehicleid);
	if(vv == 400 || vv == 406 || vv == 413 || vv == 414 || vv == 422 || vv == 423
	|| vv == 424 || vv == 427 || vv == 428 || vv == 430 || vv == 431 || vv == 432
	|| vv == 433 || vv == 434 || vv == 437 || vv == 440 || vv == 442 || vv == 444
	|| vv == 446 || vv == 452 || vv == 453 || vv == 455 || vv == 456 || vv == 459
	|| vv == 470 || vv == 476 || vv == 477 || vv == 478 || vv == 479 || vv == 482
	|| vv == 486 || vv == 495 || vv == 498 || vv == 499 || vv == 500 || vv == 508
	|| vv == 525 || vv == 526 || vv == 527 || vv == 529 || vv == 530 || vv == 531
	|| vv == 533 || vv == 536 || vv == 540 || vv == 543 || vv == 546 || vv == 547
	|| vv == 550 || vv == 551 || vv == 552 || vv == 554 || vv == 556 || vv == 557
	|| vv == 562 || vv == 560 || vv == 561 || vv == 565 || vv == 566 || vv == 572
	|| vv == 574 || vv == 576 || vv == 578 || vv == 579 || vv == 580 || vv == 582
	|| vv == 583 || vv == 585 || vv == 589 || vv == 600 || vv == 411 || vv == 415 || vv == 429 || vv == 480 || vv == 451 || vv == 539 || vv == 541
	|| vv == 567 || vv == 568 || vv == 571 || vv == 581 || vv == 586 || vv == 604 || vv == 461 || vv == 463 || vv == 468 || vv == 494
	|| vv == 502 || vv == 503 || vv == 506 || vv == 521 || vv == 522
	|| vv == 523 || vv == 534 || vv == 535 || vv == 542 || vv == 549
	|| vv == 555 || vv == 558 || vv == 559 || vv == 587 || vv == 602)
	{
	    vtext = "Diesel";
 	}
 	else if(vv == 401 || vv == 402 || vv == 403 || vv == 404
 	|| vv == 405 || vv == 408 || vv == 409 || vv == 410 || vv == 412 || vv == 416
 	|| vv == 418 || vv == 419 || vv == 420 || vv == 421 || vv == 426 || vv == 436
 	|| vv == 438 || vv == 439 || vv == 443 || vv == 445 || vv == 448 || vv == 454
 	|| vv == 457 || vv == 458 || vv == 462 || vv == 466 || vv == 467 || vv == 471
	|| vv == 472 || vv == 473 || vv == 474 || vv == 475 || vv == 483 || vv == 484
	|| vv == 485 || vv == 489 || vv == 490 || vv == 491 || vv == 492 || vv == 493
	|| vv == 496 || vv == 504 || vv == 505 || vv == 507 || vv == 514 || vv == 515
	|| vv == 516 || vv == 517 || vv == 518 || vv == 524 || vv == 528 || vv == 532
	|| vv == 545 || vv == 573 || vv == 575 || vv == 588 || vv == 595 || vv == 596
	|| vv == 597 || vv == 598 || vv == 599 || vv == 537 || vv == 601
	|| vv == 603 || vv == 417 || vv == 425 || vv == 460 || vv == 469 || vv == 487
	|| vv == 488 || vv == 497 || vv == 511 || vv == 512
	|| vv == 513 || vv == 519 || vv == 520 || vv == 548
	|| vv == 553 || vv == 563 || vv == 577 || vv == 592 || vv == 593)
 	{
 	    vtext = "Benzin";
  	}
 	return vtext;
}

Alles anzeigen

Code

stock Float:GetGasMax(vehmodel)
{
	if(vehmodel >= 400 && vehmodel <= 611)
	{
	    return VollerTank[(vehmodel-400)];
	}
	return 100.0;
}

Jeffry · 21. September 2015

Ok, dann das Callback so:

C

public OnPlayerEnterDynamicVehicle(playerid, vehicleid, ispassenger)
{
    for(new i=0; i<MAX_AUTOHAUSCA; i++)
    {
        printf("vehicleid: %d | fahrzeugerstellt: %d", vehicleid, AInfo[i][fahrzeugerstellt]);
        if(vehicleid == AInfo[i][fahrzeugerstellt])
        {
            new stringauto[512];
            printf("%d -> %s",vehicleid, VehicleName[GetDynamicVehicleModel(vehicleid)-400]);
            format(stringauto,sizeof(stringauto),"{C30000}Fahrzeugname: {FAFAFA}%s\n{C30000}Preis: {FAFAFA}%.2f$\n{C30000}Tankvolumen: {FAFAFA}%.2f\n{C30000}Tankart: {FAFAFA}%s\n\nWillst du dieses Fahrzeug kaufen?",VehicleName[GetDynamicVehicleModel(vehicleid)-400],AInfo[i][fahrzeugpreis],GetGasMax(GetDynamicVehicleModel(vehicleid)),GetPlayerDynamicVehicleType(vehicleid));
            ShowPlayerDialog(playerid,DIALOG_KAUFAUTO,DIALOG_STYLE_MSGBOX,"Autohaus",stringauto,"Kaufen","Abbrechen");
            break;
        }
    }
    return 1;
}

Alles anzeigen

Und füge diese neue Funktion in deinen Gamemode ein:

C

GetPlayerDynamicVehicleType(vehicleid)
{
    new vtext[20];
    format(vtext, sizeof(vtext), "Unbekannt");
    new vv = GetDynamicVehicleModel(vehicleid);
    if(vv == 400 || vv == 406 || vv == 413 || vv == 414 || vv == 422 || vv == 423
    || vv == 424 || vv == 427 || vv == 428 || vv == 430 || vv == 431 || vv == 432
    || vv == 433 || vv == 434 || vv == 437 || vv == 440 || vv == 442 || vv == 444
    || vv == 446 || vv == 452 || vv == 453 || vv == 455 || vv == 456 || vv == 459
    || vv == 470 || vv == 476 || vv == 477 || vv == 478 || vv == 479 || vv == 482
    || vv == 486 || vv == 495 || vv == 498 || vv == 499 || vv == 500 || vv == 508
    || vv == 525 || vv == 526 || vv == 527 || vv == 529 || vv == 530 || vv == 531
    || vv == 533 || vv == 536 || vv == 540 || vv == 543 || vv == 546 || vv == 547
    || vv == 550 || vv == 551 || vv == 552 || vv == 554 || vv == 556 || vv == 557
    || vv == 562 || vv == 560 || vv == 561 || vv == 565 || vv == 566 || vv == 572
    || vv == 574 || vv == 576 || vv == 578 || vv == 579 || vv == 580 || vv == 582
    || vv == 583 || vv == 585 || vv == 589 || vv == 600 || vv == 411 || vv == 415 || vv == 429 || vv == 480 || vv == 451 || vv == 539 || vv == 541
    || vv == 567 || vv == 568 || vv == 571 || vv == 581 || vv == 586 || vv == 604 || vv == 461 || vv == 463 || vv == 468 || vv == 494
    || vv == 502 || vv == 503 || vv == 506 || vv == 521 || vv == 522
    || vv == 523 || vv == 534 || vv == 535 || vv == 542 || vv == 549
    || vv == 555 || vv == 558 || vv == 559 || vv == 587 || vv == 602)
    {
        vtext = "Diesel";
     }
     else if(vv == 401 || vv == 402 || vv == 403 || vv == 404
     || vv == 405 || vv == 408 || vv == 409 || vv == 410 || vv == 412 || vv == 416
     || vv == 418 || vv == 419 || vv == 420 || vv == 421 || vv == 426 || vv == 436
     || vv == 438 || vv == 439 || vv == 443 || vv == 445 || vv == 448 || vv == 454
     || vv == 457 || vv == 458 || vv == 462 || vv == 466 || vv == 467 || vv == 471
    || vv == 472 || vv == 473 || vv == 474 || vv == 475 || vv == 483 || vv == 484
    || vv == 485 || vv == 489 || vv == 490 || vv == 491 || vv == 492 || vv == 493
    || vv == 496 || vv == 504 || vv == 505 || vv == 507 || vv == 514 || vv == 515
    || vv == 516 || vv == 517 || vv == 518 || vv == 524 || vv == 528 || vv == 532
    || vv == 545 || vv == 573 || vv == 575 || vv == 588 || vv == 595 || vv == 596
    || vv == 597 || vv == 598 || vv == 599 || vv == 537 || vv == 601
    || vv == 603 || vv == 417 || vv == 425 || vv == 460 || vv == 469 || vv == 487
    || vv == 488 || vv == 497 || vv == 511 || vv == 512
    || vv == 513 || vv == 519 || vv == 520 || vv == 548
    || vv == 553 || vv == 563 || vv == 577 || vv == 592 || vv == 593)
     {
         vtext = "Benzin";
      }
     return vtext;
}

Alles anzeigen

M_McGarrett · 21. September 2015

Komisch, es funktioniert... Bin dir sehr dankbar für deine hilfe

breadfish.de 12. März 2022