WPF Problem z bindowniem :: 4programmers.net

Rejestracja:ponad 8 lat
Ostatnio:prawie 4 lata
Postów:37

0

Witam, mam poblem aplikacja nie chce się odpalić i wywala błąd:

System.NullReferenceException occurred
Message=Odwołanie do obiektu nie zostało ustawione na wystąpienie obiektu.

Poradzcie coś bo już próbuję to rozkminić 4h :/

Kopiuj

private ICommand _dodajKwoteCommand;

        public ICommand DodajKwote
        {
            get
            {
                if (_dodajKwoteCommand == null)
                    _dodajKwoteCommand = new RelayCommand(
                        (object argument) =>
                        {
                            decimal kwota = decimal.Parse(argument.ToString());
                            model.Dodaj(kwota);
                            OnPropertyChanged("Suma");
                        }
                        ,
                        (object argument) =>
                        {
                            decimal kwota = decimal.Parse(argument.ToString()); // tu wywala błąd
                            return CzyLancuchKwotyJestPoprawny(kwota);
                        }
                        );
                return _dodajKwoteCommand;
            }
        }

Kopiuj

<Window x:Class="AsystentZakupow.MainWindow"
        xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
        xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
        xmlns:d="http://schemas.microsoft.com/expression/blend/2008"
        xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
        xmlns:local="clr-namespace:AsystentZakupow"
        xmlns:mw="clr-namespace:AsystentZakupow.ViewModel"
        mc:Ignorable="d"
        Title="Asystent Zakupow" Height="200" Width="200">
    <Window.DataContext>
        <mw:ModelWidoku />
    </Window.DataContext>
    <Window.Resources>
        <local:Converter x:Key="boolToBrush" />
        <local:ConverterStringToDoublew x:Key="stringToDecimal" />
    </Window.Resources>
    <Grid>
        <Grid.RowDefinitions>
            <RowDefinition Height="1*"/>
            <RowDefinition Height="1*"/>
            <RowDefinition Height="1*"/>
        </Grid.RowDefinitions>
        <TextBlock HorizontalAlignment="Left" Grid.Row="0" FontSize="25" 
                   Foreground="Navy" Margin="10"
                   VerticalAlignment="Top">
            Suma: 
            <Run Foreground="Black" FontFamily="Courier New" Text="{Binding Suma, Mode=OneWay}"/>
        </TextBlock>
        <TextBox x:Name="tbKwota" Margin="10" Grid.Row="1" TextAlignment="Right" 
                 Foreground="{Binding ElementName=btnDodaj, Path=IsEnabled, Mode=OneWay, Converter={StaticResource boolToBrush}}"
                 FontFamily="Courier New" Text="0" FontSize="30"/>
        <Button x:Name="btnDodaj" 
                Margin="10" Grid.Row="2" 
                Command="{Binding DodajKwote}"
                CommandParameter="{Binding ElementName=tbKwota, Path=Text, Converter={StaticResource stringToDecimal}}"
                Content="Dodaj" FontSize="20" />
    </Grid>
</Window>

edytowany 1x, ostatnio: Marcino24 2017-04-01 13:08

cSharpKazik

2017-04-01 12:58

cSharpKazik2017-04-01 12:58

Rejestracja:ponad 8 lat
Ostatnio:około 2 lata
Postów:117

0

Kiedyś robiłem ten program.

Zamień sobie to:

Kopiuj

decimal kwota = decimal.Parse(argument.ToString()); // tu wywala błąd
return CzyLancuchKwotyJestPoprawny(kwota);

na to:

Kopiuj

return CzyLancuchKwotyJestPoprawny((string)argument);

i powinno być ok.

CzyLanuchKwotyJestPoprawny u mnie wyglądal tak:

Kopiuj

