сабж

Код

abstract boolean break byte class case catch char const continue default do double else extends final finally float for goto if implements import instanceof int interface long native new package private public protected return transient short static super this switch synchronized throw throws try void volatile while byvalue cast future generic inner outer rest var strictfp true false null operator assert

Это сообщение отредактировал(а) Domestic Cat - 2.9.2004, 00:09

Хорошо бы услышать мнение по поводу предлагаемой мной посветки,
см тут : http://forum.vingrad.ru/index.php?act=ST&f...=15#entry208613

{ import, package } // imports, crimson

{ throws, throw, catch, try, finally, assert } // exceptions, red

{ transient, volatile, native, static, strictfp, final, abstract, default, public, protected, private} // modifiers, darkblue

{ class, interface, void, boolean, short, int, float, double, byte, char, long, super, this, instanceof, new} // types, blue

{while, do, for, if, else, switch, return, case, break, continue} // control flow, blue

{false, true, null} // purple

{extends, implements} // inheritance, blue

{synchronized} // threads, red

{byvalue, cast, future, generic, const, rest, var, goto, outer, inner, operator} // other, royalblue

String literal: "This is string literal" , green

Numbers: 32 + 17 - 27 = 22 , red

Comments: // This is comment teal

Это сообщение отредактировал(а) Domestic Cat - 2.9.2004, 23:19

А не слишком ли сложная подсветка?

Цитата(DarkDS @ 1.9.2004, 14:28)

А не слишком ли сложная подсветка?

Оттенки 2х цветов - синего и красного (кроме зеленых стрингов). Но вообще да, подправлю сейчас.

Красиво, движек не напрягает, без разницы как слово предтавить, в смысле все слова подходят под одну лексему: идентификатор, а дальше в хештаблицу глянуть за стилем

Ну а сейчас как?

Лудше. Намного лудше. Единственный перебор по моему - числа. Их лудше черными оставить

Цитата(DarkDS @ 1.9.2004, 14:44)

Лудше. Намного лудше. Единственный перебор по моему - числа. Их лудше черными оставить

O ! Самый раз. Может, все малоиспользуемые в rojalblue перекрасить (rest, goto, ...)

Да в общем то перекрашивать их не нужно. Так как если перекрасить, то чутка запутывать будут

Ну все равно никто ими не пользуется. А так будут выделяться.
Нада будет вопрос про rest и var в "Маленький тест" сунуть

test:

Код

/**     * Saves given record in database file if it has no owner and date     * to which the record relates is valid. <code>String</code> array record is     * record to be saved, except of the last element which is     * the number of record.     * Method uses fillowing schema:     * <ol>     * <li>Lock record.</li>     * <li>Check owner, if room has been sold, throw <code>AccessDeniedException</code></li>     * <li>Unlock record</li></ol>     *     * @param   record <code>String</code> array containing data and record number     * @throws  AccessDeniedException thrown if client tries to book room     *          which has been booked already, or record date is not valid     * @throws  RecordNotFoundException thrown if such record does not exist     * @throws  RemoteException thrown if RMI fails     */    public void setRecord(String[] record) throws   AccessDeniedException,                                                    RecordNotFoundException,                                                    RemoteException    {        // Last element of record array is record ID, see comments in constructor        int actualLength = record.length - 1;        int recordNumber = Integer.parseInt(record[actualLength], 10);        String[] rec = new String[actualLength];        System.arraycopy(record, 0, rec, 0, actualLength);                    /*         * Check time limit. CSR cannot book rooms with date in the past;         * and may book room if recordDate - today < timeLimit         * (2 days, in original variant).         */        String stringRecordDate = rec[rec.length - 2];        GregorianCalendar recordDate = new GregorianCalendar();        StringTokenizer st = new StringTokenizer(stringRecordDate, "/");        int year = Integer.parseInt(st.nextToken(), 10);        int month = Integer.parseInt(st.nextToken(), 10);        int day = Integer.parseInt(st.nextToken(), 10);        recordDate.set(year, month - 1, day, 12, 0, 0); // Noon        GregorianCalendar today = new GregorianCalendar();                if (today.after(recordDate))        {            throw new AccessDeniedException("Time limit");        }                // Check time limit for future        if (timeLimit != Start.NO_TIME_LIMIT)        {            GregorianCalendar dateToCompare;            dateToCompare = new GregorianCalendar();            dateToCompare.add(Calendar.DATE, timeLimit);            if (!dateToCompare.after(recordDate))              {                throw new AccessDeniedException("Time limit");            }              }                /*         * Although the owner field was checked by client, we have to         * check it again, in the case if somebody have already booked this room.         * Notice: Clients database (what CSR sees in JTable) is updated         * each time it books room or performs search.         */        boolean flag = false;        manager.lock(recordNumber); // Locking record        String[] temp = (String[]) database.elementAt(recordNumber);        if (!temp[temp.length - 2].trim().equals(""))        {            flag = true;        }        else        {            data.update(recordNumber, rec);            database.set(recordNumber, record);        }        manager.unlock(recordNumber); // Unlocking record        if (flag)        {            // Must not throw Exception before unlock            throw new AccessDeniedException("Room has been booked already");        }    } // End of setRecord

Млин, а что с отступами? Они пропадают или их не было?

Цитата(Zandr @ 2.9.2004, 05:42)

Млин, а что с отступами? Они пропадают или их не было?

Что с ними? Конкретнее плиз И какой у тебя броузер?
По моему сотступами всё ОК.
Подсветка даже восстанавливает табулятор

Дык эта, Opera 7.23j
Добавлено @ 13:08
А в IE 5.00 (W2k Pro Ru SP4), конкретно на этой странице:
Строка 210 - ошибка - Объект не поддерживает это свойство или метод
далее
Строка 259 - ошибка - this.endOfLineComment.0.0 - не является объектом
С отступами все OK, но цветов нет вообще...

Знаю - броузеры староваты... А менять-то как не охота...

Цитата(Zandr @ 2.9.2004, 12:02)

Дык эта, Opera 7.23j

Да, старая Опера действительно не сдвигает к сож. ничего поделать нельзя, так как глюк в опере, а не в скрипте... Скрипт получает уже ХТМЛ без пробелов....

Сейчас опера 7.5 по моему... Там уже всё намного лучше

Цитата(Zandr @ 2.9.2004, 12:02)

А в IE 5.00 (W2k Pro Ru SP4), конкретно на этой странице:....

Где ж мне эксплорер такой взять? Ни чего обещать не могу, но постараюсь конечно посмотреть...

Дело в том, что чем старее броузеры которые поддерживаются, тем запутаннее и сложнее (медленнее) код.. Поэтому например нетскейп 4 отпадает... А по идее скрипт предусмотрен так, что бы на не поддерживающих броузерах код вообще не подсвечивался и оставался как раньше...