VPF::Сортировка warning - Форум программистов

KeyLord

Дата 3.6.2008, 16:46 (ссылка)

(нет голосов)

Загрузка ...

Новичок

Профиль
Группа: Участник
Сообщений: 4
Регистрация: 3.6.2008

Репутация: нет
Всего: нет

Приветствую.
Есть задание:
Ввести n строк с консоли. Упорядочить и вывести строки в порядке возрастания их длин

вот мой код:

Код


    public void SortMaxLen(ArrayList<String> als) {
        sort(als, new Comparator() {
        public int compare(Object o1, Object o2) {
                String o1s, o2s;
                o1s = (String) o1;
                o2s = (String) o2;
                if (o1s.length() > o2s.length()) return 1;
                if (o1s.length() < o2s.length()) return -1;
                return 0;
        }
        });
        System.out.println(als);
    }

работает правильно, но идея выбрасывает варнинг: uses unchecked or unsafe operations
проясните, пожалуйста, для меня, что здесь не так?

Старовъръ

Дата 3.6.2008, 16:52 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 491
Регистрация: 8.5.2008

Репутация: 4
Всего: 10

Возможно следовало бы вначале проверить являются ли объекты о1 и о2 строками, а потом приводить типы

Код


if(o1 instanceof String)
  o1s = (String)o1;

Если нет, то укажи строку, в которой выбивается это предупреждение.

--------------------

My GitHub blog, Java Professionals Course, Blog on quality & management, Lib for randomized testing

fixxer

Дата 3.6.2008, 17:29 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 672
Регистрация: 14.9.2006
Где: Саратов, Россия

Репутация: 6
Всего: 27

Ели версия джавы >= 5, то имхо лучше так:

Код


    public void SortMaxLen(ArrayList<String> als) {
        sort(als, new Comparator<String>() {
        public int compare(String o1, String o2) {
                if (o1.length() > o2.length()) return 1;
                if (o1.length() < o2.length()) return -1;
                return 0;
        }
        });
        System.out.println(als);
    }

Это сообщение отредактировал(а) fixxer - 3.6.2008, 17:29

--------------------

niasilil

Дата 3.6.2008, 17:32 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 325
Регистрация: 4.6.2007
Где: USA

Репутация: 8
Всего: 9

Это скорее всего изза компаратора. Он же generics изспользует, а у тебя raw type. Вот компилятор и недоволен.

Код


    public void SortMaxLen(ArrayList<String> als) {
        sort(als, new Comparator<String>() {

            public int compare(String o1, String o2) {
...

--------------------

SCJP 5.0, SCJD

KeyLord

Дата 3.6.2008, 22:35 (ссылка)

(нет голосов)

Загрузка ...

Новичок

Профиль
Группа: Участник
Сообщений: 4
Регистрация: 3.6.2008

Репутация: нет
Всего: нет

fixxer, niasilil, спасибо, сразу написать String я не додумался.

m1st

Дата 16.4.2012, 23:06 (ссылка)

(нет голосов)

Загрузка ...

Шустрый

Профиль
Группа: Участник
Сообщений: 73
Регистрация: 1.11.2006

Репутация: нет
Всего: нет

Решение этого задания есть тут: http://forum.vingrad.ru/forum/act-ST/f-104...6/unread-1.html

0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема \| Java: Общие вопросы \| Следующая тема »