Проблема с кодом

[BlackSmith]

попробуй так SELECT * FROM character_mail WHERE char_name= ? ....

[incognito]

нету введения переменой _activeChar во втором методе Оо не проще объединить 2 метода в один=)?!

[SunnyX]

Вместо того, что бы пальцы гнуть, лучше бы на код внимательно посмотрели.

В SQL строковые переменных необходимо обрамлять одинарными кавычками.
Так:

Код:

String query = "SELECT * FROM character_mail WHERE recipient = '"+ _activeChar.getName() + "' ORDER BY author ASC";

Добавлено через 1 минуту
А самый лучший вариант, как и сказал BlackSmith: использовать конструктор, который сам генерирует часть кода, а ты через функции подставляешь свои значения.

[VISTALL]

спс всем, все красавцы, все мучают, всунуть в String переменные для запроса, дурной вопрос а зачем тогда "PreparedStatement"?a

Пол дня уже бьюсь над одним куском, не работает и всё тут
[src=#java]
public Mail(L2PcInstance character)
{
_activeChar = character;
}

public Mail()
{
if (_activeChar == null)
return;
Connection con = null;
try
{
con = L2DatabaseFactory.getInstance().getConnection();
String query = "SELECT * FROM character_mail WHERE recipient="+ _activeChar.getName() +" ORDER BY author ASC";
PreparedStatement statement = con.prepareStatement(query);
ResultSet rset = statement.executeQuery();
while (rset.next())
[/src]
Одно НО - это то, что проблема находится тут

Код:

SELECT * FROM character_mail WHERE recipient="+ _activeChar.getName() +" ORDER BY author ASC

В этом уверен на 100%. Может есть какой-нибудь другой реальный способ селекции из базы по Имени (char_name) или Номеру Объекта (obj_id)?
P.S.: Для особо одарённых, вылетает ошибка NullPointerException

[src=#java]
public Mail(L2PcInstance character)
{
_activeChar = character;
}

public Mail()
{
if (_activeChar == null)
return;
Connection con = null;
try
{
con = L2DatabaseFactory.getInstance().getConnection();
PreparedStatement statement = con.prepareStatement("SELECT * FROM character_mail WHERE recipient=? ORDER BY author ASC");
statement.setString(1, _activeChar.getName());
ResultSet rset = statement.executeQuery();
while (rset.next())
[/src]