Skip to content

Conversation

@NiSuKe
Copy link
Collaborator

@NiSuKe NiSuKe commented Nov 19, 2018

No description provided.

@InfaSysKey InfaSysKey changed the title aqui estan las encuestas ordenadas alfabéticamente Encuestas ordenadas alfabéticamente. Nov 26, 2018
Copy link
Collaborator

@InfaSysKey InfaSysKey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Corregir algunos fallos.

import android.widget.Toast;

import java.util.ArrayList;
import java.util.Collections;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

¿ Necesaria implementación ? Ya esta importada en la clase necesaria.


public int[] getQuestions() {
return questions;
public int[] getQuestions() {return questions;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Quizá con un espacio puede quedar más limpio el código.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Better leave things as they are if there is no meaningful change. Try to revert these lines

APIService.OnResponse<SurveysModel> onSurveysResponse = new APIService.OnResponse<SurveysModel>() {
@Override
public void onSuccess(SurveysModel surveysModel) {
Collections.sort(surveysModel.getSurveys(), new Comparator<SurveyModel>() {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

¿Algún comentario de como funciona?

@InfaSysKey InfaSysKey changed the title Encuestas ordenadas alfabéticamente. Surveys ordered alphabetically. Nov 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants