Беда с парсингом - Программирование Android

@Tro0P · Регистрация: 14.11.2015

в приложении при запуске в onCreate, должны динамически создаваться кнопки исходя из количества эл-то String [], этот массив возвращается в результате метода парсинга сайта новостей. Метод рабочий, проверял в java SE приложении. при запуске выдаёт ошибку, не могу понять причины

Java

  protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);
        Toolbar toolbar = (Toolbar) findViewById(R.id.toolbar);
        setSupportActionBar(toolbar);
 
        FloatingActionButton fab = (FloatingActionButton) findViewById(R.id.fab);
        fab.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {
                Snackbar.make(view, "Replace with your own action", Snackbar.LENGTH_LONG)
                        .setAction("Action", null).show();
            }
        });
        try {
            News_Parsing newsParsing=new News_Parsing();
            Document doc = Jsoup.connect("http://ria.ru/export/rss2/politics/index.xml").timeout(5000).get();
            Elements news = doc.select("item title");
            Title=new String[newsParsing.parsingTitle(news).length];
            Title=newsParsing.parsingTitle(news);
 
        //Elements news = doc.select("ss");
        lMain=(LinearLayout)  findViewById(R.id.lMain);
       // z=new int[20];
        newBtn=new Button[Title.length];
        for(int k = 0; k < Title.length; k++){
            newBtn[k] = new Button(this);
            newBtn[k].setText(Title[k]);
            // button.setOnClickListener(listener);
            newBtn[k].setId(View.generateViewId());
            newBtn[k].setOnClickListener(this);
            lMain.addView(newBtn[k], 200, 500);
        }}
        catch (IOException e){
            e.printStackTrace();
        }
    }

@Mikalai · 26.11.2015, 01:11

Tro0P, нельзя обращаться в сеть в основном потоке. Используй AsyncTask

@Tro0P · 29.11.2015, 18:28 **[ТС]**

переделал след. образом

Java

 protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);
        Toolbar toolbar = (Toolbar) findViewById(R.id.toolbar);
        setSupportActionBar(toolbar);
 
        FloatingActionButton fab = (FloatingActionButton) findViewById(R.id.fab);
        fab.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {
                Snackbar.make(view, "Replace with your own action", Snackbar.LENGTH_LONG)
                        .setAction("Action", null).show();
            }
        });
 
        lMain=(LinearLayout) findViewById(R.id.lMain);
        scroll=(ScrollView) findViewById(R.id.scroll);
        new AsyncT().execute();
 
    }
 
    @Override
    public void onClick(View v) {
        for (int i=0;i<button.length;i++){
            if(v.getId()==button[i].getId()){
                button[i].setText(""+i);
            }
        }
 
    }
 
    class AsyncT extends AsyncTask<Void,Void,Void> {
        @Override
        protected Void doInBackground(Void... params) {
            return null;
        }
 
        protected void onPostExecute (Void result){
            Document doc;
            doc=null;
            try{
                doc = Jsoup.connect("http://ria.ru/export/rss2/politics/index.xml").timeout(5000).get();
            } catch (IOException e){
                e.printStackTrace();
            }
            Elements news=doc.select("title");
            newsPars=new NewsPars();
            title=newsPars.parsingTitle(news);
            title=newsPars.parsingTitle(news);
            for (int i=0;i<title.length;i++){
                button=new Button[title.length];
                button[i]=new Button(pc.vorgul.testnewsparser.MainActivity.this);
                button[i].setText(title[i]);
                button[i].setId(View.generateViewId());
                button[i].setOnClickListener(pc.vorgul.testnewsparser.MainActivity.this);
                lMain.addView(button[i],200,200);
            }
        }
    }

ошибка выскакивает на строке

Java

1	doc = Jsoup.connect("http://ria.ru/export/rss2/politics/index.xml").timeout(5000).get();

выбрасывается исключение android.os.NetworkOnMainThreadException, подскажите как решить проблему

@Mikalai · 29.11.2015, 18:35

Jsoup.connect нужно вызывать в doInBackground

@dajver · 29.11.2015, 18:42

Сообщение от Mikalai

Jsoup.connect нужно вызывать в doInBackground

Да по сути перенести весь код в doInBackground

@Tro0P · 29.11.2015, 18:58 **[ТС]**

переделал так:

Java

 class AsyncT extends AsyncTask<Void,Void,Void> {
        @Override
        protected Void doInBackground(Void... params) {
            doc=null;
            try{
                doc = Jsoup.connect("http://ria.ru/export/rss2/politics/index.xml").timeout(5000).get();
            } catch (IOException e){
                e.printStackTrace();
            }
            return null;
        }
 
        protected void onPostExecute (Void result){
           /* Document doc;
            doc=null;
            try{
                doc = Jsoup.connect("http://ria.ru/export/rss2/politics/index.xml").timeout(5000).get();
            } catch (IOException e){
                e.printStackTrace();
            }*/
            Elements news=doc.select("title");
            newsPars=new NewsPars();
            title=newsPars.parsingTitle(news);
            title=newsPars.parsingTitle(news);
            for (int i=0;i<title.length;i++){
                button=new Button[title.length];
                button[i]=new Button(pc.vorgul.testnewsparser.MainActivity.this);
                button[i].setText(title[i]);
                button[i].setId(View.generateViewId());
                button[i].setOnClickListener(pc.vorgul.testnewsparser.MainActivity.this);
                lMain.addView(button[i],200,200);
            }
        }
    }

Добавлено через 11 минут
разобрался, в андроид манифесте не был разрешен доступ к интернету.
Сори за глупые ошибки) по сути первое приложение делаю, спасибо за помощь

@Mikalai 262 / 238 / 95 Регистрация: 11.01.2015 Сообщений: 666
	26.11.2015, 01:11	#2
	Tro0P, нельзя обращаться в сеть в основном потоке. Используй AsyncTask 1

@Mikalai 262 / 238 / 95 Регистрация: 11.01.2015 Сообщений: 666
	29.11.2015, 18:35	#4
	Jsoup.connect нужно вызывать в doInBackground 0

@dajver 410 / 274 / 18 Регистрация: 18.09.2010 Сообщений: 1,114
	29.11.2015, 18:42	#5
	Сообщение от Mikalai Jsoup.connect нужно вызывать в doInBackground Да по сути перенести весь код в doInBackground 0

	29.11.2015, 18:58

Опции темы