Что не так в задаче на основы ООП (Python)

@ITsss_jstn · Регистрация: 12.12.2023

Студворк — интернет-сервис помощи студентам

Что не так в задаче по основам ООП (python)?

задачка с Яндекс Хэндбука

Рассмотрим объект «Программист», который задаётся именем, должностью и количеством отработанных часов. Каждая должность имеет собственный оклад (заработную плату за час работы). В нашей импровизированной компании существуют 3 должности:

Junior — с окладом 10 тугриков в час;

Middle — с окладом 15 тугриков в час;

Senior — с окладом 20 тугриков в час по умолчанию и +1 тугрик за каждое новое повышение.

Напишите класс Programmer, который инициализируется именем и должностью (отработка у нового работника равна нулю). Класс реализует следующие методы:

work(time) — отмечает новую отработку в количестве часов time;

rise() — повышает программиста;

info() — возвращает строку для бухгалтерии в формате: <имя> <количество отработанных часов>ч. <накопленная зарплата>тгр.

Примечание

Ваше решение должно содержать только классы и функции. В решении не должно быть вызовов инициализации требуемых классов.

Пример

Ввод:

programmer = Programmer('Васильев Иван', 'Junior')

programmer.work(750)

print(programmer.info())

programmer.rise()

programmer.work(500)

print(programmer.info())

programmer.rise()

programmer.work(250)

print(programmer.info())

programmer.rise()

programmer.work(250)

print(programmer.info())

Вывод:

Васильев Иван 750ч. 7500тгр.

Васильев Иван 1250ч. 15000тгр.

Васильев Иван 1500ч. 20000тгр.

Васильев Иван 1750ч. 25250тгр.

Прикрепляю свой код

Python
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
class Programmer:
 
    def __init__(self, name, role, up=0, n=20, rr=0, sm=0, time=0):
        self.up = up #переменная уровней должностей, начиная с 0 - Junior
        self.role = role #Дожность
        self.name = name #имя работника
        self.n = n #зарплата сеньоров(тгр/ч), получивших повышение(назвал Milord)
        self.rr = rr #суммарное кол-во отработанных часов 
        self.sm = sm #переменная для подсчёта всех заработанных денег работником
        self.time = time # часы работы
 
    def work(self, timee):
        self.time = timee 
        self.rr = self.rr + self.time
  
    def rise(self):
        if self.role == 'Middle':
            self.up = 1
        if self.role == 'Senior':
            self.up = 2
        self.up += 1
        if self.up == 1:
            self.role = 'Middle'
        if self.up == 2:
            self.role = 'Senior'
        if self.up >= 3:
            self.role = 'Milord'
            self.n += 1
        
    def info(self):
        if self.role == 'Junior':
            self.sm = self.time * 10
            return f'{self.name} {self.time}ч. {self.sm}тгр.'
        if self.role == 'Middle':
            self.sm = (self.time * 15) + self.sm
            return f'{self.name} {self.rr}ч. {self.sm}тгр.'
        if self.role == 'Senior':
            self.sm = self.sm + (self.time * 20)
            return f'{self.name} {self.rr}ч. {self.sm}тгр.'
        if self.role == 'Milord':
            self.sm = (self.time * self.n) + self.sm
            return f'{self.name} {self.rr}ч. {self.sm}тгр.'

Итак, первые 4 проверки(включая указанную в примере) решение проходит, но дальше не принимает

Прошу помощи у знающих, что может быть не так?

P.s: просьба отнестись ко всем косякам с понимаем, ведь я - нуб

@Semen-Semenich · 12.12.2023, 22:20

ITsss_jstn,

Сообщение от ITsss_jstn

rise() — повышает программиста;

и каким образом? при вызове этого метода звание а соответственно и зарплата должны меняться на повышение. еще общая сумма должна вычисляются не от общего времени а от времени в каждой должности. то есть 5 часов * 10 в должности Junior плюс 3ч * 15 в должности Middle после повышения. но это мое мнение и как вариант ошибки. попробуй по примеру просчитать так ли это.

@idealist · 13.12.2023, 02:00

Python
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
class Programmer:
    salary_dict = {'Junior': 10, 'Middle': 15, 'Senior': 20}
 
    def __init__(self, name, position):
        self.name = name
        self.position = position
        self.time = 0
        self.salary = Programmer.salary_dict[position]
        self.pay = 0
 
    def work(self, time):
        self.time += time
        self.pay += time * self.salary
 
    def rise(self):
        match self.position:
            case 'Junior':
                self.position = 'Middle'
                self.salary = Programmer.salary_dict[self.position]
            case 'Middle':
                self.position = 'Senior'
                self.salary = Programmer.salary_dict[self.position]
            case 'Senior':
                self.salary += 1
 
    def info(self):
        return f'{self.name} {self.time}ч. {self.pay}тгр.'
 
