как поддерживать переменные куки и сеансы с jsoup? - PullRequest
Новая
       27

как поддерживать переменные куки и сеансы с jsoup?

5 голосов
/ 11 октября 2011 
public boolean isGood(String path)
{
    if (p != path)
    {
        good = false;
    }

    if (good)
    {
        try 
        {
            Connection connection = Jsoup.connect(path);
            Map<String, String> cookys = Jsoup.connect(path).response().cookies();

            if (cookys != cookies)
                cookies = cookys;

            for (Entry<String, String> cookie : cookies.entrySet()) 
            {
                connection.cookie(cookie.getKey(), cookie.getValue());
            }

            Doc = connection.get();
            good = true;
        }
        catch (Exception e) 
        {
            rstring = e.getMessage().toString();
            good = false;
        }
    }
    else
    {
        try
        {
            Response response = Jsoup.connect(path).execute();
            cookies = response.cookies();
            Doc = response.parse();
            good = true;
        }
        catch (Exception e) 
        {
            rstring = e.getMessage().toString();
            good = false;
        } 
    }       
    return good;
}

Этот метод не верен. То, что я пытаюсь выяснить, - это способ не знать, какие куки будут существовать, иметь возможность обрабатывать изменения куки, а также поддерживать сеансы.

Я пишу приложение для моего форума, посвященного простым машинам, и оно меняет конфигурацию файлов cookie, когда вы щелкаете по кнопке для некоторого пользовательского поведения.

Но если приложение хорошо работает для моего сайта, я собирался опубликовать версию, которую другие могли бы использовать для других форумов.

Я знаю, что я иду в правильном направлении, но логика вроде как пинает меня в зад.

Любой совет будет принят с благодарностью.

Ответы [ 2 ]

13 голосов
/ 11 октября 2011

Этот код очень запутанный.Поток нелогичен, а обработка исключений плохая.Сравнения ссылок на объекты, такие как if (p != path) и if (cookys != cookies), не имеют никакого смысла.Для сравнения содержимого объекта вам нужно использовать метод equals().

Кстати, я понимаю, что вы хотите поддерживать файлы cookie в нескольких последующих запросах Jsoup в том же домене.В этом случае вам необходимо в основном придерживаться следующего потока: 

Map<String, String> cookies = new HashMap<String, String>();

// First request.
Connection connection1 = Jsoup.connect(url1);
for (Entry<String, String> cookie : cookies.entrySet()) {
    connection1.cookie(cookie.getKey(), cookie.getValue());
}
Response response1 = connection1.execute();
cookies.putAll(response1.cookies());
Document document1 = response1.parse();
// ...

// Second request.
Connection connection2 = Jsoup.connect(url2);
for (Entry<String, String> cookie : cookies.entrySet()) {
    connection2.cookie(cookie.getKey(), cookie.getValue());
}
Response response2 = connection2.execute();
cookies.putAll(response2.cookies());
Document document2 = response2.parse();
// ...

// Third request.
Connection connection3 = Jsoup.connect(url3);
for (Entry<String, String> cookie : cookies.entrySet()) {
    connection3.cookie(cookie.getKey(), cookie.getValue());
}
Response response3 = connection3.execute();
cookies.putAll(response3.cookies());
Document document3 = response3.parse();
// ...

// Etc.

Это можно изменить следующим образом: 

private Map<String, String> cookies = new HashMap<String, String>();

public Document get(url) throws IOException {
    Connection connection = Jsoup.connect(url);
    for (Entry<String, String> cookie : cookies.entrySet()) {
        connection.cookie(cookie.getKey(), cookie.getValue());
    }
    Response response = connection.execute();
    cookies.putAll(response.cookies());
    return response.parse();
}

, который можно использоватьas 

YourJsoupWrapper jsoupWrapper = new YourJsoupWrapper();

Document document1 = jsoupWrapper.get(url1);
// ...

Document document2 = jsoupWrapper.get(url2);
// ...

Document document3 = jsoupWrapper.get(url3);
// ...

Обратите внимание, что в следующем выпуске Jsoup 1.6.2 будет добавлен новый метод Connection#cookies(Map), который должен делать этот цикл for каждый раз лишним.

1 голос
/ 17 ноября 2013

+ 1 для BalusC

Я кое-что изменил в вашем коде, и он работает для меня, поэтому вы получаете куки с сайта и только потом получаете Документ 

public Document get(String url) throws IOException {
    Connection connection = Jsoup.connect(url).userAgent("Mozilla/5.0 (Windows; U; WindowsNT 5.1; en-US; rv1.8.1.6) Gecko/20070725 Firefox/2.0.0.6");
    Connection.Response response = connection.execute();
    connection.cookies(response.cookies());
    return connection.get();
}
...