Select απο Table HELP!!!Java

[guybrush07]

Ρε παιδιά help me δεν βγάζω άκρη...έχω ένα πίνακα με 2 στήλες ID και names
θέλω επιλέγοντας το ID μου εμφανίζει το όνομα που πρέπει
Δείτε τι έχω κάνει ...μη με κράξετε αρχάριος.

package Mypack;

import java.sql.Connection;
import java.sql.DriverManager;
import java.sql.Statement;
import javax.annotation.Resource;
import javax.jws.WebMethod;
import javax.jws.WebParam;
import javax.jws.WebService;
import javax.sql.DataSource;
import java.sql.ResultSet;


@WebService(serviceName = "MTEL")
public class MTEL {


    @Resource(name = "data1")
    private DataSource data1;


    @WebMethod(operationName = "Search")
    public String Search(@WebParam(name = "numb") String numb) {
        try {
            Class.forName("com.mysql.jdbc.Driver");
            Connection con = DriverManager.getConnection("jdbc:mysql://192.168.1.3:3306/mydata", "root", "");
            Statement stmt = con.createStatement();
            
String sql = "SELECT name FROM people WHERE ID='" + numb + "";          
            ResultSet rs = stmt.executeQuery(sql);
            while (rs.next()) {
                uname = rs.getString("name");
                int id = rs.getInt("ID");
                System.out.println(uname);


            }






        } catch (Exception e) {
            System.out.println(e.getMessage());
        }
        return uname;
    }
}

Ότι και αν έχω δοκιμάσει παίρνω πάντα null σαν αποτέλεσμα στο http://localhost:.............

[ownagE_]

Είναι σωστή η IP που δίνεις στο connection string;
Αν ο SQL server τρέχει στο ίδιο PC που τρέχει η εφαρμογή τότε θα πρέπει να βάλεις 127.0.0.1 ή localhost.

Και 2 σημεία που είδα στα γρήγορα:

String sql = "SELECT name FROM people WHERE ID='" + numb + "";      
Δεν θα 'πρεπε να 'ναι ως εξής..;
String sql = "SELECT name FROM people WHERE ID='" + numb + "'";

..και..

while (rs.next()) {
                uname = rs.getString("name");
                int id = rs.getInt("ID");    // είναι σωστό αυτό; αφου νομίζω το resultset περιέχει μόνο τα names! (select name ..)
                System.out.println(uname);


            }

[nikospara]

Καλησπέρα,

Επιπλέον των παραπάνω, μήν κάνεις ΠΟΤΕ string concatenation δεδομένων που δίνει ο χρήστης για δημιουργία SQL queries. Διάβασε για SQL injection.

Πιο σωστός τρόπος εν προκειμένω είναι:

import java.sql.PreparedStatement;
...
PreparedStatement stmt = con.prepareStatement("SELECT id, name FROM people WHERE id=?"); // προσέθεσα το id στο SELECT
int id = Integer.parseInt(numb);
stmt.setInt(1, id);
ResultSet rs = stmt.executeQuery();
...

Εξίσου σημαντικό είναι να διασφαλισεις ότι θα κλείνεις πάντα το Connection, ως εξής:

Connection conn = null;
try {
    ...
}
catch(...) {
    ...
}
finally {
    if( conn != null ) {
        try{ conn.close(); }
        catch(Exception ignore) {}
    }
}

(Αν θές να είσαι πιο εντάξει πρέπει να κάνεις το ίδιο και για τα PreparedStatement, ResultSet, δηλαδή να τα κλείνεις το finally.)

Όταν είσαι εντάξει με αυτά, δές και για connection pooling/datasources αντί να φτιάχνεις το java.sql.Connection εξαρχής (και να έχεις κωδικούς πρόσβασης hardcoded).

Καλή επιτυχία