55+ в ИТ

[M. Ridcully]

[map][/map]

У мужика нормальный читаемый код, скорее всего для данной задачи можно обойтись перебором. Меня напрягло, чтотне взбрасывается exception, если в корзине обнаружен item which is null.

Проблема в том, что вы примерно схожего возраста. Поэтому для вас это норм код. Для меня - нет, хоть я себя и чувствую стариком. Приемлемый 30-летний погромист просто блеванет

На самом деле 30-летний погромист учился кодить на питоне, поэтому он выдаст чтото вроде

Code: Select all

 
 
 def isInCart(item, cart):
   return len(filter(lambda x: x.getId() == item.getId(), cart.getItems())) > 0
 
 

Что под капотом тот же обход листа

Уж больно коряво, если вы про Питон.
Лучше уж

Code: Select all

def is_in_cart(item, cart):
    return item.getId() in (it.getId() for it in cart.getItems())

Или даже так, если объекты по-людски написаны:

Code: Select all

def is_in_cart(item, cart):
    return item in cart.getItems()

[Komissar]

“A вы друзья, как ни садитесь...”

[Zachet]

[map][/map]

У мужика нормальный читаемый код, скорее всего для данной задачи можно обойтись перебором. Меня напрягло, чтотне взбрасывается exception, если в корзине обнаружен item which is null.

Проблема в том, что вы примерно схожего возраста. Поэтому для вас это норм код. Для меня - нет, хоть я себя и чувствую стариком. Приемлемый 30-летний погромист просто блеванет

На самом деле 30-летний погромист учился кодить на питоне, поэтому он выдаст чтото вроде

Code: Select all

 
 
 def isInCart(item, cart):
   return len(filter(lambda x: x.getId() == item.getId(), cart.getItems())) > 0
 
 

Что под капотом тот же обход листа

Уж больно коряво, если вы про Питон.
Лучше уж

Code: Select all

def is_in_cart(item, cart):
    return item.getId() in (it.getId() for it in cart.getItems())

Или даже так, если объекты по-людски написаны:

Code: Select all

def is_in_cart(item, cart):
    return item in cart.getItems()

Не принципиально. Суть в том что 30-летний погромист напишет на питоне обход листа в одну строчку, и забудет. Пока Комиссар будет выяснять, не подменили-ли инопланетяне у него в карте один айтем на нулл.

[KinDzaDza]

[map][/map]

У мужика нормальный читаемый код, скорее всего для данной задачи можно обойтись перебором. Меня напрягло, чтотне взбрасывается exception, если в корзине обнаружен item which is null.

Проблема в том, что вы примерно схожего возраста. Поэтому для вас это норм код. Для меня - нет, хоть я себя и чувствую стариком. Приемлемый 30-летний погромист просто блеванет

На самом деле 30-летний погромист учился кодить на питоне, поэтому он выдаст чтото вроде

Code: Select all

 
 
 def isInCart(item, cart):
   return len(filter(lambda x: x.getId() == item.getId(), cart.getItems())) > 0
 
 

Что под капотом тот же обход листа

Уж больно коряво, если вы про Питон.
Лучше уж

Code: Select all

def is_in_cart(item, cart):
    return item.getId() in (it.getId() for it in cart.getItems())

Или даже так, если объекты по-людски написаны:

Code: Select all

def is_in_cart(item, cart):
    return item in cart.getItems()

Не принципиально. Суть в том что 30-летний погромист напишет на питоне обход листа в одну строчку, и забудет. Пока Комиссар будет выяснять, не подменили-ли инопланетяне у него в карте один айтем на нулл.

Зато у Комиссара приложение не начнёт падать в продакшн с нуль пойнтер эксепшен когда какое-нибудь другое молодое дарование внезапно(tm) поменяет код по добавлению айтемов в эту корзину.
Хотя, конечно, мы ту обсуждаем сферического коня в вакууме. Без знаний как за кадром устроен этот cart из примера, что там с айтемами (как там например имплементирован метод equal() и т.п.) - все остальное гадание на кофейной гуще. Фреймворки тоже не боги писали, как мне помнится классный библиотечный binary search в Java как раз на 100500 классно гнал фуфло.

[M. Ridcully]

Хотя, конечно, мы ту обсуждаем сферического коня в вакууме.

Дык, чисто потрындеть да своей гениальностью блеснуть, как водится.

[АццкоМото]

