Ihr solltet alle nochmal in die Scriptschule gehen. 
namespace
das ist unnötig unterlass es einfach 
@TE
public OnDialogResponse(playerid, dialogid, response, listitem, inputtext[])
{
switch(dialogid)
{
case DIALOG_SMARKT:
{
switch(listitem)
{
case 0:
{
if(GetPlayerMoney(playerid) <3400) return SendClientMessage(playerid,ROT,"Du hast nicht genug Geld.");
GivePlayerMoney(playerid,-3400);
GivePlayerWeapon(playerid,31,400);
}
case 1:
{
if(GetPlayerMoney(playerid) <3250) return SendClientMessage(playerid,ROT,"Du hast nicht genug Geld.");
GivePlayerMoney(playerid,-3250);
GivePlayerWeapon(playerid,30,400);
}
case 2:
{
if(GetPlayerMoney(playerid) <2400) return SendClientMessage(playerid,ROT,"Du hast nicht genug Geld.");
GivePlayerMoney(playerid,-2400);
GivePlayerWeapon(playerid,29,400);
}
}
}
}
return 1;
}
Das sollte dein problem lösen