Diskuze: Vlastní třída
V předchozím kvízu, Test znalostí C# .NET online, jsme si ověřili nabyté zkušenosti z kurzu.
Petr Nymsa:15.9.2013 20:34
Prosím, přečti si zdejší tutoriály .... tímhle to nikam nevede.
Nemáš páru vůbec o čem se zde hovoří 
.. je pro zajímavost .. postni sem
ten kód at třeba pomůžeme
Jan Vargovský:15.9.2013 22:34
Soutěž o nejpekelnější kód ? Kdybys alespoň znal AND, OR operátory
... Zkus si prosím tě přečíst raději tutoriály a pak si budeš trhat
vlasy, co jsi to vůbec napsal
Petr Nymsa:16.9.2013 8:12
To je opravdu katastrofa. Přečti si ty zdejší tutoriály. Ty tvé "voidy"
stejně v jiném programu fungovat nebudou například ten "void" co určuje
stisknutou klávesu. A to jednodušše proto že to je callback události
KeyDown na Formu 
Petr Nymsa:16.9.2013 8:40
Ano používej partial pokud to má smysl. V tvém případě to smysl nemá.
Podívej se na tutoriály o OOP a sám poznáš že by js to mohl napsat mnohem
lépe 
Například jsem zkoušel využít && pro
Luboš Běhounek Satik:16.9.2013 8:50
Myslím, že by ten kód mohl klidně zůstat v jednom souboru, zatím to
není složitý kód
Zaměř se raději na kód metody rectangle, dala by se napsat asi na 5
řádků .
Luboš Běhounek Satik:16.9.2013 8:52
Bylo by to takhle:
if (rPlayer.IntersectsWith(r1) || rPlayer.IntersectsWith(r2))
{
start();
}Ale doporučuji to přepsat úplně
Jan Vargovský:16.9.2013 14:09
Nemůžeš spojovat parametry, musíš spojit ty výsledky. Když se zamyslíš nad tím kódem, tak i když spustíš hned u první podmínky ten start, tak se potom testujou ty všechny. (je to zbytečné - ale to je jen takový detail)
Jestli si pamatuju, jsi ten člověk, co dělal ty microsoftácké tutoriály.
Takže jestli ano, tak několik rad:
Ty všechny labely můžeš získat skrz foreach cyklus z formuláře :
foreach (Control c in Controls)
{
if (c is Label)
{
// TODO
}
}Pozn. Kdyby to bylo v panelu, tak by jsi akorát volal panel.Controls místo Controls.
Každý label má vlastnost ClientRectangle (velikost toho labelu), takže si je stačí uložit někde na začátku programu a jen snima pracovat.
Vytvořil by sis privátní list:
private List<Rectangle> rectangles = new List<Rectangle>();přidal do listu v konstruktoru všechny ty velikosti a souřadnice těch labelů
foreach (Control c in Controls)
{
if (c is Label)
{
rectangles.Add(c.ClientRectangle);
}
}A pak si jen napsal metodu, která ti proiteruje toho playera se zbytkem těch rectanglu.
private bool IsThrough(Rectangle playerRectangle, params Rectangle[] rectangles)
{
foreach (Rectangle r in rectangles)
{
if (playerRectangle.IntersectsWith(r))
return true;
}
return false;
}Takže tvůj kód by byl
if (IsThrough(label1.ClientRectangle, rectangles.ToArray()))
{
start();
}Tvůj kód je pak univerzální, že ať předěláš své bludiště jakkoliv (bylo by těm těch labelů třeba 1000) tak nebudeš vytvářet r1 - r1000 (kdybys použil alespoň pole), ale jednoduše si pomůžeš.
Snad ti to pomůže
Petr Nymsa:16.9.2013 17:08
Problémem který tu všichni řešíme je ten, že zřejmě nechápeš
přesně co se tam děje Pokud
to potřebuješ splácat narychlo, opisuj. Pokud tomu chceš porozumět přečti
si zdejší tutoriály
noName:17.9.2013 12:24
Stále mi to nejede 
Můžeš
prosím vložit ty kódy do mého kódu?
Labely jsou umístěny na panel1
using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Windows.Forms;
using System.Diagnostics;
namespace Maze
{
public partial class Form1 : Form
{
public Form1()
{
InitializeComponent();
start();
}
// STOPWATCH:
public Stopwatch watch = new Stopwatch();
// START:
public void start()
{
player.Location = new Point(0, 575);
watch.Reset();
watch.Start();
}
// CONTROL:
public void Form1_KeyDown(object sender, KeyEventArgs e)
{
// UP:
if (e.KeyCode == Keys.W)
{
player.Top -= 10;
}
// DOWN:
if (e.KeyCode == Keys.S)
{
player.Top += 10;
}
// LEFT:
if (e.KeyCode == Keys.A)
{
player.Left -= 10;
}
// RIGHT:
if (e.KeyCode == Keys.D)
{
player.Left += 10;
}
}
// FINISH:
public void finish()
{
watch.Stop();
string time = watch.Elapsed.Seconds.ToString();
MessageBox.Show("Congratulations! " + "Your time: " + time + " seconds.");
}
}
}Předem děkuji.
Zobrazeno 50 zpráv z 67.