Зато у Комиссара приложение не начнёт падать в продакшн с нуль пойнтер эксепшен когда какое-нибудь другое молодое дарование внезапно(tm) поменяет код по добавлению айтемов в эту корзину.

сюрпрайз: для этого придуманы не только аннотации типа @NonNull, которые решают проблему частично, но и йызыки тиипа Котлина, которые решают проблему полностью. И молодое дарование наверняка умеет этим пользоваться, а старперы будут и дальше надувать щеки "а вот если вдруг"

На всякий случай напомню, что идея Комми - проверять на нуль и кидать эксепшн. Не NPE, а другой. Т.е. по факту у неопытного писюка будет код такой, что нарушить null-safety будет невозможно, а у Комми - такой, что не падает со стыдным NPE, но падает с KommyVerifiedThatHereWeHaveANullPointerInCollectionAndYouAllSuckException

[M. Ridcully]

А вообще всех за пояс заткнет Кумар Дипшитович, который честно поймает null pointer и в обработчике сделает молчаливый возврат безо всякой диагностики. И подумает: "у меня ничего не падает, а вы поебитесь отлаживать".

[Komissar]

Зато у Комиссара приложение не начнёт падать в продакшн с нуль пойнтер эксепшен когда какое-нибудь другое молодое дарование внезапно(tm) поменяет код по добавлению айтемов в эту корзину.

сюрпрайз: для этого придуманы не только аннотации типа @NonNull, которые решают проблему частично, но и йызыки тиипа Котлина, которые решают проблему полностью. И молодое дарование наверняка умеет этим пользоваться, а старперы будут и дальше надувать щеки "а вот если вдруг"

На всякий случай напомню, что идея Комми - проверять на нуль и кидать эксепшн. Не NPE, а другой. Т.е. по факту у неопытного писюка будет код такой, что нарушить null-safety будет невозможно, а у Комми - такой, что не падает со стыдным NPE, но падает с KommyVerifiedThatHereWeHaveANullPointerInCollectionAndYouAllSuckException

Мои эксепшны выводятся на экран PM со словами "ЗА ВАМИ УЖЕ ВЫЕХАЛИ"

[АццкоМото]

Зато у Комиссара приложение не начнёт падать в продакшн с нуль пойнтер эксепшен когда какое-нибудь другое молодое дарование внезапно(tm) поменяет код по добавлению айтемов в эту корзину.

сюрпрайз: для этого придуманы не только аннотации типа @NonNull, которые решают проблему частично, но и йызыки тиипа Котлина, которые решают проблему полностью. И молодое дарование наверняка умеет этим пользоваться, а старперы будут и дальше надувать щеки "а вот если вдруг"

На всякий случай напомню, что идея Комми - проверять на нуль и кидать эксепшн. Не NPE, а другой. Т.е. по факту у неопытного писюка будет код такой, что нарушить null-safety будет невозможно, а у Комми - такой, что не падает со стыдным NPE, но падает с KommyVerifiedThatHereWeHaveANullPointerInCollectionAndYouAllSuckException

Мои эксепшны выводятся на экран PM со словами "ЗА ВАМИ УЖЕ ВЫЕХАЛИ"

Ага. А когда мой код не работает, фиксят компилятор. Продолжим?

[АццкоМото]

А вообще всех за пояс заткнет Кумар Дипшитович, который честно поймает null pointer и в обработчике сделает молчаливый возврат безо всякой диагностики. И подумает: "у меня ничего не падает, а вы поебитесь отлаживать".

Кстати, в данном конкретном случае это не самый плохой вариант: нуль ведь это не то, что мы ищем. Т.е. по факту можно не ловить npe а просто проверить на ноль и пропустить

[M. Ridcully]

А вообще всех за пояс заткнет Кумар Дипшитович, который честно поймает null pointer и в обработчике сделает молчаливый возврат безо всякой диагностики. И подумает: "у меня ничего не падает, а вы поебитесь отлаживать".

Кстати, в данном конкретном случае это не самый плохой вариант: нуль ведь это не то, что мы ищем. Т.е. по факту можно не ловить npe а просто проверить на ноль и пропустить

А кто говорил про "пропустить элемент"? Return false сразу и всё.
Кумар не лыком шит, пускай результат зависит от порядка элементов в getItems()!

[Komissar]

