pklgn / dotnet-practice-labs Goto Github PK

View Code? Open in Web Editor NEW

0.0 0.0 0.0 1.42 MB

These are my dotnet practice labs

C# 49.15% TSQL 2.77% HTML 8.87% CSS 8.18% JavaScript 5.06% TypeScript 25.97%

dotnet-practice-labs's People

Contributors

Watchers

dotnet-practice-labs's Issues

Некорректный тип проекта

dotnet-practice-labs/lab4/SpaceObjects/DatabaseProvider/DatabaseProvider.csproj

Line 4 in 0aa8efd

DatabaseProvider должен быть библиотекой классов (а не запускаемым проектом). Это можно перенастроить в .csproj файле. Также нужно удалить точку входа (Program.cs)

Решение со звёздочками

dotnet-practice-labs/lab7/src/components/Reviewer/StarGrade/StarGrade.tsx

Lines 10 to 51 in 0aa8efd

 function StarGrade(props: StarGradeProps) { 

 const { currentGrade, maxGrade, setGrade } = props; 

 // TODO: add isGrading or smth like that to commit user input 

 const filledStars = Array.from({ length: currentGrade }, () => ( 

 <Star state="filled" /> 

 )); 

 const defaultStars = Array.from({ length: maxGrade - currentGrade }, () => ( 

 <Star state="default" /> 

 )); 

 function handleMouseOver(event: React.SyntheticEvent<HTMLDivElement>) { 

 if (event.currentTarget.dataset && event.currentTarget.dataset.index) { 

 setGrade(Number(event.currentTarget.dataset.index) + 1); 

 } 

 } 

 return ( 

 <div className={styles.starGrades}> 

 {filledStars.length !== 0 && 

 filledStars.map((filledStar, index) => ( 

 <div 

 data-index={JSON.stringify(index)} 

 key={index} 

 onMouseOver={handleMouseOver} 

 > 

 {filledStar} 

 </div> 

 ))} 

 {defaultStars.length !== 0 && 

 defaultStars.map((defaultStar, index) => ( 

 <div 

 data-index={JSON.stringify(index + filledStars.length)} 

 key={index + filledStars.length} 

 onMouseOver={handleMouseOver} 

 > 

 {defaultStar} 

 </div> 

 ))} 

 </div> 

 ); 

 }

Решение с dataset мне лично не очень нравится. У нас тут дз было про React и TS, а тут нативные JS методы. Но в целом норм.
На самом деле, тебе это только усложнило код. Также index в key это не очень хорошая практика, но тоже приемлемо.

Ошибка форматирования

dotnet-practice-labs/lab4/SpaceObjects/Core/Models/Planet.cs

Lines 19 to 30 in 0aa8efd

 public override string ToString() 

 { 

 return 

 $@"Planet {Name} 

  Diameter: {Diameter} 

  Mass: {Mass} 

  Surface temperature: {SurfaceTemperature} 

  Discovery date: {DiscoveryDate} 

 Planet Type: {PlanetTypeId} 

 Orbits star: {StarId} 

 "; 

 }

Пропсы это не тип для использования

dotnet-practice-labs/lab7/src/components/Reviewer/Reviewer.tsx

Line 12 in 0aa8efd

grade: GradeProps;

Привет!
Пропсы нужны для указания того, что ты передаешь в компонент. Ты не можешь их экспортировать и использовать как тип.
Зато ты можешь создать тип Grade и использовать его как пропсы type GradeProps = Grade!

Задание стилей элементу

dotnet-practice-labs/lab7/src/components/Common/Textarea/Textarea.tsx

Lines 14 to 20 in 0aa8efd

 useEffect(() => { 

 if (textareaRef.current) { 

 textareaRef.current.style.height = "inherit"; 

 const scrollHeight = textareaRef.current.scrollHeight; 

 textareaRef.current.style.height = scrollHeight + "px"; 

 } 

 });

В реакте мы фундаментально стараемся как меньше взаимодействовать с DOM, потому что подобные действия разрывают наш флоу работы приложения, и из-за этого оно работает медленнее и менее понятно. Также это недекларативно.
В таком случае обычно выносят всё что тебе нужно в классы в css файл, создают состояние типа isTextAreaHightlighted и в className пришут className={`${isTextAreaHightlighted && "textArea_highlighted"}`}
В данном же случае, тебе надо задавать высоту динамически. Ты можешь сделать состояние textAreaHeight типа number, менять его в handleTextChange, а в компоненте сделать переменную textAreaStyles, где и задавать стили, потом передавать это в пропсы.
Примерно так.

А почему все результаты функций через out?

dotnet-practice-labs/lab1/PersonalInfoReader/Program.cs

Line 4 in bf0cdd6

void RunPersonDialog(TextReader input, TextWriter output, out Person person)

Ошибка форматирования

dotnet-practice-labs/lab4/SpaceObjects/Core/Models/Star.cs

Lines 22 to 33 in 0aa8efd

 public override string ToString() 

 { 

 return 

 $@"Star {Name} 

  Diameter: {Diameter} 

  Mass: {Mass} 

  Magnitude: {Magnitude} 

  Temperature: {Temperature} 

  Discovery date: {DiscoveryDate} 

 Star Type: {StarTypeId} 

 "; 

 }