public bool CzyLancuchKwotyJestPoprawny(string s)
        {
            if (string.IsNullOrWhiteSpace(s))
            {
                return false;
            }

            decimal kwota;

            if (!decimal.TryParse(s, out kwota))
            {
                return false;
            }
            else
            {
                return model.CzyKwotaJestPoprawna(kwota);
            }
        }

Marcino24

2017-04-01 13:09

Marcino242017-04-01 13:09

Rejestracja:ponad 8 lat
Ostatnio:prawie 4 lata
Postów:37

0

Tak już próbowałem i to samo. A powiedz mi czy w XAMLu masz tak samo? Bo właśnie wydaję mi się że to właśnie tam jest przyczyna błędu.

cSharpKazik

2017-04-01 19:08

cSharpKazik2017-04-01 19:08

Rejestracja:ponad 8 lat
Ostatnio:około 2 lata
Postów:117

0

A jak wygląda u Ciebie metoda "CzyLanuchKwotyJestPoprawny" ?

grzesiek51114

2017-04-01 21:16

~~grzesiek51114~~2017-04-01 21:16

Rejestracja:ponad 11 lat
Ostatnio:ponad 4 lata
Postów:2442

1

A po co Ty w ogóle przekazujesz kontrolkę jako parametr przycisku zamiast zwyczajnie zbindować stringa do TextBox.Text? Zupełnie niepotrzebnie komplikujesz sobie kod:

Kopiuj

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.ComponentModel;
using System.Windows.Input;

namespace _4programmersWPF
{
    public class MainWindowVM : INotifyPropertyChanged
    {
        private string text;
        public string Text
        {
            get { return this.text; }
            set
            {
                this.text = value;
                this.OnPropertyChanged(nameof(this.Text));
            }
        }

        private decimal salary;

        public ICommand PressMe
        {
            get
            {
                return new RelayCommand(() =>
                {
                    //  Some code...
                },

                () => decimal.TryParse(this.Text, out this.salary));
            }
        }

        public event PropertyChangedEventHandler PropertyChanged;
        public void OnPropertyChanged(string property)
        {
            this.PropertyChanged?.Invoke(this, new PropertyChangedEventArgs(property));
        }
    }
}

Kopiuj

<Window x:Class="_4programmersWPF.MainWindow"
        xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
        xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
        xmlns:d="http://schemas.microsoft.com/expression/blend/2008"
        xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
        xmlns:local="clr-namespace:_4programmersWPF"
        xmlns:vm="clr-namespace:_4programmersWPF"
        mc:Ignorable="d"
        Title="4prog test app"
        Name="Window" WindowStartupLocation="CenterScreen" SizeToContent="WidthAndHeight">
  <Window.DataContext>
    <vm:MainWindowVM/>
  </Window.DataContext>
  <StackPanel>
    <TextBox Text="{Binding Text,UpdateSourceTrigger=PropertyChanged}"
             FontSize="14" Margin="5"/>
    <Button Command="{Binding PressMe}"
            Content="Press me!" FontSize="14" Margin="5"/>
  </StackPanel>
</Window>

Możesz pójść nawet o krok dalej i zbindować do textboksa własność typu decimal. Walidacja nastąpi automatycznie; pole zaświeci się na czerwono kiedy wpiszesz nieprawidłową wartość.

edytowany 1x, ostatnio: ~~grzesiek51114~~ 2017-04-01 21:18

Marcino24

2017-04-02 09:41

Marcino242017-04-02 09:41

Rejestracja:ponad 8 lat
Ostatnio:prawie 4 lata
Postów:37

0

W zadaniu w pewnej książce było, żeby zrobic konwerter string na decimal a to co ty podałeś to jest całkiem inna ścieżka. Mógłbyś coś poradzić jak to zrobić na konwerterze?

grzesiek51114

2017-04-02 11:06

~~grzesiek51114~~2017-04-02 11:06

Rejestracja:ponad 11 lat
Ostatnio:ponad 4 lata
Postów:2442

1