А вообще всех за пояс заткнет Кумар Дипшитович, который честно поймает null pointer и в обработчике сделает молчаливый возврат безо всякой диагностики. И подумает: "у меня ничего не падает, а вы поебитесь отлаживать".

Кстати, в данном конкретном случае это не самый плохой вариант: нуль ведь это не то, что мы ищем. Т.е. по факту можно не ловить npe а просто проверить на ноль и пропустить

А кто говорил про "пропустить элемент"? Return false сразу и всё.
Кумар не лыком шит, пускай результат зависит от порядка элементов в getItems()!

по уму так надо проверять, не подсунул ли Кумар нулевой карт.

[АццкоМото]

А вообще всех за пояс заткнет Кумар Дипшитович, который честно поймает null pointer и в обработчике сделает молчаливый возврат безо всякой диагностики. И подумает: "у меня ничего не падает, а вы поебитесь отлаживать".

Кстати, в данном конкретном случае это не самый плохой вариант: нуль ведь это не то, что мы ищем. Т.е. по факту можно не ловить npe а просто проверить на ноль и пропустить

А кто говорил про "пропустить элемент"? Return false сразу и всё.
Кумар не лыком шит, пускай результат зависит от порядка элементов в getItems()!

А, ну так-то да. попадаются очень талантливые люди (ц)

[АццкоМото]

по уму так надо проверять, не подсунул ли Кумар нулевой карт.

по уму надо проверять, не подсунули ли нам Кумара!

[KinDzaDza]

по уму так надо проверять, не подсунул ли Кумар нулевой карт.

по уму надо проверять, не подсунули ли нам Кумара!

У нас как-то одному перцу вообше было запрешено что-либо коммитить в обший репозиторий. Только после code review его начальник выборочно сам коммитил шедевры эпистолярного жанра.

[Lisa]

по уму так надо проверять, не подсунул ли Кумар нулевой карт.

по уму надо проверять, не подсунули ли нам Кумара!

У нас как-то одному перцу вообше было запрешено что-либо коммитить в обший репозиторий. Только после code review его начальник выборочно сам коммитил шедевры эпистолярного жанра.

Выгнать не проще было?

[Komissar]

Выгнать не проще было?

а если он из одной деревни с начальником?

[ystar]

по уму так надо проверять, не подсунул ли Кумар нулевой карт.

по уму надо проверять, не подсунули ли нам Кумара!

У нас как-то одному перцу вообше было запрешено что-либо коммитить в обший репозиторий. Только после code review его начальник выборочно сам коммитил шедевры эпистолярного жанра.

Выгнать не проще было?

выгнать? тут только если бюджет заканчивается увольняют, либо что-то типа "мне не нравится diversity" или харазмент какой. за то что ты не работаешь - здесь не выгоняют.

[Lisa]

У нас как-то одному перцу вообше было запрешено что-либо коммитить в обший репозиторий. Только после code review его начальник выборочно сам коммитил шедевры эпистолярного жанра.

Выгнать не проще было?

выгнать? тут только если бюджет заканчивается увольняют, либо что-то типа "мне не нравится diversity" или харазмент какой. за то что ты не работаешь - здесь не выгоняют.

"Тут" и "здесь" это где? Везде, где я работала, плохих работников увольняли не задумываясь.

[NYgal]

Муж искал работу 2 года назад, 60 ровно было. Нашел через знакомых, но и другие варианты были. Этот - лучший по сумме плюшек.
До этого сидел дома 2 года по нашему совместному решению: надо было с ребенком на дом обучении посидеть, он и до того сидел, но он работал из дому до сокращения. Контора его переводила, но мы не захотели в очередной раз, и на этот раз лавочка закрылась.
Контракторствует. Он cissp, не кодировщик, но кодирует понемножку , если ему удобнее кодировать.
Ребенку еще 3 года в школе, так что мы сидим на попе ровно.
Я сейчас практически живу Комми дрим: работаю, ковыряясь потихоньку, данные там данные сям, короче дейта говернанс в госконторе. Люди в целом от терпимых до очень хороших, особых революций нет вроде, тихо по сравнению с индустрией.

В будущем морковка пенсия.
Можно бросить и жить на ренту, но ребенок все равно привязан к школе , и пенсии жалко. Муж по моей просьбе тоже получил тайтл на работу в госслужбе, периодически его зовут на интервью, но он не обязан принимать их оферы, ему пока, тьфу-тьфу-тьфу, на его месте неплохо. Кроме денег плюс - гибкий график, чего у меня,увы, нет.

