holounic/university

списывать нехорошо!

Hi there 👋

I'm a:

• 🗺️ Research Engineer @ Google DeepMind
• 🌈 Bs in Applied Mathematics & CS @ ITMO University
• 🚘 DeepMind Research Engineer intern April 2023 - September 2023
• 🌯 Google SWE intern October 2022 - February 2023
• 🦏 Yandex Middle Java Developer April 2022 - September 2022
• 🦄 Jetbrains Deep RL intern October 2020 - January 2022
• 🥦 Google Summer 2020 STEP intern
• 💚 Fan of discrete mathematics
• 🌻 Wrote Bs thesis on Bayesian optimization
• 🐷 RL works!

Btw sometimes I joke funny.

university's Issues

Excess variable

university/paradigms/semester-1/some-code/markup/MarkableBlock.java

Line 6 in 5e269e5

protected String specifier = "";

You store this variable in every instance of the every heir.
It would be better to do void toMarkup(StringBuilder result, String border) , so as not to store extra variables.
Just call this method in the heirs with the desired border.
(Some people got a ban from Korneev for this thing)

Че? Каво?

university/paradigms/semester-1/some-code/markup/MarkupBlock.java

Line 15 in 4efe281

builder.append(this.textField.get(idx).toMarkdown());

Во-первых почему нельзя было написать this.textField.get(i).toMarkdown(builder)?
Тогда можно просто совсем стереть String toMarkdown().

Во-вторых зачем делать textField.get(i)?
Тебе могут подсунуть LinkedList и проигрыш, потому что будет работать за квадрат. Лучше написать foreach, который гарантированно работает за линию при любой реализации интерфейста List.

За первый пункт скорее всего минус, а вот за второй БанБанБан.

markup.Paragraph

university/paradigms/semester-1/some-code/markup/Paragraph.java

Line 5 in ba309ab

public Paragraph(List<TextBlock> content) {

Paragraph object should not be able to store Paragraphs, but it can.