Ale uwierz mi, że naprawdę nie potrzebujesz tam żadnego konwertera. Interfejsu IValueConverter używa się takich przypadkach jak ten tutaj: http://www.wpf-tutorial.com/data-binding/value-conversion-with-ivalueconverter/

U Ciebie wystarczy zwykłe bindowanie.

No ale jak już to...:

Kopiuj

using System;
using System.Collections.Generic;
using System.Globalization;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Data;

namespace _4programmersWPF
{
    public class Converter : IValueConverter
    {
        public object Convert(object value, Type targetType, object parameter, CultureInfo culture)
        {
            var result = decimal.Zero;
            decimal.TryParse((value as string), out result);
            return result;
        }

        public object ConvertBack(object value, Type targetType, object parameter, CultureInfo culture)
        {
            return string.Format("{0}", value != null ? (decimal)value : 0);
        }
    }
}

Kopiuj

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.ComponentModel;
using System.Windows.Input;

namespace _4programmersWPF
{
    public class MainWindowVM : INotifyPropertyChanged
    {
        private decimal salary;
        public decimal Salary
        {
            get { return this.salary; }
            set
            {
                this.salary = value;
                this.OnPropertyChanged(nameof(this.Salary));
            }
        }

        private string message;
        public string Message
        {
            get { return this.message; }
            set
            {
                this.message = value;
                this.OnPropertyChanged(nameof(this.Message));
            }
        }

        public ICommand PressMe
        {
            get
            {
                return new RelayCommandParametrized(p =>
                {
                    this.Salary = (decimal)p; // Przy konwerterze masz pewność, że tutaj nie sypnie Ci wyjątkiem.
                    this.Message = string.Format("{0}", this.Salary);
                });
            }
        }

        public event PropertyChangedEventHandler PropertyChanged;
        public void OnPropertyChanged(string property)
        {
            this.PropertyChanged?.Invoke(this, new PropertyChangedEventArgs(property));
        }
    }
}

Kopiuj

<Window x:Class="_4programmersWPF.MainWindow"
        xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
        xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
        xmlns:d="http://schemas.microsoft.com/expression/blend/2008"
        xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
        xmlns:local="clr-namespace:_4programmersWPF"
        xmlns:vm="clr-namespace:_4programmersWPF"
        mc:Ignorable="d"
        Title="4prog test app"
        Name="Window" WindowStartupLocation="CenterScreen" SizeToContent="WidthAndHeight">
  <Window.DataContext>
    <vm:MainWindowVM/>
  </Window.DataContext>
  <Window.Resources>
    <vm:Converter x:Key="Converter"/>
  </Window.Resources>
  <StackPanel>
    <TextBox Margin="5" Name="tbx"/>
    <Button Content="Press me!" Command="{Binding PressMe}"
            CommandParameter="{Binding ElementName=tbx,
            Path=Text,Converter={StaticResource Converter}}"
            Margin="5"/>
    <Label Content="{Binding Message, UpdateSourceTrigger=PropertyChanged}"
           Margin="5"/>
  </StackPanel>
</Window>

...i masz teraz taką mini walidację danych. Problem w tym, że stosowaniem konwerterów do takich rzeczy trochę zaciemniasz sobie kod, bo zwyczajnie można to po prostu zbindować.

edytowany 2x, ostatnio: ~~grzesiek51114~~ 2017-04-02 11:41

Marcino24

2017-04-03 18:42

Marcino242017-04-03 18:42

Rejestracja:ponad 8 lat
Ostatnio:prawie 4 lata
Postów:37

0

Dzięki za wytłumaczenie problemu. Nie wiem czemu w książce uczą żeby zrobić konwerter string na double jak takiej praktyki się nie stosuje. Teraz już przynajmniej wiem że tak się nie robi. ;)

~~grzesiek51114~~ 2017-04-03 18:45

Książka pewnie próbuje wytłumaczyć Ci na przykładzie jak działają takie konwertery ale lepszym przykładem jest jednak ten z linku, który podałem.

