3

Sou novo no desenvolvimento Android, estou usando SQLite como banco. Preciso fazer um SELECT, e retornar os valores de três colunas da minha tabela. Depois preciso setar os atributos do meu objeto com os valores de retorno desse SELECT .

1- Eu possuo um objeto chamado ImageColor, conforme código abaixo:

public class ColorImage {

    private int RedColor;
    private int GreenColor;
    private int BlueColor;
    private int Id;
    private String Nome;

   //Getters e Setters
}

2 - Minha tabela no banco está dessa forma:

inserir a descrição da imagem aqui

Eu quero fazer um SELECT e retornar os valores do RED, GREEN e BLUE.

o meu método que faz o SELECT e retorna um objeto do tipo ImageColor e esse:

public ColorImage SelectedColor(String nome){

    ColorImage color = new ColorImage();

    String selectQuery = "SELECT red, green, blue FROM COLOR WHERE nome =" + nome;

    SQLiteDatabase banco = this.getWritableDatabase();

    Cursor cursor = banco.rawQuery(selectQuery, null);

    color.setRedColor(cursor.getInt(0));
    color.setGreenColor(cursor.getInt(1));
    color.setBlueColor(cursor.getInt(2));

    return color;
}

Porém não está funcionando, não sei o que está errado. Alguém sabe o que está errado?

1
  • Não retorna valor, não sei o que está errado.
    – Mik3i4a5
    13/11/2017 às 21:23

1 Resposta 1

7

A construção de comandos SQL dinâmicos com concatenação nem sempre produz o resultado esperado/válido.

Neste caso faltam as plicas. Elas são necessárias quando se usa uma string como valor. A forma correcta será assim:

String selectQuery = "SELECT red, green, blue FROM COLOR WHERE nome = " + "'" + nome + "'";

Para evitar este tipo de erros o indicado é usar placeholders para receber as partes dinâmicas e fornecer em separado os valores a usar. O método garantirá a correcta atribuição dos valores aos placeholders.

String selectQuery = "SELECT red, green, blue FROM COLOR WHERE nome = ?";
...
...
Cursor cursor = db.rawQuery(selectQuery, new String[] { nome });

Não esqueça de chamar cursor.moveToFirst() antes de utilizar o cursor.

public ColorImage SelectedColor(String nome){

    ColorImage color = new ColorImage();

    String selectQuery = "SELECT red, green, blue FROM COLOR WHERE nome = ?";

    SQLiteDatabase banco = this.getWritableDatabase();

    Cursor cursor = db.rawQuery(selectQuery, new String[] { nome });
    if(cursor.moveToFirst()){
        color.setRedColor(cursor.getInt(0));
        color.setGreenColor(cursor.getInt(1));
        color.setBlueColor(cursor.getInt(2));
    }
    cursor.close();

    return color;
}

Melhor ainda será usar um dos métodos query()

Cursor query(String table, 
             String[] columns, 
             String selection, 
             String[] selectionArgs, 
             String groupBy, 
             String having, 
             String orderBy)

No seu caso ficaria assim:

Cursor query("COLOR", 
             new String[]{"red", "green", "blue"}, 
             String "nome = ?", 
             new String[]{nome}, 
             null, 
             null, 
             null);
4
  • obrigado por sua resposta. Fiz exatamente como você disse, mas o App fecha por erro
    – Mik3i4a5
    13/11/2017 às 23:06
  • E qual é o erro?
    – ramaral
    14/11/2017 às 12:24
  • Consegui resolver o erro, mas ele sempre retorna zero para qualquer cor que eu procure. Ele não entra no if( cursor.moveToFirst() )
    – Mik3i4a5
    14/11/2017 às 21:26
  • Se não entra no ifé porque o cursor vem vazio. Verifique se existe algum registo que tenha um valor na coluna nome exactamente igual ao valor que está a passar ao método.
    – ramaral
    14/11/2017 às 22:53

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .