Ensimmäinen koodivinkkini :D.Ajattelin tämän tänne lisätä vaikka tämä nyt ei mikään älyttömän monimutkainen ole sillä tuntuu että nykyään täällä putkassa ei pahemmin ole aloittelioille suunnattuja vinkkejä.Exe tulee kuhan kerkeen laittaa...
Edit: Kommentteja lisäilty
Dim Merkki As Char Dim Pelaaja As Integer Private Sub Form1_Load(ByVal sender As System.Object, ByVal e As System.EventArgs) Handles MyBase.Load AloitaUusiPeli() End Sub 'Määritellään nappien tehtävät Private Sub Button1_Click(ByVal sender As System.Object, ByVal e As System.EventArgs) Handles Button1.Click Button1.Text = Merkki Button1.Enabled = False TarkistaVoittaja() End Sub Private Sub Button2_Click(ByVal sender As System.Object, ByVal e As System.EventArgs) Handles Button2.Click Button2.Text = Merkki Button2.Enabled = False TarkistaVoittaja() End Sub Private Sub Button3_Click(ByVal sender As System.Object, ByVal e As System.EventArgs) Handles Button3.Click Button3.Text = Merkki Button3.Enabled = False TarkistaVoittaja() End Sub Private Sub Button4_Click(ByVal sender As System.Object, ByVal e As System.EventArgs) Handles Button4.Click Button4.Text = Merkki Button4.Enabled = False TarkistaVoittaja() End Sub Private Sub Button5_Click(ByVal sender As System.Object, ByVal e As System.EventArgs) Handles Button5.Click Button5.Text = Merkki Button5.Enabled = False TarkistaVoittaja() End Sub Private Sub Button6_Click(ByVal sender As System.Object, ByVal e As System.EventArgs) Handles Button6.Click Button6.Text = Merkki Button6.Enabled = False TarkistaVoittaja() End Sub Private Sub Button7_Click(ByVal sender As System.Object, ByVal e As System.EventArgs) Handles Button7.Click Button7.Text = Merkki Button7.Enabled = False TarkistaVoittaja() End Sub Private Sub Button8_Click(ByVal sender As System.Object, ByVal e As System.EventArgs) Handles Button8.Click Button8.Text = Merkki Button8.Enabled = False TarkistaVoittaja() End Sub Private Sub Button9_Click(ByVal sender As System.Object, ByVal e As System.EventArgs) Handles Button9.Click Button9.Text = Merkki Button9.Enabled = False TarkistaVoittaja() End Sub Private Sub Button12_Click(ByVal sender As System.Object, ByVal e As System.EventArgs) Handles Button12.Click Button12.Text = Merkki Button12.Enabled = False TarkistaVoittaja() End Sub Private Sub Button13_Click(ByVal sender As System.Object, ByVal e As System.EventArgs) Handles Button13.Click Button13.Text = Merkki Button13.Enabled = False TarkistaVoittaja() End Sub Private Sub Button14_Click(ByVal sender As System.Object, ByVal e As System.EventArgs) Handles Button14.Click Button14.Text = Merkki Button14.Enabled = False TarkistaVoittaja() End Sub Private Sub Button15_Click(ByVal sender As System.Object, ByVal e As System.EventArgs) Handles Button15.Click Button15.Text = Merkki Button15.Enabled = False TarkistaVoittaja() End Sub Private Sub Button16_Click(ByVal sender As System.Object, ByVal e As System.EventArgs) Handles Button16.Click Button16.Text = Merkki Button16.Enabled = False TarkistaVoittaja() End Sub Private Sub Button17_Click(ByVal sender As System.Object, ByVal e As System.EventArgs) Handles Button17.Click Button17.Text = Merkki Button17.Enabled = False TarkistaVoittaja() End Sub Private Sub Button18_Click(ByVal sender As System.Object, ByVal e As System.EventArgs) Handles Button18.Click Button18.Text = Merkki Button18.Enabled = False TarkistaVoittaja() End Sub Private Sub Button10_Click(ByVal sender As System.Object, ByVal e As System.EventArgs) Handles Button10.Click AloitaUusiPeli() End Sub Private Sub Button11_Click(ByVal sender As System.Object, ByVal e As System.EventArgs) Handles Button11.Click End End Sub Private Sub SeuraavaPelaaja() If Pelaaja = 1 Then Merkki = "X" Pelaaja = 2 ElseIf Pelaaja = 2 Then Merkki = "0" Pelaaja = 1 End If 'Näytetään kene vuoro Label2.Text = "Pelaaja " & Pelaaja & " : ' " & Merkki & " ' " End Sub Private Sub NäytäVoittaja() 'näytä voittoviesti Label2.Text = "pelaaja ' " & Merkki & " ' Hakkasi toisen pelaajan pystyyn!Ottakaa uusinta matsi!!" 'sammuta kaikki napit Button1.Enabled = False Button2.Enabled = False Button3.Enabled = False Button4.Enabled = False Button5.Enabled = False Button6.Enabled = False Button7.Enabled = False Button8.Enabled = False Button9.Enabled = False Button12.Enabled = False Button13.Enabled = False Button14.Enabled = False Button15.Enabled = False Button16.Enabled = False Button17.Enabled = False Button18.Enabled = False End Sub Private Sub TarkistaVoittaja() 'tarkista onko voittorivejä If Len(Button1.Text & Button2.Text & Button3.Text & Button12.Text) = 4 And _ Button1.Text = Button2.Text And Button2.Text = Button3.Text And Button3.Text = Button12.Text Then Button1.BackColor = Color.Yellow Button2.BackColor = Color.Yellow Button3.BackColor = Color.Yellow Button12.BackColor = Color.Yellow NäytäVoittaja() MsgBox("Hah liian helppoa") ElseIf Len(Button4.Text & Button5.Text & Button6.Text & Button13.Text) = 4 And _ Button4.Text = Button5.Text And Button5.Text = Button6.Text And Button6.Text = Button13.Text Then Button4.BackColor = Color.Yellow Button5.BackColor = Color.Yellow Button6.BackColor = Color.Yellow Button13.BackColor = Color.Yellow NäytäVoittaja() ElseIf Len(Button7.Text & Button8.Text & Button9.Text & Button14.Text) = 4 And _ Button7.Text = Button8.Text And Button8.Text = Button9.Text And Button9.Text = Button14.Text Then Button7.BackColor = Color.Yellow Button8.BackColor = Color.Yellow Button9.BackColor = Color.Yellow Button14.BackColor = Color.Yellow NäytäVoittaja() ElseIf Len(Button1.Text & Button4.Text & Button7.Text & Button15.Text) = 4 And _ Button1.Text = Button4.Text And Button4.Text = Button7.Text And Button7.Text = Button15.Text Then Button1.BackColor = Color.Yellow Button4.BackColor = Color.Yellow Button7.BackColor = Color.Yellow Button15.BackColor = Color.Yellow NäytäVoittaja() ElseIf Len(Button2.Text & Button5.Text & Button8.Text & Button16.Text) = 4 And _ Button2.Text = Button5.Text And Button5.Text = Button8.Text And Button8.Text = Button16.Text Then Button2.BackColor = Color.Yellow Button5.BackColor = Color.Yellow Button8.BackColor = Color.Yellow Button16.BackColor = Color.Yellow NäytäVoittaja() ElseIf Len(Button3.Text & Button6.Text & Button9.Text & Button17.Text) = 4 And _ Button3.Text = Button6.Text And Button6.Text = Button9.Text And Button9.Text = Button17.Text Then Button3.BackColor = Color.Yellow Button6.BackColor = Color.Yellow Button9.BackColor = Color.Yellow Button17.BackColor = Color.Yellow NäytäVoittaja() ElseIf Len(Button1.Text & Button5.Text & Button9.Text & Button18.Text) = 4 And _ Button1.Text = Button5.Text And Button5.Text = Button9.Text And Button9.Text = Button18.Text Then Button1.BackColor = Color.Yellow Button5.BackColor = Color.Yellow Button9.BackColor = Color.Yellow Button18.BackColor = Color.Yellow NäytäVoittaja() ElseIf Len(Button1.Text & Button5.Text & Button9.Text & Button18.Text) = 4 And _ Button1.Text = Button5.Text And Button5.Text = Button9.Text And Button9.Text = Button18.Text Then Button1.BackColor = Color.Yellow Button5.BackColor = Color.Yellow Button9.BackColor = Color.Yellow Button18.BackColor = Color.Yellow NäytäVoittaja() ElseIf Len(Button15.Text & Button16.Text & Button17.Text & Button18.Text) = 4 And _ Button15.Text = Button16.Text And Button16.Text = Button17.Text And Button17.Text = Button18.Text Then Button15.BackColor = Color.Yellow Button16.BackColor = Color.Yellow Button17.BackColor = Color.Yellow Button18.BackColor = Color.Yellow NäytäVoittaja() ElseIf Len(Button12.Text & Button6.Text & Button8.Text & Button15.Text) = 4 And _ Button12.Text = Button6.Text And Button6.Text = Button8.Text And Button8.Text = Button15.Text Then Button12.BackColor = Color.Yellow Button6.BackColor = Color.Yellow Button8.BackColor = Color.Yellow Button15.BackColor = Color.Yellow NäytäVoittaja() ElseIf Len(Button12.Text & Button13.Text & Button14.Text & Button18.Text) = 4 And _ Button12.Text = Button13.Text And Button13.Text = Button14.Text And Button14.Text = Button18.Text Then Button12.BackColor = Color.Yellow Button13.BackColor = Color.Yellow Button14.BackColor = Color.Yellow Button18.BackColor = Color.Yellow NäytäVoittaja() ElseIf Len(Button12.Text & Button13.Text & Button14.Text & Button18.Text) = 4 And _ Button12.Text = Button13.Text And Button13.Text = Button14.Text And Button14.Text = Button18.Text Then Button12.BackColor = Color.Yellow Button13.BackColor = Color.Yellow Button14.BackColor = Color.Yellow Button18.BackColor = Color.Yellow NäytäVoittaja() Else SeuraavaPelaaja() End If End Sub Private Sub AloitaUusiPeli() 'Napit taas käyttöön Button1.Enabled = True Button2.Enabled = True Button3.Enabled = True Button4.Enabled = True Button5.Enabled = True Button6.Enabled = True Button7.Enabled = True Button8.Enabled = True Button9.Enabled = True Button12.Enabled = True Button13.Enabled = True Button14.Enabled = True Button15.Enabled = True Button16.Enabled = True Button17.Enabled = True Button18.Enabled = True 'Nollataan nappien tekstit/merkit Button1.Text = "" Button2.Text = "" Button3.Text = "" Button4.Text = "" Button5.Text = "" Button6.Text = "" Button7.Text = "" Button8.Text = "" Button9.Text = "" Button12.Text = "" Button13.Text = "" Button14.Text = "" Button15.Text = "" Button16.Text = "" Button17.Text = "" Button18.Text = "" 'Muutetaan nappien taustaväri Button1.BackColor = Color.Lime Button2.BackColor = Color.Lime Button3.BackColor = Color.Lime Button4.BackColor = Color.Lime Button5.BackColor = Color.Lime Button6.BackColor = Color.Lime Button7.BackColor = Color.Lime Button8.BackColor = Color.Lime Button9.BackColor = Color.Lime Button12.BackColor = Color.Lime Button13.BackColor = Color.Lime Button14.BackColor = Color.Lime Button15.BackColor = Color.Lime Button16.BackColor = Color.Lime Button17.BackColor = Color.Lime Button18.BackColor = Color.Lime Pelaaja = 2 Merkki = "0" SeuraavaPelaaja() End Sub Private Sub Button19_Click(ByVal sender As System.Object, ByVal e As System.EventArgs) Handles Button19.Click 'Ohje... MsgBox("Pelin ideana on saada 'X' merkkejä tai '0' merkkejä neljä peräkkäin.Painamalla 'Otetaan uusiksi' peli alkaa alusta.Painamalla 'Blaah tylsää' peli loppuu") End Sub End Class
Aloittelijalle suunnatussa koodivinkissä sopii olla kommentteja enemmän kuin 3.
kommentteja lisäilty...
"Yksinkertainen"
Kannattaisi tässäkin käyttää indeksointia sen sijaan, että tekisi tuhottomasti eri nimisiä kontrolleja.
hunajavohveli kirjoitti:
Kannattaisi tässäkin käyttää indeksointia sen sijaan, että tekisi tuhottomasti eri nimisiä kontrolleja.
Kyllä. Ihan yhtä helppoa se ei tosin ole kuin wanhassa VB:ssä:
http://msdn.microsoft.com/library/default.asp?
Aihe on jo aika vanha, joten et voi enää vastata siihen.