Marcino24

2017-06-09 22:24

Marcino242017-06-09 22:24

Rejestracja:ponad 8 lat
Ostatnio:prawie 4 lata
Postów:37

0

Miałbym jeszcze jeden problem, co prawda inny projekt ale myślę że mogę go napisać w tym temacie. Mam coś takiego
title

I teraz jeżeli kliknę na pozycję z listy to mam bindowanie do textboxów a wykonuję to poprzez selectValue np dla osoby:

Kopiuj

uzupełnienie listy oraz binodwanie SelectedValue
<ListView x:Name="lvPersonsList" SelectedValue="{Binding SelectedPerson, UpdateSourceTrigger=PropertyChanged}" 
                              ItemsSource="{Binding Path=PersonsList}" Grid.Row="1">

bindowanie do text boxów danych z SelectedValue
<Label Content="Imie "/>
                <TextBox x:Name="txbNameEdit" Text="{Binding SelectedPerson.PersonID.Name}" VerticalContentAlignment="Center" Height="40" />
                <Label Content="Nazwisko "/>
                <TextBox x:Name="txbSureNameEdit" Text="{Binding SelectedPerson.PersonID.SureName}" VerticalContentAlignment="Center" Height="40" />

Jak zrobić by po zmianie tekstu w okienku textboxa był on przypisywany do jakiegoś property żeby się go dało później go dalej przetworzyć (np wysłać update do bazy danych) ?

Ktos

2017-06-10 08:49

Ktos2017-06-10 08:49

Moderator

Rejestracja:prawie 23 lata
Ostatnio:około 10 godzin

0

Prawdopodobnie wystarczy ci binding w trybie TwoWay:

Kopiuj

<TextBox x:Name="txbNameEdit" Text="{Binding SelectedPerson.PersonID.Name, Mode=TwoWay}" VerticalContentAlignment="Center" Height="40" />

Teraz automatycznie Name zostanie zaktualizowane kiedy TextBox z nim związany straci focus.

Marcino24

2017-06-10 12:01

Marcino242017-06-10 12:01

Rejestracja:ponad 8 lat
Ostatnio:prawie 4 lata
Postów:37

0

No dobra, ale teraz tak robię to na obiektowej bazie danych i żeby wyszukać stare dane osobowe muszę mu wpisać stare wartości a tu mi podmieni stare na te zmienione z textboxa. Jak zrobić żeby był jakiś tempName oraz tempSureName do których zostaną przypisane oryginalne wartości?

edytowany 1x, ostatnio: Marcino24 2017-06-10 13:08

Marcino24

2017-06-10 17:50

Marcino242017-06-10 17:50

Rejestracja:ponad 8 lat
Ostatnio:prawie 4 lata
Postów:37

0

Próbowałem coś takiego:

Kopiuj

 private Person _selectedPerson;

        public Person SelectedPerson
        {
            get { return _selectedPerson; }
            set
            {
                _selectedPerson = value;
                _selectedPerson1 = value;
                OnPropertyChanged(nameof(this.SelectedPerson));
            }
        }

        private Person_selectedPerson1;

        public Person SelectedPerson1
        {
            get { return _selectedPerson1; }
            set
            {
                _selectedPerson1 = value;
                OnPropertyChanged(nameof(this.SelectedPerson1));
            }
        }

i teraz mam inny problem, dlaczego gdy teraz w teztboxie zmieniam wartość dla SelectedPerson1.Name to zmienia się też automatycznie dla SelectedPerson.Name ?

edytowany 2x, ostatnio: Marcino24 2017-06-12 05:16

WPF Problem z bindowniem

Marcino24

cSharpKazik

Marcino24

cSharpKazik

grzesiek51114

Marcino24

grzesiek51114

Marcino24

Marcino24

Ktos

Marcino24

Marcino24

Praca dla programistów

Forum dyskusyjne

Sprawy administracyjne

O nas

Skontaktuj się z nami