rik/jlibredis
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Small refactoring to separate commands from connection

Browse Source
This commit is contained in:
Jonathan Leibiusky
2010-06-11 09:53:26 -03:00
parent 7e6ed0af41
commit 0f3217ff51
2 changed files with 7 additions and 64 deletions
Download Patch File Download Diff File Expand all files Collapse all files

48
src/main/java/redis/clients/jedis/Client.java
View File

@@ -6,16 +6,14 @@ import java.io.OutputStream;
import java.net.Socket;
import java.net.UnknownHostException;
import redis.clients.jedis.Protocol;
public class Client {
private String host;
private int port = Protocol.DEFAULT_PORT;
private Socket socket;
private boolean connected = false;
private Protocol protocol = new Protocol();
private OutputStream outputStream;
private InputStream inputStream;
protected String host;
protected int port = Protocol.DEFAULT_PORT;
protected Socket socket;
protected boolean connected = false;
protected Protocol protocol = new Protocol();
protected OutputStream outputStream;
protected InputStream inputStream;
public Client(String host) {
super();
@@ -47,17 +45,6 @@ public class Client {
public Client() {
}
public String ping() {
String command = protocol.buildCommand("PING");
try {
outputStream.write(command.getBytes());
return protocol.getSingleLineReply(inputStream);
} catch (IOException e) {
// TODO Not sure what to do here
return null;
}
}
public void connect() throws UnknownHostException, IOException {
if (!connected) {
socket = new Socket(host, port);
@@ -82,25 +69,4 @@ public class Client {
return connected;
}
public String set(String key, String value) {
String command = protocol.buildCommand("SET", key, value);
try {
outputStream.write(command.getBytes());
return protocol.getSingleLineReply(inputStream);
} catch (IOException e) {
// TODO Not sure what to do here
return null;
}
}
public String get(String key) {
String command = protocol.buildCommand("GET", key);
try {
outputStream.write(command.getBytes());
return protocol.getBulkReply(inputStream);
} catch (IOException e) {
// TODO Not sure what to do here
return null;
}
}
}

23
src/test/java/redis/clients/jedis/tests/ClientTest.java
View File

@@ -23,29 +23,6 @@ public class ClientTest extends Assert {
client.disconnect();
}
@Test
public void ping() throws UnknownHostException, IOException {
client.setHost("localhost");
client.connect();
assertTrue(client.isConnected());
String status = client.ping();
assertEquals("PONG", status);
}
@Test
public void setAndGet() throws UnknownHostException, IOException {
client.setHost("localhost");
client.connect();
String status = client.set("foo", "bar");
assertEquals("OK", status);
String value = client.get("foo");
assertEquals("bar", value);
}
@Test(expected = UnknownHostException.class)
public void checkUnkownHost() throws UnknownHostException, IOException {
client.setHost("someunknownhost");