Приватные методы тоже с большой буквы

dotnet-practice-labs/lab6/WebAppApi/Services/UserService.cs

Line 84 in bf0cdd6

private User getUser(int id)

Не прописан foreign key

dotnet-practice-labs/lab3/SpaceObjectsCreate.sql

Line 11 in 0aa8efd

[id_planet_type] INT,

В полях внешних ключей нет команды создания внешнего ключа. Таблицы де-факто не связаны

Можно использовать String.Builder

dotnet-practice-labs/lab1/PersonalInfoReader/Program.cs

Line 54 in bf0cdd6

$@"[name]: {_name}

Незачем передавать callback

dotnet-practice-labs/lab7/src/components/Reviewer/Reviewer.tsx

Lines 37 to 39 in 0aa8efd

 setText(() => ""); 

 setRating(() => 0); 

 setReviews(() => [review]);

Ты можешь передавать туда callback, но он нужен, чтобы из параметров получать текущее состояние.
В твоём случае, это лишь усложнение. Можешь просто устанавливать значение и будет то же самое.

Модификаторы доступа у классов и полей класса

dotnet-practice-labs/lab1/PersonalInfoReader/Program.cs

Line 36 in bf0cdd6

class Person

Добавить обработку деления на 0 в 5-й лабе

Сейчас падает ошибка :-(

Nullable поля

dotnet-practice-labs/lab3/SpaceObjectsCreate.sql

Line 11 in 0aa8efd

[id_planet_type] INT,

Not null прописано только у первичных ключей, у остальных полей не указано. Необходимо определить, какие поля точно не могут быть null (например название планеты) и прописать им not null

Модификаторы доступа для полей класса

dotnet-practice-labs/lab2/Shapes/Circle.cs

Line 8 in bf0cdd6

Point _center;

Style в props

dotnet-practice-labs/lab7/src/components/Reviewer/Rating/Rating.tsx

Line 10 in 0aa8efd

style?: any;

Так как ты это используешь это не style, а className типа string.

Необходимо добавить модификторы доступа для всех методов

dotnet-practice-labs/lab1/BubbleSort/Program.cs

Line 6 in bf0cdd6

static void Main()

Ошибка форматирования

dotnet-practice-labs/lab4/SpaceObjects/DatabaseProvider/Repositories/Implementations/StarTypeRepository.cs

Lines 16 to 17 in d519b7a

 } 

 public StarType GetById(int id)

Перед строками, которые NVARCHAR необходимо ставить букву N

dotnet-practice-labs/lab3/SpaceObjectsInsert.sql

Line 8 in bf0cdd6

 ('Газовый гигант', 'Планеты, в значительной степени состоящие из газа, и значительно более массивные, чем планеты земной группы'), 

N'русский текст'

	function StarGrade(props: StarGradeProps) {
	const { currentGrade, maxGrade, setGrade } = props;

	// TODO: add isGrading or smth like that to commit user input
	const filledStars = Array.from({ length: currentGrade }, () => (
	<Star state="filled" />
	));
	const defaultStars = Array.from({ length: maxGrade - currentGrade }, () => (
	<Star state="default" />
	));

	function handleMouseOver(event: React.SyntheticEvent<HTMLDivElement>) {
	if (event.currentTarget.dataset && event.currentTarget.dataset.index) {
	setGrade(Number(event.currentTarget.dataset.index) + 1);
	}
	}

	return (
	<div className={styles.starGrades}>
	{filledStars.length !== 0 &&
	filledStars.map((filledStar, index) => (
	<div
	data-index={JSON.stringify(index)}
	key={index}
	onMouseOver={handleMouseOver}
	>
	{filledStar}
	</div>
	))}
	{defaultStars.length !== 0 &&
	defaultStars.map((defaultStar, index) => (
	<div
	data-index={JSON.stringify(index + filledStars.length)}
	key={index + filledStars.length}
	onMouseOver={handleMouseOver}
	>
	{defaultStar}
	</div>
	))}
	</div>
	);
	}

	public override string ToString()
	{
	return
	$@"Planet {Name}
	Diameter: {Diameter}
	Mass: {Mass}
	Surface temperature: {SurfaceTemperature}
	Discovery date: {DiscoveryDate}
	Planet Type: {PlanetTypeId}
	Orbits star: {StarId}
	";
	}

	useEffect(() => {
	if (textareaRef.current) {
	textareaRef.current.style.height = "inherit";
	const scrollHeight = textareaRef.current.scrollHeight;
	textareaRef.current.style.height = scrollHeight + "px";
	}
	});

	public override string ToString()
	{
	return
	$@"Star {Name}
	Diameter: {Diameter}
	Mass: {Mass}
	Magnitude: {Magnitude}
	Temperature: {Temperature}
	Discovery date: {DiscoveryDate}
	Star Type: {StarTypeId}
	";
	}

	setText(() => "");
	setRating(() => 0);
	setReviews(() => [review]);

pklgn / dotnet-practice-labs Goto Github PK

dotnet-practice-labs's People

Contributors

Watchers

dotnet-practice-labs's Issues

Recommend Projects

Recommend Topics

Recommend Org

Jobs