programmer = Programmer('Васильев Иван', 'Junior')
programmer.work(750)
print(programmer.info())
programmer.rise()
programmer.work(500)
print(programmer.info())
programmer.rise()
programmer.work(250)
print(programmer.info())
programmer.rise()
programmer.work(250)
print(programmer.info())

А вообще, такие наборы ветвлений

Python
1
2
3
4
5
6
7
8
9
10
11
12
13
def info(self):
        if self.role == 'Junior':
            self.sm = self.time * 10
            return f'{self.name} {self.time}ч. {self.sm}тгр.'
        if self.role == 'Middle':
            self.sm = (self.time * 15) + self.sm
            return f'{self.name} {self.rr}ч. {self.sm}тгр.'
        if self.role == 'Senior':
            self.sm = self.sm + (self.time * 20)
            return f'{self.name} {self.rr}ч. {self.sm}тгр.'
        if self.role == 'Milord':
            self.sm = (self.time * self.n) + self.sm
            return f'{self.name} {self.rr}ч. {self.sm}тгр.'

противоречат принципам ООП, здесь полиморфизм должен срабатывать. То, что в функции rise пришлось это использовать - это недостаток проекта, предложенного авторами задачи. В реале все должности должны были бы быть подклассами класса "Программист".

@idealist · 13.12.2023, 11:56

Еще вариант:

Python
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
class Programmer:
    positions = ['Junior', 'Middle', 'Senior']
    salaries = [10, 15, 20]
 
    def __init__(self, name, position):
        self.name = name
        self.position_ind = Programmer.positions.index(position)
        self.time = 0
        self.bonus = 0
        self.pay = 0
 
    def work(self, time):
        self.time += time
        self.pay += time * (Programmer.salaries[self.position_ind] + self.bonus)
 
    def rise(self):
        try:
            Programmer.positions[self.position_ind + 1]
            self.position_ind += 1
        except:
            self.bonus += 1
 
    def info(self):
        return f'{self.name} {self.time}ч. {self.pay}тгр.'
 
programmer = Programmer('Васильев Иван', 'Junior')
programmer.work(750)
print(programmer.info())
programmer.rise()
programmer.work(500)
print(programmer.info())
programmer.rise()
programmer.work(250)
print(programmer.info())
programmer.rise()
programmer.work(250)
print(programmer.info())

Новые блоги и статьи Все статьи Все блоги /
Уведомление о неверно выбранном значении справочника Maks 06.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "НарядПутевка", разработанного в конфигурации КА2. Задача: уведомлять пользователя, если в документе выбран неверный склад. . .	Установка Qt Creator для C и C++: ставим среду, CMake и MinGW без фреймворка Qt 8Observer8 05.04.2026 Среду разработки Qt Creator можно установить без фреймворка Qt. Есть отдельный репозиторий для этой среды: https:/ / github. com/ qt-creator/ qt-creator, где можно скачать установщик, на вкладке Releases:. . .	AkelPad-скрипты, структуры, и немного лирики.. testuser2 05.04.2026 Такая программа, как AkelPad существует уже давно, и также давно существуют скрипты под нее. Тем не менее, прога живет, периодически что-то не спеша дополняется, улучшается. Что меня в первую очередь. . .	Отображение реквизитов в документе по условию и контроль их заполнения Maks 04.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "ПланированиеСпецтехники", разработанного в конфигурации КА2. Данный документ берёт данные из другого нетипового документа. . .
Фото всей Земли с борта корабля Orion миссии Artemis II kumehtar 04.04.2026 Это первое подобное фото сделанное человеком за 50 лет. Снимок называют новым вариантом легендарной фотографии «The Blue Marble» 1972 года, сделанной с борта корабля «Аполлон-17». Новое фото. . .	Вывод диалогового окна перед закрытием, если документ не проведён Maks 04.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "СписаниеМатериалов", разработанного в конфигурации КА2. Задача: реализовать программный контроль на предмет проведения документа. . .	Программный контроль заполнения реквизитов табличной части документа Maks 02.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "СписаниеМатериалов", разработанного в конфигурации КА2. Задача: 1. Реализовать контроль заполнения реквизита. . .	wmic не является внутренней или внешней командой Maks 02.04.2026 Решение: DISM / Online / Add-Capability / CapabilityName:WMIC~~~~ Отсюда: https:/ / winitpro. ru/ index. php/ 2025/ 02/ 14/ komanda-wmic-ne-naydena/