Это больно, когда хочется лето на лаче проводить. Когда контракторствовала, жто луше удавалось.
Муж пока сам бросать работу не планирует.
Есть еще проекты в планах.

[Komissar]

Это больно, когда хочется лето на лаче проводить. Когда контракторствовала, жто луше удавалось.
Муж пока сам бросать работу не планирует.
Есть еще проекты в планах.

Ничего, мать, держись. Выйдем оба на пенсию, будем лето у тебя на даче на Кейпе, а зиму у меня на даче в Сьерре.

А муж... ну, пусть работает и дальше!

[rtogan]

Я не понял, там похоже ошибка в логике.
Должно быть
boolean isItemInCart(Item item) {
for (int i=0; i<cart.size(); i++) {
if (cart.getItem(i).getId() == item.getId() {
return true;
}
}
return false;
}

Да, return false не на месте. Издержки написания спросоня и без ide

Но смысл, надеюсь, понятен.

а как надо?

ну там HashMap например. всякие RxJava делают то же самое куда элегантнее или там java streams. два последних подхода могут быть спорными, но тупой перебор да еще итакой многословный...

HashMap для корзины, которой в среднем 2-3, пусть даже 10 предметов? Это будет гораздо медленнее, чем линейный перебор. Не говоря уже о том, что неоправданно сделает код менее ясным.

[rtogan]

ну там HashMap например. всякие RxJava делают то же самое куда элегантнее или там java streams. два последних подхода могут быть спорными, но тупой перебор да еще итакой многословный...

Извините, но код который вы привели не говорит, что для хранения данных в корзине использовалось что либо наподобии хеш таблиц или любого рода упорядоченные списки. Я лично решил, что это обычный не упорядоченый список в котором лежат объекты. Кстати для хранения небольшого количества данных (несколько десятков) - а в обычной корзине покупателя их обычно бывает всего несколько штук, вряд ли имеет место использовать что либо другое. Так что обычный линейный перебор - оптимальное решение. ИМХО.

+ 111111111111

[АццкоМото]

HashMap для корзины, которой в среднем 2-3, пусть даже 10 предметов? Это будет гораздо медленнее, чем линейный перебор. Не говоря уже о том, что неоправданно сделает код менее ясным.

Суть как раз в том, чтобы сделать код более ясным. Скорость - в данном случае дело десятое. Код ондроедный и будет там потрачено 1 мс или 10 мс - шерифа не колышит, а юзер не заметит. Опять же, если речь пойдет о миллионах элементов, такой человек будет продолжать писать так, как привык.

Иными словами: когда кошка пьет из унитаза, ничего противозаконного не происходит. Но все равно неприятно.

[NYgal]

Это больно, когда хочется лето на лаче проводить. Когда контракторствовала, жто луше удавалось.
Муж пока сам бросать работу не планирует.
Есть еще проекты в планах.

Ничего, мать, держись. Выйдем оба на пенсию, будем лето у тебя на даче на Кейпе, а зиму у меня на даче в Сьерре.

А муж... ну, пусть работает и дальше!

А если серьезно, то иногда напрягает такая скованность со свободным временем.
Едем с мужем в Бруклин, например, после того, как хорошо отдохнули на даче, а меня так и дергает типа повернуть на запад и сразу на месяц- два неспешно через весь континент и обратно.
Теоретически через 3 года, как ребенок выйдет из школьного возраста - запросто.
А практически... 4 недели отпуска, ну, может больше с переработками, пенсия 16% зарплаты после 10 лет, 40 - после 20, наследуемая пожизненно бенефишиари, которым может быть ребенок.
Жалко забить на несколько миллионов.
Подруга зовет в Аспен - ну не на 3 дня же ехать?
Дачу в Пуэрто Рико не купишь - на 7 дней в году и заморачиваться с удаленной сдачей?
С другой стороны - работа то ненапряжная, и врядли попрут по старости. Можно честно рассчитывать до своих 67 досидеть спокойно.
Таким жалко бросаться.
Вот так мысленно и мечусь
Да еще мама, блестящий была врач, забила на работу чуть старше меня сегодняшней, года 54 ей было. Деградация, деменция в 65.
Не хочу. Надеюсь, работа убережет.
А там и таблетку придумают
Так что орбайтен!