[Cornknight]

С точки зрения языка код написан правильно? Или надо что-то изменить? Класс Element определён на месте? »

Код у тебя скорей всего приведён не полностью. Поэтому трудно сказать что на месте, а что нет. Могу только предположить, что Element у тебя является внутренним классом по отношению к другому (то есть он определён внутри другого класса). А внутренним классам Java запрещает иметь static элементы, поэтому компилятор и ругается. Top level type - это как раз про внешний класс. Объяви весь класс Element как static или вынеси его за пределы внешнего класса и всё будет OK. Только когда будешь выносить, учти, что по правилам в одном файле может быть только один public класс, поэтому Element либо не должен быть public, либо нужно его вынести в отдельный файл.

Поехали дальше

static в данном контексте вообще не требуется. Более того, сортировка не получится, потому что static поля являются глобальными по отношению ко всему классу, то есть для каждого экземпляра класса их значение всегда будет одним и тем же.

Сравнивать так: if (a[j].Кey < a[jmin].Кey) нельзя, потому что Key у тебя String, а не число. И вообще кey у тебя с маленькой буквы, а в Java в отличие от Паскаля идентификаторы чувствительны к регистру. key и Key - разные вещи.

Для пары "ключ - данные" есть специальные типы, например HashMap, и свой собственный тип создавать не надо. В HashMap, например можно добыть данные по ключу, получать список ключей и много ещё чего полезного там есть.

Для сортировки в принципе не требуется писать специальных методов, в Java есть встроенные механизмы. Например, вот - http://www.javaportal.ru/java/articles/sort.html

Вроде всё.